Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review of existing tests' ability to be automated #443

Open
4 of 16 tasks
zcorpan opened this issue Jun 9, 2021 · 2 comments
Open
4 of 16 tasks

Review of existing tests' ability to be automated #443

zcorpan opened this issue Jun 9, 2021 · 2 comments
Labels
Agenda+Automation For discussion during the next teleconference related to test automation AT-Automation Related to assistive technology automation workstream tests About assistive technology tests

Comments

@zcorpan
Copy link
Member

zcorpan commented Jun 9, 2021

Background

In the ARIA-AT Automation Design Doc, we discussed that the existing ARIA AT tests may need some changes to allow for automation without ambiguity in the test instructions. This issue reviews the existing tests and suggests changes we think are necessary for automation. These changes could make the tests clearer for manual testers as well.

View these suggestions as a starting point for discussion; they are not set in stone.

alert

Test instructions look OK wrt automation.

breadcrumb

Test instructions look OK wrt automation.

checkbox

PR: #447 (WIP)

  • Should split “navigate forwards” and “navigate backwards” to separate tests.
  • For tests that navigate forwards, insert a link before the checkbox and set focus to it with the setup script. Similar to checkbox-tri-state.
  • For tests that navigate backwards, insert a link after the checkbox and set focus to it with the setup script. Similar to checkbox-tri-state.
  • Remove “Note: it should be in the unchecked state.” from the instructions. This is covered in the success criteria.

checkbox-tri-state

  • Remove “Note: it should be in the partially checked state.” from the instructions. This is covered in the success criteria.
  • Should use command sequences: "You may need to press certain commands (such as Down Arrow) multiple times"

combobox-autocomplete-both-updated

  • Should use command sequences: "You may need to press certain commands (such as Down Arrow) multiple times"

combobox-select-only

  • Should use command sequences: "You may need to press certain commands (such as Down Arrow) multiple times"

command-button

Test instructions look OK wrt automation.

disclosure-faq

  • Remove “Note: it should be in the collapsed state.” from the instructions. This is covered in the success criteria.
  • Should use command sequences: “you may need to press Down Arrow multiple times”

disclosure-navigation

  • Should use command sequences: "you may need to press certain commands (such as Down Arrow) multiple times"

horizontal-slider

This test is currently skipped in automation because key press instructions are ambiguous.

menu-button-actions

Test instructions look OK wrt automation.

menu-button-actions-active-descendant

Test instructions look OK wrt automation.

menubar-editor

  • Split “Up Arrow / Down Arrow” into separate tests, with a link before or after that has initial focus.

minimal-data-grid

  • Uses command sequences ("T followed by Down Arrow"), but not fully? "you may need to press certain commands (such as Down Arrow) multiple times"

modal-dialog

Test instructions look OK wrt automation.

radiogroup-aria-activedescendant

  • Should use command sequences: “you may need to press Down Arrow multiple times”

radiogroup-roving-tabindex

  • Should use command sequences: “you may need to press Down Arrow multiple times”

rating-slider

This test is currently skipped in automation because key press instructions are ambiguous.

tabs-manual-activation

  • Should use command sequences: "You may need to press certain commands (such as Down Arrow) multiple times"

toggle-button

Test instructions look OK wrt automation.

vertical-temperature-slider

This test is currently skipped in automation because key press instructions are ambiguous.

@zcorpan zcorpan added tests About assistive technology tests AT-Automation Related to assistive technology automation workstream labels Jun 9, 2021
@jscholes
Copy link
Contributor

jscholes commented Jun 9, 2021

“Insure NVDA is in browse mode by pressing Escape. Note: This command has no effect if NVDA is already in browse mode.”

See #382.

@zcorpan
Copy link
Member Author

zcorpan commented Nov 16, 2021

I've updated the OP with my findings for the current state of the tests.

The main thing is that some of the tests aren't using command sequences yet.

menubar-editor also uses the old "Up Arrow / Down Arrow" pattern in one test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+Automation For discussion during the next teleconference related to test automation AT-Automation Related to assistive technology automation workstream tests About assistive technology tests
Projects
None yet
Development

No branches or pull requests

3 participants