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

Command line should provide an option for run timeout #437

Open
vinaychandra opened this issue Jul 4, 2018 · 3 comments
Open

Command line should provide an option for run timeout #437

vinaychandra opened this issue Jul 4, 2018 · 3 comments

Comments

@vinaychandra
Copy link

The current --timeout parameter when invoking command line sets the per-test timeout. Can we also have an option for the full timeout for the test run similar to RunConfiguration.Test SessionTimeout from vstest?

@ChrisMaddock
Copy link
Member

@vinaychandra

Can I ask what the motivation for this is?

I would have thought the main purpose of timeout is to kill particularly tests which may be flaky/unreliable. That doesn't really work the same way across an entire test run. What did you want to do with this functionality?

@mikkelbu
Copy link
Member

I'll try to provide some motivation.

At my work we have prior had problems on certain old CI setups which have a max timeout for each step in the pipeline. Sometimes the tests took a lot longer than expected - or a lot of tests where added when we merged in new functionality - making the test-step fail with a timeout and no information from nunit, since the process was just killed. Of cause we could just set the timeout to something very large, but a few times we have also run into tests that did not terminate :(.

Hence, it would be nice if the console could be stopped more controlled, so that it would be possible to get a (partial) result and perhaps also some indication of the last/current test executing.

@CharliePoole
Copy link
Collaborator

Pretty easy to create a run-level timeout. Partial output is harder. Tentatively adding this to 4.0 milestone, but "we'll see."

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

No branches or pull requests

4 participants