1872903 - Add an exception from a11y_checks for a click on a web content in Desktop UI

Status: RESOLVED FIXED

(Firefox :: General, task, P3)

Description

[Anna Yeddi [:ayeddi]]

The browser/base/content/test/forms/browser_selectpopup_large.js sends a click outside on a web content. We do not want to test arbitrary web content, since we do not support remote documents like in-content elements with a11y_checks of browser mochitests at the moment, thus we add an exception from the a11y_checks for this tests via setEnv and remove the fail-if notation from its test manifest.

A11y_checks output is the following:

Result	Path	Error	El ID	El tagName	El className	Stack trace
FAIL	browser/base/content/test/forms/browser_selectpopup_large.js	Interactive elements must be labeled	ContentSelectDropdown	menulist		"
chrome://mochikit/content/browser-test.js:test_ok:1583
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:a11yFail:339
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:assertLabelled:510
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:assertCanBeClicked:670
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:runIfA11YChecks/this.AccessibilityUtils:635
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:handleClick:751
chrome://mochikit/content/tests/SimpleTest/EventUtils.js:synthesizeMouseAtPoint:621
chrome://mochitests/content/browser/browser/base/content/test/forms/browser_selectpopup_large.js:performLargePopupTests:177
chrome://mochitests/content/browser/browser/base/content/test/forms/browser_selectpopup_large.js:test_large_popup:300
chrome://mochikit/content/browser-test.js:handleTask:1134
chrome://mochikit/content/browser-test.js:_runTaskBasedTest:1206
chrome://mochikit/content/browser-test.js:Tester_execTest:1348
chrome://mochikit/content/browser-test.js:nextTest/<:1123
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1058
"

Comment 1

[Anna Yeddi [:ayeddi]]

Attachment: Bug 1872903 - Add an exception from a11y_checks for clicks on a web content in Desktop UI mochitests. r=mconley,florian

The browser/base/content/test/forms/browser_selectpopup_large.js sends a click outside on a web content. We do not want to test arbitrary web content, since we do not support remote documents like in-content elements with a11y_checks of browser mochitests at the moment, thus we add an exception from the a11y_checks for this tests via setEnv and remove the fail-if notation from its test manifest.

Comment 2

[Anna Yeddi [:ayeddi]]

The same exception is needed for the browser/base/content/test/forms/browser_selectpopup_toplevel.js - <select> element - in Desktop UI tests that is also failing the labelRule of a11y_checks.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c7be36ee0f24
Add an exception from a11y_checks for clicks on a web content in Desktop UI mochitests. r=mconley

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/c7be36ee0f24