Modify colors support

Author: ehmicky

README.md

@@ -47,7 +47,7 @@ logProcessErrors(options)
 - [`log` `{function}`](#custom-logging)
 - [`getLevel` `{function}`](#log-level)
 - [`getMessage` `{function}`](#log-message)
-- [`colors` `{boolean}`](#log-message) (default: `false`)
+- [`colors` `{boolean}`](#log-message) (default: `true`)
 - [`skipEvent` `{function}`](#skipping-events)
 - [`exitOn` `{string[]}`](#process-exit) (default: `['uncaughtException']`)
 

src/colors.js

@@ -1,24 +1,11 @@
 'use strict'
 
 const chalk = require('chalk')
-const { stdout: { level } = {} } = require('supports-color')
 
-const getColors = function({ opts }) {
-  const chalkOpts = getChalkOpts({ opts })
-  return chalk.constructor(chalkOpts)
-}
-
-// Can disable or force colors with `opts.colors`.
-// chalk will automatically disable colors if output does not support it.
-const getChalkOpts = function({ opts: { colors } }) {
-  if (colors === false) {
-    return { enabled: false }
-  }
-
-  // Chalk `level` must be set otherwise `enabled` will be noop if terminal
-  // does not support color (e.g. during a shell file redirection)
-  // However it should only be set if it is `undefined` or `0`
-  return { enabled: true, level: level || 2 }
+const getColors = function({ opts: { colors } }) {
+  // Can disable colors with `opts.colors`.
+  // chalk will automatically disable colors if output does not support it.
+  return chalk.constructor({ enabled: colors })
 }
 
 module.exports = {

src/options.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const { validate } = require('jest-validate')
-const supportsColor = require('supports-color')
 
 const { defaultGetLevel } = require('./level')
 const { defaultGetMessage } = require('./message')
@@ -23,7 +22,7 @@ const DEFAULT_OPTS = {
   getLevel: defaultGetLevel,
   getMessage: defaultGetMessage,
   log: defaultLog,
-  colors: Boolean(supportsColor.stdout),
+  colors: true,
   exitOn: ['uncaughtException'],
 }
 

test/colors.js

@@ -19,21 +19,6 @@ repeatEvents((prefix, { eventName, emitEvent }) => {
     stopLogging()
   })
 
-  test(`${prefix} should allow forcing colorizing default opts.getMessage()`, async t => {
-    const { stopLogging, log } = startLogging({
-      log: 'spy',
-      colors: true,
-      eventName,
-    })
-
-    await emitEvent()
-
-    t.true(log.calledOnce)
-    t.true(hasAnsi(log.firstCall.args[0]))
-
-    stopLogging()
-  })
-
   test(`${prefix} should allow disabling colorizing default opts.getMessage()`, async t => {
     const { stopLogging, log } = startLogging({
       log: 'spy',