Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
AjaxTest is consistently failing today, don't think it's due to a recent commit, so it might be something on DrupalCI itself.
We've also had 'bad test days' with other tests, so maybe it's a symptom of whatever that is.
If there's no clear fix for it, we might need to skip the test and then open an other critical issue to re-enable it.
Example:
https://www.drupal.org/pift-ci-job/2332343
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#4 | 3267754-4.patch | 1.96 KB | lauriii |
#4 | 3267754-4-50x.patch | 3.69 KB | lauriii |
|
Comments
Comment #2
SpokjeLooking at the deploy time/date of the new Chromedriver here #3208793-11: Update Chrome container to use newer version and the fail times/dates of https://www.drupal.org/pift-ci-job/2332343 my Spidey-Senses are tingling?
Comment #3
lauriiiI think we should skip
\Drupal\FunctionalJavascriptTests\Ajax\AjaxTest
for now. I am running into multiple random failures locally in that test which makes it difficult to try to come up with a fix.Comment #4
lauriiiI was able to get the test to pass with this. I'm now thinking that the other random failures I was running into, were only happening when running the test with xdebug.
Comment #5
alexpottThe patch fixes this locally - nice one @lauriii
Comment #6
alexpottCommitted and pushed e03a5e415e to 10.0.x and 7b59ff8649 to 9.4.x and 95f6f37d1a to 9.3.x. Thanks!
Committed this to all branches that are currently active as the recent Chromedriver update is affecting them all.
Leaving to be ported to 9.2.x by a release manager so security release tests pass.
Comment #10
xjmWe'll have a regular 9.2.x release (no SA) coming out soon for the yarn updates, so backported to 9.2.x as well. Thanks!