Skip to content
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

[dotnet] add browser output from selenium manager to options #12405

Merged
merged 4 commits into from
Jul 24, 2023

Conversation

titusfortner
Copy link
Member

@titusfortner titusfortner commented Jul 23, 2023

Status

Draft because:

Description

Sets browser location on Options class if supported (Chrome, Edge, Firefox only)

  • Errors on Chrome & Edge if browser version invalid (Not backwards compatible, but GOOD!)
  • Uses Dev/Beta/Canary/Nightly if found on system and specified by name as browserVersion
  • Downloads Stable Chrome For Testing if Chrome not installed on System.

Motivation and Context

  • This ensures that the browser Selenium Manager is checking against is the one actually getting used by the driver

Notes

This is probably not the right way to do this, but it is the easy way. Store the value in the class, mark it not implemented if not implemented and get / set as necessary.

@titusfortner titusfortner changed the title Dotnet sm browser [dotnet] add browser output from selenium manager to options Jul 23, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (1d459cd) 56.99% compared to head (94ec997) 56.99%.

❗ Current head 94ec997 differs from pull request most recent head eb56c27. Consider uploading reports for the commit eb56c27 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk   #12405   +/-   ##
=======================================
  Coverage   56.99%   56.99%           
=======================================
  Files          86       86           
  Lines        5336     5336           
  Branches      193      193           
=======================================
  Hits         3041     3041           
  Misses       2102     2102           
  Partials      193      193           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@titusfortner titusfortner marked this pull request as ready for review July 24, 2023 16:06
@titusfortner titusfortner requested a review from diemol July 24, 2023 16:06
@diemol diemol merged commit e86c3cf into trunk Jul 24, 2023
@diemol diemol deleted the dotnet_sm_browser branch July 24, 2023 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants