Skip to content

Disable 'Starts from desktop shortcut' system test#14294

Merged
feerrenrut merged 1 commit intomasterfrom
systest-disableStartFromHotkey
Oct 26, 2022
Merged

Disable 'Starts from desktop shortcut' system test#14294
feerrenrut merged 1 commit intomasterfrom
systest-disableStartFromHotkey

Conversation

@feerrenrut
Copy link
Copy Markdown
Contributor

Link to issue number:

None, related to work in #14054

Summary of the issue:

Test 'Starts from desktop shortcut' relies on NVDA to be running to send the hotkey.
NVDA running interferes with the logic of the test.
The test intermittently fails, see: #14054

Description of user facing changes

None

Description of development approach

Disable the test until this can be addressed, issue #14293 has been opened to resolve this and re-enable the test.

Testing strategy:

Run on appveyor.

Known issues with pull request:

NVDA starting from hotkey will not be automatically tested.

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
  • Security precautions taken.

Test 'Starts from desktop shortcut' relies on NVDA to be running to
send the hotkey. This interferes with the logic of the test.

Disable test until this can be addressed.
@feerrenrut feerrenrut merged commit f606c89 into master Oct 26, 2022
@feerrenrut feerrenrut deleted the systest-disableStartFromHotkey branch October 26, 2022 04:09
@nvaccessAuto nvaccessAuto added this to the 2023.1 milestone Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants