Merge d3595a0 into c9f5c8c

.github/workflows/node.js.yml

@@ -3,13 +3,14 @@
 
 name: Node.js CI
 
+
 on:
   push:
     branches: [ master ]
   pull_request:
     branches: [ master ]
 
-jobs: 
+jobs:
   build:
 
     runs-on: ubuntu-latest
@@ -33,5 +34,19 @@ jobs:
 
     - run: npm ci
     - run: npm run lint
-    - run: npm run coverage
-    - run: jest --runInBand
+
+    - if: ${{ startsWith(matrix.node-version, '10.') || startsWith(matrix.node-version, '12.') }}
+      run: npm t
+
+    - if: ${{ startsWith(matrix.node-version, '14.') }}
+      env:
+        COVERALLS_REPO_TOKEN: "${{ secrets.COVERALLS_REPO_TOKEN }}"
+        COVERALLS_GIT_BRANCH: "${{ github.ref }}"
+      run: npm run coverage
+
+    # - if: ${{ startsWith(matrix.node-version, '14.') }}
+    #   name: Coveralls GitHub Action
+    #   uses: coverallsapp/github-action@v1.1.2
+    #   with:
+    #     github-token: ${{ secrets.github_token }}
+    #     path-to-lcov: ./coverage/lcov.info

package.json

@@ -10,8 +10,7 @@
     "test-debug": "node --inspect-brk ./node_modules/jest/bin/jest.js --no-cache --runInBand -t \"ObjectAttribute\"",
     "coverage": "jest --coverage --runInBand --coverageReporters=text-lcov | coveralls",
     "validate": "npm run lint && npm test",
-    "remote": "git remote rm eddyerburgh && git remote",
-    "travis": "npm run lint && npm run coverage && jest --runInBand"
+    "remote": "git remote rm eddyerburgh && git remote"
   },
   "ManifestComments": [
     "Pinned jest to 24.9.0. 25.1.0 is broken on Windows. Waiting for issue #9459 to be resolved."

tests/src/helpers.test.js

@@ -1,3 +1,11 @@
+test('Must contain at least one test', () => {
+  expect(1)
+    .toEqual(1);
+});
+
+/*
+THIS WAS INTENTIONALLY COMMENTED OUT TO SEE IF COVERALLS LOWERS TEST COVERAGE ON THE BRANCH PROPERLY
+
 const helpers = require('../../src/helpers.js');
 
 describe('Helpers', () => {
@@ -55,3 +63,4 @@ describe('Helpers', () => {
     });
   });
 });
+*/

tests/src/index.test.js

@@ -1,3 +1,10 @@
+test('Must have at least 1 test', () => {
+  expect(1)
+    .toEqual(1);
+});
+
+/*
+INTENTIONALLY COMMENTED OUT TO LOWER COVERAGE PERCENT
 const helpers = require('../helpers.js');
 const index = require('../../index.js');
 
@@ -46,3 +53,4 @@ describe('Index', () => {
       .toMatchSnapshot();
   });
 });
+*/