Skip to content

Slow test reporter #2166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Aug 6, 2025
Merged

Slow test reporter #2166

merged 8 commits into from
Aug 6, 2025

Conversation

hadley
Copy link
Member

@hadley hadley commented Aug 4, 2025

Initial prototype by claude, but lots of iteration by me.

Fixes #1466

@hadley hadley requested a review from gaborcsardi August 5, 2025 16:34
@hadley
Copy link
Member Author

hadley commented Aug 5, 2025

@gaborcsardi would you mind taking a look at this one? It'd be useful if you'd try running it on a couple of projects because it's hard to test the output.

@gaborcsardi
Copy link
Member

I like it!

── Slowest tests: ───────────────────────────────────────────────────────────────────────────────────────
[4.19s] test-watcher.R: watcher works correctly
[1.73s] test-parallel-crash.R: crash
[1.17s] test-parallel-errors.R: error in parallel setup code
[0.86s] test-parallel.R: snapshots
[0.83s] test-parallel.R: snapshots are removed if test file has no snapshots
[0.82s] test-parallel.R: new snapshots are added
[0.76s] test-parallel.R: ok
[0.75s] test-parallel.R: snapshots are removed if test file is removed
[0.73s] test-parallel-outside.R: error outside of test_that()
[0.68s] test-parallel.R: fail
... and 482 more slow tests

I understand where this is coming from, but to see that that I have 482 (!) more slow tests is kind of demoralizing....

@hadley hadley merged commit fa29578 into main Aug 6, 2025
11 checks passed
@hadley hadley deleted the slow-reporter branch August 6, 2025 12:56
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.

Reporter for finding slow tests
2 participants