@remix-run/test 0.2.0 → 0.3.0

package/README.md

@@ -41,13 +41,14 @@ Run tests with the CLI:
 remix test
 ```
 
-By default, `remix test` discovers all files matching `**/*.test{,.e2e}.{ts,tsx}`. Pass a glob as the first positional argument to override:
+By default, `remix test` discovers all files matching `**/*.test{,.e2e}.{ts,tsx}`. Pass one or more globs as positional arguments to override:
 
 ```sh
 remix test "src/**/*.test.ts"
+remix test "src/**/*.test.ts" "tests/**/*.test.tsx"
 ```
 
-Or, you may control via the `glob.test` config field/CLI arg.
+Or, you may control via the `glob.test` config field/CLI arg. Each `glob.*` field accepts a single string or an array of patterns, and `--glob.*` flags can be repeated on the CLI.
 
 If you install `@remix-run/test` directly instead of the umbrella `remix` package, the same runner is available as `remix-test`:
 
@@ -76,15 +77,18 @@ export default {
   // Max number of concurrent test workers (default `os.availableParallelism()`)
   concurrency: 2,
 
+  // Pool for server and E2E test files ("forks", "threads")
+  pool: 'forks',
+
   // Code coverage options
   coverage: {
     // Enable coverage reporting
     enabled: true,
     // Output directory (default: ".coverage")
     dir: '.coverage',
-    // Glob patterns to include/exclude
-    include: ['src/**'],
-    exclude: ['src/**/*.test.ts'],
+    // Glob pattern(s) to include/exclude
+    include: 'src/**',
+    exclude: 'src/**/*.test.ts',
     // Minimum thresholds (%)
     statements: 80,
     lines: 80,
@@ -92,12 +96,13 @@ export default {
     functions: 80,
   },
 
+  // Glob pattern(s) identifying test files
   glob: {
-    // Glob pattern identifying all test files (default: "**/*.test{,.browser,.e2e}.{ts,tsx}")
+    // All test files (default: "**/*.test{,.browser,.e2e}.{ts,tsx}").
     test: '**/*.test{,.browser,.e2e}.ts',
-    // Glob pattern identifying browser test files (default: "**/*.test.browser.{ts,tsx}")
+    // Browser test files (default: "**/*.test.browser.{ts,tsx}")
     browser: '**/*.test.browser.ts',
-    // Glob pattern identifying E2E test files (default: "**/*.test.e2e.{ts,tsx}")
+    // E2E test files (default: "**/*.test.e2e.{ts,tsx}")
     e2e: '**/*.test.e2e.ts',
   },
 
@@ -114,7 +119,7 @@ export default {
     },
   },
 
-  // Comma-separated list of playwright projects to run E2E tests for
+  // Playwright project(s) to run E2E tests for
   project: 'chromium',
 
   // Test reporter ("spec", "files", "tap", "dot")
@@ -123,8 +128,8 @@ export default {
   // Path to a setup module (see Setup section below)
   setup: './test/setup.ts',
 
-  // Comma-separated list of test types to run ("server", "browser", "e2e")
-  type: 'server,browser,e2e',
+  // Test type(s) to run ("server", "browser", "e2e")
+  type: ['server', 'browser', 'e2e'],
 
   // Watch for file changes and re-run
   watch: false,
@@ -141,28 +146,29 @@ remix test --config ./tests/config.ts
 
 You may also specify any config field as a CLI flag which will take precedence over config file values:
 
-| Flag                        | Short |
-| --------------------------- | ----- |
-| `--browser.echo`            |       |
-| `--browser.open`            |       |
-| `--concurrency <n>`         | `-c`  |
-| `--coverage`                |       |
-| `--coverage.dir <path>`     |       |
-| `--coverage.include`        |       |
-| `--coverage.exclude`        |       |
-| `--coverage.statements`     |       |
-| `--coverage.lines`          |       |
-| `--coverage.branches`       |       |
-| `--coverage.functions`      |       |
-| `--glob.test`               |       |
-| `--glob.browser`            |       |
-| `--glob.e2e`                |       |
-| `--playwrightConfig <path>` |       |
-| `--project <name>`          | `-p`  |
-| `--reporter <name>`         | `-r`  |
-| `--setup <path>`            | `-s`  |
-| `--type <name>`             | `-t`  |
-| `--watch`                   | `-w`  |
+| Flag                        | Short     |
+| --------------------------- | --------- | --- |
+| `--browser.echo`            |           |
+| `--browser.open`            |           |
+| `--concurrency <n>`         | `-c`      |
+| `--coverage`                |           |
+| `--coverage.dir <path>`     |           |
+| `--coverage.include`        |           |
+| `--coverage.exclude`        |           |
+| `--coverage.statements`     |           |
+| `--coverage.lines`          |           |
+| `--coverage.branches`       |           |
+| `--coverage.functions`      |           |
+| `--glob.test`               |           |
+| `--glob.browser`            |           |
+| `--glob.e2e`                |           |
+| `--playwrightConfig <path>` |           |
+| `--pool <forks              | threads>` |     |
+| `--project <name>`          | `-p`      |
+| `--reporter <name>`         | `-r`      |
+| `--setup <path>`            | `-s`      |
+| `--type <name>`             | `-t`      |
+| `--watch`                   | `-w`      |
 
 ### Setup
 

package/dist/app/client/entry.js

@@ -167,6 +167,10 @@ function runInIframe(testFile) {
     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) {
             if (event.source !== iframe.contentWindow)

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAGA,OAAO,EACL,oBAAoB,EAIrB,MAAM,iBAAiB,CAAA;AAYxB,OAAO,EAAE,oBAAoB,EAAE,CAAA;AAE/B,MAAM,WAAW,mBAAmB;IAClC,IAAI,CAAC,EAAE,MAAM,EAAE,CAAA;IACf,GAAG,CAAC,EAAE,MAAM,CAAA;CACb;AASD,wBAAsB,YAAY,CAAC,OAAO,GAAE,mBAAwB,GAAG,OAAO,CAAC,MAAM,CAAC,CAerF"}
+{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,oBAAoB,EAIrB,MAAM,iBAAiB,CAAA;AAWxB,OAAO,EAAE,oBAAoB,EAAE,CAAA;AAK/B,MAAM,WAAW,mBAAmB;IAClC,IAAI,CAAC,EAAE,MAAM,EAAE,CAAA;IACf,GAAG,CAAC,EAAE,MAAM,CAAA;CACb;AAWD,wBAAsB,YAAY,CAAC,OAAO,GAAE,mBAAwB,GAAG,OAAO,CAAC,MAAM,CAAC,CAerF"}

package/dist/cli.js

@@ -2,15 +2,14 @@ import * as fsp from 'node:fs/promises';
 import * as path from 'node:path';
 import { getRemixTestHelpText, IS_RUNNING_FROM_SRC, loadConfig, } from "./lib/config.js";
 import { generateCombinedCoverageReport } from "./lib/coverage.js";
-import { loadPlaywrightConfig, resolveProjects } from "./lib/playwright.js";
 import { createReporter } from "./lib/reporters/index.js";
-import { runBrowserTests } from "./lib/runner-browser.js";
 import { runServerTests } from "./lib/runner.js";
 import { createWatcher } from "./lib/watcher.js";
 import { importModule } from "./lib/import-module.js";
 import { IS_BUN } from "./lib/runtime.js";
 import { isMainThread } from 'node:worker_threads';
 export { getRemixTestHelpText };
+const MISSING_PLAYWRIGHT_MESSAGE = 'Playwright is required to run browser and E2E tests. Install it with `npm i -D playwright`.';
 export async function runRemixTest(options = {}) {
     let argv = options.argv ?? process.argv.slice(2);
     let cwd = await resolveCwd(options.cwd ?? process.cwd());
@@ -119,9 +118,6 @@ async function runRemixTestInCwd(argv, cwd) {
                 browserServerFilesKey = browserFilesKey;
                 browserPort = result.port;
             }
-            let playwrightConfig = config.playwrightConfig == null || typeof config.playwrightConfig === 'string'
-                ? await loadPlaywrightConfig(config.playwrightConfig, cwd)
-                : config.playwrightConfig;
             let reporter = createReporter(config.reporter);
             let startTime = performance.now();
             let counts = {
@@ -136,6 +132,7 @@ async function runRemixTestInCwd(argv, cwd) {
                 let serverResult = await runServerTests(serverFiles, reporter, config.concurrency, 'server', {
                     coverage: config.coverage,
                     cwd,
+                    pool: config.pool,
                 });
                 counts.failed += serverResult.failed;
                 counts.passed += serverResult.passed;
@@ -145,12 +142,17 @@ async function runRemixTestInCwd(argv, cwd) {
             }
             // Run browser/e2e tests for all browsers configured by the user
             if (browserFiles.length > 0 || e2eFiles.length > 0) {
+                let { loadPlaywrightConfig, resolveProjects } = await importPlaywrightSupport();
+                let runBrowserTests = browserFiles.length > 0 ? (await importBrowserTestRunner()).runBrowserTests : undefined;
+                let playwrightConfig = config.playwrightConfig == null || typeof config.playwrightConfig === 'string'
+                    ? await loadPlaywrightConfig(config.playwrightConfig, cwd)
+                    : config.playwrightConfig;
                 let projects = resolveProjects(playwrightConfig);
                 if (config.project) {
-                    let projectNames = config.project.split(',').map((project) => project.trim());
-                    projects = projects.filter((project) => project.name && projectNames.includes(project.name));
+                    let projectNames = new Set(config.project);
+                    projects = projects.filter((project) => project.name && projectNames.has(project.name));
                     if (projects.length === 0) {
-                        throw new Error(`No playwright projects found with name(s) "${config.project}"`);
+                        throw new Error(`No playwright projects found with name(s) "${config.project.join(', ')}"`);
                     }
                 }
                 let lastBrowserResult = null;
@@ -166,7 +168,7 @@ async function runRemixTestInCwd(argv, cwd) {
                         }
                     }
                     let [browserResult, e2eResult] = await Promise.all([
-                        browserFiles.length > 0
+                        runBrowserTests != null
                             ? runBrowserTests({
                                 baseUrl: `http://localhost:${browserPort}`,
                                 console: config.browser?.echo,
