Refine build workflow

sebastianquek · sebastianquek · commit cfd736f9fdc1 · 2023-03-02T11:58:10.000+08:00
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -45,13 +45,13 @@ jobs:
           path: npm.zip
           retention-days: 1
 
-  smoke-tests-esm:
-    name: "Smoke tests (ESM)"
+  smoke-tests-commonjs:
+    name: "Smoke tests (CommonJS)"
     needs: build
     runs-on: ubuntu-22.04
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [7.x]
     steps:
       - name: Checkout repo
         uses: actions/checkout@v3
@@ -72,17 +72,17 @@ jobs:
 
       - name: Run smoke tests
         run: |
-          cd tests/smoke/esm
+          cd tests/smoke/commonjs
           npm i
           npm start
 
-  smoke-tests-commonjs:
-    name: "Smoke tests (commonjs)"
+  smoke-tests-esm:
+    name: "Smoke tests (ESM)"
     needs: build
     runs-on: ubuntu-22.04
     strategy:
       matrix:
-        node-version: [7.x]
+        node-version: [18.x]
     steps:
       - name: Checkout repo
         uses: actions/checkout@v3
@@ -103,6 +103,6 @@ jobs:
 
       - name: Run smoke tests
         run: |
-          cd tests/smoke/commonjs
+          cd tests/smoke/esm
           npm i
           npm start
diff --git a/tests/smoke/commonjs/commonjs.js b/tests/smoke/commonjs/commonjs.js
@@ -133,4 +133,7 @@ const run = async () => {
   console.log("success", process.versions.node);
 };
 
-run();
+run().catch((e) => {
+  console.error(e);
+  process.exitCode = 1;
+});
