<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 3, 2020 at 3:08 PM Christian Ehrhardt <<a href="mailto:christian.ehrhardt@canonical.com">christian.ehrhardt@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 3, 2020 at 12:38 PM Christian Ehrhardt <<a href="mailto:christian.ehrhardt@canonical.com" target="_blank">christian.ehrhardt@canonical.com</a>> wrote:<br></div><div>... </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Class III:<br>- php-http-request2<br>  - PHP Fatal error:  Declaration of HTTP_Request2_Adapter_CommonNetworkTest::setUp() must be compatible with PHPUnit\Framework\TestCase::setUp(): void in /tmp/autopkgtest.Ih0Z0B/build.kMc/src/HTTP_Request2-2.3.0/tests/Request2/Adapter/CommonNetworkTest.php on line 5 (no idea yet)<br>- php-net-ldap2 - same as the error in php-http-request2<br><br>=> TODO Class III issues are still open and free for grabbing by anyone<br></div>
</blockquote></div><div><br></div>Taking a first look at these now ...<br clear="all"></div></blockquote><div><br></div><div><br></div><div>Note: both packages are rather rarely updated. Both still refer to git://<a href="http://anonscm.debian.org">anonscm.debian.org</a><br><br>Note: There is an old patch `phpunit6_compatibility.patch` that touches the offending file already.<br><br>The code extends phpunit:<br>  abstract class HTTP_Request2_Adapter_CommonNetworkTest extends PHPUnit\Framework\TestCase<br>  <br>This is defined in phpunit src/Framework/TestCase.php:<br> 416     protected function setUp(): void                                             <br> 417     {                                                                            <br> 418     }<br> <br>vs<br><br> 122     protected function setUp()                                                   <br> <br>Ok, it seems the :void type was added and needs to be adapted for.<br><br>Trying to recreate the same in a local VM works, so we can iterate in there.<br><br>The tests will succeed if we just append the : void as needed.<br><br>But it becomes clear that they will need more work later.<br>The result is RC==0 but showing a lot of:<br>The @expectedException, @expectedExceptionCode, @expectedExceptionMessage, and @expectedExceptionMessageRegExp annotations are deprecated. They will be removed in PHPUnit 9. Refactor your test to use expectException(), expectExceptionCode(), expectExceptionMessage(), or expectExceptionMessageRegExp() instead.<br><br>We also now hit the issue rbasak looked at and has a debdiff attached.<br>=> PHP Warning:  file_put_contents(/usr/bin/.phpunit.result.cache): failed to open stream: Permission denied in /usr/share/php/PHPUnit/Runner/DefaultTestResultCache.php on line 108<br><br>The root cause change in phpunit can be seen here<br><a href="https://github.com/sebastianbergmann/phpunit/commit/f5e5add13e73933c65878f53d5a94cdc0f120cc4">https://github.com/sebastianbergmann/phpunit/commit/f5e5add13e73933c65878f53d5a94cdc0f120cc4</a><br>and explained here<br><a href="https://github.com/sebastianbergmann/phpunit/issues/3288">https://github.com/sebastianbergmann/phpunit/issues/3288</a><br><a href="https://phpunit.de/announcements/phpunit-7.html">https://phpunit.de/announcements/phpunit-7.html</a><br><br>Here are two debdiffs for bryce to consider.<br>They will need the debdiff that Rbasak provided in -proposed first.<br><br>P.S. The projects seem dormant, not sure if upstreaming is worth. Both compat v6 and v7 patches are not usptreamed either. The way it is we can retain them ... for now.<br></div></div></div>