fix: use ci to spawn tests separately

Author: robertsLando

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ on:
   pull_request:
 
 jobs:
-  test:
+  build:
     strategy:
       fail-fast: false # prevent test to stop if one fails
       matrix:
@@ -29,7 +29,38 @@ jobs:
       - if: matrix['node-version'] == '18.x' && matrix['os'] == 'ubuntu-latest'
         run: yarn lint
       - run: yarn build
-      - run: yarn test
-        env:
-          CI: true
-        timeout-minutes: 30
+  test_host:
+    strategy:
+      fail-fast: false # prevent test to stop if one fails
+      matrix:
+        node-version: [18.x, 20.x]
+        os: [ubuntu-latest, windows-latest, macos-latest]
+    uses: ./.github/workflows/test.yml
+    with:
+      node_version: ${{ matrix.node-version }}
+      os: ${{ matrix.os }}
+      npm_command: test:host
+  
+  test_18:
+    strategy:
+      fail-fast: false # prevent test to stop if one fails
+      matrix:
+        node-version: [18.x, 20.x]
+        os: [ubuntu-latest, windows-latest, macos-latest]
+    uses: ./.github/workflows/test.yml
+    with:
+      node_version: ${{ matrix.node-version }}
+      os: ${{ matrix.os }}
+      npm_command: test:18
+
+  test_20:
+    strategy:
+      fail-fast: false # prevent test to stop if one fails
+      matrix:
+        node-version: [18.x, 20.x]
+        os: [ubuntu-latest, windows-latest, macos-latest]
+    uses: ./.github/workflows/test.yml
+    with:
+      node_version: ${{ matrix.node-version }}
+      os: ${{ matrix.os }}
+      npm_command: test:20

.github/workflows/test.yml

@@ -0,0 +1,42 @@
+name: CI
+
+on:
+  workflow_call:
+    inputs:
+      node_version:
+        description: 'Node.js version'
+        type: string
+        required: true
+        default: '18.x'
+      os:
+        description: 'Operating system'
+        type: string
+        required: true
+        default: 'ubuntu-latest'
+      npm_command:
+        description: 'NPM command to run'
+        type: string
+        required: true
+        default: 'install'
+
+
+jobs:
+  test:
+    runs-on: ${{ inputs.os }}
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: ${{ inputs.node_version }}
+      - uses: actions/cache@v3
+        with:
+          path: ~/.pkg-cache/
+          key: ${{ inputs.os }}-${{ inputs.node_version }}
+
+      - run: yarn install
+
+      - run: yarn build
+      - run: yarn ${{ inputs.npm_command }}
+        env:
+          CI: true
+        timeout-minutes: 30

test/test.js

@@ -7,7 +7,6 @@ const pc = require('picocolors');
 const { globSync } = require('tinyglobby');
 const utils = require('./utils.js');
 const { spawn } = require('child_process');
-const { cpus } = require('os');
 const host = 'node' + utils.getNodeMajorVersion();
 let target = process.argv[2] || 'host';
 if (target === 'host') target = host;
@@ -19,11 +18,7 @@ if (target === 'host') target = host;
 
 const flavor = process.env.FLAVOR || process.argv[3] || 'all';
 
-// const isCI = process.env.CI === 'true';
-
-// based on tests using too high concurrency make tests slower
-const maxConcurrency =
-  flavor === 'only-npm' ? 1 : Math.max(cpus().length - 1, 2);
+const isCI = process.env.CI === 'true';
 
 console.log('');
 console.log('*************************************');
@@ -156,13 +151,20 @@ function runTest(file) {
   });
 }
 
+const clearLastLine = () => {
+  if (isCI) return;
+  process.stdout.moveCursor(0, -1); // up one line
+  process.stdout.clearLine(1); // from cursor to end
+};
+
 async function run() {
   let done = 0;
   let ok = 0;
   let failed = [];
   const start = Date.now();
 
   function addLog(log, isError = false) {
+    clearLastLine();
     if (isError) {
       console.error(log);
     } else {
@@ -174,6 +176,9 @@ async function run() {
     file = path.resolve(file);
     const startTest = Date.now();
     try {
+      if (!isCI) {
+        console.log(pc.gray(`⏳ ${file} - ${done}/${files.length}`));
+      }
       await runTest(file);
       ok++;
       addLog(
@@ -198,8 +203,8 @@ async function run() {
     done++;
   });
 
-  for (let i = 0; i < promises.length; i += maxConcurrency) {
-    await Promise.all(promises.slice(i, i + maxConcurrency).map((p) => p()));
+  for (let i = 0; i < promises.length; i++) {
+    await promises[i]();
   }
 
   const end = Date.now();
@@ -231,8 +236,6 @@ function cleanup() {
   for (const process of activeProcesses) {
     process.kill();
   }
-
-  process.exit(1);
 }
 
 process.on('SIGINT', cleanup);