From 251a1beac0f6bafd344b43fc5dbda5532c8f9e07 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Linus=20Unneb=C3=A4ck?= Date: Fri, 24 Apr 2020 16:19:39 +0100 Subject: [PATCH] Remove trailing space in error message (#99) * Remove trailing space in error message * Make sure tests covers error message --- logger.js | 2 +- test.js | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/logger.js b/logger.js index fefcda5..e67e5b7 100644 --- a/logger.js +++ b/logger.js @@ -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 diff --git a/test.js b/test.js index c33f642..b227b1c 100644 --- a/test.js +++ b/test.js @@ -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() }) })