From 6c1942923c4013b7c1e0809251f3a8d6938eda04 Mon Sep 17 00:00:00 2001 From: Marcel Overdijk Date: Sat, 31 Aug 2024 04:13:09 +0200 Subject: [PATCH] fix(logger): removing spaces from logger (#3334) --- src/middleware/logger/index.test.ts | 20 ++++++++++---------- src/middleware/logger/index.ts | 4 ++-- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/src/middleware/logger/index.test.ts b/src/middleware/logger/index.test.ts index fbb37973f..efc812544 100644 --- a/src/middleware/logger/index.test.ts +++ b/src/middleware/logger/index.test.ts @@ -34,7 +34,7 @@ describe('Logger by Middleware', () => { const res = await app.request('http://localhost/empty') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /empty \x1b[32m200\x1b[0m')).toBe(true) + expect(log.startsWith('--> GET /empty \x1b[32m200\x1b[0m')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -42,7 +42,7 @@ describe('Logger by Middleware', () => { const res = await app.request('http://localhost/short') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /short \x1b[32m200\x1b[0m')).toBe(true) + expect(log.startsWith('--> GET /short \x1b[32m200\x1b[0m')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -50,7 +50,7 @@ describe('Logger by Middleware', () => { const res = await app.request('http://localhost/long') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /long \x1b[32m200\x1b[0m')).toBe(true) + expect(log.startsWith('--> GET /long \x1b[32m200\x1b[0m')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -58,7 +58,7 @@ describe('Logger by Middleware', () => { const res = await app.request('http://localhost/seconds') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /seconds \x1b[32m200\x1b[0m')).toBe(true) + expect(log.startsWith('--> GET /seconds \x1b[32m200\x1b[0m')).toBe(true) expect(log).toMatch(/1s/) }) @@ -70,7 +70,7 @@ describe('Logger by Middleware', () => { const res = await app.request('http://localhost/notfound') expect(res).not.toBeNull() expect(res.status).toBe(404) - expect(log.startsWith(' --> GET /notfound \x1b[33m404\x1b[0m')).toBe(true) + expect(log.startsWith('--> GET /notfound \x1b[33m404\x1b[0m')).toBe(true) expect(log).toMatch(/m?s$/) }) }) @@ -111,7 +111,7 @@ describe('Logger by Middleware in NO_COLOR', () => { const res = await app.request('http://localhost/empty') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /empty 200')).toBe(true) + expect(log.startsWith('--> GET /empty 200')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -119,7 +119,7 @@ describe('Logger by Middleware in NO_COLOR', () => { const res = await app.request('http://localhost/short') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /short 200')).toBe(true) + expect(log.startsWith('--> GET /short 200')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -127,7 +127,7 @@ describe('Logger by Middleware in NO_COLOR', () => { const res = await app.request('http://localhost/long') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /long 200')).toBe(true) + expect(log.startsWith('--> GET /long 200')).toBe(true) expect(log).toMatch(/m?s$/) }) @@ -135,7 +135,7 @@ describe('Logger by Middleware in NO_COLOR', () => { const res = await app.request('http://localhost/seconds') expect(res).not.toBeNull() expect(res.status).toBe(200) - expect(log.startsWith(' --> GET /seconds 200')).toBe(true) + expect(log.startsWith('--> GET /seconds 200')).toBe(true) expect(log).toMatch(/1s/) }) @@ -147,7 +147,7 @@ describe('Logger by Middleware in NO_COLOR', () => { const res = await app.request('http://localhost/notfound') expect(res).not.toBeNull() expect(res.status).toBe(404) - expect(log.startsWith(' --> GET /notfound 404')).toBe(true) + expect(log.startsWith('--> GET /notfound 404')).toBe(true) expect(log).toMatch(/m?s$/) }) }) diff --git a/src/middleware/logger/index.ts b/src/middleware/logger/index.ts index e9a008d5d..45acd2f9b 100644 --- a/src/middleware/logger/index.ts +++ b/src/middleware/logger/index.ts @@ -55,8 +55,8 @@ function log( ) { const out = prefix === LogPrefix.Incoming - ? ` ${prefix} ${method} ${path}` - : ` ${prefix} ${method} ${path} ${colorStatus(status)} ${elapsed}` + ? `${prefix} ${method} ${path}` + : `${prefix} ${method} ${path} ${colorStatus(status)} ${elapsed}` fn(out) }