Skip to content

Print individual client test results and report timeouts as failures#4918

Merged
kgrgreer merged 1 commit intodevelopmentfrom
test-output-for-client-tests
Mar 17, 2026
Merged

Print individual client test results and report timeouts as failures#4918
kgrgreer merged 1 commit intodevelopmentfrom
test-output-for-client-tests

Conversation

@abdelaziz-mahdy
Copy link
Copy Markdown
Collaborator

Problem

Client-side tests only printed aggregate counts (PASSED: N, FAILED: N) without individual test names or assertion details. Server-side tests printed each test name with SUCCESS/FAILURE lines. When client tests timed out, the report showed TESTS: 0 with no indication of failure.

Solution

Match client test output to server test behavior: print each test name and its SUCCESS/FAILURE lines after completion. Propagate actual failure messages from the browser to the server report. Detect browser timeouts and report them as explicit failures.

Changes

  • TestBorder.js: Push FAILURE output lines from each test into testRun.failures (previously only pushed test ID)
  • TestRunnerScript.js: Add reportClientTestDetails() to print individual client test output matching server-side format
  • TestRunnerScript.js: Detect incomplete TestRun after BrowserAgent timeout and record an explicit timeout failure

@jlhughes
Copy link
Copy Markdown
Collaborator

I won't be commenting on this PR, other than to say I provided a complete alternative to client testing that I believe would have been a better starting point for additional features.
#4729

@kgrgreer kgrgreer merged commit 0359a8a into development Mar 17, 2026
1 check passed
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.

3 participants