Skip to content

Commit d8c881a

Browse files
authored
fix(lint): methodOverride cause lint fail (#1693)
1 parent e04625e commit d8c881a

File tree

11 files changed

+33
-11
lines changed

11 files changed

+33
-11
lines changed

tests/esm/fixtures/express/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ import { RegisterRoutes } from './routes.js';
1010
export const app: express.Express = express();
1111
app.use(bodyParser.urlencoded({ extended: true }) as RequestHandler);
1212
app.use(bodyParser.json() as RequestHandler);
13-
app.use(methodOverride());
13+
app.use((req, res, next) => {
14+
methodOverride()(req, res, next);
15+
});
1416
app.use((req: any, res: any, next: any) => {
1517
req.stringValue = 'fancyStringForContext';
1618
next();

tests/fixtures/custom/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@ import { RegisterRoutes } from './customRoutes';
2020
export const app: express.Express = express();
2121
app.use(bodyParser.urlencoded({ extended: true }));
2222
app.use(bodyParser.json());
23-
app.use(methodOverride());
23+
app.use((req, res, next) => {
24+
methodOverride()(req, res, next);
25+
});
2426
app.use((req: any, res: any, next: any) => {
2527
req.stringValue = 'fancyStringForContext';
2628
next();

tests/fixtures/express-dynamic-controllers/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { RegisterRoutes } from './routes';
77
export const app: express.Express = express();
88
app.use(bodyParser.urlencoded({ extended: true }));
99
app.use(bodyParser.json());
10-
app.use(methodOverride());
10+
app.use((req, res, next) => {
11+
methodOverride()(req, res, next);
12+
});
1113
app.use((req: any, res: any, next: any) => {
1214
req.stringValue = 'fancyStringForContext';
1315
next();

tests/fixtures/express-openapi3/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,9 @@ import { RegisterRoutes } from './routes';
2323
export const app: express.Express = express();
2424
app.use(bodyParser.urlencoded({ extended: true }));
2525
app.use(bodyParser.json());
26-
app.use(methodOverride());
26+
app.use((req, res, next) => {
27+
methodOverride()(req, res, next);
28+
});
2729
app.use((req: any, res: any, next: any) => {
2830
req.stringValue = 'fancyStringForContext';
2931
next();

tests/fixtures/express-router-with-custom-multer/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ export const router = express.Router();
1010
app.use('/v1', router);
1111
router.use(bodyParser.urlencoded({ extended: true }));
1212
router.use(bodyParser.json());
13-
router.use(methodOverride());
13+
router.use((req, res, next) => {
14+
methodOverride()(req, res, next);
15+
});
1416
router.use((req: any, res: any, next: any) => {
1517
req.stringValue = 'fancyStringForContext';
1618
next();

tests/fixtures/express-router/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ export const router = express.Router();
1010
app.use('/v1', router);
1111
router.use(bodyParser.urlencoded({ extended: true }));
1212
router.use(bodyParser.json());
13-
router.use(methodOverride());
13+
router.use((req, res, next) => {
14+
methodOverride()(req, res, next);
15+
});
1416
router.use((req: any, res: any, next: any) => {
1517
req.stringValue = 'fancyStringForContext';
1618
next();

tests/fixtures/express/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,9 @@ import { RegisterRoutes } from './routes';
3131
export const app: express.Express = express();
3232
app.use(bodyParser.urlencoded({ extended: true }));
3333
app.use(bodyParser.json());
34-
app.use(methodOverride());
34+
app.use((req, res, next) => {
35+
methodOverride()(req, res, next);
36+
});
3537
app.use((req: any, res: any, next: any) => {
3638
req.stringValue = 'fancyStringForContext';
3739
next();

tests/fixtures/inversify-async/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@ import { RegisterRoutes } from './routes';
99
export const app: express.Express = express();
1010
app.use(bodyParser.urlencoded({ extended: true }));
1111
app.use(bodyParser.json());
12-
app.use(methodOverride());
12+
app.use((req, res, next) => {
13+
methodOverride()(req, res, next);
14+
});
1315
RegisterRoutes(app);
1416

1517
// It's important that this come after the main routes are registered

tests/fixtures/inversify-cpg/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ import { RegisterRoutes } from './routes';
88
export const app: express.Express = express();
99
app.use(bodyParser.urlencoded({ extended: true }));
1010
app.use(bodyParser.json());
11-
app.use(methodOverride());
11+
app.use((req, res, next) => {
12+
methodOverride()(req, res, next);
13+
});
1214
RegisterRoutes(app);
1315

1416
// It's important that this come after the main routes are registered

tests/fixtures/inversify-dynamic-container/server.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ import { RegisterRoutes } from './routes';
88
export const app: express.Express = express();
99
app.use(bodyParser.urlencoded({ extended: true }));
1010
app.use(bodyParser.json());
11-
app.use(methodOverride());
11+
app.use((req, res, next) => {
12+
methodOverride()(req, res, next);
13+
});
1214
RegisterRoutes(app);
1315

1416
// It's important that this come after the main routes are registered

0 commit comments

Comments
 (0)