Skip to content

Commit 77b3a6c

Browse files
committed
test: add 60s browser Mocha timeout (parity with engine repo)
Defensive headroom for js/wasm browser tests: Mocha's 2s default can trip a heavier test under CI/parallel load before it finishes computing (the engine repo hit this with its micrograd training demo). Set in the shared root browser-test config so it applies to every module. No-op for the current fast browser suites (tokenizer/validation/trace). Verified CI-equivalent (fresh lock, 'clean assemble allTests' default mode): BUILD SUCCESSFUL, kotlinStoreYarnLock clean. With -PincludeBrowserTests: 616 browser testcases, 0 failures. kotlin-js-store/yarn.lock is regenerated per-build (not committed), so no lock change ships.
1 parent feed359 commit 77b3a6c

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

build.gradle.kts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,11 @@ subprojects {
7777
if (name == "jsBrowserTest" || name == "wasmJsBrowserTest") {
7878
failOnNoDiscoveredTests = false
7979
enabled = includeBrowserTests
80+
// Generous Mocha timeout (default is 2s): browser engines — wasmJs
81+
// especially — run slower under CI/parallel load, so a heavier test
82+
// can exceed the default before it finishes computing. Mirrors the
83+
// engine repo's per-module bump. Harmless for the fast tests here.
84+
useMocha { timeout = "60s" }
8085
}
8186
}
8287
}

0 commit comments

Comments
 (0)