Merge pull request #181 from bcomnes/node-test-runner

Convert to using the node test runner

Author: bcomnes

.github/workflows/test.yml

@@ -41,7 +41,7 @@ jobs:
         with:
           node-version: ${{ matrix.node }}
       - run: npm i
-      - run: npm run test-mocha
+      - run: npm run test
       - uses: codecov/codecov-action@v5
         if: ${{ github.event_name == 'pull_request' }}
         with:

.knip.jsonc

@@ -1,5 +1,5 @@
 {
   "$schema": "https://unpkg.com/knip@5/schema.json",
   "entry": ["lib/index.js", "bin/*/*.js", "test-workspace/tasks/*.js"],
-  "ignoreDependencies": ["yarn"]
+  "ignoreDependencies": ["yarn", "spec"]
 }

eslint.config.js

@@ -2,7 +2,7 @@
 
 module.exports = [
   ...require('neostandard')({
-    env: ['mocha'],
+    env: ['node', 'mocha'],
     ignores: require('neostandard').resolveIgnoresFromGitignore(),
     ts: true,
   }),

package.json

@@ -22,10 +22,11 @@
     "check:knip": "knip",
     "precheck": "node scripts/make-slink.js",
     "check": "node ./bin/run-p/index.js check:*",
-    "pretest-mocha": "node scripts/make-slink.js",
-    "test-mocha": "c8 mocha --timeout 120000",
-    "test": "node ./bin/run-s/index.js check test-mocha",
-    "watch": "mocha --timeout 120000 --watch --growl",
+    "pretest": "node scripts/make-slink.js",
+    "test": "node ./bin/run-s/index.js test:*",
+    "test:check": "npm run check",
+    "test:node-test": "c8 node --test --test-reporter spec --test-concurrency=1",
+    "watch": "node --watch --test",
     "version": "auto-changelog -p --template keepachangelog auto-changelog --breaking-pattern 'BREAKING CHANGE:' && git add CHANGELOG.md",
     "prepublishOnly": "git push --follow-tags && gh-release -y"
   },
@@ -47,7 +48,6 @@
     "installed-check": "^9.3.0",
     "jsdoc": "^4.0.0",
     "knip": "^5.23.3",
-    "mocha": "^11.0.1",
     "neostandard": "^0.12.0",
     "yarn": "^1.12.3"
   },

test/aggregate-output.js

@@ -9,7 +9,8 @@
 // Requirements
 // ------------------------------------------------------------------------------
 
-const assert = require('assert').strict
+const { test, describe, before, after, beforeEach } = require('node:test')
+const assert = require('node:assert/strict')
 const nodeApi = require('../lib')
 const BufferStream = require('./lib/buffer-stream')
 const util = require('./lib/util')
@@ -48,15 +49,15 @@ describe('[aggregated-output] npm-run-all', () => {
       stdout = new BufferStream()
     })
 
-    it('Node API with parallel', async () => {
+    test('Node API with parallel', async () => {
       await nodeApi(
         ['test-task:delayed first 5000', 'test-task:delayed second 1000', 'test-task:delayed third 3000'],
         { stdout, parallel: true, silent: true, aggregateOutput: true }
       )
       assert.equal(stdout.value, EXPECTED_PARALLELIZED_TEXT)
     })
 
-    it('Node API without parallel should fail', async () => {
+    test('Node API without parallel should fail', async () => {
       try {
         await nodeApi(
           ['test-task:delayed first 5000', 'test-task:delayed second 1000', 'test-task:delayed third 3000'],
@@ -68,15 +69,15 @@ describe('[aggregated-output] npm-run-all', () => {
       assert(false, 'should fail')
     })
 
-    it('npm-run-all command with parallel', async () => {
+    test('npm-run-all command with parallel', async () => {
       await runAll(
         ['--parallel', 'test-task:delayed first 5000', 'test-task:delayed second 1000', 'test-task:delayed third 3000', '--silent', '--aggregate-output'],
         stdout
       )
       assert.equal(stdout.value, EXPECTED_PARALLELIZED_TEXT)
     })
 
-    it('npm-run-all command without parallel should fail', async () => {
+    test('npm-run-all command without parallel should fail', async () => {
       try {
         await runAll(
           ['test-task:delayed first 5000', 'test-task:delayed second 1000', 'test-task:delayed third 3000', '--silent', '--aggregate-output'],
@@ -88,7 +89,7 @@ describe('[aggregated-output] npm-run-all', () => {
       assert(false, 'should fail')
     })
 
-    it('run-s command should fail', async () => {
+    test('run-s command should fail', async () => {
       try {
         await runSeq(
           ['test-task:delayed first 5000', 'test-task:delayed second 1000', 'test-task:delayed third 3000', '--silent', '--aggregate-output'],
@@ -100,7 +101,7 @@ describe('[aggregated-output] npm-run-all', () => {
       assert(false, 'should fail')
     })
 
-    it('run-p command', async () => {
+    test('run-p command', async () => {
       await runPar(
         [
           'test-task:delayed first 5000',