Switched from Travis CI to GitHub Actions

Author: JamesMessinger

.github/workflows/CI-CD.yaml

@@ -15,11 +15,8 @@ on:
   schedule:
     - cron: "0 0 1 * *"
 
-env:
-  CI: true
-
 jobs:
-  test:
+  node_tests:
     name: Node ${{ matrix.node }} on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     timeout-minutes: 10
@@ -46,7 +43,7 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      - name: Run linters
+      - name: Run linter
         run: npm run lint
 
       - name: Build the code
@@ -55,10 +52,49 @@ jobs:
       - name: Run TypeScript tests
         run: npm run test:typescript
 
-      - name: Run JavaScript tests
-        run: npm run coverage
+      - name: Run Node tests
+        run: npm run coverage:node
+
+      - name: Send code coverage results to Coveralls
+        uses: coverallsapp/github-action@v1.0.1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          parallel: true
+
+  browser_tests:
+    name: Browser Tests
+    runs-on: ${{ matrix.os }}
+    timeout-minutes: 10
+    strategy:
+      fail-fast: true
+      matrix:
+        os:
+          - ubuntu-latest     # Chrome, Firefox, Safari (via SauceLabs), Edge (via SauceLabs)
+          - windows-latest    # Internet Explorer
+
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v2
+
+      - name: Install Node
+        uses: actions/setup-node@v1
+        with:
+          node-version: 12
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build the code
+        run: npm run build
+
+      - name: Run tests
+        run: npm run coverage:browser
+        env:
+          SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
+          SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
 
       - name: Combine code coverage data into a single file
+        shell: bash
         run: |
           ls -Rlh coverage/*/lcov.info
           cat coverage/*/lcov.info > ./coverage/lcov.info
@@ -73,10 +109,39 @@ jobs:
     name: Code Coverage
     runs-on: ubuntu-latest
     timeout-minutes: 10
-    needs: test
+    needs:
+      - node_tests
+      - browser_tests
     steps:
       - name: Let Coveralls know that all tests have finished
         uses: coverallsapp/github-action@v1.0.1
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           parallel-finished: true
+
+  deploy:
+    name: Publish to NPM
+    if: github.ref == 'refs/heads/master'
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    needs:
+      - node_tests
+      - browser_tests
+
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v2
+
+      - name: Install Node
+        uses: actions/setup-node@v1
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build the code
+        run: npm run build
+
+      - name: Publish to NPM
+        uses: JS-DevTools/npm-publish@v1
+        with:
+          token: ${{ secrets.NPM_TOKEN }}

karma.conf.js

@@ -3,12 +3,17 @@
 // https://jstools.dev/karma-config/
 
 "use strict";
-const { karmaConfig } = require("karma-config");
+const { karmaConfig } = require("@jsdevtools/karma-config");
+const { host } = require("@jsdevtools/host-environment");
 
 module.exports = karmaConfig({
   sourceDir: "esm",
   fixtures: "test/fixtures/**/*.js",
   browsers: {
-    ie: true,
+    chrome: true,
+    firefox: host.os.linux,
+    safari: host.os.linux,    // SauceLabs
+    edge: host.os.linux,      // SauceLabs
+    ie: host.os.windows,
   },
 });

package.json

@@ -49,7 +49,7 @@
     "test:browser": "karma start --single-run",
     "test:typescript": "tsc --noEmit test/specs/typescript.spec.ts",
     "coverage": "npm run coverage:node && npm run coverage:browser",
-    "coverage:node": "nyc --reporter=text --reporter=lcov --report-dir coverage/node node_modules/mocha/bin/mocha",
+    "coverage:node": "nyc --reporter=text --reporter=lcov node_modules/mocha/bin/mocha",
     "coverage:browser": "npm run test:browser -- --coverage",
     "upgrade": "npm-check -u && npm audit fix",
     "bump": "bump --tag --push --all",