Give the challenge sweeps an explicit timeout

[pambrose]

Follow-up to #19, which split the challenge sweep per language but left the worst body at roughly 34s of a 60s budget by CI extrapolation.

The obvious fix would not have worked

Adding TestConfig(timeout = ..) and stopping there would have looked right while changing nothing about the actual failure mode. The #18 failure was:

kotlinx.coroutines.test.UncompletedCoroutinesError:
  After waiting for 1m, the test body did not run to completion

That comes from runTest's own default timeout, which is a separate mechanism from Kotest's test-case timeout. A Kotest timeout cannot raise it.

Reading the Ktor 3.5.1 source:

public fun testApplication(
    parentCoroutineContext: CoroutineContext = EmptyCoroutineContext,
    block: suspend ApplicationTestBuilder.() -> Unit
): TestResult = runTestWithRealTime(parentCoroutineContext) {
    runTestApplication(parentCoroutineContext, block)
}

public suspend fun runTestApplication(..)

testApplication takes no timeout parameter — the runTestWithRealTime wrapper is what imposes the 60s default. runTestApplication is the same public entry point without the wrapper, and because a Kotest test body is already a coroutine it can be awaited directly.

Change

The two sweeps await runTestApplication and declare TestConfig(timeout = CHALLENGE_SWEEP_TIMEOUT) (5 minutes). The hidden 60s ceiling is gone, so the declared timeout is the only governing limit.

Verified the timeout has teeth

Rather than assume, I temporarily set it to 1.seconds:

Test	Result
Test all Java challenges	FAILED
Test all Kotlin challenges	FAILED
other 3 tests	PASSED

kotlinx.coroutines.TimeoutCancellationException:
  Coroutine "spec-scope-.." timed out waiting for 1000 ms

That is Kotest's spec-scope timeout, not UncompletedCoroutinesError — confirming both that the declared value is honored and that the underlying mechanism actually changed. Restored to 5 minutes afterward.

Verification at 5 minutes

• All 5 tests pass (./gradlew test --rerun-tasks)
• make lint clean (kotlinter + detekt)

Note

runTestApplication carries a Ktor source comment reading "not really needed outside ktor probably". It is public API, but that phrasing suggests JetBrains sees it as niche, so it is worth knowing this is the coupling point if a future Ktor upgrade changes the test entry points. Test with correct answers and Test individual challenges still use plain testApplication; they complete in under 0.1s and need no headroom.

🤖 Generated with Claude Code