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

CommentFileSizeAuthor
#4 3267754-4.patch1.96 KBlauriii
#4 3267754-4-50x.patch3.69 KBlauriii
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

Spokje’s picture

Spokje
Primary language Dutch
CreditAttribution: Spokje at Chromatic commented

Looking 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?

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented

I 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.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Active » Needs review
FileSize
3267754-4-50x.patch3.69 KB
3267754-4.patch1.96 KB

I 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.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Needs review » Reviewed & tested by the community

The patch fixes this locally - nice one @lauriii

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Version: 10.0.x-dev » 9.2.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed 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.

  • alexpott committed e03a5e4 on 10.0.x 
    Issue #3267754 by lauriii: AjaxTest is failing

  • alexpott committed 7b59ff8 on 9.4.x 
    Issue #3267754 by lauriii: AjaxTest is failing

(cherry picked from...

  • alexpott committed 95f6f37 on 9.3.x 
    Issue #3267754 by lauriii: AjaxTest is failing

(cherry picked from...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Patch (to be ported) » Fixed

We'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!

  • xjm committed 5251c7f on 9.2.x authored by alexpott 
    Issue #3267754 by lauriii: AjaxTest is failing

(cherry picked from...

Status: Fixed » Closed (fixed)

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