-
Notifications
You must be signed in to change notification settings - Fork 35
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
Update quality-checks.md #265
base: main
Are you sure you want to change the base?
Conversation
Added BrowserStack and suggested config for endorsed accessibility test tools
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi - can we clarify the conditions we think should apply to tests passing or failing? (Not the contents of your specific tests, but the things you're testing against?)
For example, are we expecting tests to fail if certain UI elements don't exist under some browsers, or for tests to fail if buttons don't take you to the correct destination, etc, etc?
I think this is what teams are missing: a clear steer on what "good" really means for cross-browser testing.
Thanks!
Hi Andy, in NHSUK we use Chrome as a default browser when running our tests. We expect the same test to pass on almost any BrowserStack combination of device, OS or browser. This is because we use the NHSUK front end library and the UI presented should not differ between devices/browsers, other than screen size/scale etc. This is the same for elements that appear on the page and end-to-end journeys. Our automation tests are still running via our frame work, BrowserStack just gives us the ability to run our tests against different browsers/devices. BrowserStack itself isn't actually running any tests. We just use their interface to run our tests. |
There are probably a couple of things bundled in this PR, both good - I think, they are as follows:
With the above in mind
|
Added BrowserStack and suggested config for endorsed accessibility test tools