Skip to content

Commit

Permalink
disable parallel mode by default
Browse files Browse the repository at this point in the history
Having run benchmarks against our unit tests and integration tests, I've found that the unit tests do not benefit from parallel mode--and in fact, they run more slowly.  Integration tests still see a 50% performance boost, so we will use parallel tests for those only.
  • Loading branch information
boneskull committed Jun 9, 2020
1 parent 53ada74 commit dc26e90
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion .mocharc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ global:
- 'okGlobalC'
- 'callback*'
timeout: 1000
parallel: true
watch-ignore:
- '.*'
- 'docs/_dist/**'
Expand Down
2 changes: 1 addition & 1 deletion package-scripts.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ module.exports = {
integration: {
script: test(
'integration',
'--timeout 10000 --slow 3750 "test/integration/**/*.spec.js"'
'--parallel --timeout 10000 --slow 3750 "test/integration/**/*.spec.js"'
),
description: 'Run Node.js integration tests',
hiddenFromHelp: true
Expand Down

0 comments on commit dc26e90

Please sign in to comment.