feat: support Node.js 20

.github/workflows/ci.yml

@@ -31,6 +31,7 @@ jobs:
           - 14.x
           - 16.x
           - 18.x
+          - 20.x
     runs-on: ${{matrix.os}}
     steps:
       - uses: actions/checkout@v2

test/server.test.js

@@ -1094,7 +1094,9 @@
     });
 });
 
-test('error handler defers "after" event', function(t) {
+test('error handler defers "after" event', async function(t) {
+    let afterResolve;
+    let clientResolve;
     t.expect(9);
     SERVER.once('NotFound', function(req, res, err, cb) {
         t.ok(req);
@@ -1107,7 +1109,7 @@
         SERVER.once('after', function(req2, res2) {
             t.ok(req2);
             t.ok(res2);
-            t.end();
+            afterResolve();
         });
         return cb();
     });
@@ -1118,8 +1120,18 @@
     CLIENT.get('/' + uuid.v4(), function(err, _, res) {
         t.ok(err);
         t.equal(res.statusCode, 404);
-        t.end();
+        clientResolve();
     });
+
+    await Promise.all([
+        new Promise(resolve => {
+            afterResolve = resolve;
+        }),
+        new Promise(resolve => {
+            clientResolve = resolve;
+        })
+    ]);
+    t.end();
 });
 
 // eslint-disable-next-line
@@ -1916,7 +1928,9 @@
     });
 });
 
-test('should cleanup inflight requests count for 404s', function(t) {
+test('should cleanup inflight requests count for 404s', async function(t) {
+    let afterResolve;
+    let clientResolve;
     SERVER.get('/foo1', function(req, res, next) {
         t.equal(SERVER.inflightRequests(), 1);
         res.send();
@@ -1926,7 +1940,7 @@
     SERVER.on('after', function(req) {
         if (req.path() === '/doesnotexist') {
             t.equal(SERVER.inflightRequests(), 0);
-            t.end();
+            afterResolve();
         }
     });
 
@@ -1939,8 +1953,19 @@
             t.ok(err2);
             t.equal(res2.statusCode, 404);
             t.equal(SERVER.inflightRequests(), 0);
+            clientResolve();
         });
     });
+
+    await Promise.all([
+        new Promise(resolve => {
+            afterResolve = resolve;
+        }),
+        new Promise(resolve => {
+            clientResolve = resolve;
+        })
+    ]);
+    t.end();
 });
 
 test('should cleanup inflight requests count for timeouts', function(t) {
@@ -2054,7 +2079,7 @@
     });
 
     // // check /foo
     // TODO: should it contain use handlers?
     t.equal(debugInfo.routes[0].handlers[0], 'use');
     t.equal(debugInfo.routes[0].handlers[1], 'use2');
     t.equal(debugInfo.routes[0].handlers[2], 'anonymous');