Skip to content

Commit

Permalink
test: fix test-bootstrap-modules for coverage generation
Browse files Browse the repository at this point in the history
The internal binding profiler is loaded if the test is run with
NODE_V8_COVERAGE.

PR-URL: #51816
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
  • Loading branch information
joyeecheung authored and richardlau committed Mar 25, 2024
1 parent 587e70e commit dd0f164
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions test/parallel/test-bootstrap-modules.js
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,11 @@ if (process.features.inspector) {
expected.beforePreExec.add('Internal Binding inspector');
expected.beforePreExec.add('NativeModule internal/util/inspector');
expected.atRunTime.add('NativeModule internal/inspector_async_hook');

// This is loaded if the test is run with NODE_V8_COVERAGE.
if (process.env.NODE_V8_COVERAGE) {
expected.atRunTime.add('Internal Binding profiler');
}
}

const difference = (setA, setB) => {
Expand Down

0 comments on commit dd0f164

Please sign in to comment.