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

Show currently running tests in progress output #3728

Open
nohwnd opened this issue Aug 30, 2024 · 2 comments · May be fixed by #4142
Open

Show currently running tests in progress output #3728

nohwnd opened this issue Aug 30, 2024 · 2 comments · May be fixed by #4142

Comments

@nohwnd
Copy link
Member

nohwnd commented Aug 30, 2024

Summary

Part of #2162 that was not implemented, because we want to compare the experience between single dll run, and multi dll run (in dotnet test), once we have it available.

By default please also output currently running test + result of passing tests. This is also important when a test does actually hang/crash so you can see what test might have caused it.

Background and Motivation

Show which tests are running so we know when a test is hanging.

Proposed Feature

Show one line for every test in progress and timing.

Make sure the information fits on screen. Decide which tests should be cut off when there are too many to still keep the output useful.

Alternative Designs

Consider the impact of this #3639 where I think it is more appropriate to implement detection of stuck tests closer to the test framework. This also limits the amount of information we will render on screen, and we don't have to consier what will happen when tests run very fast, or how will user figure out if test is stuck.

@Evangelink
Copy link
Member

Assigning @drognanar to investigate and start suggesting some display/experience here.

@nohwnd nohwnd linked a pull request Nov 25, 2024 that will close this issue
@nohwnd
Copy link
Member Author

nohwnd commented Nov 25, 2024

Added the prototype code I did for this when we were discussing it in the linked PR.

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

Successfully merging a pull request may close this issue.

3 participants