Wednesday 27 January 2021

Re: +1 maint - phpunit 9 bootstrap proposal

Brief update:

I've gotten most everything passing now, except composer and
php-http-request2.

My best guess is that these two packages' test case failures are due to
changed behavior in array behavior, however I've not had any luck so far
diagnosing what's causing the failure. I can reproduce the failures for
php-http-request2 locally, but composer passes without issue when run
locally. None of the issues appear to be reported anywhere else, and
both packages are passing Debian's CI without issue.

I'm not sure how much more time I want to invest in debugging these, and
might consider suggesting to just disable the wayward tests just to get
phpunit across the finish line. Advice and/or help would be welcome.

Bryce

On Tue, Jan 19, 2021 at 03:55:54PM -0800, Bryce Harrington wrote:
> On Tue, Jan 12, 2021 at 05:08:26PM -0800, Bryce Harrington wrote:
> > On Tue, Jan 12, 2021 at 04:24:39PM -0800, Steve Langasek wrote:
> > > On Mon, Jan 11, 2021 at 08:48:15PM -0800, Bryce Harrington wrote:
> > > > phpunit has been stuck in a 8.5 -> 9.5 transition, which blocks other
> > > > things. I'd like to propose we back out 9.5 and bootstrap to 9.0, and
> > > > *then* to 9.5.
>
> Hi Steve,
>
> I worked a bit Friday retriggering the remaining packages, which
> got these to pass:
>
> php-guzzlehttp-psr7, phpdox, php-amqplib, php-laravel-lumen-framework,
> phpcpd
>
> Over the weekend it looks like there was a re-trigger of the whole php
> stack (or at least the phpunit-using portion of it), for phpunit 9.5.1-1
> I guess. This added a few more packages with issues, but they just
> needed retriggers.
>
> Today I reviewed the remainder, and note below what looks like needs
> done to resolve them.
>
> composer (2.0.8-2):
> - Errors:
> + Failed asserting that 'Undefined index: file' matches PCRE pattern
> "/(File format not recognized|Unrecognized archive format)/i".
> + Failed asserting that 'Undefined index: file' contains "is not a zip
> archive".
> + Undefined index: file
> - "Undefined index" suggests the test is trying to read an array
> element that doesn't exist. All three tests appear to be expecting
> an error message that isn't present in an array.
> - Not finding any indication of these errors reported upstream or in
> Debian.
> - autopkgtest passes locally for me.
> - I'm not sure what is wrong here...
>
> √ php-arthurhoaro-web-thumbnailer (2.0.3+dfsg-1):
> - Passed with phpunit 9.5.0-5, but is failing now with 9.5.1-1 on
> arm64 only
> - Failures are network issues trying to connect to gravatar.com.
> Previous failures haven't hit this before, so is unusual.
> - Re-triggered testrun as-is in case it's just a network issue
> - PASSED
>
> √ php-crypt-gpg (1.6.4-1):
> - Passed with phpunit 9.5.0-5, but is failing now with 9.5.1-1 on
> arm64 only
> - Failing testcase with error about invalid GPG password.
> - Re-triggering as-is, to rule out flaky test environment
> - PASSED
>
> php-doctrine-dbal (2.12.1-1):
> - The build logs indicate the autopkgtest failed, but the results show
> no errors, just warnings.
> - Running autopkgtest locally shows two errors in DBAL's
> PortabilityTest.php
> + Debian's 0002-Revert-Update-PHPUnit-to-9.2.patch is altering
> PortabilityTest.php. With this patch disabled, these tests pass,
> however there's then a different error, permission denied trying
> to chattr on /tmp/doctrine_failed_connection_292.db. Still, this
> looks like the right path to investigate
> - Also, there is a new 3.0.0-1 in debian experimental
>
> php-http-request2 (2.3.0-1ubuntu2):
> - Reproduced same 25 testsuite failures locally in lxc.
> Offhand, I am wondering if these are due to phpunit api changes?
> - Debian is also seeing autopkgtest failures with this package,
> although their errors look like phpunit framework issues.
> - There is a new upstream version 2.4.2 (not yet in Debian) with some
> php version updates. Might be worth pulling that version in.
>
> php-league-flysystem (1.1.3-2):
> - Timestamp discrepancy on file in filesystem. Same issue seen on
> all arch's.
> - Debian is seeing same issue in their autopkgtests.
> - Fixed in 1.1.3-3. This should sync in when it's migrated in Debian.
>
> php-net-ldap2 (2.2.0-3ubuntu3):
> - The test failures here are fixed in 2.2.0-6 from -proposed
> - The ubuntu delta is safe to drop - it's just compatibility fixups
> for earlier phpunit problems.
> - I've retriggered with php-net-ldap2 from -proposed
>
> php-twig (2.14.1-1):
> - Fixed in 2.14.1-2:
> https://salsa.debian.org/php-team/pear/twig/-/commit/77d0a0f8f6f5b5754c5752f50cbaa55b3ca07fc5
> - I've retriggered with php-twig from -proposed
>
> php-parser (4.10.4-1):
> - Data type discrepancy on armhf
> - The two armhf failures have been seen before (lp: #1878102),
> and we've disabled the tests in the past. Probably easiest path
> forward here as well.
>
> Bryce
>
>
> --
> ubuntu-devel mailing list
> ubuntu-devel@lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel

--
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel