Skip to content

Commit

Permalink
Ako/ fix coverage reporter (binary-com#9129)
Browse files Browse the repository at this point in the history
* build: update max-worker param

* build: update max worker param

* ci: remove max worker param

* ci: add a separate script to run jest on github

* build: add JEST_MAX_WORKERS as param and added a default value to it

* ci: update codecov test runner command

* Update package.json

---------

Co-authored-by: balakrishna-deriv <56330681+balakrishna-deriv@users.noreply.github.com>
  • Loading branch information
ali-hosseini-deriv and balakrishna-deriv committed Jun 27, 2023
1 parent d5c38a0 commit 6741ee5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ on:
pull_request:
branches:
- '**'
env:
JEST_MAX_WORKERS: 4
push:
branches:
- master
Expand All @@ -21,7 +23,7 @@ jobs:
npm install
npm run bootstrap
npm run build:all
npm run test:jest 4
npm run test:jest
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v3.1.1
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@
"start": "f () { lerna exec --scope=@deriv/${1:-'*'} -- npm run start ;}; f",
"test": "f () { bash ./scripts/circleci-config.test.sh && npm run test:stylelint && npm run test:eslint-all && npm run test:jest ;}; f",
"test:stylelint": "stylelint \"./packages/*/src/**/*.s(a|c)ss\"",
"test:jest": "jest --all --maxWorkers=${1:-8}",
"test:jest": "jest --all --maxWorkers=${JEST_MAX_WORKERS:-'45%'}",
"test:e2e": "cd end-to-end-test && npx playwright test",
"test:e2e-dev": "cd end-to-end-test && npx playwright test --trace on && npx playwright show-report",
"test:performance": "cd e2e_tests && jest -c ./jest.config.js --maxWorkers=2 --detectOpenHandles performance",
Expand Down

0 comments on commit 6741ee5

Please sign in to comment.