fix undefined

packages/allure-js-commons/src/sdk/utils.ts

@@ -46,13 +46,13 @@ export const stripAnsi = (str: string): string => {
 
 export const getMessageAndTraceFromError = (error: Error | { message?: string; stack?: string }): StatusDetails => {
   const { message, stack } = error;
-  const actual = "actual" in error ? serialize(error.actual) : undefined;
-  const expected = "expected" in error ? serialize(error.expected) : undefined;
+  const actual = "actual" in error && error.actual !== undefined ? { actual: serialize(error.actual) } : {};
+  const expected = "expected" in error && error.expected !== undefined ? { expected: serialize(error.expected) } : {};
   return {
     message: message ? stripAnsi(message) : undefined,
     trace: stack ? stripAnsi(stack) : undefined,
-    actual,
-    expected,
+    ...actual,
+    ...expected,
   };
 };
 

packages/allure-js-commons/test/sdk/utils.spec.ts

@@ -531,6 +531,13 @@ describe("getMessageAndTraceFromError", () => {
     });
   });
 
+  it("should ignore undefined actual value", () => {
+    const error: Error & { actual?: string } = new Error("some message");
+    error.actual = undefined;
+    const result = getMessageAndTraceFromError(error);
+    expect(result).not.toHaveProperty("actual");
+  });
+
   it("should return expected from error", () => {
     const error: Error & { expected?: string } = new Error("some message");
     error.expected = "some expected value";
@@ -539,4 +546,11 @@ describe("getMessageAndTraceFromError", () => {
       expected: "some expected value",
     });
   });
+
+  it("should ignore undefined expected value", () => {
+    const error: Error & { expected?: string } = new Error("some message");
+    error.expected = undefined;
+    const result = getMessageAndTraceFromError(error);
+    expect(result).not.toHaveProperty("expected");
+  });
 });

packages/allure-vitest/test/spec/expect.test.ts

@@ -47,4 +47,28 @@ describe("expect", () => {
       },
     ]);
   });
+
+  // this is the way vitest process errors
+  it("should add actual and expected values when regular exception is thrown", async () => {
+    const { tests } = await runVitestInlineTest(`
+    import { test, expect } from "vitest";
+
+    test("fail test", () => {
+      throw new Error("fail!")
+    });
+
+  `);
+
+    expect(tests).toHaveLength(1);
+    expect(tests).toMatchObject([
+      {
+        name: "fail test",
+        status: "broken",
+        statusDetails: {
+          expected: "undefined",
+          actual: "undefined",
+        },
+      },
+    ]);
+  });
 });