@remix-run/test 0.2.0 → 0.3.0

package/dist/lib/worker-e2e-file.js

@@ -0,0 +1,69 @@
+import { runTests } from "./executor.js";
+import { importModule } from "./import-module.js";
+import { getBrowserLauncher, getPlaywrightLaunchOptions, getPlaywrightPageOptions, } from "./playwright.js";
+import { createFailedResults } from "./worker-results.js";
+import { isRecord, parseCoverageConfig } from "./worker-server.js";
+export async function runE2ETestFile(value, onOpenResults) {
+    try {
+        let workerData = parseE2ETestWorkerData(value);
+        await importModule(workerData.file, import.meta);
+        let launcher = await getBrowserLauncher(workerData.playwrightUseOpts);
+        let opts = getPlaywrightLaunchOptions(workerData.playwrightUseOpts);
+        let browser = await launcher.launch(opts);
+        let browserClosed = false;
+        try {
+            let results = await runTests({
+                browser,
+                open: workerData.open ?? false,
+                playwrightPageOptions: getPlaywrightPageOptions(workerData.playwrightUseOpts),
+                coverage: !!workerData.coverage,
+            });
+            if (workerData.open) {
+                await onOpenResults?.(results);
+                console.log('\nBrowser is open. Press Ctrl+C to close.');
+                await new Promise((resolve) => browser.on('disconnected', () => resolve()));
+                return undefined;
+            }
+            await browser.close();
+            browserClosed = true;
+            return results;
+        }
+        finally {
+            if (!browserClosed) {
+                await browser.close();
+            }
+        }
+    }
+    catch (error) {
+        return createFailedResults(error);
+    }
+}
+function parseE2ETestWorkerData(value) {
+    if (!isRecord(value) || typeof value.file !== 'string') {
+        throw new Error('Invalid E2E test worker data');
+    }
+    return {
+        file: value.file,
+        coverage: parseCoverageConfig(value.coverage),
+        open: parseBoolean(value.open, 'open'),
+        playwrightUseOpts: parsePlaywrightUseOpts(value.playwrightUseOpts),
+    };
+}
+function parseBoolean(value, name) {
+    if (value === undefined) {
+        return undefined;
+    }
+    if (typeof value !== 'boolean') {
+        throw new Error(`Invalid E2E test worker ${name}`);
+    }
+    return value;
+}
+function parsePlaywrightUseOpts(value) {
+    if (value === undefined) {
+        return undefined;
+    }
+    if (!isRecord(value)) {
+        throw new Error('Invalid E2E test worker playwright options');
+    }
+    return value;
+}

package/dist/lib/worker-e2e.js