@@ -185,6 +187,7 @@ async function runRemixTestInCwd(argv, cwd) {
                                 projectName: project.name,
                                 coverage: config.coverage,
                                 cwd,
+                                pool: config.pool,
                             })
                             : null,
                     ]);
@@ -246,6 +249,36 @@ async function runRemixTestInCwd(argv, cwd) {
     }
     return await runPromise;
 }
+async function importPlaywrightSupport() {
+    try {
+        return await import("./lib/playwright.js");
+    }
+    catch (error) {
+        throw toPlaywrightImportError(error);
+    }
+}
+async function importBrowserTestRunner() {
+    try {
+        return await import("./lib/runner-browser.js");
+    }
+    catch (error) {
+        throw toPlaywrightImportError(error);
+    }
+}
+function toPlaywrightImportError(error) {
+    return isMissingPlaywrightImport(error) ? new Error(MISSING_PLAYWRIGHT_MESSAGE) : error;
+}
+function isMissingPlaywrightImport(error) {
+    if (!isRecord(error) || typeof error.message !== 'string') {
+        return false;
+    }
+    return ((error.code === 'ERR_MODULE_NOT_FOUND' || error.code === 'MODULE_NOT_FOUND') &&
+        (error.message.includes("Cannot find package 'playwright'") ||
+            error.message.includes("Cannot find module 'playwright'")));
+}
+function isRecord(value) {
+    return typeof value === 'object' && value !== null;
+}
 async function resolveCwd(cwd) {
     try {
         return await fsp.realpath(cwd);
@@ -257,12 +290,12 @@ async function resolveCwd(cwd) {
 async function discoverTests(config, cwd) {
     let files = await findFiles(config.glob.test, config.glob.exclude, cwd);
     if (files.length === 0) {
-        console.log(`No test files found matching pattern: ${config.glob.test}`);
+        console.log(`No test files found matching pattern: ${config.glob.test.join(', ')}`);
         return null;
     }
     let browserSet = new Set(await findFiles(config.glob.browser, config.glob.exclude, cwd));
     let e2eSet = new Set(await findFiles(config.glob.e2e, config.glob.exclude, cwd));
-    let types = new Set(config.type.split(','));
+    let types = new Set(config.type);
     let browserFiles = types.has('browser') ? files.filter((f) => browserSet.has(f)) : [];
     let e2eFiles = types.has('e2e') ? files.filter((file) => e2eSet.has(file)) : [];
     let serverFiles = types.has('server')
@@ -270,7 +303,7 @@ async function discoverTests(config, cwd) {
         : [];
     let totalFiles = browserFiles.length + serverFiles.length + e2eFiles.length;
     if (totalFiles === 0) {
-        console.log(`No test files remain after filtering for type ${config.type}`);
+        console.log(`No test files remain after filtering for type ${config.type.join(', ')}`);
         return null;
     }
     console.log(`Found ${totalFiles} test file(s) (${serverFiles.length} server, ${browserFiles.length} browser, ${e2eFiles.length} e2e)`);
@@ -281,25 +314,37 @@ async function discoverTests(config, cwd) {
         e2eFiles,
     };
 }
-async function findFiles(pattern, excludePattern, cwd) {
-    let files = [];
+async function findFiles(patterns, excludePatterns, cwd) {
+    let files = new Set();
     if (IS_BUN) {
         // Bun's `fs.promises.glob` follows symlinks and doesn't prune traversal
         // via `exclude`, so it enters pnpm symlink cycles in `node_modules`.
         // Use Bun's native Glob, which defaults to `followSymlinks: false`.
         // @ts-expect-error — bun module is only resolvable under the Bun runtime
         let { Glob } = await import('bun');
-        let glob = new Glob(pattern);
-        let excludeGlob = new Glob(excludePattern);
-        for await (let file of glob.scan({ cwd, absolute: true })) {
-            if (!excludeGlob.match(path.relative(cwd, file))) {
-                files.push(file);
+        let excludeGlobs = excludePatterns.map((p) => new Glob(p));
+        for (let pattern of patterns) {
+            let glob = new Glob(pattern);
+            for await (let file of glob.scan({ cwd, absolute: true })) {
+                let rel = toPosix(path.relative(cwd, file));
+                if (!excludeGlobs.some((eg) => eg.match(rel))) {
+                    files.add(toPosix(file));
+                }
             }
         }
-        return files;
+        return [...files];
     }
-    for await (let file of fsp.glob(pattern, { cwd, exclude: [excludePattern] })) {
-        files.push(path.resolve(cwd, file));
+    for (let pattern of patterns) {
+        for await (let file of fsp.glob(pattern, { cwd, exclude: excludePatterns })) {
+            files.add(toPosix(path.resolve(cwd, file)));
+        }
     }
-    return files;
+    return [...files];
+}
+// Normalize discovered paths so set membership across the test/browser/e2e
+// `findFiles` calls is byte-stable on every platform. Node accepts forward
+// slashes for filesystem operations on Windows, so downstream `fs.readFile`
+// etc. work without further conversion.
+function toPosix(p) {
+    return path.sep === '/' ? p : p.replace(/\\/g, '/');
 }

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-export type { RemixTestConfig } from './lib/config.ts';
+export type { RemixTestConfig, RemixTestPool } from './lib/config.ts';
 export { describe, it, suite, test, before, after, beforeEach, afterEach, beforeAll, afterAll, } from './lib/framework.ts';
 export { mock } from './lib/mock.ts';
 export type { TestContext } from './lib/context.ts';

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,YAAY,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACtD,OAAO,EACL,QAAQ,EACR,EAAE,EACF,KAAK,EACL,IAAI,EACJ,MAAM,EACN,KAAK,EACL,UAAU,EACV,SAAS,EACT,SAAS,EACT,QAAQ,GACT,MAAM,oBAAoB,CAAA;AAC3B,OAAO,EAAE,IAAI,EAAE,MAAM,eAAe,CAAA;AACpC,YAAY,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAA;AACnD,YAAY,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,YAAY,EAAE,eAAe,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAA;AACrE,OAAO,EACL,QAAQ,EACR,EAAE,EACF,KAAK,EACL,IAAI,EACJ,MAAM,EACN,KAAK,EACL,UAAU,EACV,SAAS,EACT,SAAS,EACT,QAAQ,GACT,MAAM,oBAAoB,CAAA;AAC3B,OAAO,EAAE,IAAI,EAAE,MAAM,eAAe,CAAA;AACpC,YAAY,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAA;AACnD,YAAY,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAA"}

package/dist/lib/config.d.ts

@@ -1,6 +1,7 @@
 import type { PlaywrightTestConfig } from 'playwright/test';
 export declare const IS_RUNNING_FROM_SRC: boolean;
 export declare function getBrowserTestRootDir(): string;
+export type RemixTestPool = 'forks' | 'threads';
 export interface RemixTestConfig {
     /**
      * Options for controlling the playwright browser
@@ -12,17 +13,18 @@ export interface RemixTestConfig {
         open?: boolean;
     };
     /**
-     * Glob patterns to identify test files
-     *  - `glob.test`: Glob pattern for all test files (--glob.test)
-     *  - `glob.browser`: Glob pattern for the subset of browser test files (--glob.browser)
-     *  - `glob.e2e`: Glob pattern for the subset of e2e test files (--glob.e2e)
-     *  - `glob.exclude`: Glob pattern for paths to exclude from discovery (--glob.exclude)
+     * Glob patterns to identify test files. Each field accepts a single pattern
+     * or an array of patterns; arrays are unioned during discovery.
+     *  - `glob.test`: Glob pattern(s) for all test files (--glob.test)
+     *  - `glob.browser`: Glob pattern(s) for the subset of browser test files (--glob.browser)
+     *  - `glob.e2e`: Glob pattern(s) for the subset of e2e test files (--glob.e2e)
+     *  - `glob.exclude`: Glob pattern(s) for paths to exclude from discovery (--glob.exclude)
      */
     glob?: {
-        test?: string;
-        browser?: string;
-        e2e?: string;
-        exclude?: string;
+        test?: string | string[];
+        browser?: string | string[];
+        e2e?: string | string[];
+        exclude?: string | string[];
     };
     /** Max number of concurrent test workers (--concurrency) */
     concurrency?: number | string;
@@ -32,8 +34,8 @@ export interface RemixTestConfig {
      */
     coverage?: boolean | {
         dir?: string;
-        include?: string[];
-        exclude?: string[];
+        include?: string | string[];
+        exclude?: string | string[];
         statements?: number | string;
         lines?: number | string;
         branches?: number | string;
@@ -49,12 +51,23 @@ export interface RemixTestConfig {
      * PlaywrightTestConfig object. CLI `--playwrightConfig` only accepts a file path.
      */
     playwrightConfig?: string | PlaywrightTestConfig;
-    /** Filter tests to a specific playwright project or comma-separated list of projects (--project) */
-    project?: string;
+    /**
+     * Pool used to run server and E2E test files. Forked child processes are the default,
+     * but worker threads are available for projects that prefer the previous behavior.
+     */
+    pool?: RemixTestPool;
+    /**
+     * Filter tests to specific playwright project(s) (--project). Accepts a single
+     * project name or an array of names; `--project` may be repeated on the CLI.
+     */
+    project?: string | string[];
     /** Test reporter (--reporter) */
     reporter?: string;
-    /** Comma-separated list of test types to run (--type) */
-    type?: string;
+    /**
+     * Test type(s) to run (--type). Accepts a single type or an array of types;
+     * `--type` may be repeated on the CLI. Valid values: "server", "browser", "e2e".
+     */
+    type?: string | string[];
     /** Watch mode — re-run tests on file changes (--watch) */
     watch?: boolean;
 }
@@ -74,16 +87,17 @@ export interface ResolvedRemixTestConfig {
         functions?: number;
     } | undefined;
     glob: {
-        test: string;
-        browser: string;
-        e2e: string;
-        exclude: string;
+        test: string[];
+        browser: string[];
+        e2e: string[];
+        exclude: string[];
     };
     playwrightConfig: string | PlaywrightTestConfig | undefined;
-    project: string | undefined;
+    project: string[] | undefined;
     reporter: string;
+    pool: RemixTestPool;
     setup: string | undefined;
-    type: string;
+    type: string[];
     watch: boolean;
 }
 export declare function loadConfig(args?: string[], cwd?: string): Promise<ResolvedRemixTestConfig>;

package/dist/lib/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../../src/lib/config.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,iBAAiB,CAAA;AAG3D,eAAO,MAAM,mBAAmB,SAA4D,CAAA;AAa5F,wBAAgB,qBAAqB,IAAI,MAAM,CAS9C;AAqID,MAAM,WAAW,eAAe;IAC9B;;;;OAIG;IACH,OAAO,CAAC,EAAE;QACR,IAAI,CAAC,EAAE,OAAO,CAAA;QACd,IAAI,CAAC,EAAE,OAAO,CAAA;KACf,CAAA;IACD;;;;;;OAMG;IACH,IAAI,CAAC,EAAE;QACL,IAAI,CAAC,EAAE,MAAM,CAAA;QACb,OAAO,CAAC,EAAE,MAAM,CAAA;QAChB,GAAG,CAAC,EAAE,MAAM,CAAA;QACZ,OAAO,CAAC,EAAE,MAAM,CAAA;KACjB,CAAA;IACD,4DAA4D;IAC5D,WAAW,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;IAC7B;;;OAGG;IACH,QAAQ,CAAC,EACL,OAAO,GACP;QACE,GAAG,CAAC,EAAE,MAAM,CAAA;QACZ,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,UAAU,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QAC5B,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QACvB,QAAQ,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QAC1B,SAAS,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;KAC5B,CAAA;IACL;;;OAGG;IACH,KAAK,CAAC,EAAE,MAAM,CAAA;IACd;;;OAGG;IACH,gBAAgB,CAAC,EAAE,MAAM,GAAG,oBAAoB,CAAA;IAChD,oGAAoG;IACpG,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,iCAAiC;IACjC,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,yDAAyD;IACzD,IAAI,CAAC,EAAE,MAAM,CAAA;IACb,4DAA0D;IAC1D,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB;AAED,MAAM,WAAW,uBAAuB;IACtC,OAAO,EAAE;QACP,IAAI,CAAC,EAAE,OAAO,CAAA;QACd,IAAI,CAAC,EAAE,OAAO,CAAA;KACf,CAAA;IACD,WAAW,EAAE,MAAM,CAAA;IACnB,QAAQ,EACJ;QACE,GAAG,EAAE,MAAM,CAAA;QACX,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,UAAU,CAAC,EAAE,MAAM,CAAA;QACnB,KAAK,CAAC,EAAE,MAAM,CAAA;QACd,QAAQ,CAAC,EAAE,MAAM,CAAA;QACjB,SAAS,CAAC,EAAE,MAAM,CAAA;KACnB,GACD,SAAS,CAAA;IACb,IAAI,EAAE;QACJ,IAAI,EAAE,MAAM,CAAA;QACZ,OAAO,EAAE,MAAM,CAAA;QACf,GAAG,EAAE,MAAM,CAAA;QACX,OAAO,EAAE,MAAM,CAAA;KAChB,CAAA;IACD,gBAAgB,EAAE,MAAM,GAAG,oBAAoB,GAAG,SAAS,CAAA;IAC3D,OAAO,EAAE,MAAM,GAAG,SAAS,CAAA;IAC3B,QAAQ,EAAE,MAAM,CAAA;IAChB,KAAK,EAAE,MAAM,GAAG,SAAS,CAAA;IACzB,IAAI,EAAE,MAAM,CAAA;IACZ,KAAK,EAAE,OAAO,CAAA;CACf;AAED,wBAAsB,UAAU,CAAC,IAAI,GAAE,MAAM,EAA0B,EAAE,GAAG,SAAgB,oCAK3F;AAED,wBAAgB,oBAAoB,CAAC,OAAO,GAAE,MAAM,CAAC,WAA4B,GAAG,MAAM,CAmBzF"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../../src/lib/config.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,iBAAiB,CAAA;AAG3D,eAAO,MAAM,mBAAmB,SAA4D,CAAA;AAa5F,wBAAgB,qBAAqB,IAAI,MAAM,CAS9C;AAgJD,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,SAAS,CAAA;AAE/C,MAAM,WAAW,eAAe;IAC9B;;;;OAIG;IACH,OAAO,CAAC,EAAE;QACR,IAAI,CAAC,EAAE,OAAO,CAAA;QACd,IAAI,CAAC,EAAE,OAAO,CAAA;KACf,CAAA;IACD;;;;;;;OAOG;IACH,IAAI,CAAC,EAAE;QACL,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;QACxB,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;QAC3B,GAAG,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;QACvB,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;KAC5B,CAAA;IACD,4DAA4D;IAC5D,WAAW,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;IAC7B;;;OAGG;IACH,QAAQ,CAAC,EACL,OAAO,GACP;QACE,GAAG,CAAC,EAAE,MAAM,CAAA;QACZ,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;QAC3B,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;QAC3B,UAAU,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QAC5B,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QACvB,QAAQ,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;QAC1B,SAAS,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;KAC5B,CAAA;IACL;;;OAGG;IACH,KAAK,CAAC,EAAE,MAAM,CAAA;IACd;;;OAGG;IACH,gBAAgB,CAAC,EAAE,MAAM,GAAG,oBAAoB,CAAA;IAChD;;;OAGG;IACH,IAAI,CAAC,EAAE,aAAa,CAAA;IACpB;;;OAGG;IACH,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;IAC3B,iCAAiC;IACjC,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB;;;OAGG;IACH,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAA;IACxB,4DAA0D;IAC1D,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB;AAED,MAAM,WAAW,uBAAuB;IACtC,OAAO,EAAE;QACP,IAAI,CAAC,EAAE,OAAO,CAAA;QACd,IAAI,CAAC,EAAE,OAAO,CAAA;KACf,CAAA;IACD,WAAW,EAAE,MAAM,CAAA;IACnB,QAAQ,EACJ;QACE,GAAG,EAAE,MAAM,CAAA;QACX,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAA;QAClB,UAAU,CAAC,EAAE,MAAM,CAAA;QACnB,KAAK,CAAC,EAAE,MAAM,CAAA;QACd,QAAQ,CAAC,EAAE,MAAM,CAAA;QACjB,SAAS,CAAC,EAAE,MAAM,CAAA;KACnB,GACD,SAAS,CAAA;IACb,IAAI,EAAE;QACJ,IAAI,EAAE,MAAM,EAAE,CAAA;QACd,OAAO,EAAE,MAAM,EAAE,CAAA;QACjB,GAAG,EAAE,MAAM,EAAE,CAAA;QACb,OAAO,EAAE,MAAM,EAAE,CAAA;KAClB,CAAA;IACD,gBAAgB,EAAE,MAAM,GAAG,oBAAoB,GAAG,SAAS,CAAA;IAC3D,OAAO,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;IAC7B,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,aAAa,CAAA;IACnB,KAAK,EAAE,MAAM,GAAG,SAAS,CAAA;IACzB,IAAI,EAAE,MAAM,EAAE,CAAA;IACd,KAAK,EAAE,OAAO,CAAA;CACf;AAED,wBAAsB,UAAU,CAAC,IAAI,GAAE,MAAM,EAA0B,EAAE,GAAG,SAAgB,oCAK3F;AAED,wBAAgB,oBAAoB,CAAC,OAAO,GAAE,MAAM,CAAC,WAA4B,GAAG,MAAM,CAmBzF"}

package/dist/lib/config.js

@@ -40,19 +40,23 @@ const cliOptions = {
     },
     'glob.browser': {
         type: 'string',
-        description: 'Glob pattern for browser test files',
+        multiple: true,
+        description: 'Glob pattern(s) for browser test files',
     },
     'glob.e2e': {
         type: 'string',
-        description: 'Glob pattern for E2E test files',
+        multiple: true,
+        description: 'Glob pattern(s) for E2E test files',
     },
     'glob.exclude': {
         type: 'string',
-        description: 'Glob pattern for paths to exclude from discovery',
+        multiple: true,
+        description: 'Glob pattern(s) for paths to exclude from discovery',
     },
     'glob.test': {
         type: 'string',
-        description: 'Glob pattern for all test files',
+        multiple: true,
+        description: 'Glob pattern(s) for all test files',
     },
     concurrency: {
         type: 'string',
@@ -109,7 +113,12 @@ const cliOptions = {
     project: {
         type: 'string',
         short: 'p',
-        description: 'Filter to a specific Playwright project (comma-separated)',
+        multiple: true,
+        description: 'Filter to specific Playwright project(s)',
+    },
+    pool: {
+        type: 'string',
+        description: 'Pool used to run server and E2E test files: forks, threads (default: forks)',
     },
     reporter: {
         type: 'string',
@@ -119,7 +128,8 @@ const cliOptions = {
     type: {
         type: 'string',
         short: 't',
-        description: 'Comma-separated test types to run (default: server,browser,e2e)',
+        multiple: true,
+        description: 'Test types to run (default: server, browser, e2e)',
     },
     watch: {
         type: 'boolean',
@@ -143,16 +153,17 @@ const defaultValues = {
         functions: undefined,
     },
     glob: {
-        test: '**/*.test{,.e2e,.browser}.{ts,tsx}',
-        browser: '**/*.test.browser.{ts,tsx}',
-        e2e: '**/*.test.e2e.{ts,tsx}',
-        exclude: 'node_modules/**',
+        test: ['**/*.test{,.e2e,.browser}.{ts,tsx}'],
+        browser: ['**/*.test.browser.{ts,tsx}'],
+        e2e: ['**/*.test.e2e.{ts,tsx}'],
+        exclude: ['node_modules/**'],
     },
-    reporter: process.env.CI === 'true' ? 'files' : 'spec',
-    type: 'server,browser,e2e',
-    setup: undefined,
+    pool: 'forks',
     playwrightConfig: undefined,
     project: undefined,
+    reporter: process.env.CI === 'true' ? 'files' : 'spec',
+    setup: undefined,
+    type: ['server', 'browser', 'e2e'],
     watch: false,
 };
 export async function loadConfig(args = process.argv.slice(2), cwd = process.cwd()) {
@@ -163,10 +174,10 @@ export async function loadConfig(args = process.argv.slice(2), cwd = process.cwd
 }
 export function getRemixTestHelpText(_target = process.stdout) {
     let lines = [
-        'Usage: remix-test [glob] [options]',
+        'Usage: remix-test [glob...] [options]',
         '',
         'Arguments:',
-        `  glob                     Glob pattern for test files (default: "${defaultValues.glob.test}")`,
+        `  glob                     Glob pattern(s) for test files (default: "${defaultValues.glob.test.join(', ')}")`,
         '',
         'Options:',
     ];
@@ -181,17 +192,25 @@ export function getRemixTestHelpText(_target = process.stdout) {
 function parseCliArgs(args) {
     return util.parseArgs({ args, options: cliOptions, allowPositionals: true });
 }
+function toArray(value) {
+    return Array.isArray(value) ? [...value] : [value];
+}
+function toCommaSeparatedArray(value) {
+    return toArray(value).flatMap((item) => item
+        .split(',')
+        .map((part) => part.trim())
+        .filter(Boolean));
+}
 function resolveConfig(fileConfig, { values: cliValues, positionals }) {
     let fileCoverage = typeof fileConfig.coverage === 'boolean' ? {} : fileConfig.coverage || {};
     return {
         glob: {
-            test: positionals[0] ??
-                cliValues['glob.test'] ??
-                fileConfig.glob?.test ??
-                defaultValues.glob.test,
-            browser: cliValues['glob.browser'] ?? fileConfig.glob?.browser ?? defaultValues.glob.browser,
-            e2e: cliValues['glob.e2e'] ?? fileConfig.glob?.e2e ?? defaultValues.glob.e2e,
-            exclude: cliValues['glob.exclude'] ?? fileConfig.glob?.exclude ?? defaultValues.glob.exclude,
+            test: toArray(positionals.length > 0
+                ? positionals
+                : (cliValues['glob.test'] ?? fileConfig.glob?.test ?? defaultValues.glob.test)),
+            browser: toArray(cliValues['glob.browser'] ?? fileConfig.glob?.browser ?? defaultValues.glob.browser),
+            e2e: toArray(cliValues['glob.e2e'] ?? fileConfig.glob?.e2e ?? defaultValues.glob.e2e),
+            exclude: toArray(cliValues['glob.exclude'] ?? fileConfig.glob?.exclude ?? defaultValues.glob.exclude),
         },
         browser: {
             echo: cliValues['browser.echo'] ?? fileConfig.browser?.echo ?? defaultValues.browser.echo,
@@ -201,12 +220,18 @@ function resolveConfig(fileConfig, { values: cliValues, positionals }) {
         coverage: cliValues.coverage === true || !!fileConfig.coverage
             ? {
                 dir: cliValues['coverage.dir'] ?? fileCoverage.dir ?? defaultValues.coverage.dir,
-                include: cliValues['coverage.include'] ??
-                    fileCoverage.include ??
-                    defaultValues.coverage.include,
-                exclude: cliValues['coverage.exclude'] ??
-                    fileCoverage.exclude ??
-                    defaultValues.coverage.exclude,
+                include: (() => {
+                    let raw = cliValues['coverage.include'] ??
+                        fileCoverage.include ??
+                        defaultValues.coverage.include;
+                    return raw === undefined ? undefined : toArray(raw);
+                })(),
+                exclude: (() => {
+                    let raw = cliValues['coverage.exclude'] ??
+                        fileCoverage.exclude ??
+                        defaultValues.coverage.exclude;
+                    return raw === undefined ? undefined : toArray(raw);
+                })(),
                 statements: cliValues['coverage.statements'] !== undefined
                     ? Number(cliValues['coverage.statements'])
                     : fileCoverage.statements !== undefined
@@ -231,12 +256,22 @@ function resolveConfig(fileConfig, { values: cliValues, positionals }) {
             : undefined,
         setup: cliValues.setup ?? fileConfig.setup ?? defaultValues.setup,
         playwrightConfig: cliValues.playwrightConfig ?? fileConfig.playwrightConfig ?? defaultValues.playwrightConfig,
-        project: cliValues.project ?? fileConfig.project ?? defaultValues.project,
+        pool: resolvePool(cliValues.pool ?? fileConfig.pool ?? defaultValues.pool),
+        project: (() => {
+            let raw = cliValues.project ?? fileConfig.project ?? defaultValues.project;
+            return raw === undefined ? undefined : toCommaSeparatedArray(raw);
+        })(),
         reporter: cliValues.reporter ?? fileConfig.reporter ?? defaultValues.reporter,
-        type: cliValues.type ?? fileConfig.type ?? defaultValues.type,
+        type: toCommaSeparatedArray(cliValues.type ?? fileConfig.type ?? defaultValues.type),
         watch: cliValues.watch ?? fileConfig.watch ?? defaultValues.watch,
     };
 }
+function resolvePool(value) {
+    if (value === 'forks' || value === 'threads') {
+        return value;
+    }
+    throw new Error(`Unsupported test pool "${value}". Supported pools are: forks, threads`);
+}
 async function loadConfigFile(configPath, cwd) {
     let candidates = configPath
         ? [path.resolve(cwd, configPath)]
@@ -244,12 +279,17 @@ async function loadConfigFile(configPath, cwd) {
     for (let candidate of candidates) {
         try {
             await fsp.access(candidate);
-            let mod = await importModule(candidate, import.meta);
-            return mod.default ?? mod;
         }
         catch {
-            // not found or failed to load — try next
+            // not found — try the next candidate
+            continue;
         }
+        // The file exists; let import errors propagate rather than silently
+        // falling through to defaults — that masking is what hid "Windows
+        // absolute paths aren't valid ESM specifiers" by classifying every
+        // browser test as a server test.
+        let mod = await importModule(candidate, import.meta);
+        return mod.default ?? mod;
     }
     return {};
 }