fix

Author: HerrCai0907

src/executionResult.ts

@@ -96,24 +96,28 @@ export class ExecutionResultSummary {
       this.total.toString();
     log(`\ntest case: ${rate} (success/total)\n`);
     if (this.fail !== 0) {
-      log(chalk.red("Error Message: "));
-      // sort failedInfos by testcaseName to keep stability for e2e testing
-      const failedInfosArray = Array.from(this.failedInfos.entries()).sort((a, b) => a[0].localeCompare(b[0]));
-      for (const [testcaseName, { hasCrash, assertMessages, logMessages }] of failedInfosArray) {
-        log(`  ${testcaseName}: `);
-        for (const assertMessage of assertMessages) {
-          log("    " + chalk.yellow(assertMessage));
-        }
-        if (hasCrash) {
-          log("    " + chalk.red("Test Crashed!"));
-        }
-        for (const logMessage of logMessages ?? []) {
-          log(chalk.gray(logMessage));
-        }
-        if (logMessages.length > 0) {
-          // empty line to separate test
-          log("");
-        }
+      this.#printErrorMessage(log);
+    }
+  }
+
+  #printErrorMessage(log: (msg: string) => void): void {
+    log(chalk.red("Error Message: "));
+    // sort failedInfos by testcaseName to keep stability for e2e testing
+    const failedInfosArray = Array.from(this.failedInfos.entries()).sort((a, b) => a[0].localeCompare(b[0]));
+    for (const [testcaseName, { hasCrash, assertMessages, logMessages }] of failedInfosArray) {
+      log(`  ${testcaseName}: `);
+      for (const assertMessage of assertMessages) {
+        log("    " + chalk.yellow(assertMessage));
+      }
+      if (hasCrash) {
+        log("    " + chalk.red("Test Crashed!"));
+      }
+      for (const logMessage of logMessages ?? []) {
+        log(chalk.gray(logMessage));
+      }
+      if (logMessages.length > 0) {
+        // empty line to separate test
+        log("");
       }
     }
   }

src/utils/errorTraceHandler.ts

@@ -3,6 +3,7 @@
 import { readFile } from "node:fs/promises";
 import { parseSourceMapPath } from "./wasmparser.js";
 import { BasicSourceMapConsumer, IndexedSourceMapConsumer, SourceMapConsumer } from "source-map";
+import chalk from "chalk";
 
 export interface WebAssemblyCallSite {
   functionName: string;
@@ -47,7 +48,7 @@ function getOriginLocationWithSourceMap(
 }
 
 function getWebAssemblyFunctionName(callSite: NodeJS.CallSite): string {
-  return callSite.getFunctionName() ?? `wasm-function[${callSite.getFunction()}]`;
+  return callSite.getFunctionName() ?? `wasm-function[${callSite.getFunction()?.toString() ?? "unknown"}]`;
 }
 
 function createWebAssemblyCallSite(
@@ -87,15 +88,24 @@ export interface ExecutionError {
   stacks: WebAssemblyCallSite[];
 }
 
-async function getSourceMapConsumer(sourceMapPath: string): Promise<BasicSourceMapConsumer | IndexedSourceMapConsumer> {
-  return new Promise<BasicSourceMapConsumer | IndexedSourceMapConsumer>(async (resolve, reject) => {
-    let sourceMapContent: string | null = null;
+async function getSourceMapConsumer(sourceMapPath: string | null): Promise<SourceMapHandler | null> {
+  if (sourceMapPath === null) {
+    return null;
+  }
+  const sourceMapContent: string | null = await (async () => {
     try {
-      sourceMapContent = await readFile(sourceMapPath, "utf8");
+      return await readFile(sourceMapPath, "utf8");
     } catch (error) {
-      reject(error);
-      return;
+      if (error instanceof Error) {
+        console.log(chalk.yellow(`Failed to read source map file: ${sourceMapPath} due to ${error}`));
+      }
+      return null;
     }
+  })();
+  if (sourceMapContent === null) {
+    return null;
+  }
+  return new Promise<SourceMapHandler>((resolve) => {
     SourceMapConsumer.with(sourceMapContent, null, (consumer) => {
       resolve(consumer);
     });
@@ -108,7 +118,7 @@ export async function handleWebAssemblyError(
 ): Promise<ExecutionError> {
   const wasmBuffer = await readFile(wasmPath);
   const sourceMapPath = parseSourceMapPath(wasmBuffer.buffer as ArrayBuffer);
-  const sourceMapConsumer = sourceMapPath ? await getSourceMapConsumer(sourceMapPath) : null;
+  const sourceMapConsumer: SourceMapHandler | null = await getSourceMapConsumer(sourceMapPath);
   let stacks: NodeJS.CallSite[] = [];
   const originalPrepareStackTrace = Error.prepareStackTrace;
   Error.prepareStackTrace = (_: Error, structuredStackTrace: NodeJS.CallSite[]) => {
@@ -120,6 +130,6 @@ export async function handleWebAssemblyError(
     message: error.message,
     stacks: stacks
       .map((callSite) => createWebAssemblyCallSite(callSite, { wasmPath, sourceMapConsumer }))
-      .filter((callSite) => callSite != null),
+      .filter((callSite) => callSite !== null),
   };
 }

src/utils/wasmparser.ts

@@ -89,7 +89,7 @@ export function parseSourceMapPath(buf: ArrayBuffer): string | null {
       if (sectionInfo.id !== SectionCode.Custom) {
         reader.skipSection();
       }
-    } else if (reader.state == BinaryReaderState.SOURCE_MAPPING_URL) {
+    } else if (reader.state === BinaryReaderState.SOURCE_MAPPING_URL) {
       const sectionInfo = reader.result as ISourceMappingURL;
       return new TextDecoder("utf8").decode(sectionInfo.url);
     }

tests/ts/test/executionResult.test.ts

@@ -62,8 +62,6 @@ test("equal assert failed", async () => {
 
 test("equal crash", async () => {
   const executionResult = new ExecutionResultSummary();
-  const actualString = "A long sentence for testing errorMsg.length > 160 in executionResult.ts merge function";
-  const expectString = "= A long sentence for testing errorMsg.length > 160 in executionResult.ts merge function ";
   const testcaseA: IExecutionResult = {
     fail: 1,
     total: 1,