From c5a6e38adfff020dbe42e7f0ae036d504d45d288 Mon Sep 17 00:00:00 2001
From: shmck <shawn.j.mckay@gmail.com>
Date: Sun, 12 Jan 2020 19:36:40 -0800
Subject: [PATCH 1/2] setup logger and environment

---
 src/channel/index.ts             |  3 ++-
 src/environment.ts               |  9 +++++++++
 src/services/git/index.ts        |  3 ++-
 src/services/logger/index.ts     | 14 ++++++++++++++
 src/services/testRunner/index.ts |  7 +++++--
 5 files changed, 32 insertions(+), 4 deletions(-)
 create mode 100644 src/environment.ts
 create mode 100644 src/services/logger/index.ts

diff --git a/src/channel/index.ts b/src/channel/index.ts
index db69db47..8284b269 100644
--- a/src/channel/index.ts
+++ b/src/channel/index.ts
@@ -6,6 +6,7 @@ import setupActions from '../actions/setupActions'
 import solutionActions from '../actions/solutionActions'
 import tutorialConfig from '../actions/tutorialConfig'
 import { COMMANDS } from '../editor/commands'
+import logger from '../services/logger'
 import Context from './context'
 
 interface Channel {
@@ -120,7 +121,7 @@ class Channel implements Channel {
         return
 
       default:
-        console.log(`No match for action type: ${actionType}`)
+        logger(`No match for action type: ${actionType}`)
         return
     }
   }
diff --git a/src/environment.ts b/src/environment.ts
new file mode 100644
index 00000000..ab539f9f
--- /dev/null
+++ b/src/environment.ts
@@ -0,0 +1,9 @@
+interface Environment {
+  LOG: boolean
+}
+
+const environment: Environment = {
+  LOG: (process.env.LOG || '').toLowerCase() === 'test',
+}
+
+export default environment
diff --git a/src/services/git/index.ts b/src/services/git/index.ts
index 971678bc..4f58ac5d 100644
--- a/src/services/git/index.ts
+++ b/src/services/git/index.ts
@@ -1,4 +1,5 @@
 import node from '../node'
+import logger from '../logger'
 
 const gitOrigin = 'coderoad'
 
@@ -48,7 +49,7 @@ export async function saveCommit(message: string): Promise<void> {
     console.error(stderr)
     throw new Error('Error saving progress to Git')
   }
-  console.log('save with commit & continue stdout', stdout)
+  logger(['save with commit & continue stdout', stdout])
 }
 
 export async function clear(): Promise<void> {
diff --git a/src/services/logger/index.ts b/src/services/logger/index.ts
new file mode 100644
index 00000000..42310b89
--- /dev/null
+++ b/src/services/logger/index.ts
@@ -0,0 +1,14 @@
+import enviroment from '../../environment'
+
+const logger = (message: string | string[]) => {
+  if (!enviroment.LOG) {
+    return
+  }
+  if (Array.isArray(message)) {
+    message.forEach(console.log)
+  } else {
+    console.log(message)
+  }
+}
+
+export default logger
diff --git a/src/services/testRunner/index.ts b/src/services/testRunner/index.ts
index 0f75dc86..20b507a0 100644
--- a/src/services/testRunner/index.ts
+++ b/src/services/testRunner/index.ts
@@ -1,5 +1,6 @@
 import { getOutputChannel } from '../../editor/outputChannel'
 import node from '../../services/node'
+import logger from '../../services/logger'
 import parser from './parser'
 import { debounce, throttle } from './throttle'
 
@@ -27,7 +28,8 @@ const createTestRunner = (config: TestRunnerConfig, callbacks: Callbacks) => {
     if (!startTime) {
       return
     }
-    console.log('------------------- RUN TEST -------------------')
+
+    logger('------------------- RUN TEST -------------------')
 
     // flag as running
     callbacks.onRun(payload)
@@ -44,7 +46,8 @@ const createTestRunner = (config: TestRunnerConfig, callbacks: Callbacks) => {
     if (!debounce(startTime)) {
       return
     }
-    console.log('----------------- PROCESS TEST -----------------')
+
+    logger('----------------- PROCESS TEST -----------------')
 
     const { stdout, stderr } = result
 

From 8608dcc708868a1a7891d263ec7bb3fa4dd81330 Mon Sep 17 00:00:00 2001
From: shmck <shawn.j.mckay@gmail.com>
Date: Sun, 12 Jan 2020 19:38:45 -0800
Subject: [PATCH 2/2] setup dotenv for process.env

---
 src/environment.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/environment.ts b/src/environment.ts
index ab539f9f..8e9c75d0 100644
--- a/src/environment.ts
+++ b/src/environment.ts
@@ -1,3 +1,7 @@
+require('dotenv').config({
+  path: './.env',
+})
+
 interface Environment {
   LOG: boolean
 }