Test and build in separate jobs (#351)

* Move dev/bundle tests to another folder

* Separate test:tooling

* Test and build in separate jobs

* Upload webpack dist

* Move artifacts one by one

Author: aminya

.github/workflows/ci.yml

@@ -7,26 +7,71 @@ env:
   THREADS_WORKER_INIT_TIMEOUT: 15000
 
 jobs:
-  Test:
+  # Building threads library from source
+  Build:
     if: "!contains(github.event.head_commit.message, '[skip ci]')"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
-        node_version: [ '10', '12', '14' ]
+        node_version: ['15']
         os: ['ubuntu-latest', 'macos-latest', 'windows-latest']
     steps:
       - uses: actions/checkout@v2
       - name: Setup node
-        uses: actions/setup-node@v2-beta
+        uses: actions/setup-node@v2
         with:
           node-version: ${{ matrix.node }}
 
       - name: install dependencies and build
         run: npm install
 
+      - name: Dev test
+        run: npm run test:tooling
+
+      - name: Upload artifacts
+        if: contains(matrix.os, 'ubuntu')
+        uses: actions/upload-artifact@v2
+        with:
+          path: |
+            ./bundle
+            ./dist
+            ./dist-esm
+            ./test-tooling/webpack/dist
+
+  # Testing the built files
+  Test:
+    needs: Build
+    if: "!contains(github.event.head_commit.message, '[skip ci]')"
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        node_version: [ '10', '12', '14' ]
+        os: ['ubuntu-latest', 'macos-latest', 'windows-latest']
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup node
+        uses: actions/setup-node@v2
+        with:
+          node-version: ${{ matrix.node }}
+
+      - name: install dependencies
+        run: npm install --ignore-scripts
+
+      - name: Download articats
+        uses: actions/download-artifact@v2
+
+      - name: Place built files
+        shell: bash
+        run: |
+          mkdir dist && mv artifact/dist/* ./dist
+          mkdir dist-esm && mv artifact/dist-esm/* ./dist-esm
+          mkdir bundle && mv artifact/bundle/* ./bundle
+          mkdir -p test-tooling/webpack/dist/ && mv artifact/test-tooling/webpack/dist/* ./test-tooling/webpack/dist/
+
       - name: run the tests
-        run: npm test
+        run: npm run test:library && npm run test:puppeteer:basic && npm run test:puppeteer:webpack
 
   Skip:
     if: contains(github.event.head_commit.message, '[skip ci]')

.gitignore

@@ -6,8 +6,8 @@ docs/vendor
 dist/*
 dist-*/*
 node_modules/
-test/rollup/dist/
-test/webpack/dist/
+test-tooling/rollup/dist/
+test-tooling/webpack/dist/
 test/workers/*.js
 .DS_Store
 Thumbs.db

package.json

@@ -13,10 +13,11 @@
     "build:es": "tsc -p tsconfig-esm.json",
     "postbuild": "npm run bundle",
     "bundle": "rollup -c -f umd --file=bundle/worker.js --name=threads --silent -- dist-esm/worker/bundle-entry.js",
-    "test": "npm run test:ava && npm run test:puppeteer:basic && npm run test:puppeteer:webpack",
-    "test:ava": "cross-env TS_NODE_FILES=true ava",
+    "test": "npm run test:library && npm run test:tooling && npm run test:puppeteer:basic && npm run test:puppeteer:webpack",
+    "test:library": "cross-env TS_NODE_FILES=true ava ./test/**/*.test.ts",
+    "test:tooling": "cross-env TS_NODE_FILES=true ava ./test-tooling/**/*.test.ts",
     "test:puppeteer:basic": "puppet-run --plugin=mocha --bundle=./test/workers/:workers/ --serve=./bundle/worker.js:/worker.js ./test/*.chromium*.ts",
-    "test:puppeteer:webpack": "puppet-run --serve ./test/webpack/dist/app.web/0.worker.js --serve ./test/webpack/dist/app.web/1.worker.js --plugin=mocha ./test/webpack/webpack.chromium.mocha.ts",
+    "test:puppeteer:webpack": "puppet-run --serve ./test-tooling/webpack/dist/app.web/0.worker.js --serve ./test-tooling/webpack/dist/app.web/1.worker.js --plugin=mocha ./test-tooling/webpack/webpack.chromium.mocha.ts",
     "posttest": "tslint --project .",
     "prepare": "npm run build"
   },
@@ -108,7 +109,8 @@
       "ts"
     ],
     "files": [
-      "./test/**/*.test.ts"
+      "./test/**/*.test.ts",
+      "./test-tooling/**/*.test.ts"
     ],
     "require": [
       "ts-node/register"