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

4.0.0.Beta does not correctly report the initial test run time #2477

Closed
nicojs opened this issue Sep 13, 2020 · 0 comments · Fixed by #2496
Closed

4.0.0.Beta does not correctly report the initial test run time #2477

nicojs opened this issue Sep 13, 2020 · 0 comments · Fixed by #2496
Assignees
Labels
🐛 Bug Something isn't working
Milestone

Comments

@nicojs
Copy link
Member

nicojs commented Sep 13, 2020

The beta currently does not report the initial test run correctly. For example:

10:26:23 (3511) INFO DryRunExecutor Initial test run succeeded. Ran 59 tests in 30 seconds (net 21857 ms, overhead 34 ms).

The time here (30 seconds), is the total time until the initial test run completed. It would be better if we start the timer just before we start the test run, for example, when we Starting initial test run. This may take a while.

Note: this is purely cosmetics. The timeout calculation is using the exact initial test run time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working
Projects
None yet
2 participants