fix: fix pretty print

Byndyusoft · May 25, 2021 · e8da669 · e8da669
1 parent cca890a
commit e8da669
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 11 deletions.
diff --git a/src/providers/pino-logger-options-builder.ts b/src/providers/pino-logger-options-builder.ts
@@ -42,7 +42,9 @@ export class PinoLoggerOptionsBuilder {
         ? {
             translateTime: "SYS:STANDARD",
             colorize: true,
-            ignore: Object.keys(this._base).concat("hash").join(","),
+            ignore: Object.keys(this._base)
+              .concat("errHash", "msgTemplateHash")
+              .join(","),
             suppressFlushSyncWarning: true,
           }
         : false,

diff --git a/test/unit/__snapshots__/pino-logger-options-builder.spec.ts.snap b/test/unit/__snapshots__/pino-logger-options-builder.spec.ts.snap
@@ -125,8 +125,13 @@ Object {
 exports[`pino-logger-options-builder must log object 1`] = `
 Object {
   "a": 1,
+  "env": "test",
+  "hostname": "host",
   "level": "info",
+  "name": "name",
+  "pid": 123456,
   "time": "2016-04-05T17:02:19.796Z",
+  "version": "version",
 }
 `;
 
@@ -162,7 +167,13 @@ Object {
 
 exports[`pino-logger-options-builder must override level 1`] = `
 Object {
-  "base": Object {},
+  "base": Object {
+    "env": "test",
+    "hostname": "host",
+    "name": "name",
+    "pid": 123456,
+    "version": "version",
+  },
   "formatters": Object {
     "level": [Function],
   },
@@ -171,15 +182,27 @@ Object {
   },
   "level": "warn",
   "prettyPrint": false,
-  "redact": Array [],
-  "serializers": Object {},
+  "redact": Array [
+    "req.headers.authorization",
+  ],
+  "serializers": Object {
+    "err": [Function],
+    "req": [Function],
+    "res": [Function],
+  },
   "timestamp": [Function],
 }
 `;
 
 exports[`pino-logger-options-builder must override pretty print 1`] = `
 Object {
-  "base": Object {},
+  "base": Object {
+    "env": "test",
+    "hostname": "host",
+    "name": "name",
+    "pid": 123456,
+    "version": "version",
+  },
   "formatters": Object {
     "level": [Function],
   },
@@ -189,12 +212,18 @@ Object {
   "level": "info",
   "prettyPrint": Object {
     "colorize": true,
-    "ignore": "hash",
+    "ignore": "pid,hostname,name,version,env,errHash,msgTemplateHash",
     "suppressFlushSyncWarning": true,
     "translateTime": "SYS:STANDARD",
   },
-  "redact": Array [],
-  "serializers": Object {},
+  "redact": Array [
+    "req.headers.authorization",
+  ],
+  "serializers": Object {
+    "err": [Function],
+    "req": [Function],
+    "res": [Function],
+  },
   "timestamp": [Function],
 }
 `;

diff --git a/test/unit/pino-logger-options-builder.spec.ts b/test/unit/pino-logger-options-builder.spec.ts
@@ -78,7 +78,7 @@ describe("pino-logger-options-builder", () => {
   });
 
   it("must log object", () => {
-    const builder = new PinoLoggerOptionsBuilder(false);
+    const builder = new PinoLoggerOptionsBuilder();
 
     stdout.start();
     pino(builder.build()).info({a: 1});
@@ -108,13 +108,13 @@ describe("pino-logger-options-builder", () => {
   });
 
   it("must override level", () => {
-    const builder = new PinoLoggerOptionsBuilder(false);
+    const builder = new PinoLoggerOptionsBuilder();
 
     expect(builder.withLevel(LogLevel.warn).build()).toMatchSnapshot();
   });
 
   it("must override pretty print", () => {
-    const builder = new PinoLoggerOptionsBuilder(false);
+    const builder = new PinoLoggerOptionsBuilder();
 
     expect(builder.withPrettyPrint(true).build()).toMatchSnapshot();
   });