Closed
Bug 1396880
Opened 7 years ago
Closed 7 years ago
Intermittent test_accessibility.py TestAccessibility.test_element_is_not_enabled_to_accessbility | ElementClickInterceptedException: Element <button id="button15"> is not clickable at point (1125,246.3) because it does not have pointer events enabled...
Categories
(Testing :: Marionette Client and Harness, defect, P5)
Tracking
(firefox57 fixed, firefox58 fixed)
RESOLVED
FIXED
mozilla58
People
(Reporter: intermittent-bug-filer, Assigned: whimboo)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
Filed by: hskupin [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=127762893&repo=try https://queue.taskcluster.net/v1/task/RRKhTkLsTxK733twpK2K-A/runs/0/artifacts/public/logs/live_backing.log Here the full exception summary: Intermittent test_accessibility.py TestAccessibility.test_element_is_not_enabled_to_accessbility | ElementClickInterceptedException: Element <button id="button15"> is not clickable at point (1125,246.3000030517578) because it does not have pointer events enabled, and element <body> would receive the click instead
Comment 1•7 years ago
|
||
I think probably this is a case of an incorrect test assertion now that webdriverClickElement covers some of the things accessibility also does. In https://bugzil.la/1387447 I addressed some of these cases but it is possible I missed one.
OS: Unspecified → All
Hardware: Unspecified → All
Assignee | ||
Comment 2•7 years ago
|
||
I checked the log again in more detail. So there is a JS error thrown for ElementNotInteractableError: [task 2017-09-01T15:45:15.941530Z] 15:45:15 INFO - WebDriverError@chrome://marionette/content/error.js:239:5 [task 2017-09-01T15:45:15.941865Z] 15:45:15 INFO - ElementClickInterceptedError@chrome://marionette/content/error.js:336:5 [task 2017-09-01T15:45:15.942123Z] 15:45:15 INFO - webdriverClickElement@chrome://marionette/content/interaction.js:188:11 [task 2017-09-01T15:45:15.942694Z] 15:45:15 INFO - async*interaction.clickElement@chrome://marionette/content/interaction.js:150:11 [task 2017-09-01T15:45:15.942881Z] 15:45:15 INFO - async*clickElement/<@chrome://marionette/content/listener.js:1368:14 [task 2017-09-01T15:45:15.943041Z] 15:45:15 INFO - navigate/<@chrome://marionette/content/listener.js:451:13 [task 2017-09-01T15:45:15.943390Z] 15:45:15 INFO - async*navigate@chrome://marionette/content/listener.js:450:13 [task 2017-09-01T15:45:15.943541Z] 15:45:15 INFO - clickElement@chrome://marionette/content/listener.js:1367:5 The failure then happens here: # Buttons are enabled but disabled/not-explorable via # the accessibility API and thus are not clickable via the # accessibility API. [...] self.run_element_test(self.pointer_events_none_elementIDs, lambda element: self.assertRaises(ElementNotAccessibleException, element.click)) So we expect a ElementNotAccessibleException but get a ElementNotInteractableError.
Comment 3•7 years ago
|
||
Looking at OF, this hasnt been hit in a long time so closing
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 4•7 years ago
|
||
This is not present on OF because it would need the webdriver conforming click to be enabled, which we currently do not have. This bug is still a blocker for us to get this turned on.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee | ||
Comment 5•7 years ago
|
||
I have had a look at this file and as it turns out the tests which are failing here are for buttons which have the `pointer-event: none` CSS property set. It means they basically have to fail for whatever action is performed on those, eg. a click in this case. It will basically cause a `ElementClickInterceptedException` with webdriverClick turned on, and will never raise an accessibility related exception anymore. As such those parts of test_element_is_not_enabled_to_accessbility have to be moved out to an element related test, and should not be kept in test_accessibility. Actually we have those tests in test_click.py. For now I would propose that we keep running the above failing tests only if webdriverClick is turned off. I would like to keep those tests until the legacy click has been fully removed.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → hskupin
Status: REOPENED → ASSIGNED
status-firefox57:
--- → affected
status-firefox58:
--- → affected
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8919704 -
Flags: review?(mjzffr)
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8919704 [details] Bug 1396880 - Skip not relevant checks in test_accessibility for webdriver click. https://reviewboard.mozilla.org/r/190608/#review195892
Attachment #8919704 -
Flags: review?(ato) → review+
Assignee | ||
Updated•7 years ago
|
Attachment #8919704 -
Flags: review?(mjzffr)
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2c78eb815f54 Skip not relevant checks in test_accessibility for webdriver click. r=ato
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2c78eb815f54
Status: ASSIGNED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Assignee | ||
Comment 10•7 years ago
|
||
Please uplift this test-only change to beta.
Whiteboard: [checkin-needed-beta]
Comment 11•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/5ddf0b9c8ff2
Whiteboard: [checkin-needed-beta]
Updated•1 year ago
|
Product: Testing → Remote Protocol
Comment 12•1 year ago
|
||
Moving bug to Testing::Marionette Client and Harness component per bug 1815831.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•