Fix Node.js test harness to output stdout/stderr (#485)

* Fix Node.js test harness to output stdout/stderr Follow-up to 3cb3877 Also added test assertion to ensure the XCTest output is printed to stderr * Fix test for Swift 5.10
swiftwasm · Jun 13, 2024 · 1b7daf2 · 1b7daf2
1 parent 193722e
commit 1b7daf2
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/Sources/CartonKit/Server/StaticArchive.swift b/Sources/CartonKit/Server/StaticArchive.swift
diff --git a/Tests/CartonCommandTests/TestCommandTests.swift b/Tests/CartonCommandTests/TestCommandTests.swift
@@ -43,6 +43,11 @@ final class TestCommandTests: XCTestCase {
         ["carton", "test", "--environment", "node"], packageDirectory: packageDirectory.asURL
       )
       try result.checkNonZeroExit()
+      let stdout = try result.utf8Output()
+      let stderr = try result.utf8stderrOutput()
+      let expectedContent = "Test Suite 'All tests' passed"
+      // SwiftPM changed the output destination of SwiftPM Plugin in 6.0
+      XCTAssertTrue(stdout.contains(expectedContent) || stderr.contains(expectedContent))
     }
   }
 

diff --git a/entrypoint/testNode.ts b/entrypoint/testNode.ts
@@ -46,7 +46,15 @@ const startWasiTask = async () => {
     // No JavaScriptKit module found, run the Wasm module without JSKit
   }
 
-  const wasmRunner = WasmRunner({ args: testArgs }, runtimeConstructor);
+  const wasmRunner = WasmRunner({
+    args: testArgs,
+    onStdoutLine: (line) => {
+      console.log(line);
+    },
+    onStderrLine: (line) => {
+      console.error(line);
+    },
+  }, runtimeConstructor);
   let procExitCalled = false;
 
   process.on("beforeExit", () => {