chore: replace tap by node:test (#1042)

This PR replaces Tap by node:test

Tasks:
-  [x] auth.js
-  [x] basic.js
-  [x] bridge.js
-  [x] client-pub-sub.js
-  [x] close_socket_by_other_party.js
-  [x] connect.js  (with TODO's)
-  [x] events.js
-  [x] keep-alive.js
-  [x] meta.js (with 1 TODO)
-  [x] not-blocking.js
-  [x] qos1.js
-  [x] qos2.js
-  [x] regr-21.js
-  [x] require.cjs
-  [x] retain.js (with 1 TODO)
-  [x] topics.js
-  [x] will.js
-  [x] update package.json scripts
-  [x] update package.json devDependencies
-  [x] update 'coverage' directory in ci.yml

The following devDependencies have been replaced by NodeJS native
functionality
 - @sinonjs/fake-timers
 - concat-stream
 - duplexify
 - mqtt-connection
 - tap

It adds `c8` as devDependency to facilitate test coverage.

Author: seriousme

.github/workflows/ci.yml

@@ -65,7 +65,7 @@ jobs:
         if: strategy.job-index == 0
         uses: codecov/codecov-action@v5
         with:
-          directory: .tap/report
+          directory: coverage
           fail_ci_if_error: true
           flags: unittests
           name: codecov-aedes

checkVersion.js

@@ -0,0 +1,20 @@
+// node 20 on windows has issues with node:test wildcard
+// this script is only used by package.json
+// example of usage in package.json:
+// "unit": "node checkVersion.js && npm run unit:v20win32 || npm run unit:other",
+// "unit:v20win32": "node --test --test-timeout=180000",
+// "unit:other": "node --test --test-timeout=180000 test/*.js test/*.cjs",
+
+/* c8 ignore start */
+import { platform, version, exit } from 'node:process'
+
+const major = version.split('.')[0]
+let exitCode = 1
+// node 20 on windows returns 0
+if ((major === 'v20') && platform === 'win32') {
+  exitCode = 0
+}
+// all the others 1
+console.log(`Running ${major} on ${platform} returning exitCode ${exitCode}`)
+exit(exitCode)
+/* c8 ignore stop */

package.json

@@ -12,11 +12,14 @@
 		"lint:markdown": "markdownlint docs/*.md README.md",
 		"lint:markdown:fix": "markdownlint --fix docs/*.md README.md",
 		"test": "npm run lint && npm run unit && npm run test:typescript",
-		"test:ci": "npm run lint && npm run unit:coverage && npm run test:typescript",
+		"test:ci": "npm run lint && npm run coverage && npm run test:typescript",
 		"test:report": "npm run lint && npm run unit:report && npm run test:typescript",
 		"test:typescript": "tsd",
-		"unit": "tap test/*.js",
-		"unit:coverage": "tap --coverage-report=lcov test/*.js",
+		"unit": "node checkVersion.js && npm run unit:v20win32 || npm run unit:other",
+		"unit:v20win32": "node --test --test-timeout=180000",
+		"unit:other": "node --test --test-timeout=180000 test/*.js test/*.cjs",
+		"coverage": "c8 --reporter=lcov npm run unit",
+		"coverage:report": "c8 report",
 		"license-checker": "license-checker --production --onlyAllow=\"MIT;ISC;BSD-3-Clause;BSD-2-Clause;0BSD\"",
 		"release": "read -p 'GITHUB_TOKEN: ' GITHUB_TOKEN && export GITHUB_TOKEN=$GITHUB_TOKEN && release-it --disable-metrics"
 	},
@@ -103,19 +106,15 @@
 	},
 	"devDependencies": {
 		"@fastify/pre-commit": "^2.2.0",
-		"@sinonjs/fake-timers": "^14.0.0",
-		"@types/node": "^24.0.14",
-		"concat-stream": "^2.0.0",
-		"duplexify": "^4.1.3",
-		"eslint": "^9.31.0",
+		"@types/node": "^24.2.0",
+		"c8": "^10.1.3",
+		"eslint": "^9.32.0",
 		"license-checker": "^25.0.1",
 		"markdownlint-cli": "^0.45.0",
-		"mqtt": "^5.13.2",
-		"mqtt-connection": "^4.1.0",
+		"mqtt": "^5.14.0",
 		"neostandard": "^0.12.2",
-		"release-it": "^19.0.3",
-		"tap": "^21.1.0",
-		"tsd": "^0.32.0",
+		"release-it": "^19.0.4",
+		"tsd": "^0.33.0",
 		"ws": "^8.18.3"
 	},
 	"dependencies": {
@@ -133,6 +132,7 @@
 		"uuid": "^11.1.0"
 	},
 	"peerDependencies": {
+		"aedes-persistence-level": "^9.1.2",
 		"aedes-persistence-mongodb": "^9.3.1",
 		"aedes-persistence-redis": "^11.2.1"
 	},