[ci-visibility] Support mocha parallel mode #4314
Conversation
Overall package sizeSelf size: 6.5 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
juan-fernandez
force-pushed
the
juan-fernandez/support-parallel-mocha
branch
from
8d720da
to
5ae4dec
Compare
BenchmarksBenchmark execution time: 2024-05-22 10:27:39 Comparing candidate commit ef92015 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 258 metrics, 8 unstable metrics. |
juan-fernandez
changed the title
| const testToStartLine = new WeakMap() | ||
|
|
||
| // mocha-each support | ||
| addHook({ |
There was a problem hiding this comment.
ℹ️ these two hooks are not modified at all
| // skippable and known tests. | ||
| // It is called but skipped in parallel mode. In parallel mode, we'll use | ||
| // the run-helpers hook to grab the data. | ||
| addHook({ |
There was a problem hiding this comment.
ℹ️ this hook was not modified, I just removed a warning log for parallel mode
There was a problem hiding this comment.
oh, thanks for these comments! more than 500 lines that I can ignore :-D
| file: 'lib/cli/run-helpers.js' | ||
| }, (run) => { | ||
| shimmer.wrap(run, 'runMocha', runMocha => async function () { | ||
| if (!testStartCh.hasSubscribers) { |
There was a problem hiding this comment.
ℹ️ this hook was not modified
| // In serial mode, this hook is used to set the correct async resource to the test. | ||
| // In parallel mode, the same hook is executed in the worker, so this needs to be repeated in | ||
| // mocha/worker.js | ||
| addHook({ |
There was a problem hiding this comment.
ℹ️ the hook was not modified, but we extracted the function to runnableWrapper
|
|
||
| // Only used in parallel mode (--parallel flag is passed) | ||
| // Used to generate suite events and receive test payloads from workers | ||
| addHook({ |
There was a problem hiding this comment.
|
|
||
| // Only used in parallel mode (--parallel flag is passed) | ||
| // Used to start and finish test session and test module | ||
| addHook({ |
There was a problem hiding this comment.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4314 +/- ##
==========================================
+ Coverage 85.23% 86.12% +0.89%
==========================================
Files 252 250 -2
Lines 11042 10742 -300
Branches 33 33
==========================================
- Hits 9412 9252 -160
+ Misses 1630 1490 -140 ☔ View full report in Codecov by Sentry. |
juan-fernandez
force-pushed
the
juan-fernandez/support-parallel-mocha
branch
from
4d3edbf
to
2939b88
Compare
| @@ -0,0 +1,23 @@ | |||
| // eslint-disable-next-line | |||
| const workerpool = require('workerpool') | |||
There was a problem hiding this comment.
ℹ️ to test that running workerpool outside of a test does not crash
| @@ -1,674 +1,5 @@ | |||
| const { createCoverageMap } = require('istanbul-lib-coverage') | |||
There was a problem hiding this comment.
ℹ️ this file is mostly copied to mocha/main.js
| @@ -0,0 +1,305 @@ | |||
| const { | |||
There was a problem hiding this comment.
ℹ️ this file now contains functions used both for parallel and serial mode and for the main and worker processes
| @@ -2498,44 +2613,7 @@ testFrameworks.forEach(({ | |||
| }).catch(done) | |||
| }) | |||
| }) | |||
| it('marks the test session as skipped if every suite is skipped', (done) => { | |||
There was a problem hiding this comment.
ℹ️ this test was duplicated
packages/dd-trace/test/ci-visibility/exporters/test-worker/exporter.spec.js
Outdated
Show resolved
Hide resolved
packages/dd-trace/test/ci-visibility/exporters/test-worker/exporter.spec.js
Show resolved
Hide resolved
|
ℹ️ I missed a |
|
could you have a look at the latest commit @uurien ? |
What does this PR do?
Add support for
mocha --parallel.Changes:
mocha.jsinstrumentation file into three:main.jsandworker.js, which correspond to the main process and the worker processes, andcommon.jsfor common pieces. Additionally extract some common functionality intoutils.jsworker.jsfile will only instrument tests. Suites, modules and sessions will be handled bymain.js.main.jswill mostly include everything thatmocha.jshad, but we've added two extra hooks forlib/nodejs/parallel-buffered-runner.jsandworkerpoolto support parallel mode.workerpoollibrary because we need to intercept interprocess communication between mocha's main process and the workers, so we can report the test data.Motivation
Closes #4046
We want to be able to support mocha when it's run in parallel mode: https://mochajs.org/#parallel-tests
Plugin Checklist