Skip to content

Disable failing test: ARIA_invalid_spelling_and_grammar#13320

Merged
feerrenrut merged 2 commits intomasterfrom
disableAriaInvalidTest
Feb 9, 2022
Merged

Disable failing test: ARIA_invalid_spelling_and_grammar#13320
feerrenrut merged 2 commits intomasterfrom
disableAriaInvalidTest

Conversation

@feerrenrut
Copy link
Contributor

@feerrenrut feerrenrut commented Feb 9, 2022

Link to issue number:

None
An issue created to enable again: #13321

Summary of the issue:

Prior alpha builds that passed, now fail.
It is assumed that Chrome was updated on appveyor.
The failure can be reproduced locally, by running this system test runsystemtests -i chrome -t ARIA_invalid*,
and manually running NVDA against the sample.
Further investigation is required to determine the cause of the failure.

Description of how this pull request fixes the issue:

Disable the test to prevent blocking other work.
This failure can then be investigated and rectified.

Testing strategy:

Appveyor build.

Known issues with pull request:

None

Change log entries:

None

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English

Prior alpha builds that passed, now fail.
It is assumed that Chrome was updated.
Further investigation is required to determin the cause of the failure.
Co-authored-by: Sean Budd <sean@nvaccess.org>
@AppVeyorBot

This comment was marked as outdated.

@feerrenrut feerrenrut merged commit aa2a607 into master Feb 9, 2022
@feerrenrut feerrenrut deleted the disableAriaInvalidTest branch February 9, 2022 08:57
@nvaccessAuto nvaccessAuto added this to the 2022.1 milestone Feb 9, 2022
seanbudd pushed a commit that referenced this pull request Feb 18, 2022
* Disable failing test: ARIA_invalid_spelling_and_grammar

Prior alpha builds that passed, now fail.
It is assumed that Chrome was updated.
Further investigation is required to determine the cause of the failure.
seanbudd pushed a commit that referenced this pull request Feb 21, 2022
* Disable failing test: ARIA_invalid_spelling_and_grammar

Prior alpha builds that passed, now fail.
It is assumed that Chrome was updated.
Further investigation is required to determine the cause of the failure.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants