build(examples): work around protractor typings issues and fix existing type errors

This works around the typings issues until we have a build of protractor with typings that don't
polute global types via ambient type definitions

build.sh

@@ -161,5 +161,4 @@ do
   fi
 done
 
-# TODO(i): uncomment once the protractor typings issue is resolved
+./modules/@angular/examples/build.sh
-#./modules/@angular/examples/build.sh

modules/@angular/examples/core/di/ts/contentChild/e2e_test/content_child_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../../_common/e2e_util';
 
 describe('contentChild example', () => {
   afterEach(verifyNoBrowserErrors);
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
-  let result: ElementFinder;
+  let result: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/core/di/ts/contentChild/index.html');

modules/@angular/examples/core/di/ts/contentChildren/e2e_test/content_children_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../../_common/e2e_util';
 
 describe('contentChildren example', () => {
   afterEach(verifyNoBrowserErrors);
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
-  let result: ElementFinder;
+  let result: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/core/di/ts/contentChildren/index.html');

modules/@angular/examples/core/di/ts/viewChild/e2e_test/view_child_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../../_common/e2e_util';
 
 describe('viewChild example', () => {
   afterEach(verifyNoBrowserErrors);
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
-  let result: ElementFinder;
+  let result: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/core/di/ts/viewChild/index.html');

modules/@angular/examples/core/di/ts/viewChild/view_child_example.ts

@@ -25,7 +25,7 @@ export class Pane {
     <div>Selected: {{selectedPane}}</div> 
   `,
 })
-export class ViewChildComp implements AfterViewInit {
+export class ViewChildComp {
   @ViewChild(Pane)
   set pane(v: Pane) {
     setTimeout(() => { this.selectedPane = v.id; }, 0);

modules/@angular/examples/core/di/ts/viewChildren/e2e_test/view_children_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../../_common/e2e_util';
 
 describe('viewChildren example', () => {
   afterEach(verifyNoBrowserErrors);
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
-  let result: ElementFinder;
+  let result: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/core/di/ts/viewChildren/index.html');

modules/@angular/examples/forms/ts/formBuilder/e2e_test/form_builder_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('formBuilder example', () => {
   afterEach(verifyNoBrowserErrors);
-  let inputs: ElementFinder;
+  let inputs: protractor.ElementArrayFinder;
-  let paragraphs: ElementFinder;
+  let paragraphs: protractor.ElementArrayFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/formBuilder/index.html');

modules/@angular/examples/forms/ts/nestedFormArray/e2e_test/nested_form_array_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('nestedFormArray example', () => {
   afterEach(verifyNoBrowserErrors);
-  let inputs: ElementFinder;
+  let inputs: protractor.ElementArrayFinder;
-  let buttons: ElementFinder;
+  let buttons: protractor.ElementArrayFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/nestedFormArray/index.html');
@@ -25,12 +25,12 @@ describe('nestedFormArray example', () => {
   });
 
   it('should add inputs programmatically', () => {
-    expect(browser.isElementPresent(inputs.get(2))).toBe(false);
+    expect(inputs.count()).toBe(2);
 
     buttons.get(1).click();
     inputs = element.all(by.css('input'));
 
-    expect(browser.isElementPresent(inputs.get(2))).toBe(true);
+    expect(inputs.count()).toBe(3);
   });
 
   it('should set the value programmatically', () => {

modules/@angular/examples/forms/ts/nestedFormArray/nested_form_array_example.ts

@@ -34,7 +34,7 @@ export class NestedFormArray {
     ]),
   });
 
-  get cities(): FormArray { return this.form.get('cities'); }
+  get cities(): FormArray { return this.form.get('cities') as FormArray; }
 
   addCity() { this.cities.push(new FormControl()); }
 

modules/@angular/examples/forms/ts/nestedFormGroup/e2e_test/nested_form_group_spec.ts

@@ -10,9 +10,9 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('nestedFormGroup example', () => {
   afterEach(verifyNoBrowserErrors);
-  let firstInput: ElementFinder;
+  let firstInput: protractor.ElementFinder;
-  let lastInput: ElementFinder;
+  let lastInput: protractor.ElementFinder;
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/nestedFormGroup/index.html');

modules/@angular/examples/forms/ts/ngModelGroup/e2e_test/ng_model_group_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('ngModelGroup example', () => {
   afterEach(verifyNoBrowserErrors);
-  let inputs: ElementFinder;
+  let inputs: protractor.ElementArrayFinder;
-  let buttons: ElementFinder;
+  let buttons: protractor.ElementArrayFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/ngModelGroup/index.html');

modules/@angular/examples/forms/ts/simpleForm/e2e_test/simple_form_spec.ts

@@ -10,8 +10,8 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('simpleForm example', () => {
   afterEach(verifyNoBrowserErrors);
-  let inputs: ElementFinder;
+  let inputs: protractor.ElementArrayFinder;
-  let paragraphs: ElementFinder;
+  let paragraphs: protractor.ElementArrayFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/simpleForm/index.html');

modules/@angular/examples/forms/ts/simpleFormControl/e2e_test/simple_form_control_spec.ts

@@ -12,9 +12,9 @@ describe('simpleFormControl example', () => {
   afterEach(verifyNoBrowserErrors);
 
   describe('index view', () => {
-    let input: ElementFinder;
+    let input: protractor.ElementFinder;
-    let valueP: ElementFinder;
+    let valueP: protractor.ElementFinder;
-    let statusP: ElementFinder;
+    let statusP: protractor.ElementFinder;
 
     beforeEach(() => {
       browser.get('/forms/ts/simpleFormControl/index.html');

modules/@angular/examples/forms/ts/simpleFormGroup/e2e_test/simple_form_group_spec.ts

@@ -12,8 +12,8 @@ describe('formControlName example', () => {
   afterEach(verifyNoBrowserErrors);
 
   describe('index view', () => {
-    let firstInput: ElementFinder;
+    let firstInput: protractor.ElementFinder;
-    let lastInput: ElementFinder;
+    let lastInput: protractor.ElementFinder;
 
     beforeEach(() => {
       browser.get('/forms/ts/simpleFormGroup/index.html');

modules/@angular/examples/forms/ts/simpleNgModel/e2e_test/simple_ng_model_spec.ts

@@ -10,9 +10,9 @@ import {verifyNoBrowserErrors} from '../../../../_common/e2e_util';
 
 describe('simpleNgModel example', () => {
   afterEach(verifyNoBrowserErrors);
-  let input: ElementFinder;
+  let input: protractor.ElementFinder;
-  let paragraphs: ElementFinder;
+  let paragraphs: protractor.ElementArrayFinder;
-  let button: ElementFinder;
+  let button: protractor.ElementFinder;
 
   beforeEach(() => {
     browser.get('/forms/ts/simpleNgModel/index.html');

modules/@angular/examples/tsconfig.json

@@ -16,7 +16,7 @@
     "target": "es5",
     "lib": ["es2015", "dom"],
     "skipLibCheck": true,
-    "types": ["jasmine", "protractor", "node"]
+    "types": ["jasmine", "node"]
   },
   "include": [
     "./_common/*.ts",
@@ -24,6 +24,8 @@
     "./**/test/*.ts",
     "./**/e2e_test/*.ts",
     "../../../node_modules/zone.js/dist/zone.js.d.ts",
-    "../../system.d.ts"
+    "../../system.d.ts",
+    // TODO(i): we can't use protractor's built-in typings because they contain lots of ambient definitions
+    "../../../node_modules/@types/protractor/index.d.ts",
   ]
 }

scripts/ci-lite/test_e2e.sh

@@ -37,8 +37,7 @@ if [[ ${TRAVIS} ]]; then
   sh -e /etc/init.d/xvfb start
 fi
 NODE_PATH=$NODE_PATH:./dist/all $(npm bin)/protractor ./protractor-e2e.conf.js --bundles=true
-# TODO(i): uncomment once the protractor typings issue is resolved
+NODE_PATH=$NODE_PATH:./dist/all $(npm bin)/protractor ./protractor-examples-e2e.conf.js --bundles=true
-#NODE_PATH=$NODE_PATH:./dist/all $(npm bin)/protractor ./protractor-examples-e2e.conf.js --bundles=true
 NODE_PATH=$NODE_PATH:./dist/all $(npm bin)/protractor ./protractor-perf.conf.js --bundles=true --dryrun
 echo 'travis_fold:end:test.e2e.localChrome'