Bug 260104

Summary: AX: Test style cleanup for aria-current.html
Product: WebKit Reporter: Joshua Hoffman <jhoffman23>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, andresg_22, apinheiro, cfleizach, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Joshua Hoffman 2023-08-11 15:45:54 PDT 
Adjust the test style of aria-current to be more consistent with other AX tests.
Comment 1 Radar WebKit Bug Importer 2023-08-11 15:46:05 PDT 
<rdar://problem/113775205>
Comment 2 Joshua Hoffman 2023-08-11 15:51:03 PDT 
Created attachment 467249 [details]
Patch
Comment 3 Andres Gonzalez 2023-08-14 06:44:22 PDT 
(In reply to Joshua Hoffman from comment #2)
> Created attachment 467249 [details]
> Patch

Thanks Josh. One more style guideline and we are there. Use JS templete strings instead of concatenation, e.g.:

-    output += "PASS: AXARIACurrent is " + value + "\n";

+    output += `PASS: AXARIACurrent is ${value}\n`;
Comment 4 Joshua Hoffman 2023-08-14 15:09:53 PDT 
Created attachment 467273 [details]
Patch
Comment 5 EWS 2023-08-15 09:18:19 PDT 
Committed 266913@main (bb3bc0cc2777): <https://commits.webkit.org/266913@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 467273 [details].