Change env in run_settings from string to object

Author: Karan Nagpal

bin/helpers/utils.js

@@ -269,13 +269,8 @@ exports.setUserSpecs = (bsConfig, args) => {
 exports.setTestEnvs = (bsConfig, args) => {
   let envKeys = {};
 
-  // set env vars which are defined in env key as a string
-  if(!this.isUndefined(bsConfig.run_settings.env)) {
-    let bstackJsonEnvVars = this.fixCommaSeparatedString(bsConfig.run_settings.env).split(',');
-    bstackJsonEnvVars.forEach((envVar) => {
-      let env = envVar.split("=");
-      envKeys[env[0]] = env[1];
-    });
+  if(bsConfig.run_settings.env && Object.keys(bsConfig.run_settings.env).length !== 0) {
+    envKeys = bsConfig.run_settings.env;
   }
 
   // set env vars which are defined in system_env_vars key

test/unit/bin/helpers/utils.js

@@ -525,7 +525,10 @@ describe('utils', () => {
     it('set env only from browserstack.json env param', () => {
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1, env2=value2',
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          }
         },
       };
       let args = {
@@ -540,7 +543,10 @@ describe('utils', () => {
       let argsEnv = 'env3=value3, env4=value4';
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1, env2=value2',
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          }
         },
       };
       let args = {
@@ -555,7 +561,10 @@ describe('utils', () => {
       let argsEnv = 'env1=value0, env4=value4';
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1, env2=value2',
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          }
         },
       };
       let args = {
@@ -570,7 +579,10 @@ describe('utils', () => {
       let argsEnv = 'env3=value3 , env4=value4';
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1 , env2=value2',
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          }
         },
       };
       let args = {
@@ -587,7 +599,10 @@ describe('utils', () => {
       let argsEnv = 'env3=value3 , env4=value4';
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1 , env2=value2',
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          },
           system_env_vars: ['ENV1', 'ENV2']
         },
       };
@@ -625,7 +640,10 @@ describe('utils', () => {
       let argsEnv = 'env3=value3 , env4=value4';
       let bsConfig = {
         run_settings: {
-          env: 'env1=value1 , env2=value2'
+          env: {
+            env1: 'value1',
+            env2: 'value2',
+          }
         },
       };
       let args = {