summaryrefslogtreecommitdiff
path: root/DomainDigUITests
diff options
context:
space:
mode:
authorChristian Cleberg <[email protected]>2026-07-20 18:29:18 -0500
committerChristian Cleberg <[email protected]>2026-07-20 18:50:57 -0500
commit349bec0e99f330fc1172ca9847c58f019e861f61 (patch)
tree2279c18f4dbfe64e6d0a6ce7a126982a5df66aa8 /DomainDigUITests
parenteb8a0d30da0cc150e95d44bad998c13ee2defc25 (diff)
downloaddomain-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')
-rw-r--r--DomainDigUITests/AccessibilityAuditHarness.swift12
-rw-r--r--DomainDigUITests/AccessibilityAuditTests.swift15
2 files changed, 13 insertions, 14 deletions
diff --git a/DomainDigUITests/AccessibilityAuditHarness.swift b/DomainDigUITests/AccessibilityAuditHarness.swift
index ca4d12b..08ddedd 100644
--- a/DomainDigUITests/AccessibilityAuditHarness.swift
+++ b/DomainDigUITests/AccessibilityAuditHarness.swift
@@ -85,6 +85,18 @@ enum AccessibilityAuditHarness {
timeout = nil
do {
try app.performAccessibilityAudit { issue in
+ // WCAG 1.4.3 exempts inactive components from contrast
+ // requirements, but the audit flags them anyway. Inspect's
+ // Run button is disabled until a domain is typed, so the
+ // empty state reported a contrast failure that was never a
+ // real defect. Suppressing on the rule beats driving the UI
+ // to enable the control: typing raises the keyboard, which
+ // then follows the audit onto later screens and flags the
+ // system emoji picker's category buttons.
+ if issue.auditType.contains(.contrast), issue.element?.isEnabled == false {
+ return true
+ }
+
let isEnforced = !enforcedAuditTypes.intersection(issue.auditType).isEmpty
let marker = isEnforced ? "FAIL" : "report"
// Include the element so the burndown says *what* to fix, not
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 {