facile-it / paraunit

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

Deprecation warnings in >2.0 #224

Open evang522 opened 5 months ago

evang522 commented 5 months ago

The class src/Logs/TestHook/Deprecation.php implements PHPUnit's \PHPUnit\Event\Test\DeprecationTriggeredSubscriber.

This means that every time a deprecation is triggered in PHPUnit tests, it is reported by paraunit. (In our case, it also includes deprecations triggered by vendors' code usage / dependencies). But PHPUnit itself doesn't do this kind of reporting by default.

Perhaps I've missed an important setting somewhere. Is it possible to suppress these deprecations? It becomes a bit difficult to see the results of our rather large test suites when there are so many deprecations being reported.

If it's helpful, I created a repository with minimal packages to demonstrate the deprecations with paraunit and the lack thereof in phpunit.

https://github.com/evang522/demo-paraunit-deprecations

Thanks!

Jean85 commented 5 months ago

No you're right, Paraunit's behavior was like this since the previous major, and now PHPUnit started handling deprecations explicitly and we didn't align it to that.

It probably makes sense to do that while implementing #197, since --show-deprecations is the right switch for this.