Skip to content

Eagerly print test name before execution to expose hanging tests#176

Merged
Simn merged 2 commits into
masterfrom
copilot/update-test-runner-logging
Mar 22, 2026
Merged

Eagerly print test name before execution to expose hanging tests#176
Simn merged 2 commits into
masterfrom
copilot/update-test-runner-logging

Conversation

Copilot AI commented Mar 22, 2026

Copy link
Copy Markdown
Contributor

The runner printed test names only after completion, making it impossible to identify which test was hanging — the output would simply stop with no indication of what was running.

Changes

  • printTestStart(name): Prints name ... (no trailing newline) and flushes stdout before the test runs
    • sys: Sys.print() + Sys.stdout().flush()
    • js: process.stdout.write(...)
    • other targets: trace(...) fallback (separate line)
  • printTestEnd(passed, detail): Prints just the status (OK / FAIL: ...) after the test, completing the line
  • Removed printResult(name, passed, detail) which did the full print post-execution
  • printTestStart is called before the try block so a hang is always visible in output

Output

TestBasic
  testSimple ... OK
  testDelay ...        ← visible immediately; hangs here if test blocks

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update test runner to log currently executing test before execution Eagerly print test name before execution to expose hanging tests Mar 22, 2026
Copilot AI requested a review from Simn March 22, 2026 10:42
Copilot finished work on behalf of Simn March 22, 2026 10:42
@Simn Simn marked this pull request as ready for review March 22, 2026 11:00
@Simn Simn merged commit 5d63c00 into master Mar 22, 2026
50 checks passed
@Simn Simn deleted the copilot/update-test-runner-logging branch March 22, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants