Merge pull request #2742 from MontaGhanmy/remove_coverage_folder_upload

🐳 [Jest] Removed coverage folder upload
linagora · Feb 14, 2023 · 49c0bf9 · 49c0bf9
2 parents f79f696 + 6074fa5
commit 49c0bf9
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -23,11 +23,6 @@ jobs:
         run: cd twake && docker-compose -f docker-compose.tests.yml run -e NODE_OPTIONS=--unhandled-rejections=warn node npm run test:merge:json
       - name: generate coverage summary text
         run: cd twake && docker-compose -f docker-compose.tests.yml run -e NODE_OPTIONS=--unhandled-rejections=warn node npm run test:merge:text
-      - name: upload test coverage report
-        uses: actions/upload-artifact@v2
-        with:
-          name: coverage-report-${{ github.event.pull_request.number }}
-          path: twake/coverage/
       - name: Jest Coverage Comment
         uses: MishaKav/jest-coverage-comment@main
         with:

diff --git a/twake/backend/node/package.json b/twake/backend/node/package.json
@@ -27,7 +27,7 @@
     "test:unit": "jest test/unit --forceExit --coverage --detectOpenHandles --maxWorkers=1 --testTimeout=30000 --verbose false > coverage/coverage-report.txt",
     "test:unit:watch": "npm run test:unit -- --watchAll --verbose false | pino-pretty",
     "test:merge:json": "npx istanbul report --dir coverage/merged --include 'coverage/**/coverage-final.json' json-summary",
-    "test:merge:text": "npx istanbul report --dir coverage/merged --include 'coverage/**/coverage-final.json' text",
+    "test:merge:text": "npx istanbul report --dir coverage/merged --include 'coverage/**/coverage-final.json' text > /coverage/merged/coverage-report.txt",
     "kill": "kill $(lsof -t -i:3000) | exit 0"
   },
   "jest": {