feat(hybrid): set ng12hybrid flag in the config (#3452)

Author: sjelin

lib/browser.ts

@@ -1005,8 +1005,10 @@ export class ProtractorBrowser extends Webdriver {
    * Create a new instance of Browser by wrapping a webdriver instance.
    *
    * @param {webdriver.WebDriver} webdriver The configured webdriver instance.
-   * @param {string=} opt_baseUrl A URL to prepend to relative gets.
-   * @param {boolean=} opt_untrackOutstandingTimeouts Whether Browser should
+   * @param {string=} baseUrl A URL to prepend to relative gets.
+   * @param {string=} rootElement The css selector for the element which is the
+   *     root of the Angular app.
+   * @param {boolean=} untrackOutstandingTimeouts Whether Browser should
    *     stop tracking outstanding $timeouts.
    * @returns {Browser} a new Browser instance
    */

lib/config.ts

@@ -567,6 +567,15 @@ export interface Config {
      */
     disableEnvironmentOverrides?: boolean;
 
+    /**
+     * Tells Protractor to interpret any angular apps it comes across as hybrid
+     * angular1/angular2 apps (i.e. apps using ngUpgrade)
+     * Defaults to `false`
+     *
+     * @type {boolean}
+     */
+    ng12Hybrid?: boolean;
+
     seleniumArgs?: Array<any>;
     configDir?: string;
     troubleshoot?: boolean;

lib/configParser.ts

@@ -41,6 +41,7 @@ export class ConfigParser {
       noGlobals: false,
       plugins: [],
       skipSourceMapSupport: false,
+      ng12Hybrid: false
     };
   }
 

lib/runner.ts

@@ -231,6 +231,9 @@ export class Runner extends EventEmitter {
     if (config.useAllAngular2AppRoots) {
       browser_.useAllAngular2AppRoots();
     }
+    if (config.ng12Hybrid) {
+      browser_.ng12Hybrid = config.ng12Hybrid;
+    }
 
     browser_.ready =
         driver.manage().timeouts().setScriptTimeout(config.allScriptsTimeout);

spec/hybrid/async_spec.js

@@ -1,16 +1,10 @@
 describe('async angular1/2 hybrid using ngUpgrade application', function() {
   beforeEach(function() {
-    browser.ng12Hybrid = true;
-
     browser.get('/hybrid');
   });
 
-  afterEach(function() {
-    browser.ng12Hybrid = false;
-  });
-
-  it('should propertly load the page', function() {
-    expect($('h1').getText()).toEqual('My App');
+  it('should set browser flag via config', function() {
+    expect(browser.ng12Hybrid).toBe(true);
   });
 
   it('should be able to click buttons and wait for $timeout', function() {
@@ -29,4 +23,10 @@ describe('async angular1/2 hybrid using ngUpgrade application', function() {
     ng1Btn.click();
     expect(ng1Btn.getText()).toEqual('Click Count: 1');
   });
+
+  it('should use the flag on the browser object', function() {
+    browser.ng12Hybrid = false;
+    browser.get('/ng2'); // will time out if Protractor expects hybrid
+    browser.ng12Hybrid = true;
+  });
 });

spec/hybridConf.js

@@ -14,5 +14,7 @@ exports.config = {
 
   baseUrl: env.baseUrl,
 
-  rootElement: 'body'
+  rootElement: 'body',
+
+  ng12Hybrid: true
 };