diff options
| author | Christian Cleberg <[email protected]> | 2026-07-20 18:29:18 -0500 |
|---|---|---|
| committer | Christian Cleberg <[email protected]> | 2026-07-20 18:50:57 -0500 |
| commit | 349bec0e99f330fc1172ca9847c58f019e861f61 (patch) | |
| tree | 2279c18f4dbfe64e6d0a6ce7a126982a5df66aa8 /DomainDigUITests/AccessibilityAuditTests.swift | |
| parent | eb8a0d30da0cc150e95d44bad998c13ee2defc25 (diff) | |
| download | domain-dig-349bec0e99f330fc1172ca9847c58f019e861f61.tar.gz domain-dig-349bec0e99f330fc1172ca9847c58f019e861f61.tar.bz2 domain-dig-349bec0e99f330fc1172ca9847c58f019e861f61.zip | |
fix(a11y): suppress disabled-control contrast findings on the rule
WCAG 1.4.3 exempts inactive components from contrast requirements, so
Inspect's Run button — disabled until a domain is typed — was reporting a
contrast failure that was never a real defect.
The first attempt typed a domain to enable the button. That worked for
the single-screen test but raised the keyboard, which then followed the
audit onto every later screen in the Dynamic Type sweep and reported nine
phantom hit-region findings per screen against the system emoji picker's
category buttons. Suppressing on the rule instead — drop contrast
findings whose element reports isEnabled == false — fixes it everywhere
with no UI manipulation.
Also documents that simulator keyboard state persists across runs, so a
dirty simulator inflates the burndown with system-UI findings. Erase and
re-run before believing anything that names system UI.
Diffstat (limited to 'DomainDigUITests/AccessibilityAuditTests.swift')
| -rw-r--r-- | DomainDigUITests/AccessibilityAuditTests.swift | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/DomainDigUITests/AccessibilityAuditTests.swift b/DomainDigUITests/AccessibilityAuditTests.swift index 63ae584..6a90bac 100644 --- a/DomainDigUITests/AccessibilityAuditTests.swift +++ b/DomainDigUITests/AccessibilityAuditTests.swift @@ -17,20 +17,7 @@ final class AccessibilityAuditTests: XCTestCase { // MARK: Per-screen audits func testInspectScreen() throws { - let app = AccessibilityAuditHarness.launch() - app.selectRootTab("Inspect") - - // Type a domain so the Run button is enabled. A disabled control has no - // contrast requirement under WCAG 1.4.3, but the audit still flags it, - // so auditing the empty state would report a false positive forever. - let field = app.textFields.firstMatch - if field.waitForExistence(timeout: 5) { - field.tap() - field.typeText("example.com") - } - - let audited = try AccessibilityAuditHarness.audit(app, screen: "inspect", test: self) - try XCTSkipUnless(audited, "Audit did not complete in time for Inspect") + try auditRootTab("Inspect") } func testDashboardScreen() throws { |
