Skip to content

Commit

Permalink
test: update trace events test expectations
Browse files Browse the repository at this point in the history
V8 9.2 doesn't emit the "V8.ScriptCompiler" event anymore.
Use "V8.GCScavenger" instead.

PR-URL: nodejs#38990
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Robert Nagy <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
  • Loading branch information
targos committed Jul 20, 2021
1 parent 6ceefa7 commit db0ca87
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/parallel/test-trace-events-all.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ proc.once('exit', common.mustCall(() => {
return false;
if (trace.cat !== 'v8')
return false;
if (trace.name !== 'V8.ScriptCompiler')
if (trace.name !== 'V8.GCScavenger')
return false;
return true;
}));
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-trace-events-v8.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ proc.once('exit', common.mustCall(() => {
return false;
if (trace.cat !== 'v8')
return false;
if (trace.name !== 'V8.ScriptCompiler')
if (trace.name !== 'V8.GCScavenger')
return false;
return true;
}));
Expand Down

0 comments on commit db0ca87

Please sign in to comment.