Tuesday 19 January 2021

Re: +1 maint - phpunit 9 bootstrap proposal

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