- Issue created by @catch
- Status changed to Needs review
3 months ago 1:31am 19 August 2024 - Status changed to RTBC
3 months ago 1:39pm 19 August 2024 - πΊπΈUnited States smustgrave
I'm willing to bet that this change didn't break nightwatch. Neat solution for skipping those tests
- Status changed to Fixed
3 months ago 7:56pm 20 August 2024 - π«π·France nod_ Lille
Committed and pushed f06e0f3f47 to 11.x and 7135f09a09 to 11.0.x and 9a0f1ff644 to 10.4.x and e53b039258 to 10.3.x. Thanks!
- π«π·France nod_ Lille
seems it's causing issues in 10.4.x and 10.3.x reverting them for now
ex: https://git.drupalcode.org/project/drupal/-/pipelines/260708/test_report... - Assigned to spokje
- Status changed to Downport
3 months ago 5:45am 22 August 2024 - π³π±Netherlands spokje
Spokje β changed the visibility of the branch 3468829-speed-up-workspacescontentmoderationstatetest to active.
- π³π±Netherlands spokje
Spokje β changed the visibility of the branch 3468829-speed-up-workspacescontentmoderationstatetest to hidden.
- Issue was unassigned.
- Status changed to Needs review
3 months ago 7:00am 22 August 2024 - π³π±Netherlands spokje
Looks like we hit an interesting difference between PHPUnit 9 and 10.
In 9, for a test to be picked up by its class'
setUp()
it has to be defined in that class, even though there's also an implementation in its base-class.
In 10, when there's a definition in the base-class, thesetUp()
of the extending class will pick up on each test, even when only defined in the base-class.Added some empty-bodied (except some explanatory comment) tests-to-be-skipped and all seems well now in the land of 10.x
- π«π·France nod_ Lille
Perfect, thanks.
Committed and pushed 7f653a7d6b to 10.4.x and b3ac46aee4 to 10.3.x. Thanks!
- Status changed to Fixed
3 months ago 8:12am 22 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.