Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3267823-2.patch | 1.39 KB | alexpott |
| #4 | 3267823-4.patch | 761 bytes | alexpott |
| #2 | 3267823-2-pass.patch | 3.15 KB | alexpott |
| #3 | fail.jpg | 66.54 KB | alexpott |
| #2 | 3267823-2-fail.patch | 1.76 KB | alexpott |
Comments
Comment #2
alexpottComment #3
alexpott#2 is not working. Let's skip the test for now and revisit.
Here's screenshot of a pass
... and fail...
Why the difference?
Comment #4
alexpottHere's a patch to skip the test
Comment #5
spokjeOn "Spot-the-difference" in #3:
- The QuickEdit-popup with "Llama" has "Body ->" as a prefix in the Pass
INSTA-EDIT: O _why_ the difference, not _what's_ :/
Comment #6
longwaveLet's just add this to the skip list for now and revisit it later.
On the bright side this problem will go away when we remove Quick Edit from core :)
Comment #7
spokjeSince quickedit is on it's way out (of core) anyway,
- and the failing test does make all tested versions fail,
- and skipping the whole test would prevent that.
RTCB if TestBot agrees, which it basically already did in #2
Comment #8
xjmI'm fine with skipping the test for now, but we should file a followup issue to look into it. Drupal 9.4 has to support QuickEdit until November 2023.
Comment #9
longwaveWe should probably do it as part of #3267247: [meta] Fix and re-enable tests skipped for random failures where we are planning to fix other tests we skipped this week.
Comment #11
alexpott@longwave well amusingly these test fails are caused by the update of chromedriver.
Comment #15
xjmOkay, committed to all four branches. We can leave this issue open (but downgrade it to major) to get the test working again.
Comment #16
xjmAh, if there's already a followup, that's fine too. Thanks!
Comment #17
xjmComment #18
xjmThis was like an 80% fail rate, not 100%, so tagging accordingly.