Fix linting

Author: ehmicky

src/exit.js

@@ -32,8 +32,8 @@ export const exitProcess = function ({ name, opts: { exitOn } }) {
   }, EXIT_TIMEOUT)
 }
 
-export const EXIT_TIMEOUT = 3000
-export const EXIT_STATUS = 1
+const EXIT_TIMEOUT = 3000
+const EXIT_STATUS = 1
 
 export const validateExitOn = function ({ exitOn }) {
   if (exitOn === undefined) {

src/level.js

@@ -2,8 +2,9 @@ import { emitWarning } from 'process'
 
 import filterObj from 'filter-obj'
 import { multipleValidOptions } from 'jest-validate'
+import mapObj from 'map-obj'
 
-import { result, mapValues } from './utils.js'
+import { result } from './utils.js'
 
 // Retrieve error's log level
 export const getLevel = function ({ opts, name, error }) {
@@ -36,7 +37,10 @@ export const applyDefaultLevels = function ({
     return { ...DEFAULT_LEVEL, ...levelA }
   }
 
-  const defaultLevels = mapValues(DEFAULT_LEVEL, () => defaultLevel)
+  const defaultLevels = mapObj(DEFAULT_LEVEL, (eventName) => [
+    eventName,
+    defaultLevel,
+  ])
   return { ...defaultLevels, ...levelA }
 }
 
@@ -46,12 +50,12 @@ const isDefined = function (key, value) {
 
 // Use during options validation
 export const getExampleLevels = function () {
-  return mapValues(DEFAULT_LEVEL, getExampleLevel)
+  return mapObj(DEFAULT_LEVEL, getExampleLevel)
 }
 
-const getExampleLevel = function (level) {
+const getExampleLevel = function (eventName, level) {
   // eslint-disable-next-line no-empty-function
-  return multipleValidOptions(level, () => {})
+  return [eventName, multipleValidOptions(level, () => {})]
 }
 
 export const validateLevels = function ({ level }) {
@@ -77,7 +81,7 @@ const isValidLevel = function ({ level }) {
 const LEVELS_ARR = ['debug', 'info', 'warn', 'error', 'silent', 'default']
 const LEVELS = new Set(LEVELS_ARR)
 
-export const DEFAULT_LEVEL = {
+const DEFAULT_LEVEL = {
   default: 'error',
   uncaughtException: 'error',
   warning: 'warn',

src/limit.js

@@ -47,4 +47,4 @@ const ERROR_MESSAGE = (name) =>
 const ERROR_NAME = 'LogProcessErrors'
 const ERROR_CODE = 'TooManyErrors'
 
-export const MAX_EVENTS = 100
+const MAX_EVENTS = 100

src/utils.js

@@ -1,10 +1,3 @@
-import mapObj from 'map-obj'
-
-// Like Lodash mapValues()
-export const mapValues = function (object, mapper) {
-  return mapObj(object, (key, value) => [key, mapper(value, key)])
-}
-
 // Like Lodash result(), but faster
 export const result = function (val, ...args) {
   if (typeof val !== 'function') {

test/exit.js

@@ -6,16 +6,15 @@ import test from 'ava'
 import sinon from 'sinon'
 import { each } from 'test-each'
 
-// Required directly because this is exposed through documentation, but not
-// through code
-import { EXIT_TIMEOUT, EXIT_STATUS } from '../src/exit.js'
-
 import { EVENTS } from './helpers/events/main.js'
 import { startLogging } from './helpers/init.js'
 import { removeProcessListeners } from './helpers/remove.js'
 
 const pNextTick = promisify(process.nextTick)
 
+const EXIT_TIMEOUT = 3000
+const EXIT_STATUS = 1
+
 removeProcessListeners()
 
 // Stub `process.exit()`

test/helpers/events/main.js

@@ -1,7 +1,4 @@
-// Required directly because this is exposed through documentation, but not
-// through code
-import { DEFAULT_LEVEL } from '../../../src/level.js'
-import { mapValues } from '../../../src/utils.js'
+import mapObj from 'map-obj'
 
 import { multipleResolves } from './multiple_resolves.js'
 import { rejectionHandled } from './rejection_handled.js'
@@ -10,7 +7,7 @@ import { unhandledRejection } from './unhandled_rejection.js'
 import { warning } from './warning.js'
 
 const getEventsMap = function () {
-  return mapValues(EVENTS_SIMPLE_MAP, getEvent)
+  return mapObj(EVENTS_SIMPLE_MAP, getEvent)
 }
 
 const EVENTS_SIMPLE_MAP = {
@@ -21,10 +18,13 @@ const EVENTS_SIMPLE_MAP = {
   warning,
 }
 
-const getEvent = function (emit, eventName) {
+const getEvent = function (eventName, emit) {
   const emitMany = emitEvents.bind(null, emit)
   const defaultLevel = DEFAULT_LEVEL[eventName]
-  return { title: eventName, eventName, emit, emitMany, defaultLevel }
+  return [
+    eventName,
+    { title: eventName, eventName, emit, emitMany, defaultLevel },
+  ]
 }
 
 // Emit several emits in parallel
@@ -33,6 +33,15 @@ export const emitEvents = async function (emit, maxEvents) {
   await Promise.all(array)
 }
 
+const DEFAULT_LEVEL = {
+  default: 'error',
+  uncaughtException: 'error',
+  warning: 'warn',
+  unhandledRejection: 'error',
+  rejectionHandled: 'error',
+  multipleResolves: 'info',
+}
+
 // Map of all possible events, with related information and helper methods
 export const EVENTS_MAP = getEventsMap()
 

test/helpers/init.js

@@ -1,7 +1,7 @@
+import mapObj from 'map-obj'
 import sinon from 'sinon'
 
 import logProcessErrors from '../../src/main.js'
-import { mapValues } from '../../src/utils.js'
 
 // Call `logProcessErrors()` then return spied objects and `stopLogging()`
 export const startLogging = function ({ eventName, log, level, ...opts } = {}) {
@@ -45,7 +45,10 @@ const getLevel = function ({ level, eventName }) {
 
   const levelA = level === undefined ? { default: 'default' } : level
 
-  return mapValues(levelA, (levelB) => onlyEvent.bind(null, levelB, eventName))
+  return mapObj(levelA, (key, levelB) => [
+    key,
+    onlyEvent.bind(null, levelB, eventName),
+  ])
 }
 
 const onlyEvent = function (level, eventName, error) {

test/limit.js

@@ -1,15 +1,13 @@
 import test from 'ava'
 import { each } from 'test-each'
 
-// Required directly because this is exposed through documentation, but not
-// through code
-import { MAX_EVENTS } from '../src/limit.js'
-
 import { EVENTS } from './helpers/events/main.js'
 import { startLogging } from './helpers/init.js'
 import { removeProcessListeners } from './helpers/remove.js'
 import { stubStackTraceRandom, unstubStackTrace } from './helpers/stack.js'
 
+const MAX_EVENTS = 100
+
 removeProcessListeners()
 
 each(EVENTS, ({ title }, { eventName, emit, emitMany }) => {