Skip to content

control high verbosity nvda logging during system tests with appveyor environment var#14055

Merged
feerrenrut merged 3 commits intomasterfrom
highverbosityLoggingViaEnvVar
Sep 6, 2022
Merged

control high verbosity nvda logging during system tests with appveyor environment var#14055
feerrenrut merged 3 commits intomasterfrom
highverbosityLoggingViaEnvVar

Conversation

@feerrenrut
Copy link
Copy Markdown
Contributor

Link to issue number:

Follow on from #14054

Summary of the issue:

When tests are failing in an unusual way, it may be helpful to be able to review verbose debug logging for interaction with MSAA or UIA.

Description of user facing changes

For developers:
VERBOSE_SYSTEM_TEST_LOGGING can be set (to "true") via Appveyor settings to enable high verbosity NVDA logging.

Description of development approach

When the environment variable VERBOSE_SYSTEM_TEST_LOGGING is set on appveyor, the system tests are started with an extra parameter (verboseDebugLogging='True') which enables the advanced logging categories in NVDA.

Testing strategy:

  • Run system tests locally with and without the argument
  • Run system tests on Appveyor with and without the argument.

Known issues with pull request:

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.

@feerrenrut
Copy link
Copy Markdown
Contributor Author

@feerrenrut
Copy link
Copy Markdown
Contributor Author

@feerrenrut feerrenrut force-pushed the highverbosityLoggingViaEnvVar branch from 82225f3 to f5d2454 Compare September 6, 2022 03:03
@feerrenrut feerrenrut changed the base branch from systemTests-foregroundFixes to master September 6, 2022 03:04
@feerrenrut feerrenrut marked this pull request as ready for review September 6, 2022 03:27
@feerrenrut feerrenrut requested a review from a team as a code owner September 6, 2022 03:27
@feerrenrut feerrenrut requested a review from seanbudd September 6, 2022 03:27
@feerrenrut feerrenrut merged commit 119c4da into master Sep 6, 2022
@feerrenrut feerrenrut deleted the highverbosityLoggingViaEnvVar branch September 6, 2022 11:32
@nvaccessAuto nvaccessAuto added this to the 2022.4 milestone Sep 6, 2022
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.

3 participants