Exceptions during the setUp() or tearDown() method of a test are not handled

A very similar fix was made as part of 🐛 SQLite testing in GitlabCI - apache2(98)Address already in use Needs review but it was taken out in favor of this existing issue.
Exceptions thrown in setUp and tearDown are now caught and therefore reported properly to the CI system.

Came here from 🐛 SQLite testing in GitlabCI - apache2(98)Address already in use Needs review after a review. I think we can RTBC this one here as well! I checked both the PIFT and the GitLab CI results and all seem fine.

Also wanted to say that #2 showed the issue in a very understandable way :)

I have reviewed the MR and it looks good to me. Compared with the #2, the $this->tearDown(); is also moved to the try-catch block, but I think this solution is better, because we probably want also tearDown() to run without any exceptions. If there are any problems, it would be better to stop the execution with the exception, instead of "seeing what will happen" next.

Tests are also green. Adding a tag. Thanks all!

Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.