Skip to content

Commit cc7231f

Browse files
committed
Add opts.log()
1 parent aea3e47 commit cc7231f

File tree

5 files changed

+19
-11
lines changed

5 files changed

+19
-11
lines changed

src/default.js

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,12 @@
22

33
const { defaultGetLevel } = require('./level')
44
const { defaultGetMessage } = require('./message')
5-
6-
// Default `opts.handlerFunc`
7-
const defaultHandler = function(message, level) {
8-
// eslint-disable-next-line no-restricted-globals, no-console
9-
console[level](message)
10-
}
5+
const { defaultLog } = require('./log')
116

127
const DEFAULT_OPTS = {
138
getLevel: defaultGetLevel,
149
getMessage: defaultGetMessage,
15-
handlerFunc: defaultHandler,
10+
log: defaultLog,
1611
exitOnExceptions: true,
1712
}
1813

src/handle.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ const { getMessage } = require('./message')
99
// Generic event handler for all events.
1010
const handleEvent = async function({
1111
opts,
12-
opts: { handlerFunc, exitOnExceptions },
1312
eventName,
1413
error,
1514
promise,
@@ -29,15 +28,15 @@ const handleEvent = async function({
2928
const level = getLevel({ opts, info })
3029
const message = getMessage({ opts, info, level })
3130

32-
handlerFunc(message, level, info)
31+
opts.log(message, level, info)
3332

34-
exitProcess({ eventName, exitOnExceptions })
33+
exitProcess({ eventName, opts })
3534
}
3635

3736
// Exit process on `uncaughtException`
3837
// See https://nodejs.org/api/process.html#process_warning_using_uncaughtexception_correctly
3938
// Can be disabled with `opts.exitOnExceptions: false`
40-
const exitProcess = function({ eventName, exitOnExceptions }) {
39+
const exitProcess = function({ eventName, opts: { exitOnExceptions } }) {
4140
if (eventName !== 'uncaughtException' || !exitOnExceptions) {
4241
return
4342
}

src/level.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ const { platform } = require('process')
44

55
const { red, yellow } = require('chalk')
66

7+
// Retrieve error's `level`
78
const getLevel = function({ opts, info }) {
89
const level = opts.getLevel(info)
910

@@ -29,6 +30,7 @@ const LEVELS = {
2930
},
3031
}
3132

33+
// Default `opts.getLevel()`
3234
const defaultGetLevel = function({ eventName }) {
3335
if (eventName === 'warning') {
3436
return 'warn'

src/log.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
'use strict'
2+
3+
// Default `opts.log`
4+
const defaultLog = function(message, level) {
5+
// eslint-disable-next-line no-restricted-globals, no-console
6+
console[level](message)
7+
}
8+
9+
module.exports = {
10+
defaultLog,
11+
}

src/message.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ const getMessage = function({ opts, info, level }) {
1313
return messageA
1414
}
1515

16+
// Default `opts.getMessage()`
1617
const defaultGetMessage = function({
1718
eventName,
1819
promiseState,

0 commit comments

Comments
 (0)