Printing environment variables before jest test command (#852)

wix · Jul 25, 2018 · dff3082 · dff3082
1 parent 1daa65d
commit dff3082
Showing 1 changed file with 18 additions and 5 deletions.
diff --git a/detox/local-cli/detox-test.js b/detox/local-cli/detox-test.js
@@ -129,7 +129,7 @@ function runJest() {
   const platformString = platform ? shellQuote(`--testNamePattern=^((?!${getPlatformSpecificString(platform)}).)*$`) : '';
   const binPath = path.join('node_modules', '.bin', 'jest');
   const command = `${binPath} ${testFolder} ${configFile} --maxWorkers=${program.workers} ${platformString}`;
-  const env = Object.assign({}, process.env, {
+  const detoxEnvironmentVariables = {
     configuration: program.configuration,
     loglevel: program.loglevel,
     cleanup: program.cleanup,
@@ -140,15 +140,28 @@ function runJest() {
     recordLogs: program.recordLogs,
     takeScreenshots: program.takeScreenshots,
     recordVideos: program.recordVideos,
-  });
+  };
 
-  console.log(command);
+  console.log(printEnvironmentVariables(detoxEnvironmentVariables) + command);
   cp.execSync(command, {
     stdio: 'inherit',
-    env,
+    env: {
+      ...process.env,
+      ...detoxEnvironmentVariables,
+    },
   });
 }
 
+function printEnvironmentVariables(envObject) {
+  return Object.entries(envObject).reduce((cli, [key, value]) => {
+    if (value === null || value === undefined || value === '') {
+      return cli;
+    }
+
+    return `${cli}${key}=${JSON.stringify(value)} `;
+  }, '');
+}
+
 function getDefaultRunnerConfig() {
   let defaultConfig;
   switch (runner) {
@@ -193,4 +206,4 @@ function shellQuote(input) {
   return process.platform !== 'win32' ? `'${input}'` : `"${input}"`;
 }
 
-run();
+run();