Skip to content

Commit

Permalink
fix(test): use only a single timeout for op sanitizers (denoland#20042)
Browse files Browse the repository at this point in the history
Chipping away at making tests faster. Appears we don't need double
timeout before sanitizing ops. This should cut baseline cost of running a test
by half.
  • Loading branch information
bartlomieju authored Aug 4, 2023
1 parent 433ecc9 commit 53ab6c0
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion cli/js/40_testing.js
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,6 @@ function assertOps(fn) {
// cleared can actually be removed from resource table, otherwise
// false positives may occur (https://github.com/denoland/deno/issues/4591)
await opSanitizerDelay();
await opSanitizerDelay();
}
const post = core.metrics();
const postTraces = new Map(core.opCallTraces);
Expand Down

0 comments on commit 53ab6c0

Please sign in to comment.