Test plan for the Switch Example Using HTML Checkbox Input

[IsaDC]

Preview Tests

Preview of Switch Example Using HTML Checkbox Input Test Plan | For Pattern: switch-checkbox

[mcking65]

@IsaDC

This plan is ready except for one problem: the setAttribute method call for setting aria-checked to 'true' is not working in the scripts for tests 2, 4, 6, and 8. I looked at the js in the generated HTML; it looks good to me. I don't know why setAttribute is not working.

[mcking65]

@howard-e

Do you see why the setAttribute specified on line 26 in this file is not setting aria-checked to 'true'?

[howard-e]

@mcking65 switch-checkbox is using <input type=checkbox> which controls it's own checked attribute. The other, switch and switch-button operated differently in being controlled by aria-checked. I'm thinking the following is what's wanted here:

const switchEl = testPageDocument.querySelector('[role="switch"]');
switchEl.checked = true;
switchEl.setAttribute('aria-checked', 'true');

edit: Going back through the roles guidance on the switch pattern page, thinking the switch-checkbox WAI example could be updated to include aria-checked to follow that guidance

---

striking below thought:

Commented aria-checked in the above snippet because it's not mentioned on the example page. Should it still be part of this setup script and assertions?

[IsaDC]

Hello @mcking65 and @howard-e
I've run into an issue with the automated checks.

First, I merged the latest from master into this branch. I then tried to add the checked HTML attribute, and while there were no local validation issues, the build failed. The error log seems to point to a problem in support.json.

Any ideas what might be causing this?

[howard-e]

@IsaDC the error was reporting unexpected whitespace in the support.json. I just merged #1309 into master so merging master into this branch again will address that.

[IsaDC]

Thanks much @howard-e