Stop clearing console when testing

Shopify · Sep 6, 2018 · 6b81586 · 6b81586
1 parent 6725d30
commit 6b81586
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 4 deletions.
diff --git a/packages/slate-analytics/index.js b/packages/slate-analytics/index.js
@@ -49,7 +49,12 @@ async function init() {
       event('slate-analytics:renew-consent-true', config);
     }
 
-    clearConsole();
+    if (
+      process.env.NODE_ENV !== 'test' && // eslint-disable-line no-process-env
+      process.env.NODE_ENV !== 'test-slate-analytics' // eslint-disable-line no-process-env
+    ) {
+      clearConsole();
+    }
     console.log(`Thanks for helping improve the Slate development experience!`);
   }
 

diff --git a/packages/slate-analytics/prompt.js b/packages/slate-analytics/prompt.js
@@ -14,7 +14,11 @@ const question = {
 };
 
 function forNewConsent() {
-  clearConsole();
+  // eslint-disable-next-line no-process-env
+  if (process.env.NODE_ENV !== 'test') {
+    clearConsole();
+  }
+
   console.log(
     wrap(
       '👋  Welcome to Slate! During the alpha, we would like to gather usage analytics, such as interactions with Slate commands, performance reports, and error occurances. The data does not include any sensitive information. The detailed list of data we gather can be found at:',

diff --git a/packages/slate-tools/cli/commands/start.js b/packages/slate-tools/cli/commands/start.js
@@ -66,15 +66,22 @@ Promise.all([
   });
 
 function onCompilerCompile() {
-  clearConsole();
+  // eslint-disable-next-line no-process-env
+  if (process.env.NODE_ENV !== 'test') {
+    clearConsole();
+  }
   spinner.start();
 }
 
 function onCompilerDone(stats) {
   const statsJson = stats.toJson({}, true);
 
   spinner.stop();
-  clearConsole();
+
+  // eslint-disable-next-line no-process-env
+  if (process.env.NODE_ENV !== 'test') {
+    clearConsole();
+  }
 
   if (statsJson.errors.length) {
     event('slate-tools:start:compile-errors', {