chore(e2e): update to latest protractor version (#2608)

* 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

Author: devversion

.gitignore

@@ -33,3 +33,4 @@
 npm-debug.log
 testem.log
 /.chrome
+yarn.lock

e2e/components/dialog/dialog.e2e.ts

@@ -102,6 +102,6 @@ describe('dialog', () => {
 
   // TODO(crisbeto): should be moved to a common util. copied from the menu e2e setup.
   function expectFocusOn(el: any): void {
-    expect(browser.driver.switchTo().activeElement().getInnerHtml()).toBe(el.getInnerHtml());
+    expect(browser.driver.switchTo().activeElement().getId()).toBe(el.getId());
   }
 });

e2e/components/menu/menu-page.ts

@@ -39,8 +39,7 @@ export class MenuPage {
 
   // TODO(kara): move to common testing utility
   expectFocusOn(el: any): void {
-    expect(browser.driver.switchTo().activeElement().getInnerHtml())
-        .toBe(el.getInnerHtml());
+    expect(browser.driver.switchTo().activeElement().getId()).toBe(el.getId());
   }
 
   expectMenuPresent(expected: boolean) {

package.json

@@ -48,12 +48,10 @@
     "@types/merge2": "0.0.28",
     "@types/minimist": "^1.1.28",
     "@types/node": "^6.0.34",
-    "@types/protractor": "^4.0.0",
     "@types/run-sequence": "0.0.27",
     "@types/rx": "^2.5.33",
-    "@types/selenium-webdriver": "2.53.36",
     "axe-core": "^2.0.7",
-    "axe-webdriverjs": "^0.4.0",
+    "axe-webdriverjs": "^0.5.0",
     "conventional-changelog": "^1.1.0",
     "dgeni": "^0.4.2",
     "dgeni-packages": "^0.16.2",
@@ -91,16 +89,16 @@
     "merge2": "^1.0.2",
     "minimist": "^1.2.0",
     "node-sass": "^3.4.2",
-    "protractor": "^4.0.8",
+    "protractor": "^5.0.0",
     "resolve-bin": "^0.4.0",
     "run-sequence": "^1.2.2",
     "sass": "^0.5.0",
-    "selenium-webdriver": "2.53.3",
+    "selenium-webdriver": "^3.0.1",
     "strip-ansi": "^3.0.0",
     "stylelint": "^7.7.0",
     "symlink-or-copy": "^1.0.1",
     "travis-after-modes": "0.0.7",
-    "ts-node": "^0.7.3",
+    "ts-node": "^2.0.0",
     "tslint": "^3.13.0",
     "typedoc": "^0.5.1",
     "typescript": "~2.0.10",

test/protractor.conf.js

@@ -41,6 +41,8 @@ if (process.env['TRAVIS']) {
   config.sauceKey = key;
   config.capabilities = {
     'browserName': 'chrome',
+    'version': 'latest',
+    "chromedriverVersion": "2.26",
     'tunnel-identifier': process.env['TRAVIS_JOB_ID'],
     'build': process.env['TRAVIS_JOB_ID'],
     'name': 'Material 2 E2E Tests',
@@ -53,11 +55,6 @@ if (process.env['TRAVIS']) {
     'recordVideo': false,
     'recordScreenshots': false
   };
-
-  // Because Protractor runs selenium instances concurrently in the CI and each instance
-  // will create a bloat of logs we don't want to show any info messages.
-  // Until protractor#1451 is solved, manually disable all info messages.
-  require('protractor/built/logger').Logger.prototype.info = () => {};
 }