Skip to content

Use Selenium Without Saucelabs in Drone#63606

Closed
Hamms wants to merge 49 commits into
stagingfrom
elijah/selenium-without-sauce-in-drone
Closed

Use Selenium Without Saucelabs in Drone#63606
Hamms wants to merge 49 commits into
stagingfrom
elijah/selenium-without-sauce-in-drone

Conversation

@Hamms

@Hamms Hamms commented Jan 30, 2025

Copy link
Copy Markdown
Contributor

As part of our efforts to be more responsible with our usage of Saucelabs, explore using a self-hosted selenium container as the target for tests in drone.

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

Hamms added 22 commits January 30, 2025 11:37
As part of our efforts to be more responsible with our usage of
Saucelabs, explore using a self-hosted selenium container as the target
for tests in drone.
…-drone' into elijah/selenium-without-sauce-in-drone
Extract some common functionality from the SauceLabs-specific
Selenium logic into our generic Selenium helper, and streamline and
clean up some miscellaneous associated code. All in preparation for
[eventually adding an alternate remote Selenium
browser](#63606)
Hamms added a commit that referenced this pull request Mar 3, 2025
* Refactor Selenium Connection Logic

Extract some common functionality from the SauceLabs-specific
Selenium logic into our generic Selenium helper, and streamline and
clean up some miscellaneous associated code. All in preparation for
[eventually adding an alternate remote Selenium
browser](#63606)

* only local browsers should default to headless

* add ENV override for saucelabs url

* avoid setting window-size on mobile browsers; be more consistent with existing implementation

* [test ios] isolate references to $http_client to a single file

* give global variable a more specific name (also kick off a test run without ios devices)
@Hamms

Hamms commented Apr 15, 2025

Copy link
Copy Markdown
Contributor Author

Closing in favor of #65064

@Hamms Hamms closed this Apr 15, 2025
@Hamms Hamms deleted the elijah/selenium-without-sauce-in-drone branch April 15, 2025 21:59
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.

1 participant