chore(test): use native promises instead of q onCleanUp

scripts/test.js

@@ -7,14 +7,11 @@ var passingTests = [
   'node built/cli.js spec/basicConf.js',
   // 'node built/cli.js spec/basicConf.js --useBlockingProxy',
   'node built/cli.js spec/multiConf.js',
-  // 'node built/cli.js spec/altRootConf.js',
-  // 'node built/cli.js spec/inferRootConf.js',
-  // 'node built/cli.js spec/multiConf.js',
   'node built/cli.js spec/altRootConf.js',
   'node built/cli.js spec/inferRootConf.js',
-  // 'node built/cli.js spec/onCleanUpAsyncReturnValueConf.js',
-  // 'node built/cli.js spec/onCleanUpNoReturnValueConf.js',
-  // 'node built/cli.js spec/onCleanUpSyncReturnValueConf.js',
+  'node built/cli.js spec/onCleanUpAsyncReturnValueConf.js',
+  'node built/cli.js spec/onCleanUpNoReturnValueConf.js',
+  'node built/cli.js spec/onCleanUpSyncReturnValueConf.js',
   'node built/cli.js spec/onPrepareConf.js',
   'node built/cli.js spec/onPrepareFileConf.js',
   'node built/cli.js spec/onPreparePromiseConf.js',

spec/onCleanUp/onCleanUp_spec.js

@@ -1,5 +1,5 @@
-describe('onCleanUp function in the config', function() {
-  it('should not be affected by tests', function() {
+describe('onCleanUp function in the config', () => {
+  it('should not be affected by tests', () => {
     expect(true).toBe(true);
   });
 });

spec/onCleanUpAsyncReturnValueConf.js

@@ -1,9 +1,9 @@
-var env = require('./environment.js');
-var q = require('q');
+const env = require('./environment.js');
 
 // Test that onCleanUp actions are performed.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 
@@ -15,11 +15,9 @@ exports.config = {
 
   baseUrl: env.baseUrl + '/ng1/',
 
-  onCleanUp: function(exitCode) {
-    var deferred = q.defer();
-    setTimeout(function() {
-      deferred.resolve(exitCode);
-    }, 500);
-    return deferred.promise;
+  onCleanUp: async(exitCode) => {
+    return await new Promise(resolve => {
+      setTimeout(resolve(exitCode), 500);
+    });
   }
 };

spec/onCleanUpNoReturnValueConf.js

@@ -1,8 +1,9 @@
-var env = require('./environment.js');
+const env = require('./environment.js');
 
 // Test that onCleanUp actions are performed.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 
@@ -14,7 +15,7 @@ exports.config = {
 
   baseUrl: env.baseUrl + '/ng1/',
 
-  onCleanUp: function(/* exitCode */) {
+  onCleanUp: (/* exitCode */) => {
     // no return
   }
 };

spec/onCleanUpSyncReturnValueConf.js

@@ -1,8 +1,9 @@
-var env = require('./environment.js');
+const env = require('./environment.js');
 
 // Test that onCleanUp actions are performed.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 
@@ -14,7 +15,7 @@ exports.config = {
 
   baseUrl: env.baseUrl + '/ng1/',
 
-  onCleanUp: function(exitCode) {
+  onCleanUp: (exitCode) => {
     return exitCode;
   }
 };