Closed Bug 1864226 Opened 5 months ago Closed 5 months ago

Investigate if a test of a click on a static toaster panel in browser_protectionsUI.js in Desktop UI needs to be updated to pass a11y_checks

Categories

(Firefox :: General, task, P3)

task

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: ayeddi, Assigned: ayeddi)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

There is failing Tier 2 a11y-checks for the click on a toast message panel that is testing the main panel would be shown (after the toast disappears) - but for the main panel to appear, the keyboard-only user could activate the toolbar button that is accessible and thus this alternative way to access the panel does not have to provide keyboard accessible way. Plus, the toast is not actionable, it has no controls and it serves as a status message for a user.

We could skip this specific test for a11y-checks and re-enable the a11y-checks in the test manifest.

Failing test info:

FAIL	browser/base/content/test/protectionsUI/browser_protectionsUI.js	Node is not accessible via accessibility API	protections-popup-mainView-panel-header	box	panel-header panel-header-with-info-button	"
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:assertCanBeClicked:658
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:runIfA11YChecks/this.AccessibilityUtils:635
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:handleClick:751
chrome://mochitests/content/browser/browser/base/content/test/protectionsUI/browser_protectionsUI.js:testToggleSwitchFlow:434
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"

We intentionally turn off a11y_checks, because the following click is targeting static toast message that's not meant to be interactive and is not expected to be accessible. The toaster times out by itself and the main panel is accessible via the toolbarbutton that provides an alternative way to repeat the flow of the test case.

We are removing the fail-if/skip-if annotations from the test manifest that was added by bug 1854228 before the investigation in the meta bug 1854233.

Pushed by ayeddi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0f3651957c2d
SetEnv to skip one click on a static toaster in a Desktop UI test and re-enable a11y-checks. r=mconley
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: