Skip to content

Show test progress while the tests run - #4206

Merged
bbatsov merged 4 commits into
masterfrom
stream-test-results
Oct 2, 2026
Merged

bbatsov merged 4 commits into
masterfrom
stream-test-results

Conversation

@bbatsov

@bbatsov bbatsov commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Failures show up in the test report as each var finishes, and the echo area keeps a running count, instead of everything arriving when the run is over. The streaming comes from cider-nrepl 0.63.0 (clojure-emacs/cider-nrepl#1051), which is now the injected version; with an older cider-nrepl nothing changes.

Also stops the report from echoing "Mark set" for every expected/actual value it renders, which buried the run summary.

Fixes #696

It streams test progress while the tests run, which the test report is
about to use, and runs the tests with Orchard's new orchard.test.
The test report renders each var's non-passing assertions and unescapes
newlines over the whole buffer in one go.  Pulling both into helpers lets
the failures of a run in progress be appended to the report the same way.
With cider-test-stream-results on (the default), test requests ask
cider-nrepl to stream its progress.  The echo area then shows the running
totals after each test var, and a failing var is added to the test
report as soon as it's done, so it can be looked at while the rest of the
run continues.  The final report replaces that buffer's contents without
popping it up again.

A cider-nrepl without streaming ignores the request, and the report
arrives at the end as before.
insert-rectangle pushes the mark, so rendering each expected/actual value
echoed "Mark set" and buried the run's summary in the echo area.
@bbatsov
bbatsov force-pushed the stream-test-results branch from 744983b to 62f3e66 Compare October 2, 2026 11:05
@bbatsov
bbatsov merged commit 2baf573 into master Oct 2, 2026
18 checks passed
@bbatsov
bbatsov deleted the stream-test-results branch October 2, 2026 11:15
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.

Faster cider-test feedback

1 participant