refactor(tests): clean up

tests/tests/errors/error-handler-second-callback.js

@@ -20,7 +20,7 @@ app.use((req, res, next) => {
     res.status(400).json({ error: 'test' });
 });
 
-app.get('/test3', (req, res) => {
+app.get('/test3', (req, res, next) => {
     next(new Error('test'));
 });
 

tests/tests/routers/complex-routers.js

@@ -3,7 +3,7 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
+const router = express.Router();
 
 router.get('/test', (req, res) => {
     console.log('found router', req.params.test);

tests/tests/routers/empty-router.js

@@ -3,7 +3,7 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
+const router = express.Router();
 
 router.get('/', (req, res) => {
     res.send('test2');

tests/tests/routers/mergeparams.js

@@ -3,8 +3,8 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router({ mergeParams: true });
-const router2 = new express.Router({ mergeParams: true });
+const router = express.Router({ mergeParams: true });
+const router2 = express.Router({ mergeParams: true });
 
 app.use('/:param1', router);
 router.use('/:param2', router2);

tests/tests/routers/nested-routers.js

@@ -4,9 +4,9 @@ const express = require("express");
 
 const app = express();
 app.set("etag", false);
-const router = new express.Router();
-const router2 = new express.Router();
-const router3 = new express.Router();
+const router = express.Router();
+const router2 = express.Router();
+const router3 = express.Router();
 
 router.get('/test', (req, res) => {
     res.send('test2');

tests/tests/routers/router-function.js

@@ -3,7 +3,7 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
+const router = express.Router();
 
 router.get('/test', (req, res) => {
     res.send('test');

tests/tests/routers/routers.js

@@ -3,8 +3,8 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
-const router2 = new express.Router();
+const router = express.Router();
+const router2 = express.Router();
 const router3 = express.Router();
 
 router.get('/test', (req, res) => {

tests/tests/routers/simple-routers.js

@@ -3,7 +3,7 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
+const router = express.Router();
 
 router.get('/:test', (req, res, next) => {
     console.log(req.url, req.params.test);

tests/tests/routers/trailing-slashes.js

@@ -3,7 +3,7 @@
 const express = require("express");
 
 const app = express();
-const router = new express.Router();
+const router = express.Router();
 
 router.get("/", (req, res) => {
     res.send("main");

tests/tests/routing/options-router.js

@@ -18,7 +18,7 @@ app.use((req, res, next) => {
 app.listen(13333, async () => {
     console.log('Server is running on port 13333');
 
-    res = await fetch('http://localhost:13333/router/test', { method: 'OPTIONS' });
+    let res = await fetch('http://localhost:13333/router/test', { method: 'OPTIONS' });
     console.log(await res.text(), res.status, res.headers.get('allow'));
 
     res = await fetch('http://localhost:13333/router/test2', { method: 'OPTIONS' });

tests/tests/send-file/headers.js

@@ -1,7 +1,6 @@
 // must support headers option
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/if-unmodified-since.js

@@ -1,7 +1,6 @@
 // must support If-Unmodified-Since header
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/immutable.js

@@ -1,7 +1,6 @@
 // must support immutable option
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/last-modified.js

@@ -1,7 +1,6 @@
 // must support lastModified option
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/max-age.js

@@ -1,7 +1,6 @@
 // must support maxAge option
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/path-traversal.js

@@ -1,7 +1,6 @@
 // must prevent path traversal attacks
 
 const express = require("express");
-const path = require("path");
 
 const app = express();
 

tests/tests/send-file/range.js

@@ -1,7 +1,6 @@
 // must support Range header
 
 const express = require("express");
-const path = require("path");
 
 const app = express();