Wednesday 3 February 2021

Re: +1 maint - phpunit 9 bootstrap proposal

On Wed, Jan 27, 2021 at 05:50:14PM -0800, Bryce Harrington wrote:
> On Wed, Jan 27, 2021 at 01:40:48PM -0800, Bryce Harrington wrote:
> > 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.
>
> For composer, I've filed:
> https://github.com/composer/composer/issues/9654

I've uploaded the fix for composer and it's now passing its tests.

Now just php-http-request2 remains for unblocking phpunit.

Bryce

> > 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
>
> --
> 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