Skip to content

changed order for preventing some errors#11397

Merged
christian-bromann merged 9 commits intowebdriverio:mainfrom
udarrr:main
Oct 11, 2023
Merged

changed order for preventing some errors#11397
christian-bromann merged 9 commits intowebdriverio:mainfrom
udarrr:main

Conversation

@udarrr
Copy link
Member

@udarrr udarrr commented Oct 10, 2023

Proposed changes

I just would like to suggest to check existence of chromedriver version before downloading and prevent the error when version of browser don't match with driver version each time when we're starting test execution.

2023-10-10T18:14:01.637Z WARN webdriver: Couldn't download Chromedriver v117.0.5938.150: Download failed: server returned code 404. URL: https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/117.0.5938.150/win64/chromedriver-win64.zip, trying to find known good version...
2023-10-10T18:14:02.270Z INFO webdriver: Download of Chromedriver v117.0.5938.149 was successful

@erwinheitzman
Copy link
Member

@christian-bromann you know the driver a bit better than me, do you know if this also needs to be applied to the other drivers?

@christian-bromann
Copy link
Member

do you know if this also needs to be applied to the other drivers?

No, only Chrome has the restriction where we have to have the same driver version as the Chrome version.

Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@christian-bromann christian-bromann added the PR: Polish 💅 PRs that contain improvements on existing features label Oct 11, 2023
@christian-bromann christian-bromann merged commit 9253f99 into webdriverio:main Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Polish 💅 PRs that contain improvements on existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants