diff options
| author | Christian Cleberg <[email protected]> | 2026-07-24 22:58:44 -0500 |
|---|---|---|
| committer | Christian Cleberg <[email protected]> | 2026-07-24 23:04:14 -0500 |
| commit | 7001577e6798530c943ac4cf03258bc172927a67 (patch) | |
| tree | 5ff0810726c5894764bdc38be71a5258422d17c6 /DomainDigUITests | |
| parent | a7d154e1db9fd9904b2255db889144ad15aac2f2 (diff) | |
| download | domain-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')
| -rw-r--r-- | DomainDigUITests/AccessibilityAuditHarness.swift | 8 |
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 |
