facile-it / paraunit

Run PHPUnit tests in parallel
https://engineering.facile.it/paraunit/
Apache License 2.0
134 stars 14 forks source link

Phpunit 10 support #162

Closed Jean85 closed 2 years ago

Jean85 commented 2 years ago

This is a tentative, simple approach to support the future PHPUnit 10.

Old hooks that Paraunit leveraged to read test results are being removed, but the new event system is really cool and it probably makes for an easy swap.

The current code should work already, I should only have to fix/rework the tests.

Jean85 commented 2 years ago

We need to drop support for PHP 8.0 due to https://github.com/sebastianbergmann/phpunit/commit/95b61586c5d9b90bcaa516ddca0af00efc408940

It's better if we start supporting Symfony 6 first, and maybe drop 3.4 in the meantime, since it probably wouldn't work.

Jean85 commented 2 years ago

Blocked by https://github.com/sebastianbergmann/phpunit/issues/4985. I'll try to dig in there.

Jean85 commented 2 years ago

With 1e02861 I'm stumbling on a complicated issue: when a test outcome is risky, both Passed and Risky events are fired, and with the current architecture this means doubling the captured tests outcome.

I'll need to decouple this somehow.