test console writer in bun too

Author: enisdenjo

packages/logger/tests/__snapshots__/writers.test.ts.snap

@@ -2,96 +2,21 @@
 
 exports[`ConsoleLogWriter should color levels and keys 1`] = `
 [
-  "TRC hi 
-  hello: {
-    dear: "world"
-    try: [
-      "num"
-      1
-      2
-    ]
-  }",
-  "DBG hi 
-  hello: {
-    dear: "world"
-    try: [
-      "num"
-      1
-      2
-    ]
-  }",
-  "INF hi 
-  hello: {
-    dear: "world"
-    try: [
-      "num"
-      1
-      2
-    ]
-  }",
-  "WRN hi 
-  hello: {
-    dear: "world"
-    try: [
-      "num"
-      1
-      2
-    ]
-  }",
-  "ERR hi 
-  hello: {
-    dear: "world"
-    try: [
-      "num"
-      1
-      2
-    ]
-  }",
+  ""\\u001b[36mTRC\\u001b[0m \\u001b[1mhi\\u001b[0m \\n  \\u001b[35mhello:\\u001b[0m \\u001b[35m{\\u001b[0m\\n    \\u001b[35mdear:\\u001b[0m \\"world\\"\\n    \\u001b[35mtry:\\u001b[0m \\u001b[35m[\\u001b[0m\\n      \\"num\\"\\n      1\\n      2\\n    \\u001b[35m]\\u001b[0m\\n  \\u001b[35m}\\u001b[0m"",
+  ""\\u001b[90mDBG\\u001b[0m \\u001b[1mhi\\u001b[0m \\n  \\u001b[35mhello:\\u001b[0m \\u001b[35m{\\u001b[0m\\n    \\u001b[35mdear:\\u001b[0m \\"world\\"\\n    \\u001b[35mtry:\\u001b[0m \\u001b[35m[\\u001b[0m\\n      \\"num\\"\\n      1\\n      2\\n    \\u001b[35m]\\u001b[0m\\n  \\u001b[35m}\\u001b[0m"",
+  ""\\u001b[32mINF\\u001b[0m \\u001b[1mhi\\u001b[0m \\n  \\u001b[35mhello:\\u001b[0m \\u001b[35m{\\u001b[0m\\n    \\u001b[35mdear:\\u001b[0m \\"world\\"\\n    \\u001b[35mtry:\\u001b[0m \\u001b[35m[\\u001b[0m\\n      \\"num\\"\\n      1\\n      2\\n    \\u001b[35m]\\u001b[0m\\n  \\u001b[35m}\\u001b[0m"",
+  ""\\u001b[33mWRN\\u001b[0m \\u001b[1mhi\\u001b[0m \\n  \\u001b[35mhello:\\u001b[0m \\u001b[35m{\\u001b[0m\\n    \\u001b[35mdear:\\u001b[0m \\"world\\"\\n    \\u001b[35mtry:\\u001b[0m \\u001b[35m[\\u001b[0m\\n      \\"num\\"\\n      1\\n      2\\n    \\u001b[35m]\\u001b[0m\\n  \\u001b[35m}\\u001b[0m"",
+  ""\\u001b[41;39mERR\\u001b[0m \\u001b[1mhi\\u001b[0m \\n  \\u001b[35mhello:\\u001b[0m \\u001b[35m{\\u001b[0m\\n    \\u001b[35mdear:\\u001b[0m \\"world\\"\\n    \\u001b[35mtry:\\u001b[0m \\u001b[35m[\\u001b[0m\\n      \\"num\\"\\n      1\\n      2\\n    \\u001b[35m]\\u001b[0m\\n  \\u001b[35m}\\u001b[0m"",
 ]
 `;
 
 exports[`ConsoleLogWriter should pretty print the attributes 1`] = `
 [
-  "TRC obj 
-  a: 1
-  b: 2",
-  "DBG arr 
-  "a"
-  "b"
-  "c"",
-  "INF nested 
-  a: {
-    b: {
-      c: {
-        d: 1
-      }
-    }
-  }",
-  "WRN arr objs 
-  {
-    a: 1
-  }
-  {
-    b: 2
-  }",
-  "ERR multlinestring 
-  str: "a
-  b
-  c"
-  err: {
-    stack: "<stack>"
-    message: "woah!"
-    name: "Error"
-    class: "Error"
-  }",
-  "INF graphql 
-  query: "
-  {
-    hi(howMany: 1) {
-      hello
-      world
-    }
-  }
-  "",
+  ""TRC obj \\n  a: 1\\n  b: 2"",
+  ""DBG arr \\n  \\"a\\"\\n  \\"b\\"\\n  \\"c\\""",
+  ""INF nested \\n  a: {\\n    b: {\\n      c: {\\n        d: 1\\n      }\\n    }\\n  }"",
+  ""WRN arr objs \\n  {\\n    a: 1\\n  }\\n  {\\n    b: 2\\n  }"",
+  ""ERR multlinestring \\n  str: \\"a\\n  b\\n  c\\"\\n  err: {\\n    message: \\"woah!\\"\\n  }"",
+  ""INF graphql \\n  query: \\"\\n  {\\n    hi(howMany: 1) {\\n      hello\\n      world\\n    }\\n  }\\n  \\""",
 ]
 `;

packages/logger/tests/writers.test.ts

@@ -1,27 +1,27 @@
 import { describe, expect, it } from 'vitest';
 import { Logger } from '../src/logger';
-import { ConsoleLogWriter, ConsoleLogWriterOptions } from '../src/writers';
-import { stableError } from './utils';
+import {
+  ConsoleLogWriter,
+  ConsoleLogWriterOptions,
+  jsonStringify,
+} from '../src/writers';
 
-describe.skipIf(
-  // bun is serialising the snapshots differently. the object keys are out of order...
-  globalThis.Bun,
-)('ConsoleLogWriter', () => {
+describe('ConsoleLogWriter', () => {
   function createTConsoleLogger(opts?: Partial<ConsoleLogWriterOptions>) {
     const logs: string[] = [];
     const writer = new ConsoleLogWriter({
       console: {
         debug: (...args: unknown[]) => {
-          logs.push(args.join(' '));
+          logs.push(args.map((arg) => jsonStringify(arg)).join(' '));
         },
         info: (...args: unknown[]) => {
-          logs.push(args.join(' '));
+          logs.push(args.map((arg) => jsonStringify(arg)).join(' '));
         },
         warn: (...args: unknown[]) => {
-          logs.push(args.join(' '));
+          logs.push(args.map((arg) => jsonStringify(arg)).join(' '));
         },
         error: (...args: unknown[]) => {
-          logs.push(args.join(' '));
+          logs.push(args.map((arg) => jsonStringify(arg)).join(' '));
         },
       },
       noTimestamp: true,
@@ -38,10 +38,7 @@ describe.skipIf(
     log.debug(['a', 'b', 'c'], 'arr');
     log.info({ a: { b: { c: { d: 1 } } } }, 'nested');
     log.warn([{ a: 1 }, { b: 2 }], 'arr objs');
-    log.error(
-      { str: 'a\nb\nc', err: stableError(new Error('woah!')) },
-      'multlinestring',
-    );
+    log.error({ str: 'a\nb\nc', err: { message: 'woah!' } }, 'multlinestring');
 
     log.info(
       {