Closed Bug 1859026 Opened 6 months ago Closed 2 months ago

Investigate if a test of #identity-permission-box in Desktop UI is a non-user-facing and needs to be excluded from a11y_checks

Categories

(Firefox :: General, task, P3)

task

Tracking

()

RESOLVED FIXED
120 Branch
Tracking Status
firefox120 --- fixed
firefox124 --- fixed

People

(Reporter: ayeddi, Assigned: ayeddi)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(2 files)

There are 2 failing tests for the #identity-permission-box that appear to test not user-facing behavior: the test is sending a click to box.identity-box-button while it is visually hidden with display:none; and it aims to confirm there are no more permissions left for this site. This test target appears to be not done by a user and it does not have to confirm the accessibility of this control.

a11y_checks failing data:

FAIL	"browser/base/content/test/permissions/browser_permissions.js

The click is sent to a display:none element (or while visibility of the permissions toolbarbutton is being changed)"	Node is not accessible via accessibility API	identity-permission-box	box	identity-box-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/permissions/head.js:openPermissionPopup:17
chrome://mochitests/content/browser/browser/base/content/test/permissions/browser_permissions.js:testMainViewVisible/<:63
resource://testing-common/BrowserTestUtils.sys.mjs:withNewTab:146
chrome://mochitests/content/browser/browser/base/content/test/permissions/browser_permissions.js:testMainViewVisible:26
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
"
FAIL	"browser/base/content/test/permissions/browser_site_scoped_permissions.js

The click is sent to a display:none element (or while visibility of the permissions toolbarbutton is being changed)"	Node is not accessible via accessibility API	identity-permission-box	box	identity-box-button	"
chrome://mochikit/content/browser-test.js:test_ok:1583
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:a11yFail:342
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:assertCanBeClicked:659
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:runIfA11YChecks/this.AccessibilityUtils:638
chrome://mochikit/content/tests/SimpleTest/AccessibilityUtils.js:handleEvent:741
chrome://mochitests/content/browser/browser/base/content/test/permissions/head.js:openPermissionPopup:17
chrome://mochitests/content/browser/browser/base/content/test/permissions/browser_site_scoped_permissions.js:testSiteScopedPermissionSubdomainAffectsBaseDomain/<:49
resource://testing-common/BrowserTestUtils.sys.mjs:withNewTab:146
chrome://mochitests/content/browser/browser/base/content/test/permissions/browser_site_scoped_permissions.js:testSiteScopedPermissionSubdomainAffectsBaseDomain:24
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 these tests are expected to click a toolbar button that may be already hidden with "display:none;". This test target is not meant to be done by a user and could be inaccessible

Thus, we are excluding this click event from accessibility checks and removing the fail-if annotation that was added by bug 1854228 and bug 1858037 before the investigation in the meta bug 1854233.

Assignee: nobody → ayeddi
Status: NEW → ASSIGNED
Pushed by ayeddi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f6ee9b37c547
Disable a11y_check for two non-user-facing tests of #identity-permission-box in Desktop UI. r=Jamie,mconley
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

We intentionally exclude this click from the a11y_check, because this test case (as the ones in part 1 patch D190979) is expected to click a toolbar button that may be already hidden with "display:none;". The permissions panel anchor is hidden because the last permission was removed, however we force opening the panel anyways in order to test that the list has been properly emptied.

Pushed by ayeddi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eeb5df11b720
Pt 2. Disable a11y_check for one non-user-facing test of #identity-permission-box in Desktop UI. r=mconley
Status: REOPENED → RESOLVED
Closed: 6 months ago2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: