[random test failure] Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest

Not sure what happened there, dup of 📌 Skip Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest Fixed

Wait no we need this one, it's the follow-up to the other one.

The skip issue is now in, so no need for this one to be postponed.

Thanks,
-Derek

is there any update on this issue, as this is creating tests to fail frequently now.

@deepakm MediaLibraryTest::testButton() is skipped so can't fail - this issue was opened to unskip it when it's reliable again, which has not happened yet. Which test is failing frequently now?

MediaLibraryTest::testButton() - this tests fail when we uncomment the code for skipping the test, this has dependency on 1 of our custom phpunit tests written because we are also doing similar functionality to test insert media button on ckeditor5 editor.

Had to remove the "item selected" assertions as that text no longer seems to exist in the CKE widget. Running the repeat test to see how many failures we get.

The test has passed 200/200 times. It's not clear from the previous issue where the failure was since the old CI jobs no longer exist so it's hard to tell if the random failure is indeed fixed

Is this not what it was testing though?

@smustgrave I don't see that text when using media library in CKE5, it probably got hidden at some point as that's more for fields with specific cardinality (it doesn't make sense when embedding media)

Oh you're right, odd but guess it makes sense since you can only grab 1 at a time in ckeditor.

Committed/pushed to 11.x and cherry-picked to 11.2.x/10.6.x/10.5.x, thanks!

catch → closed merge request !12999

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