ci: fix errors in ci github action for node 8 and 9 (#523)

* ci: fix errors in ci github action for node 8 and 9

* ci: remove minor version on node 22

* ci: remove minor versions

* 🧪 skip query test on node 21

Co-authored-by: Chris de Almeida <[email protected]>

---------

Co-authored-by: Jon Church <[email protected]>
Co-authored-by: Chris de Almeida <[email protected]>

Author: 3 people

.github/workflows/ci.yml

@@ -32,6 +32,9 @@ jobs:
         - Node.js 17.x
         - Node.js 18.x
         - Node.js 19.x
+        - Node.js 20.x
+        - Node.js 21.x
+        - Node.js 22.x
 
         include:
         - name: Node.js 0.8
@@ -48,77 +51,86 @@ jobs:
           npm-i: [email protected] [email protected] [email protected]
 
         - name: io.js 1.x
-          node-version: "1.8"
+          node-version: "1"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: io.js 2.x
-          node-version: "2.5"
+          node-version: "2"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: io.js 3.x
-          node-version: "3.3"
+          node-version: "3"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: Node.js 4.x
-          node-version: "4.9"
+          node-version: "4"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: Node.js 5.x
-          node-version: "5.12"
+          node-version: "5"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: Node.js 6.x
-          node-version: "6.17"
+          node-version: "6"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: Node.js 7.x
-          node-version: "7.10"
+          node-version: "7"
           npm-i: [email protected] [email protected] [email protected]
 
         - name: Node.js 8.x
-          node-version: "8.17"
-          npm-i: [email protected]
+          node-version: "8"
+          npm-i: [email protected] [email protected]
 
         - name: Node.js 9.x
-          node-version: "9.11"
-          npm-i: [email protected]
+          node-version: "9"
+          npm-i: [email protected] [email protected]
 
         - name: Node.js 10.x
-          node-version: "10.24"
+          node-version: "10"
           npm-i: [email protected]
 
         - name: Node.js 11.x
-          node-version: "11.15"
+          node-version: "11"
           npm-i: [email protected]
 
         - name: Node.js 12.x
-          node-version: "12.22"
+          node-version: "12"
           npm-i: [email protected]
 
         - name: Node.js 13.x
-          node-version: "13.14"
+          node-version: "13"
           npm-i: [email protected]
 
         - name: Node.js 14.x
-          node-version: "14.21"
+          node-version: "14"
 
         - name: Node.js 15.x
-          node-version: "15.14"
+          node-version: "15"
 
         - name: Node.js 16.x
-          node-version: "16.19"
+          node-version: "16"
 
         - name: Node.js 17.x
-          node-version: "17.9"
+          node-version: "17"
 
         - name: Node.js 18.x
-          node-version: "18.14"
+          node-version: "18"
 
         - name: Node.js 19.x
-          node-version: "19.7"
+          node-version: "19"
+
+        - name: Node.js 20.x
+          node-version: "20"
+
+        - name: Node.js 21.x
+          node-version: "21"
+
+        - name: Node.js 22.x
+          node-version: "22"
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
 
     - name: Install Node.js ${{ matrix.node-version }}
       shell: bash -eo pipefail -l {0}
@@ -209,7 +221,7 @@ jobs:
     needs: test
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
 
     - name: Install lcov
       shell: bash

test/body-parser.js

@@ -73,13 +73,25 @@ describe('bodyParser()', function () {
       })
     })
 
+    function getMajorVersion (versionString) {
+      return versionString.split('.')[0]
+    }
+
+    function shouldSkipQuery (versionString) {
+      // Skipping HTTP QUERY tests on Node 21, it is reported in http.METHODS on 21.7.2 but not supported
+      // update this implementation to run on supported versions of 21 once they exist
+      // upstream tracking https://github.com/nodejs/node/issues/51562
+      // express tracking issue: https://github.com/expressjs/express/issues/5615
+      return getMajorVersion(versionString) === '21'
+    }
+
     methods.slice().sort().forEach(function (method) {
-      if (method === 'connect') {
-        // except CONNECT
-        return
-      }
+      if (method === 'connect') return
 
       it('should support ' + method.toUpperCase() + ' requests', function (done) {
+        if (method === 'query' && shouldSkipQuery(process.versions.node)) {
+          this.skip()
+        }
         request(this.server)[method]('/')
           .set('Content-Type', 'application/json')
           .set('Content-Length', '15')