Open ArturT opened 5 years ago
Just note: It's possible now to set tests timing to 0 seconds. Knapsack Pro API will accept such time. When tests will be split in the queue then tests with 0 seconds will be served one by one similar as test file with not recorded time.
Use 0 seconds as execution time when a test file was not executed by a test runner (Cypress/Jest)
This issue solves 2 problems:
Should be done on @knapsack-pro/core
level as described in the PR description.
This should help with edge case when test file cannot be executed by test runner and because of that we won't record timing of that test file.
Edge case could happen when:
In such case, we could assume the test file has some default timing
like 0.1sof 0 seconds.Solution idea
We should improve this part of code:
Related
A similar issue was solved here: https://github.com/KnapsackPro/knapsack_pro-ruby/pull/71 (this is old PR, we should use 0s as default time for not executed tests).