[ci] run unit tests in test job, collect coverage in parallel

2BAD · Nov 28, 2023 · d9539bb · d9539bb
1 parent 534007b
commit d9539bb
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 10 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -7,13 +7,16 @@ on:
       - '*.md'
 
 jobs:
-  test:
+  build:
     name: node@${{ matrix.node-version }}
     runs-on: ubuntu-latest
 
     strategy:
       matrix:
-        node-version: ['18', '20', '21']
+        node-version:
+          - 18
+          - 20
+          - 21
 
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
@@ -22,11 +25,7 @@ jobs:
       - uses: actions/setup-node@8f152de45cc393bb48ce5d89d36b731f54556e65 # v4
         with:
           node-version: ${{ matrix.node-version }}
+
       - run: npm ci
       - run: npm run check
-      - run: npm run test:unit
-      - uses: codecov/codecov-action@eaaf4bedf32dbdc6b720b63067d99c4d77d6047d # v3
-        with:
-          token: ${{ secrets.CODECOV_TOKEN }}
-          flags: unit
       - run: npm run build
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -1,10 +1,12 @@
 name: Publish
+
 on:
   release:
     types: [published]
 
 jobs:
   publish:
+    name: publish
     runs-on: ubuntu-latest
 
     permissions:
@@ -13,10 +15,13 @@ jobs:
 
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
+        with:
+          persist-credentials: false
       - uses: actions/setup-node@8f152de45cc393bb48ce5d89d36b731f54556e65 # v4
         with:
           node-version-file: 'package.json'
           registry-url: 'https://registry.npmjs.org'
+
       - run: npm ci
       - run: npm publish --provenance --access public
         env:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,6 +10,9 @@ on:
 
 jobs:
   test:
+    name: ${{ matrix.os-release }}-node@${{ matrix.node-version }}
+    runs-on: ${{ matrix.os-release }}
+
     strategy:
       matrix:
         node-version:
@@ -20,9 +23,6 @@ jobs:
           - ubuntu-latest
           - windows-latest
 
-    runs-on: ${{ matrix.os-release }}
-    name: ${{ matrix.os-release }}-node@${{ matrix.node-version }}
-
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
         with:
@@ -31,6 +31,20 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
 
+      - run: npm ci
+      - run: npm run test
+
+  coverage:
+    name: collect coverage
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
+        with:
+          persist-credentials: false
+      - uses: actions/setup-node@8f152de45cc393bb48ce5d89d36b731f54556e65 # v4
+        with:
+          node-version-file: 'package.json'
+
       - run: npm ci
 
       - run: npm run test:unit