Use a single 10-minute timeout for all VS Code integration tests#3323
Use a single 10-minute timeout for all VS Code integration tests#3323minestarks wants to merge 1 commit into
Conversation
|
Is there a danger the entire test run will time out and we won't get useful results? |
Yes, the entire test run could time out if we have enough tests, but I would call that still a useful result, because you'd be able to see from the logs that all your tests are timing out. |
As we discussed offline, some CI systems don't give you artifacts if the whole run times out since artifact upload has to happen within the timeout period. I don't know if that's true of this one. |
Fixes #3267
Short, ad-hoc timeouts could trip on slow machines and cause flaky failures. Timeouts should only guard against hung tests, never be short enough for a slow machine to hit.