Skip to content

Commit

Permalink
Remove trailing space in error message (#99)
Browse files Browse the repository at this point in the history
* Remove trailing space in error message

* Make sure tests covers error message
  • Loading branch information
LinusU committed Apr 24, 2020
1 parent 9d17dce commit 251a1be
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion logger.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ function pinoLogger (opts, stream) {
delete opts.autoLogging

var successMessage = opts.customSuccessMessage || function () { return 'request completed' }
var errorMessage = opts.customErrorMessage || function () { return 'request errored ' }
var errorMessage = opts.customErrorMessage || function () { return 'request errored' }
delete opts.customSuccessfulMessage
delete opts.customErroredMessage

Expand Down
1 change: 1 addition & 0 deletions test.js
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,7 @@ test('responseTime for errored request', function (t) {

dest.on('data', function (line) {
t.ok(line.responseTime >= 0, 'responseTime is defined')
t.equal(line.msg, 'request errored', 'message is set')
t.end()
})
})
Expand Down

0 comments on commit 251a1be

Please sign in to comment.