- Issue created by @acbramley
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
- Assigned to spokje
- π³π±Netherlands spokje
Let's first proof that there still is a random test failure.
Here's a patch that runs _only_
\Drupal\FunctionalJavascriptTests\Tests\JSWebAssertTest
500 times. - last update
over 1 year ago 1 pass, 2 fail - π³π±Netherlands spokje
The normal routine to prove a random failure is fixed is to run the failing patch and the patch with the fix at the same time, whilst the latter has to have ~8000 - 10.000 failure free runs to prove it's credibility.
So let's do that here.
- last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass, 2 fail - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - last update
over 1 year ago 1 pass - π³π±Netherlands spokje
25.000 runs is fine with me, here's the actual patch
- last update
over 1 year ago 29,486 pass - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 5:16pm 16 June 2023 - Status changed to RTBC
over 1 year ago 9:32pm 16 June 2023 - πΊπΈUnited States smustgrave
Improvement looks good. Thanks @Spokje for explaining too!
- last update
over 1 year ago 29,499 pass - Status changed to Fixed
over 1 year ago 8:54pm 19 June 2023 - π¬π§United Kingdom catch
Really nice find!
Committed/pushed to 11.x and cherry-picked back through to 9.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.