Skip to content

build: fix e2e tests failing due to chromedriver release (#14461) - #14475

Merged
mmalerba merged 1 commit into
angular:7.1.xfrom
devversion:follow-iup/e2e-tests-7-1-x
Dec 11, 2018
Merged

build: fix e2e tests failing due to chromedriver release (#14461)#14475
mmalerba merged 1 commit into
angular:7.1.xfrom
devversion:follow-iup/e2e-tests-7-1-x

Conversation

@devversion

Copy link
Copy Markdown
Member

Apparently the webdriver-manager always uses the latest version of the chromedriver. This is problematic because the CI does not always use the latest chrome version that works with the chromedriver from webdriver-manager.

In order to fix this, we lock the webdriver manager version to a specific version. Similarly to how it's done for angular/angular.

Note that this also restricts which Chrome versions can be run locally with the e2e tests, but the chromedriver range seems to always cover at least the latest two versions (which works for us)

Apparently the webdriver-manager always uses the _latest_ version of the chromedriver. This is problematic because the CI does not always use the latest chrome version that _works_ with the chromedriver from `webdriver-manager`.
In order to fix this, we lock the webdriver manager version to a specific version. Similarly to how it's done for `angular/angular`.
Note that this also restricts which Chrome versions can be run locally with the e2e tests, but the chromedriver range seems to always cover at least the latest two versions (which works for us)
@devversiondevversion added pr: merge safe merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release labels Dec 11, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 11, 2018
@devversion

Copy link
Copy Markdown
MemberAuthor

Caretaker note: Only targetting patch because the one that landed in master does not apply cleanly.

@mmalerba
mmalerba merged commit bddddea into angular:7.1.xDec 11, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yesPR author has agreed to Google's Contributor License Agreementmerge: caretaker noteAlert the caretaker performing the merge to check the PR for an out of normal action needed or notetarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@devversion@googlebot@mmalerba