chore(test): move noGlobals/noGlobals_spec off of the control flow

scripts/test.js

@@ -36,7 +36,7 @@ var passingTests = [
  'node built/cli.js spec/getCapabilitiesConf.js',
  'node built/cli.js spec/controlLockConf.js',
  'node built/cli.js spec/customFramework.js',
- // 'node built/cli.js spec/noGlobalsConf.js',
+ 'node built/cli.js spec/noGlobalsConf.js',
  // 'node built/cli.js spec/angular2Conf.js',
  'node built/cli.js spec/hybridConf.js',
  'node built/cli.js spec/built/noCFBasicConf.js',

spec/noGlobals/noGlobals_spec.js

@@ -1,7 +1,7 @@
-describe('configuration with no globals', function() {
- var URL = '/ng2/#/async';
+describe('configuration with no globals', () => {
+ const URL = '/ng2/#/async';
 
- it('should have objects belonging to protractor namespace', function() {
+ it('should have objects belonging to protractor namespace', () => {
  expect(typeof protractor).toEqual('object');
  expect(typeof protractor.browser).toEqual('object');
  expect(typeof protractor.$).toEqual('function');
@@ -11,7 +11,7 @@ describe('configuration with no globals', function() {
  expect(typeof protractor.By).toEqual('object');
  });
 
- it('should not have other globals', function() {
+ it('should not have other globals', () => {
  expect(typeof browser).toEqual('undefined');
  expect(typeof $).toEqual('undefined');
  expect(typeof $$).toEqual('undefined');
@@ -20,11 +20,11 @@ describe('configuration with no globals', function() {
  expect(typeof By).toEqual('undefined');
  });
 
- it('should be able to use methods under the protractor namespace', function() {
- protractor.browser.get(URL);
- var increment = protractor.$('#increment');
+ it('should be able to use methods under the protractor namespace', async () => {
+ await protractor.browser.get(URL);
+ const increment = protractor.$('#increment');
  expect(typeof increment).toEqual('object');
- increment.$('.action').click();
- expect(increment.$('.val').getText()).toEqual('1');
+ await increment.$('.action').click();
+ expect(await increment.$('.val').getText()).toEqual('1');
  });
 });

spec/noGlobalsConf.js

@@ -1,9 +1,9 @@
-var env = require('./environment');
+const env = require('./environment');
 
 // This is the configuration for a smoke test for an Angular2 application.
 exports.config = {
-
  seleniumAddress: env.seleniumAddress,
+ SELENIUM_PROMISE_MANAGER: false,
 
  framework: 'jasmine',