- πΊπΈUnited States neclimdul Houston, TX
Pretty sure I figured out what's causing this. I'll follow up with some documentation but the details are mostly in the summary. Mostly you just need to run phpunit directly and you see the failure. If you run run-tests which splits everything into its own process you won't.
- πΊπΈUnited States Beakerboy Kentucky
Yes, one test must be modifying a file and not correctly cleaning itself after the test. This leaked test data is then contaminating a future test.
- πΊπΈUnited States neclimdul Houston, TX
woops, got distracted and didn't follow up on those notes and lost all my changes from debugging this. From memory, ConfigTest::testMixedCaseConfigCleanupPackages was interacting with the global composer instance or something. The simple fix was just adding
@runInSeparateProcess
to the test and I think that resolved it. - Merge request !5624Issue #3161320: Drupal\Tests\Composer\Plugin\VendorHardening\ConfigTest... β (Closed) created by neclimdul
- Status changed to Needs review
about 1 year ago 4:01pm 30 November 2023 - πΊπΈUnited States neclimdul Houston, TX
Well hello old me. Found this again testing PHPUnit 10 changes. Don't think its technically currently a blocker but gets in the way of testing so would be good to get out of the way since its such an easy fix.
- Status changed to RTBC
about 1 year ago 5:22pm 30 November 2023 - πΊπΈUnited States smustgrave
Simple fix, updated issue summary to include proposed fix.
-
larowlan β
committed 5c3083ef on 10.2.x
Issue #3161320 by neclimdul, Beakerboy, smustgrave: Drupal\Tests\...
-
larowlan β
committed 5c3083ef on 10.2.x
-
larowlan β
committed 5b0361d8 on 11.x
Issue #3161320 by neclimdul, Beakerboy, smustgrave: Drupal\Tests\...
-
larowlan β
committed 5b0361d8 on 11.x
- Status changed to Fixed
about 1 year ago 5:17am 22 December 2023 - π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Committed to 11.x and backported to 10.2.x
Nice find, simple fix
Automatically closed - issue fixed for 2 weeks with no activity.