From 1eb5ecb56337e2d5116e74aa4ef3dbd4df01f669 Mon Sep 17 00:00:00 2001 From: Meghan Denny Date: Sat, 10 Aug 2024 00:53:23 -0700 Subject: [PATCH] ci: fix setInterval.test.js on windows (#13213) --- test/js/web/timers/setInterval.test.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/test/js/web/timers/setInterval.test.js b/test/js/web/timers/setInterval.test.js index 2654cc6254..d9eb1c5fef 100644 --- a/test/js/web/timers/setInterval.test.js +++ b/test/js/web/timers/setInterval.test.js @@ -1,6 +1,7 @@ import { it, expect } from "bun:test"; import { join } from "path"; -import "harness"; +import { isWindows } from "harness"; + it("setInterval", async () => { var counter = 0; var start; @@ -110,7 +111,10 @@ it("setInterval runs with at least the delay time", () => { it("setInterval doesn't leak memory", () => { expect([`run`, join(import.meta.dir, "setInterval-leak-fixture.js")]).toRun(); -}, 30_000); +}, !isWindows ? 30_000 : 90_000); +// ✓ setInterval doesn't leak memory [9930.00ms] +// ✓ setInterval doesn't leak memory [80188.00ms] +// TODO: investigate this discrepancy further it("setInterval doesn't run when cancelled after being scheduled", () => { expect([`run`, join(import.meta.dir, "setInterval-cancel-fixture.js")]).toRun();