Skip Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest

Yes, please!

Do we need a follow-up to fix it and an @todo comment here so we do t forget to unskip it someday?

@dww think the referenced ticket 🐛 [random test failure] Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest Active should be good?

Going ahead and marking it because "Waiting for branch to pass" means more random failures are on the way.

Happy for #2 to land-as is. Was asking if we prefer/"need" this, instead?

Leaving RTBC per evolving standards around this when posting nit-fixes. 😅

The last submitted patch, 2: 3351596.patch, failed testing. View results →

Hah, whoops. 😅 But at least #2 also randomly failed, so I don't feel so bad uploading yet another patch here. 😂 Now with passing commit-code-check.sh

Committed and pushed 56a7a8e28c to 10.1.x and 60af95fd8e to 10.0.x. Thanks!

I'll see you again over in 🐛 [random test failure] Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest Active

Sweet, thanks! 9.5.x backport? Patch #7 applies cleanly there, too...

As discussed in Slack: the fails only seem to be affecting 10.1.x and 10.0.x, so I am holding off on backport so we can then use 9.5.x as a kind of canary - if the builds start failing there as well, either we really did break something, or it might give us the clue as to the root cause of all this.

This test does fail on 9.5.x and 9.4.x; I have emails for it going back to CKE5's addition to core, including for those branches. I have had it fail for security advisories' private test runs. There just aren't as many failures for it because there aren't as many commits to those branches.

Therefore, I will backport this to 9.5.x and 9.4.x as well.

Automatically closed - issue fixed for 2 weeks with no activity.