summaryrefslogtreecommitdiff
path: root/DomainDigUITests/AccessibilityAuditHarness.swift
diff options
context:
space:
mode:
authorChristian Cleberg <[email protected]>2026-07-24 22:58:44 -0500
committerChristian Cleberg <[email protected]>2026-07-24 23:04:14 -0500
commit7001577e6798530c943ac4cf03258bc172927a67 (patch)
tree5ff0810726c5894764bdc38be71a5258422d17c6 /DomainDigUITests/AccessibilityAuditHarness.swift
parenta7d154e1db9fd9904b2255db889144ad15aac2f2 (diff)
downloaddomain-dig-7001577e6798530c943ac4cf03258bc172927a67.tar.gz
domain-dig-7001577e6798530c943ac4cf03258bc172927a67.tar.bz2
domain-dig-7001577e6798530c943ac4cf03258bc172927a67.zip
chore: clear actionable SonarCloud smells (shell [[, merged if)
- Scripts/audit-a11y.sh: replace `[ ... ]` with `[[ ... ]]` in the seven shell conditionals SonarCloud flagged (shelldre:S7688). The script is bash (`#!/usr/bin/env bash`), so `[[` is safe and avoids word-splitting/globbing footguns. The jq `[ .devices | ... ]` filter is untouched — it is not a shell test. - AccessibilityAuditHarness.swift: fold the nested `if` in the noise classifier into a single conditional (swift:S1066); behavior unchanged. The remaining open SonarCloud issues (S1075 https literals, S115 external-JSON CodingKeys, S107 parameter counts, S3087 callback nesting) are false positives or intentional and are being resolved as Won't Fix / Safe in SonarCloud, consistent with the v4.8.3 static-analysis policy in RELEASE_ROADMAP.md.
Diffstat (limited to 'DomainDigUITests/AccessibilityAuditHarness.swift')
-rw-r--r--DomainDigUITests/AccessibilityAuditHarness.swift8
1 files changed, 4 insertions, 4 deletions
diff --git a/DomainDigUITests/AccessibilityAuditHarness.swift b/DomainDigUITests/AccessibilityAuditHarness.swift
index 24f900e..58583e1 100644
--- a/DomainDigUITests/AccessibilityAuditHarness.swift
+++ b/DomainDigUITests/AccessibilityAuditHarness.swift
@@ -184,10 +184,10 @@ enum AccessibilityAuditHarness {
// and the search field's hit region at accessibility sizes is the
// system's own control. Reading `elementType` here is safe; reading
// `frame` is not (it kills element attribution for the whole audit).
- if let type = issue.element?.elementType, type == .searchField || type == .textField {
- if issue.auditType.contains(.textClipped) || issue.auditType.contains(.hitRegion) {
- return "system field placeholder/hit region, length-independent"
- }
+ if let type = issue.element?.elementType,
+ type == .searchField || type == .textField,
+ issue.auditType.contains(.textClipped) || issue.auditType.contains(.hitRegion) {
+ return "system field placeholder/hit region, length-independent"
}
// Unattributed clipped-text/dynamic-type findings. Bisection showed the