Skip to content

Commit 12e35dc

Browse files
authored
fix router.use to accept middleware arrays and add test (#47)
1 parent e07288d commit 12e35dc

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

lib/router/sequential.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ module.exports = (config = {}) => {
8080
middlewares = [prefix, ...middlewares]
8181
prefix = '/'
8282
}
83-
_use.call(router, prefix, middlewares)
83+
_use.call(router, prefix, ...middlewares)
8484

8585
// Optimized nested router detection - check first middleware only
8686
const firstMiddleware = middlewares[0]

tests/middleware-registration.test.js

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,25 @@ describe('0http - Middlewares Registration', () => {
5959
})
6060
})
6161

62+
it('should support array-based middleware registration', async () => {
63+
router.use('/arr', [m2, m3])
64+
65+
router.get('/arr/hello', (req, res, next) => {
66+
res.end(JSON.stringify(res.body))
67+
})
68+
69+
await request(baseUrl)
70+
.get('/arr/hello')
71+
.expect(200)
72+
.then((response) => {
73+
const payload = JSON.parse(response.text)
74+
75+
expect(payload[0]).to.equal('m1')
76+
expect(payload[1]).to.equal('m2')
77+
expect(payload[2]).to.equal('m3')
78+
})
79+
})
80+
6281
it('should successfully terminate the service', async () => {
6382
server.close()
6483
})

0 commit comments

Comments
 (0)