Problem/Motivation
In #3417805: Use Symfony Process to spawn subprocesses in PhpUnitTestRunner we added a new timeout for tests. I think this was incorrect for the following reasons:
- We have no idea how long contrib and custom tests take (on whichever CI run on or a local machine).
- Long debug sessions now are impossible
- The process time last for the whole test - if a test class contains lots of tests this will not allow you to run the whole test locally.
Steps to reproduce
Proposed resolution
- If we really want to enforce some timeout on tests then it should apply on the test level method.
- It should be introduced first as a notice so people can adjust things.
- It should be configurable.
But personally I think we should not do this. CI have timeouts and locally you can kill the test. This is not an improvement.