test(jco): add extended test suite for niche regression tests

This commit adds an extended test suites that uses components in
`jco/test/components` to ensure Jco functionality is working properly.

While most components that are tested with Jco are written in either
JS or Rust, all other components (e.g. C++, Python) will be stored in
`jco/test/components` and built/used from there by relevant tests.

Author: vados-cosmonic

packages/jco/package.json

@@ -71,6 +71,7 @@
         "lint:fix": "oxlint --fix",
         "test": "vitest run -c test/vitest.ts",
         "test:lts": "vitest run -c test/vitest.lts.ts",
+        "test:extended": "vitest run -c test/vitest.extended.ts",
         "prepack": "cargo xtask build release"
     },
     "dependencies": {

packages/jco/test/common.js

@@ -29,6 +29,9 @@ export const P3_COMPONENT_FIXTURES_DIR = join(COMPONENT_FIXTURES_DIR, "p3");
 /** Path to built custom rust components (i.e. output of `cargo xtask build-test-components`) */
 export const LOCAL_TEST_COMPONENTS_DIR = join(COMPONENT_FIXTURES_DIR, "../../output/rust-test-components");
 
+/** Path to built extended test components (i.e. output of `just build` run in `jco/test/components`) */
+export const EXTENDED_TEST_COMPONENTS_DIR = fileURLToPath(new URL("../../../test/components/output", import.meta.url));
+
 /**
  * Retrieve a list of all component fixtures
  *

packages/jco/test/extended/jco-issue-1380.js

@@ -0,0 +1,18 @@
+import { join } from "node:path";
+
+import { suite, test, assert } from "vitest";
+
+import { exec, jcoPath, fileExists } from "../helpers.js";
+import { EXTENDED_TEST_COMPONENTS_DIR } from "../common.js";
+
+suite("jco-issue-1380", () => {
+    test("composed component runs", async () => {
+        const combinedComponentPath = join(EXTENDED_TEST_COMPONENTS_DIR, "jco-issue-1380/composed.wasm");
+        assert(await fileExists(combinedComponentPath), "built composed component must be in place");
+
+        const { stdout, stderr } = await exec(jcoPath, "run", combinedComponentPath);
+
+        assert.strictEqual(stdout, "");
+        assert.strictEqual(stderr, "");
+    });
+});

packages/jco/test/helpers.js

@@ -662,3 +662,9 @@ export async function getCurrentWitComponentVersion() {
     CURRENT_WIT_COMPONENT_VERSION = version;
     return CURRENT_WIT_COMPONENT_VERSION;
 }
+
+export async function fileExists(p) {
+    return stat(p)
+        .then((f) => f.isFile())
+        .catch(() => false);
+}

packages/jco/test/vitest.extended.ts

@@ -0,0 +1,30 @@
+import { availableParallelism } from "node:os";
+
+import { defineConfig } from "vitest/config";
+
+const DEFAULT_TIMEOUT_MS = 1000 * 60 * 1; // 60s
+const CI_DEFAULT_TIMEOUT_MS = 1000 * 60 * 3; // 1m
+
+const REPORTERS = process.env.GITHUB_ACTIONS ? ["verbose", "github-actions"] : ["verbose"];
+const JSPI_EXEC_ARGV = "Suspending" in WebAssembly ? [] : ["--experimental-wasm-jspi"];
+
+export default defineConfig({
+    test: {
+        reporters: REPORTERS,
+        maxConcurrency: Math.max(availableParallelism() / 2, 5),
+        disableConsoleIntercept: true,
+        printConsoleTrace: true,
+        passWithNoTests: false,
+        setupFiles: ["test/meta-resolve-stub.ts"],
+        include: ["test/extended/**/*.js"],
+        testTimeout: process.env.CI ? CI_DEFAULT_TIMEOUT_MS : DEFAULT_TIMEOUT_MS,
+        hookTimeout: process.env.CI ? CI_DEFAULT_TIMEOUT_MS : DEFAULT_TIMEOUT_MS,
+        teardownTimeout: process.env.CI ? CI_DEFAULT_TIMEOUT_MS : DEFAULT_TIMEOUT_MS,
+        pool: "forks",
+        poolOptions: {
+            forks: {
+                execArgv: [...JSPI_EXEC_ARGV, "--stack-trace-limit=100"],
+            },
+        },
+    },
+});

packages/jco/test/vitest.lts.ts

@@ -24,6 +24,7 @@ export default defineConfig({
         setupFiles: ["test/meta-resolve-stub.ts"],
         include: ["test/*.js"],
         exclude: [
+            "test/extended/*",
             "test/output/*",
             "test/fixtures/*",
             "test/common.js",

packages/jco/test/vitest.ts

@@ -18,6 +18,7 @@ export default defineConfig({
         setupFiles: ["test/meta-resolve-stub.ts"],
         include: ["test/**/*.js"],
         exclude: [
+            "test/extended/*",
             "test/output/*",
             "test/fixtures/*",
             "test/common.js",

test/components/cpp/jco-issue-1380/caller/component.cpp

@@ -4,10 +4,7 @@
 #include <iostream>
 
 std::expected<void, wit::Void> exports::wasi::cli::run::Run() {
-  std::cout << "[dbg] run: minimal begin" << std::endl;
   auto h = ::test::jco_bug::iface::OpenTemp();
-  std::cout << "[dbg] run: before append" << std::endl;
   h.Append("|x|");
-  std::cout << "[dbg] run: after append" << std::endl;
   return {};
 }

test/components/python/jco-issue-1380/callee/component.py

@@ -15,32 +15,21 @@ class Handle(iface.Handle):
     def __init__(self, path: str) -> None:
         self._path = path
         Handle._storage.setdefault(self._path, "")
-        print(f"[dbg] py: Handle.__init__ path={self._path}")
 
     def append(self, data: str) -> None:
-        print(f"[dbg] py: Handle.append begin path={self._path} data={data}")
         Handle._storage[self._path] = Handle._storage.get(self._path, "") + data
-        print(
-            f"[dbg] py: Handle.append end path={self._path} size={len(Handle._storage[self._path])}"
-        )
 
 
 class Iface(exports.Iface):
     def open_temp(self) -> Handle:
-        print("[dbg] py: open_temp begin")
         Handle._seq += 1
         h = Handle(f"mem://h-{Handle._seq}")
-        print(f"[dbg] py: open_temp end path={h._path}")
         return h
 
 
 class Run(exports.Run):
     def run(self) -> None:
-        print("[dbg] py: run begin")
         api = Handle()
-        print("[dbg] py: run open_temp")
         h = api.open_temp()
-        print("[dbg] py: run append")
         h.append("|py-run|")
-        print("[dbg] py: run end")
         return None