@@ -1,49 +1,13 @@
-import { workerData, parentPort } from 'node:worker_threads';
-import { runTests } from "./executor.js";
-import { importModule } from "./import-module.js";
-import { getBrowserLauncher, getPlaywrightLaunchOptions, getPlaywrightPageOptions, } from "./playwright.js";
-try {
-    await importModule(workerData.file, import.meta);
-    let launcher = await getBrowserLauncher(workerData.playwrightUseOpts);
-    let opts = getPlaywrightLaunchOptions(workerData.playwrightUseOpts);
-    let browser = await launcher.launch(opts);
-    try {
-        let results = await runTests({
-            browser,
-            open: workerData.open,
-            playwrightPageOptions: getPlaywrightPageOptions(workerData.playwrightUseOpts),
-            coverage: workerData.coverage,
-        });
-        parentPort.postMessage(results);
-        if (workerData.open) {
-            console.log('\nBrowser is open. Press Ctrl+C to close.');
-            await new Promise((resolve) => browser.on('disconnected', () => resolve()));
-        }
-    }
-    finally {
-        await browser.close();
-    }
-    process.exit(0);
+import { parentPort, workerData } from 'node:worker_threads';
+import { runE2ETestFile } from "./worker-e2e-file.js";
+if (!parentPort) {
+    throw new Error('E2E test worker is missing a parent port');
 }
-catch (e) {
-    let results = {
-        passed: 0,
-        failed: 1,
-        skipped: 0,
-        todo: 0,
-        tests: [
-            {
-                name: '',
-                suiteName: '',
-                status: 'failed',
-                duration: 0,
-                error: {
-                    message: e instanceof Error ? e.message : String(e),
-                    stack: e instanceof Error ? e.stack : undefined,
-                },
-            },
-        ],
-    };
-    parentPort.postMessage(results);
-    process.exit(0);
+const port = parentPort;
+const results = await runE2ETestFile(workerData, (openResults) => {
+    port.postMessage(openResults);
+});
+if (results) {
+    port.postMessage(results);
 }
+process.exit(0);

package/dist/lib/worker-process.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=worker-process.d.ts.map

package/dist/lib/worker-process.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"worker-process.d.ts","sourceRoot":"","sources":["../../src/lib/worker-process.ts"],"names":[],"mappings":""}

package/dist/lib/worker-process.js

@@ -0,0 +1,55 @@
+import { runE2ETestFile } from "./worker-e2e-file.js";
+import { runServerTestFile } from "./worker-server.js";
+import { createFailedResults } from "./worker-results.js";
+const workerData = await readWorkerData();
+const results = await runWorkerProcessFile(workerData);
+if (results) {
+    await sendResults(results);
+}
+if (process.connected) {
+    process.disconnect();
+}
+process.exitCode = 0;
+function readWorkerData() {
+    return new Promise((resolve, reject) => {
+        function cleanup() {
+            process.off('message', onMessage);
+            process.off('disconnect', onDisconnect);
+        }
+        function onMessage(value) {
+            cleanup();
+            resolve(value);
+        }
+        function onDisconnect() {
+            cleanup();
+            reject(new Error('Test worker process disconnected'));
+        }
+        process.once('message', onMessage);
+        process.once('disconnect', onDisconnect);
+    });
+}
+async function runWorkerProcessFile(value) {
+    try {
+        if (!isRecord(value) || (value.type !== 'server' && value.type !== 'e2e')) {
+            throw new Error('Invalid test worker process data');
+        }
+        return value.type === 'e2e'
+            ? await runE2ETestFile(value, sendResults)
+            : await runServerTestFile(value);
+    }
+    catch (error) {
+        return createFailedResults(error);
+    }
+}
+async function sendResults(results) {
+    if (!process.send) {
+        throw new Error('Test worker process is missing an IPC channel');
+    }
+    let send = process.send.bind(process);
+    await new Promise((resolve, reject) => {
+        send(results, undefined, undefined, (error) => (error ? reject(error) : resolve()));
+    });
+}
+function isRecord(value) {
+    return typeof value === 'object' && value !== null;
+}

package/dist/lib/worker-results.d.ts

@@ -0,0 +1,3 @@
+import type { TestResults } from './reporters/results.ts';
+export declare function createFailedResults(error: unknown): TestResults;
+//# sourceMappingURL=worker-results.d.ts.map

package/dist/lib/worker-results.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"worker-results.d.ts","sourceRoot":"","sources":["../../src/lib/worker-results.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AAEzD,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,WAAW,CAmB/D"}

package/dist/lib/worker-results.js

@@ -0,0 +1,20 @@
+export function createFailedResults(error) {
+    return {
+        passed: 0,
+        failed: 1,
+        skipped: 0,
+        todo: 0,
+        tests: [
+            {
+                name: '',
+                suiteName: '',
+                status: 'failed',
+                duration: 0,
+                error: {
+                    message: error instanceof Error ? error.message : String(error),
+                    stack: error instanceof Error ? error.stack : undefined,
+                },
+            },
+        ],
+    };
+}

package/dist/lib/worker-server.d.ts

@@ -0,0 +1,10 @@
+import type { CoverageConfig } from './coverage.ts';
+import type { TestResults } from './reporters/results.ts';
+export interface ServerTestWorkerData {
+    file: string;
+    coverage?: CoverageConfig;
+}
+export declare function runServerTestFile(value: unknown): Promise<TestResults>;
+export declare function isRecord(value: unknown): value is Record<string, unknown>;
+export declare function parseCoverageConfig(value: unknown): CoverageConfig | undefined;
+//# sourceMappingURL=worker-server.d.ts.map

package/dist/lib/worker-server.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"worker-server.d.ts","sourceRoot":"","sources":["../../src/lib/worker-server.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,eAAe,CAAA;AACnD,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AAKzD,MAAM,WAAW,oBAAoB;IACnC,IAAI,EAAE,MAAM,CAAA;IACZ,QAAQ,CAAC,EAAE,cAAc,CAAA;CAC1B;AAED,wBAAsB,iBAAiB,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAAC,WAAW,CAAC,CAiC5E;AAaD,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEzE;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,cAAc,GAAG,SAAS,CA2B9E"}

package/dist/lib/worker-server.js

@@ -0,0 +1,113 @@
+var __rewriteRelativeImportExtension = (this && this.__rewriteRelativeImportExtension) || function (path, preserveJsx) {
+    if (typeof path === "string" && /^\.\.?\//.test(path)) {
+        return path.replace(/\.(tsx)$|((?:\.d)?)((?:\.[^./]+?)?)\.([cm]?)ts$/i, function (m, tsx, d, ext, cm) {
+            return tsx ? preserveJsx ? ".jsx" : ".js" : d && (!ext || !cm) ? m : (d + ext + "." + cm.toLowerCase() + "js");
+        });
+    }
+    return path;
+};
+import * as mod from 'node:module';
+import { IS_RUNNING_FROM_SRC } from "./config.js";
+import { importModule } from "./import-module.js";
+import { runTests } from "./executor.js";
+import { IS_BUN } from "./runtime.js";
+import { createFailedResults } from "./worker-results.js";
+export async function runServerTestFile(value) {
+    let workerData;
+    try {
+        workerData = parseServerTestWorkerData(value);
+        // When coverage is enabled in Node, we use a coverage-friendly TypeScript loader which
+        // replaces tsx's minified transformation with a non-minified esbuild transform
+        // so V8 coverage byte offsets align with readable source lines. This hook runs
+        // before the inherited tsx hook (hooks are LIFO), so it intercepts .ts imports and
+        // short-circuits before tsx transforms them.
+        if (workerData.coverage && !IS_BUN) {
+            // Ensure we load the right file whether we're running in the monorepo (TS) or
+            // from a published package (JS)
+            let ext = IS_RUNNING_FROM_SRC ? '.ts' : '.js';
+            mod.register(new URL(`./coverage-loader${ext}`, import.meta.url), import.meta.url);
+            await import(__rewriteRelativeImportExtension(workerData.file));
+        }
+        else {
+            await importModule(workerData.file, import.meta);
+        }
+        let results = await runTests();
+        await takeCoverage(workerData.coverage);
+        return results;
+    }
+    catch (e) {
+        try {
+            await takeCoverage(workerData?.coverage);
+        }
+        catch (coverageError) {
+            e = coverageError;
+        }
+        return createFailedResults(e);
+    }
+}
+function parseServerTestWorkerData(value) {
+    if (!isRecord(value) || typeof value.file !== 'string') {
+        throw new Error('Invalid server test worker data');
+    }
+    return {
+        file: value.file,
+        coverage: parseCoverageConfig(value.coverage),
+    };
+}
+export function isRecord(value) {
+    return typeof value === 'object' && value !== null;
+}
+export function parseCoverageConfig(value) {
+    if (value === undefined) {
+        return undefined;
+    }
+    if (!isRecord(value) || typeof value.dir !== 'string') {
+        throw new Error('Invalid server test worker coverage config');
+    }
+    let coverage = {
+        dir: value.dir,
+    };
+    let include = parseStringArray(value.include, 'include');
+    let exclude = parseStringArray(value.exclude, 'exclude');
+    let statements = parseNumber(value.statements, 'statements');
+    let lines = parseNumber(value.lines, 'lines');
+    let branches = parseNumber(value.branches, 'branches');
+    let functions = parseNumber(value.functions, 'functions');
+    if (include)
+        coverage.include = include;
+    if (exclude)
+        coverage.exclude = exclude;
+    if (statements !== undefined)
+        coverage.statements = statements;
+    if (lines !== undefined)
+        coverage.lines = lines;
+    if (branches !== undefined)
+        coverage.branches = branches;
+    if (functions !== undefined)
+        coverage.functions = functions;
+    return coverage;
+}
+function parseStringArray(value, name) {
+    if (value === undefined) {
+        return undefined;
+    }
+    if (!Array.isArray(value) || value.some((item) => typeof item !== 'string')) {
+        throw new Error(`Invalid server test worker coverage ${name}`);
+    }
+    return value;
+}
+function parseNumber(value, name) {
+    if (value === undefined) {
+        return undefined;
+    }
+    if (typeof value !== 'number') {
+        throw new Error(`Invalid server test worker coverage ${name}`);
+    }
+    return value;
+}
+async function takeCoverage(coverage) {
+    if (coverage && !IS_BUN) {
+        let v8 = await import('node:v8');
+        v8.takeCoverage();
+    }
+}

package/dist/lib/worker.js

@@ -1,57 +1,8 @@
-var __rewriteRelativeImportExtension = (this && this.__rewriteRelativeImportExtension) || function (path, preserveJsx) {
-    if (typeof path === "string" && /^\.\.?\//.test(path)) {
-        return path.replace(/\.(tsx)$|((?:\.d)?)((?:\.[^./]+?)?)\.([cm]?)ts$/i, function (m, tsx, d, ext, cm) {
-            return tsx ? preserveJsx ? ".jsx" : ".js" : d && (!ext || !cm) ? m : (d + ext + "." + cm.toLowerCase() + "js");
-        });
-    }
-    return path;
-};
-import * as mod from 'node:module';
-import * as path from 'node:path';
 import { parentPort, workerData } from 'node:worker_threads';
-import { runTests } from "./executor.js";
-import { importModule } from "./import-module.js";
-import { IS_BUN } from "./runtime.js";
-import { IS_RUNNING_FROM_SRC } from "./config.js";
-try {
-    // When coverage is enabled in Node, we use a coverage-friendly TypeScript loader which
-    // replaces tsx's minified transformation with a non-minified esbuild transform
-    // so V8 coverage byte offsets align with readable source lines. This hook runs
-    // before the inherited tsx hook (hooks are LIFO), so it intercepts .ts imports and
-    // short-circuits before tsx transforms them.
-    if (workerData.coverage && !IS_BUN) {
-        // Ensure we load the right file whether we're running in the monorepo (TS) or
-        // from a published package (JS)
-        let ext = IS_RUNNING_FROM_SRC ? '.ts' : '.js';
-        mod.register(new URL(`./coverage-loader${ext}`, import.meta.url), import.meta.url);
-        await import(__rewriteRelativeImportExtension(workerData.file));
-    }
-    else {
-        await importModule(workerData.file, import.meta);
-    }
-    let results = await runTests();
-    parentPort.postMessage(results);
-    process.exit(0);
-}
-catch (e) {
-    let results = {
-        passed: 0,
-        failed: 1,
-        skipped: 0,
-        todo: 0,
-        tests: [
-            {
-                name: '',
-                suiteName: '',
-                status: 'failed',
-                duration: 0,
-                error: {
-                    message: e instanceof Error ? e.message : String(e),
-                    stack: e instanceof Error ? e.stack : undefined,
-                },
-            },
-        ],
-    };
-    parentPort.postMessage(results);
-    process.exit(0);
+import { runServerTestFile } from "./worker-server.js";
+if (!parentPort) {
+    throw new Error('Server test worker is missing a parent port');
 }
+const results = await runServerTestFile(workerData);
+parentPort.postMessage(results);
+process.exit(0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remix-run/test",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "A test framework for JavaScript and TypeScript projects",
   "author": "Shopify Inc.",
   "license": "MIT",
@@ -66,9 +66,9 @@
     "@typescript/native-preview": "7.0.0-dev.20251125.1",
     "decamelize": "^6.0.1",
     "playwright": "^1.59.0",
-    "@remix-run/assert": "^0.1.0",
-    "@remix-run/node-fetch-server": "^0.13.0",
-    "@remix-run/ui": "^0.1.0"
+    "@remix-run/node-fetch-server": "^0.13.1",
+    "@remix-run/assert": "^0.2.0",
+    "@remix-run/ui": "^0.1.1"
   },
   "keywords": [
     "testing",

package/src/app/client/entry.ts

@@ -185,6 +185,10 @@ function runInIframe(testFile: string): Promise<FileResults> {
   return new Promise((resolve) => {
     let iframe = document.createElement('iframe')
     iframe.src = `/iframe?file=${encodeURIComponent(testFile)}`
+    // Make the iframe as big so we don't get unintentional scrolling in test UIs
+    let parentBody = iframe.contentWindow?.document.body
+    iframe.width = Math.max(parentBody?.scrollWidth ?? 0, 800).toString()
+    iframe.height = Math.max(Math.round((parentBody?.scrollHeight ?? 0) / 2), 400).toString()
     document.body.appendChild(iframe)
 
     function onMessage(event: MessageEvent) {