Fix ".use" middlewares registration (#24)

* adding cross-env module for cross OS testing
* fixing .use middlewares registration

Author: jkyberneees

lib/router/sequential.js

@@ -34,7 +34,7 @@ module.exports = (config = {}) => {
 
   router.use = (prefix, ...middlewares) => {
     if (typeof prefix === 'function') {
-      middlewares = [prefix]
+      middlewares = [prefix, ...middlewares]
       prefix = '/'
     }
     _use.call(router, prefix, middlewares)

package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "lint": "npx standard",
     "format": "npx standard --fix",
-    "test": "PORT=3000 NODE_ENV=testing npx nyc --check-coverage --lines 85 node ./node_modules/mocha/bin/mocha tests/*.test.js"
+    "test": "cross-env PORT=3000 NODE_ENV=testing npx nyc --check-coverage --lines 85 node ./node_modules/mocha/bin/mocha tests/*.test.js"
   },
   "repository": {
     "type": "git",
@@ -30,6 +30,7 @@
   "devDependencies": {
     "body-parser": "^1.19.1",
     "chai": "^4.3.6",
+    "cross-env": "^7.0.3",
     "find-my-way": "^5.2.0",
     "mocha": "^9.2.0",
     "nyc": "^15.1.0",
@@ -46,4 +47,4 @@
     "regexparam": "^2.0.0",
     "trouter": "^3.2.0"
   }
-}
+}

tests/middleware-registration.test.js

@@ -0,0 +1,65 @@
+/* global describe, it */
+const expect = require('chai').expect
+const request = require('supertest')
+
+describe('0http Web Framework - Middlewares Registration', () => {
+  const baseUrl = 'http://localhost:' + process.env.PORT
+
+  const { router, server } = require('../index')({
+    router: require('../lib/router/sequential')()
+  })
+
+  const m0 = (req, res, next) => {
+    res.body = []
+
+    return next()
+  }
+
+  const m1 = (req, res, next) => {
+    res.body.push('m1')
+
+    return next()
+  }
+
+  const m2 = (req, res, next) => {
+    res.body.push('m2')
+
+    return next()
+  }
+
+  const m3 = (req, res, next) => {
+    res.body.push('m3')
+
+    return next()
+  }
+
+  it('should successfully register middlewares', (done) => {
+    router.use(m0, m1)
+    router.use('/v1', m2, m3)
+
+    router.get('/v1/hello', (req, res, next) => {
+      res.end(JSON.stringify(res.body))
+    })
+
+    server.listen(~~process.env.PORT, err => {
+      if (!err) done()
+    })
+  })
+
+  it('should hit middlewares', async () => {
+    await request(baseUrl)
+      .get('/v1/hello')
+      .expect(200)
+      .then((response) => {
+        const payload = JSON.parse(response.text)
+
+        expect(payload[0]).to.equal('m1')
+        expect(payload[1]).to.equal('m2')
+        expect(payload[2]).to.equal('m3')
+      })
+  })
+
+  it('should successfully terminate the service', async () => {
+    server.close()
+  })
+})