Skip to content
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

feat(forge test): improve UI when running tests #7605

Open
Evalir opened this issue Apr 9, 2024 — with Linear · 2 comments
Open

feat(forge test): improve UI when running tests #7605

Evalir opened this issue Apr 9, 2024 — with Linear · 2 comments
Labels
A-testing Area: testing Cmd-forge-test Command: forge test T-feature Type: feature

Comments

Copy link
Member

Evalir commented Apr 9, 2024

new spinners and UI for forge test, see also Cargo (status logs like "checking"), cargo-nextest spinner and streaming test output

  • we don't group per "test suite" (contract), they're just like an extra separator like in Rust tests (crate::mod_a::mod_b::test_name)
  • main thread handles individual test events instead of just test results
    • keeps a buffer of currently running tests with a spinner for each, and a general spinner with counter for all the tests
    • events include test start, fuzz/invariant update counter, and test result, keyed by ID
    • once a test is done, the spinner line is replaced with the pass/fail result, logs, traces etc
@grandizzy
Copy link
Collaborator

I think this can be closed as #7914 was merged. Pending suppressing of proptest messages proptest-rs/proptest#453

@DaniPopes
Copy link
Member

That's only for fuzz/invariants. We want to refactor the entire output of the test runner.

@zerosnacks zerosnacks added T-feature Type: feature A-testing Area: testing Cmd-forge-test Command: forge test labels Jul 15, 2024
@zerosnacks zerosnacks changed the title Forge test UI Revamp feat(forge test): improve UI when running tests Jul 15, 2024
@zerosnacks zerosnacks added this to the v1.0.0 milestone Jul 26, 2024
@grandizzy grandizzy removed this from the v1.0.0 milestone Oct 1, 2024
@github-project-automation github-project-automation bot moved this to Todo in Foundry Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing Area: testing Cmd-forge-test Command: forge test T-feature Type: feature
Projects
Archived in project
Development

No branches or pull requests

4 participants