Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Selenium script updates #1276

Merged
merged 25 commits into from
Jul 4, 2021
Merged

Selenium script updates #1276

merged 25 commits into from
Jul 4, 2021

Conversation

queengooborg
Copy link
Collaborator

@queengooborg queengooborg commented Jul 4, 2021

This PR performs a number of updates to the Selenium script to resolve some issues caused by some new custom tests, along with existing tests, due to permissions requirements. This PR also performs a few other little updates to make the script a little nicer, including the following:

  • Increase test timeout
  • Turn the browser-specific ignores into arrays rather than strings
  • Remove logic to remap Safari 6.1
  • Allow mic, camera, geolocation, and notifications for Chrome, Edge and Firefox
  • Simplify code to ignore the Safari x.0 versions on BrowserStack
  • Increase verbosity of tests and add more comments
  • Encode the commas in URLs for EdgeHTML versions of Edge (due to some odd issues)
  • Add a command line argument to reverse the browser's version order in testing (oldest-to-newest)
  • Catch results upload failures

This should fix #1230 with both the timeout increase and the permissions, since the holes are caused by timeouts (sometimes due to an on-screen prompt).

@queengooborg
Copy link
Collaborator Author

It seems that Firefox is struggling at the moment. I'm unsure of whether it's something to do with this script's updates, or if it's the recent custom tests we've added, but I feel it's probably this script. Will need to look into it some more.

@queengooborg queengooborg merged commit 637208c into main Jul 4, 2021
@queengooborg queengooborg deleted the selenium-updates branch July 4, 2021 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Selenium script leaves holes in the data that the update script gets tricked by
1 participant