-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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(e2e): update to latest protractor version #2608
chore(e2e): update to latest protractor version #2608
Conversation
8c937a4
to
47842a5
Compare
* Updates to the latest Protractor version, which now uses Selenium V3 * The new types for Protractor v5 include the fix for the missing `chord` function (See DefinitelyTyped/DefinitelyTyped@123690b) * Methods like `innerHtml` have been removed in Protractor v5 (https://github.com/angular/protractor/blob/master/CHANGELOG.md)
245e4d8
to
4d734a6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any idea what the new e2e failure is?
@@ -33,3 +33,4 @@ | |||
npm-debug.log | |||
testem.log | |||
/.chrome | |||
yarn.lock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought that the yarn.lock
was was supposed to be checked into source control?
https://yarnpkg.com/en/docs/yarn-lock#toc-check-into-source-control
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is - (depends on the project) and then I would need to commit it to the Material 2 repository?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fine with me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, let's handle that in another PR when we also lock some more dependencies.
@jelbourn Saucelabs was using a super-old chromedriver version, which didn't work properly with Chrome v55. Specified the latest version and everything passes now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* chore(e2e): update to latest protractor version * Updates to the latest Protractor version, which now uses Selenium V3 * The new types for Protractor v5 include the fix for the missing `chord` function (See DefinitelyTyped/DefinitelyTyped@123690b) * Methods like `innerHtml` have been removed in Protractor v5 (https://github.com/angular/protractor/blob/master/CHANGELOG.md) * Test later chromedriver version
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Updates to the latest Protractor version, which now uses Selenium V3
The new types for Protractor v5 include the fix for the missing
chord
function(See DefinitelyTyped/DefinitelyTyped@123690b)
Methods like
innerHtml
have been removed in Protractor v5 (https://github.com/angular/protractor/blob/master/CHANGELOG.md)Due to an Protractor issue I just noticed that we don't run the latest Chrome version for e2e tests.
Closes #2384