address pr reviews

Author: meowbmw

.prettierignore

@@ -6,6 +6,6 @@
 /tests/ts/fixture
 /third_party
 /tests/cpp/lit/**/*.json
-/tests/ts/e2e/default.test.ts
+/tests/as/default.test.ts
 /.cache
 /.github/CODEOWNERS

jest.config.ts

@@ -15,7 +15,6 @@ const config: JestConfigWithTsJest = {
   moduleNameMapper: {
     "^(\\.{1,2}/.*)\\.js$": "$1",
   },
-  modulePathIgnorePatterns: ["tests/ts/e2e"],
   testPathIgnorePatterns: ["./dist"],
   testTimeout: 10000,
 };

src/core/execute.ts

@@ -30,7 +30,6 @@ function nodeExecutor(wasms: string[], outFolder: string, imports: Imports) {
         ...userDefinedImportsObject,
       } as ASImports;
       const binary = await readFile(wasm);
-      // getting a list of wasm import functions
       const importList = await parseWasmImports(binary);
       // supplying default function here, so no more need to define all of them in as-test.js
       for (const imp of importList) {

src/utils/import.ts

@@ -88,18 +88,16 @@ export function covInstruFunc(wasm: string) {
   return { covInstrument };
 }
 
+// list imports of a given wasm binary (buffer)
+// importList format should be as follows：
+// [
+//   { module: 'env', name: 'memory', kind: 'memory' },
+//   { module: 'env', name: 'myFunction', kind: 'function' },
+//   ...
+// ]
 export async function parseWasmImports(binary: Buffer) {
-  // list imports of a given wasm binary (buffer)
   const mod = await WebAssembly.compile(binary);
-
   const importList = WebAssembly.Module.imports(mod);
 
-  // importList format should be as follows：
-  // [
-  //   { module: 'env', name: 'memory', kind: 'memory' },
-  //   { module: 'env', name: 'myFunction', kind: 'function' },
-  //   ...
-  // ]
-
   return importList;
 }

tests/ts/e2e/default.test.ts renamed to tests/as/default.test.ts

@@ -1,4 +1,4 @@
-import { describe, test, expect, endTest } from "../../../assembly";
+import { describe, test, expect, endTest } from "../../assembly";
 
 // Default Stub Injection Tests
 // Making sure that external functions can supplied with an empty default function so that users will be free of supplying their own