Closed (fixed)
Project:
Drupal core
Version:
9.2.x-dev
Component:
quickedit.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 Mar 2022 at 15:36 UTC
Updated:
20 Mar 2022 at 09:29 UTC
Jump to comment: Most recent, Most recent file
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.