Skip to content

[rust] Use latest browser from cache when browser path is not discovered - #13283

Merged
bonigarcia merged 3 commits into
trunkfrom
sm_latest_browser_from_cache
Dec 11, 2023
Merged

[rust] Use latest browser from cache when browser path is not discovered#13283
bonigarcia merged 3 commits into
trunkfrom
sm_latest_browser_from_cache

Conversation

@bonigarcia

Copy link
Copy Markdown
Member

Description

This PR allows Selenium Manager to discover the latest browser version available in cache if there is some problem (e.g., the browser version cannot be discovered using online endpoints).

Motivation and Context

Currently, there is a fallback to get the latest driver version in the cache when some error happens. This PR do the same, but for browsers. It will allow to avoid problems like the reported in #13238 (i.e., a temporal glitch in the Edge endpoint response).

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@bonigarcia bonigarcia added the C-rust Rust code is mostly Selenium Manager label Dec 11, 2023
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7cbfdbd) 57.99% compared to head (efc51b3) 57.99%.
Report is 2 commits behind head on trunk.

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

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk   #13283   +/-   ##
=======================================
  Coverage   57.99%   57.99%           
=======================================
  Files          88       88           
  Lines        5338     5338           
  Branches      224      224           
=======================================
  Hits         3096     3096           
  Misses       2018     2018           
  Partials      224      224           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bonigarcia
bonigarcia merged commit 82f7cf7 into trunk Dec 11, 2023
@bonigarcia
bonigarcia deleted the sm_latest_browser_from_cache branch December 11, 2023 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-rust Rust code is mostly Selenium Manager

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants