Repository navigation
"PHPUnit\TextUI\ResultPrinter" is internal and may change without further notice. #26
Description
Hello, I Love this printer! We just recently updated to PHPUnit 8.5.13 and started receiving errors in our github actions due to your printer. You can see here that the tests all pass beautifully using your printer with phpunit 7.5.10 but that the exact same tests on the same code show the following warning on the updated phpunit:
PHPUnit 8.5.13 by Sebastian Bergmann and contributors.
Testing
Other deprecation notices (2)
2x: The "PHPUnit\TextUI\ResultPrinter" class is considered internal This class is not covered by the backward compatibility promise for PHPUnit. It may change without further notice. You should not use it from "mheap\GithubActionsReporter\Printer8".
Error: Process completed with exit code 1.
We are stuck with the updated phpunit version due to a dependency on Drupal for our project.
Activity
mheap commentedon Feb 28, 2021
Hey @laceysanderson! It's been a while but I finally found time to sit down and investigate this. Whilst we do depend on an internal class, I'm not sure what the alternative is here.
I tried to reproduce your issue and the error shown isn't produced by the printer or by PHPUnit. I bootstrapped a test project with the following:
And when I ran the tests:
Are you running any static analysis tools that could be impacting the build?
wdoug commentedon Jul 16, 2021
If I'm reading this correctly, it seems like there might have been an update so this is now an interface instead of a class that needs to be extended
mheap commentedon Sep 6, 2021
I've just taken another pass at this and it looks as though it's to do with Symfony's PHPUnit bridge (thanks for the link @wdoug)!
Unfortunately in PHPUnit 8 I have to extend the
ResultPrinterclass even though it's internalFrom Sebastien:
This issue was resolved by the addition of
DefaultResultPrinterin PHPUnit 9, which this printer extends.I think the only solution here is to switch to PHPUnit 9, or run PHPUnit without using the Symfony bridge. In the mean time, I'll close and pin this issue in case it helps anyone else in the future