test: fix type error and extend coverage

Author: christoph-fricke

src/log-record.test.ts

@@ -30,6 +30,14 @@ describe("LogRecord", () => {
     expect(record.date).toStrictEqual(date);
   });
 
+  // Some helpers for the following test cases
+  function sum(a: number, b: number) {
+    return a + b;
+  }
+  const stackErr = new Error("Stack error");
+  const testErr = Error("Test error");
+  testErr.stack = undefined;
+
   const cases: [args: unknown[], parsed: string][] = [
     [[], ""],
     [["log message"], "log message"],
@@ -42,21 +50,16 @@ describe("LogRecord", () => {
     [[{ key: 1, key2: 2 }], '{"key":1,"key2":2}'],
     [[null], "null"],
     [[undefined, 1], "undefined 1"],
-    [
-      [
-        function sum(a: number, b: number) {
-          return a + b;
-        },
-      ],
-      "[Function sum]",
-    ],
+    [[sum], "[Function sum]"],
     [
       [
         [1, 2, 3],
         [4, 5, 6],
       ],
       "[1,2,3] [4,5,6]",
     ],
+    [[stackErr], stackErr.stack as string],
+    [[testErr], "Test error"],
   ];
 
   it.each(cases)('Given arguments "%s" are parsed to "%s"', (args, parsed) => {

src/log-record.ts

@@ -70,7 +70,6 @@ function asString(data: unknown): string {
     typeof data === "number" ||
     typeof data === "bigint" ||
     typeof data === "boolean" ||
-    typeof data === "undefined" ||
     typeof data === "symbol"
   ) {
     return String(data);

src/test-helper/log-record-builder.ts

@@ -7,7 +7,8 @@ export function buildLogRecord(overrides?: Partial<ILogRecord>): ILogRecord {
     date: new Date(),
     level: LogLevel.DEBUG,
     levelName: getLogLevelName(overrides?.level ?? LogLevel.DEBUG),
-    message: "Test message",
+    args: ["message", 123],
+    message: "message 123",
     metadata: { test: "Test meta" },
     ...overrides,
   };