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

Restore streaming test output to nargo test #4489

Closed
TomAFrench opened this issue Mar 6, 2024 · 1 comment · Fixed by #6728
Closed

Restore streaming test output to nargo test #4489

TomAFrench opened this issue Mar 6, 2024 · 1 comment · Fixed by #6728

Comments

@TomAFrench
Copy link
Member

TomAFrench commented Mar 6, 2024

We removed the behaviour where tests would be printed to the console as they were started and completed as this would have broken when tests were made to run in parallel in #4484.

This has the downside of when there's a package with many tests such that running the tests take a long time, nargo appears to hang until all of the tests are run and then they're all printed at once. We should improve how we print out the test results so that we can display the test results as they come in.

@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Mar 6, 2024
@TomAFrench
Copy link
Member Author

One option for this would be to use https://github.com/LukasKalbertodt/libtest-mimic as a test harness but we could at the very least pull some code from it without using it as a proper dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant