Added colors to winston (#244)

* Added colors to winston

* Update logger.js

Author: Blackbaud-SteveBrush

cli/e2e.js

@@ -3,7 +3,7 @@
 
 const path = require('path');
 const spawn = require('cross-spawn');
-const logger = require('winston');
+const logger = require('../utils/logger');
 const portfinder = require('portfinder');
 const HttpServer = require('http-server');
 const selenium = require('selenium-standalone');

cli/serve.js

@@ -1,9 +1,9 @@
 /*jshint node: true*/
 'use strict';
 
-const logger = require('winston');
 const portfinder = require('portfinder');
 
+const logger = require('../utils/logger');
 const assetsProcessor = require('../lib/assets-processor');
 
 /**

cli/utils/run-compiler.js

@@ -1,7 +1,7 @@
 /*jshint node: true*/
 'use strict';
 
-const logger = require('winston');
+const logger = require('../../utils/logger');
 
 const runCompiler = (webpack, config) => {
   const compiler = webpack(config);

cli/version.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const path = require('path');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 /**
  * Returns the version from package.json.

config/sky-pages/sky-pages.config.js

@@ -4,7 +4,7 @@
 const fs = require('fs-extra');
 const path = require('path');
 const merge = require('merge');
-const logger = require('winston');
+const logger = require('../../utils/logger');
 
 /**
  * Resolves a path given a root path and an array-like arguments object.

config/webpack/serve.webpack.config.js

@@ -5,7 +5,7 @@ const fs = require('fs');
 const path = require('path');
 const util = require('util');
 const open = require('open');
-const logger = require('winston');
+const logger = require('../../utils/logger');
 const webpackMerge = require('webpack-merge');
 const NamedModulesPlugin = require('webpack/lib/NamedModulesPlugin');
 const LoaderOptionsPlugin = require('webpack/lib/LoaderOptionsPlugin');

index.js

@@ -1,9 +1,9 @@
 /*jshint node: true*/
 'use strict';
 
-const logger = require('winston');
 const webpack = require('webpack');
 const WebpackDevServer = require('webpack-dev-server');
+const logger = require('./utils/logger');
 const config = require('./config/sky-pages/sky-pages.config');
 
 module.exports = {

loader/sky-processor/index.js

@@ -1,7 +1,7 @@
 /*jshint node: true*/
 'use strict';
 
-const logger = require('winston');
+const logger = require('../../utils/logger');
 let plugins;
 
 const getPluginContents = (skyPagesConfig) => {

loader/sky-tslint/program.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const tslint = require('tslint');
-const logger = require('winston');
+const logger = require('../../utils/logger');
 let _program;
 
 const getProgram = (tsconfigPath) => {

test/cli-build-public-library.spec.js

@@ -4,7 +4,7 @@
 const mock = require('mock-require');
 const fs = require('fs-extra');
 const rimraf = require('rimraf');
-const logger = require('winston');
+const logger = require('../utils/logger');
 const skyPagesConfigUtil = require('../config/sky-pages/sky-pages.config');
 
 describe('cli build-public-library', () => {

test/cli-build.spec.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
+const logger = require('../utils/logger');
 const assetsProcessor = require('../lib/assets-processor');
 const runtimeUtils = require('../utils/runtime-test-utils');
 

test/cli-e2e.spec.js

@@ -3,8 +3,8 @@
 
 const path = require('path');
 const mock = require('mock-require');
-const logger = require('winston');
 const selenium = require('selenium-standalone');
+const logger = require('../utils/logger');
 
 describe('cli e2e', () => {
   const PORT = 1234;

test/cli-serve.spec.js

@@ -2,8 +2,8 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
 const portfinder = require('portfinder');
+const logger = require('../utils/logger');
 
 describe('cli serve', () => {
 

test/cli-utils-run-compiler.spec.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 describe('cli utils run compiler', () => {
   const requirePath = '../cli/utils/run-compiler';

test/cli-version.spec.js

@@ -3,7 +3,7 @@
 
 const path = require('path');
 const proxyquire = require('proxyquire');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 describe('cli version', () => {
   it('should return the version from package.json', () => {

test/config-sky-pages.spec.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const fs = require('fs-extra');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 describe('config sky-pages', () => {
 

test/config-webpack-serve.spec.js

@@ -2,8 +2,8 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
 const urlLibrary = require('url');
+const logger = require('../utils/logger');
 const runtimeUtils = require('../utils/runtime-test-utils');
 
 describe('config webpack serve', () => {

test/loader-processor.spec.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 describe('SKY UX processor Webpack loader', () => {
   const preloaderPath = '../loader/sky-processor/preload';

test/sky-pages-out-skyux2.spec.js

@@ -2,7 +2,7 @@
 'use strict';
 
 const mock = require('mock-require');
-const logger = require('winston');
+const logger = require('../utils/logger');
 
 describe('@blackbaud/skyux-builder', () => {
 

test/utils-logger.spec.js

@@ -0,0 +1,29 @@
+/*jshint jasmine: true, node: true */
+'use strict';
+
+const mock = require('mock-require');
+
+describe('logger', () => {
+  afterEach(() => {
+    mock.stopAll();
+  });
+
+  it('should configure a custom transport', () => {
+    let _transports;
+    let _colorize = false;
+    mock('winston', {
+      Logger: function (opts) {
+        _transports = opts.transports;
+      },
+      transports: {
+        Console: function (opts) {
+          _colorize = opts.colorize;
+        }
+      }
+    });
+
+    const logger = mock.reRequire('../utils/logger');
+    expect(_colorize).toEqual(true);
+    expect(_transports).toBeDefined();
+  });
+});

utils/logger.js

@@ -0,0 +1,17 @@
+/*jshint node: true */
+'use strict';
+
+const winston = require('winston');
+
+const logger = new winston.Logger({
+  transports: [
+    new winston.transports.Console({
+      level: 'debug',
+      handleExceptions: true,
+      json: false,
+      colorize: true
+    })
+  ]
+});
+
+module.exports = logger;