- Issue created by @jcandan
- ๐ช๐ธSpain fjgarlin
Iโve checked core-dev dependencies around phpunit and not much changed i the last versions so it really seems to be an upstream issue.
- ๐ช๐ธSpain fjgarlin
Iโve triggered some more pipelines here https://git.drupalcode.org/project/gitlab_templates/-/pipelines/568930
Itโs late where I am but I thought Iโd ag least post the link above in case itโs useful for investigation in the meantime.
- ๐ช๐ธSpain fjgarlin
Yeah, the one using phpunit and the mentioned version hang.
The ones using an earlier version of phpunit or using run-tests.sh do not.
- ๐ช๐ธSpain fjgarlin
Maybe try setting the โโdebugโ param for phpunit.
variables: _PHPUNIT_EXTRA: "--debug"
- ๐ฌ๐งUnited Kingdom scott_euser
With
--debug
on it just seems to hang on 'Child process started' https://git.drupalcode.org/project/page_to_pdf/-/jobs/6162154 - no further debug info there unfortunately to see where the child process is hanging. - ๐ฌ๐งUnited Kingdom scott_euser
Confirmed locking to 11.5.28 solves it https://git.drupalcode.org/project/page_to_pdf/-/merge_requests/30/diffs?commit_id=1f61289de256f556304b775a958584f89422ae40 (temp of course) - so its definitely the change in 11.5.28...11.5.29 somewhere in here https://github.com/sebastianbergmann/phpunit/compare/11.5.28...11.5.29
- ๐ช๐ธSpain fjgarlin
I know itโs not a minimal and isolated test but itโd be great if we also post the anime in the upstream phpunit issue.
- ๐ฎ๐ชIreland lostcarpark
Also encountering on Navigation Extra Tools module in โจ Use Icon Api for adding icons in Navigation items Active . Got tests to run by adding
"phpunit/phpunit": "<=11.5.28"
torequire-dev
section of composer.json, but don't want to merge that. - ๐ช๐ธSpain fjgarlin
I commented with the upstream issue with #7 and #8 (thanks!). I'm going to try to use the "conflict" feature of composer to see if we can prevent this version of phpunit to be used.
- ๐ช๐ธSpain fjgarlin
Before (using 11.5.29): https://git.drupalcode.org/project/keycdn/-/jobs/6160780 (stopped it at 10 min)
After (using 11.5.28): https://git.drupalcode.org/project/keycdn/-/jobs/6163600 (worked) -
fjgarlin โ
committed 0bb52ab6 on main
Issue #3540591 by fjgarlin, jcandan, scott_euser, lostcarpark: PHPUnit...
-
fjgarlin โ
committed 0bb52ab6 on main
- ๐ช๐ธSpain fjgarlin
This is merged now, and it's pushed to all contrib as well.
- ๐ช๐ธSpain fjgarlin
Note that this is just a band-aid, we'll need to remove this once the upstream issue is fixed.
- ๐ช๐ธSpain fjgarlin
Created ๐ Remove conflict section on expand_composer about phpunit Postponed for when the upstream issue is fixed.
- ๐จ๐ฆCanada danrod Ottawa
Thanks a lot, I was having problems with the MRs not being able to be merged to the master branches.