ci(coverage): fix coverage reporting

Modify npm script "report-coverage" to first run the tests and only then report the coverage to coveralls
omnibrain · Nov 27, 2020 · 0d74f58 · 0d74f58
1 parent 7aa6632
commit 0d74f58
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -22,7 +22,7 @@ jobs:
       deploy:
         provider: script
         skip_cleanup: true
-        script: yarn deploy-pages && yarn semantic-release
+        script: yarn report-coverage && yarn deploy-pages && yarn semantic-release
 
 branches:
   except:

diff --git a/package.json b/package.json
@@ -35,10 +35,11 @@
     "start": "rollup -c rollup.config.ts -w",
     "test": "jest --coverage",
     "test:watch": "jest --coverage --watch",
-    "test:prod": "npm run lint && npm run test -- --no-cache",
+    "test:prod": "run-s lint test",
     "deploy-pages": "run-s build publish-pages",
     "publish-pages": "ts-node tools/gh-pages-publish",
-    "report-coverage": "cat ./coverage/lcov.info | coveralls",
+    "report-coverage": "run-s test publish-coverage",
+    "publish-coverage": "cat ./coverage/lcov.info | coveralls",
     "commit": "git-cz",
     "semantic-release": "semantic-release",
     "precommit": "lint-staged"