Merge pull request #86 from BitGo/WP-5365-hide-logs-test-runs

chore(mbe): hide winston/http logs in test runs

Author: pranavjain97

package.json

@@ -10,7 +10,7 @@
   "scripts": {
     "start": "nodemon",
     "build": "yarn tsc --build --incremental --verbose . && cp package.json dist/",
-    "test": "mocha --require ts-node/register 'src/**/__tests__/**/*.test.ts'",
+    "test": "NODE_ENV=test mocha --require ts-node/register 'src/**/__tests__/**/*.test.ts'",
     "test:watch": "mocha --require ts-node/register --watch 'src/**/__tests__/**/*.test.ts'",
     "test:coverage": "nyc mocha --require ts-node/register 'src/**/__tests__/**/*.test.ts'",
     "lint": "eslint --quiet --ignore-pattern scripts/bump-version.ts .",

src/logger.ts

@@ -11,6 +11,7 @@ const levels = {
 
 // Define level based on environment
 const level = () => {
+  if (process.env.NODE_ENV === 'test' || process.env.LOG_LEVEL === 'silent') return 'silent';
   const env = process.env.NODE_ENV || 'development';
   const isDevelopment = env === 'development';
   return isDevelopment ? 'debug' : 'warn';

src/shared/appUtils.ts

@@ -16,6 +16,9 @@ import { Config, TlsMode } from '../shared/types';
  */
 export function setupLogging(app: express.Application, config: Config): void {
   // Set up morgan for logging, with optional logging into a file
+  if (process.env.NODE_ENV === 'test' || process.env.LOG_LEVEL === 'silent') {
+    return;
+  }
   let middleware;
   if (config.logFile) {
     // create a write stream (in append mode)