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

chore: release v4.6.1 #696

Closed
wants to merge 10 commits into from
Closed

chore: release v4.6.1 #696

wants to merge 10 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Mar 22, 2023

Automated changes by create-pull-request GitHub action

4.6.1 (2023-03-22)

Bug Fixes

  • cli: increase --timeout default to 90 seconds (to match --help) (#683) (d11481b)
  • work with extremely large results (#667) (395d5fc)
  • webdriverio: deprecate @wdio/sync support (#677) (6121e55)

straker and others added 9 commits January 24, 2023 14:41
chore: merge master into develop
* fix: work with extremely large partial results

* fix tests

* try again
* chore(ci): update selenium-webdriver to latest

* fix tests

* udpate webdriverjs
The webdriverio CI job was failing with the error message:

    session not created: This version of ChromeDriver only supports Chrome version 109
    Current browser version is 111.0.5563.64 with binary path /usr/bin/google-chrome

Despite the job running the following steps:

    - browser-tools/install-chrome
    - browser-tools/install-chromedriver
    - run: npm run coverage --prefix=packages/webdriverio

And browser-tools/install-chromedriver automatically installing the
right version of chromedriver based on the previously installed chrome
version.[1]

This was because browser-tools/install-chromedriver installed the
chromedriver under /usr/local/bin/chromedriver while
packages/webdriverio/src/test.ts was using the chromedriver from the
chromedriver NPM package (which is currently set to ^109.0.0 in
package.json).

This commit changes the webdriverio test code to instead use the
system-wide installed chromedriver (that has been installed via
browser-tools/install-chromedriver) in a CI environment.

[1]: https://circleci.com/developer/orbs/orb/circleci/browser-tools#commands-install-chromedriver
…683)

The timeout was decreased in 6f5498e
to 20 seconds (apparently accidentally since that commit was classfied
as a refactor ... and it also didn't update the --help text accordingly).

This commit changes it back to 90 seconds to match the --help text.
* refactor(webdriverjs): make code more DRY

* refactor: make tests respect CHROMEDRIVER_PATH
@github-actions github-actions bot requested a review from a team as a code owner March 22, 2023 13:56
docs: fix changelog
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.

5 participants