ci: update scripts and ci.yml to handle coverage better

Author: Michael Perrotte

.github/workflows/ci.yml

@@ -16,7 +16,7 @@
 #   coverage will need to be adjusted.
 #
 ################################################################################
-name: Node CI
+name: node-ci
 
 on: [push, pull_request]
 
@@ -60,7 +60,8 @@ jobs:
       #   Simply run the tests for the project.
       ################################################################################
       - name: Run tests
-        run: npm test
+        if: github.event_name != 'push' || matrix.node-version != '12.x' || matrix.os != 'ubuntu-latest'
+        run: npm test -- --no-coverage
 
       ################################################################################
       # Run coverage check
@@ -76,7 +77,7 @@ jobs:
       ################################################################################
       - name: Run coverage report
         if: github.event_name == 'push' && matrix.node-version == '12.x' && matrix.os == 'ubuntu-latest'
-        run: npm run coverage
+        run: npm test
         env:
           # The environment variable name is leveraged by `tap`
           COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}

package.json

@@ -9,8 +9,7 @@
     "postversion": "npm publish",
     "postpublish": "git push origin --follow-tags",
     "lint": "standard",
-    "test": "tap --no-coverage",
-    "coverage": "tap"
+    "test": "tap"
   },
   "devDependencies": {
     "nock": "^12.0.1",