chore(test): use native promises instead of q on onPrepare

Author: cnishina

scripts/test.js

@@ -12,10 +12,10 @@ var passingTests = [
   // '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',
-  // 'node built/cli.js spec/onPreparePromiseFileConf.js',
+  'node built/cli.js spec/onPrepareConf.js',
+  'node built/cli.js spec/onPrepareFileConf.js',
+  'node built/cli.js spec/onPreparePromiseConf.js',
+  'node built/cli.js spec/onPreparePromiseFileConf.js',
   // 'node built/cli.js spec/mochaConf.js',
   // 'node built/cli.js spec/withLoginConf.js',
   // 'node built/cli.js spec/suitesConf.js --suite okmany',

spec/onPrepare/asyncstartup.js

@@ -1,5 +1,6 @@
-var q = require('q');
-
-module.exports = q.fcall(function() {
+module.exports = async() => {
   browser.params.password = '12345';
-}).delay(1000);
+  return await new Promise((resolve, _) => {
+    setTimeout(resolve, 1000);
+  });
+}

spec/onPrepare/onPrepare_spec.js

@@ -1,5 +1,5 @@
-describe('onPrepare function in the config', function() {
-  it('should have a special variable set in onPrepare', function() {
+describe('onPrepare function in the config', () => {
+  it('should have a special variable set in onPrepare', () => {
     expect(browser.params.password).toEqual('12345');
   });
 });

spec/onPrepareConf.js

@@ -1,10 +1,11 @@
 // Configuration using a function in onPrepare to set a parameter before
 // testing.
-var env = require('./environment.js');
+const env = require('./environment.js');
 
 // The main suite of Protractor tests.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 
@@ -16,7 +17,7 @@ exports.config = {
 
   baseUrl: env.baseUrl + '/ng1/',
 
-  onPrepare: function() {
+  onPrepare: () => {
     browser.params.password = '12345';
   }
 };

spec/onPrepareFileConf.js

@@ -1,9 +1,10 @@
-var env = require('./environment.js');
+const env = require('./environment.js');
 
 // Configuration using a string in onPrepare to load a file with code to
 // execute once before tests.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 

spec/onPreparePromiseConf.js

@@ -1,11 +1,12 @@
 // Configuration using a function in onPrepare to set a parameter before
 // testing.
-var env = require('./environment.js');
+const env = require('./environment.js');
 var q = require('q');
 
 // The main suite of Protractor tests.
 exports.config = {
   mockSelenium: true,
+  SELENIUM_PROMISE_MANAGER: false,
 
   framework: 'jasmine',
 
@@ -17,9 +18,10 @@ exports.config = {
 
   baseUrl: env.baseUrl + '/ng1/',
 
-  onPrepare: function() {
-    return q.fcall(function() {
-      browser.params.password = '12345';
-    }).delay(1000);
+  onPrepare: async() => {
+    browser.params.password = '12345';
+    return await new Promise((resolve, _) => {
+      setTimeout(resolve, 1000);
+    });
   }
 };