@hughescr/stryker-bun-runner 1.1.0 → 1.1.2

package/README.md

@@ -100,6 +100,17 @@ bunx stryker run
 
 - **Sequential execution required** - Tests run with `--concurrency=1` to ensure accurate coverage tracking. This is slower than parallel execution but necessary for correct test-to-mutant correlation.
 
+## Concurrent Tests
+
+This plugin automatically patches `describe.concurrent()`, `test.concurrent()`, and `it.concurrent()` to run sequentially during mutation testing. Your tests will work without modification.
+
+**Why?** Coverage tracking requires knowing which test exercised which code. With concurrent execution, the `beforeEach` hook assigns test IDs in the order tests *start*, but coverage is recorded in the order tests *complete*. These orders differ with concurrency, causing coverage to be attributed to the wrong tests.
+
+**What this means:**
+- ✅ Your `.concurrent()` tests work automatically with Stryker
+- ✅ Normal test runs (without Stryker) still use concurrent execution
+- ⏱️ Mutation testing runs are slower due to sequential execution
+
 ## License
 
 Apache-2.0

package/dist/index.js

@@ -2946,7 +2946,6 @@ async function runBunTests(options) {
   if (options.sequentialMode) {
     args.push("--concurrency=1");
   }
-  args.push("--no-randomize");
   if (options.bunArgs && options.bunArgs.length > 0) {
     args.push(...options.bunArgs);
   }
@@ -3766,7 +3765,7 @@ class BunTestRunner {
           id: fullName,
           name: fullName,
           fileName: normalizeTestFilePath(testInfo.url),
-          startPosition: undefined,
+          startPosition: testInfo.line !== undefined ? { line: testInfo.line, column: 0 } : undefined,
           status: TestStatus.Failed,
           failureMessage: parsedTest?.failureMessage ?? testInfo.error?.message ?? "Test failed",
           timeSpentMs: elapsed
@@ -3777,7 +3776,7 @@ class BunTestRunner {
           id: fullName,
           name: fullName,
           fileName: normalizeTestFilePath(testInfo.url),
-          startPosition: undefined,
+          startPosition: testInfo.line !== undefined ? { line: testInfo.line, column: 0 } : undefined,
           status: TestStatus.Skipped,
           timeSpentMs: elapsed
         };
@@ -3786,7 +3785,7 @@ class BunTestRunner {
         id: fullName,
         name: fullName,
         fileName: normalizeTestFilePath(testInfo.url),
-        startPosition: undefined,
+        startPosition: testInfo.line !== undefined ? { line: testInfo.line, column: 0 } : undefined,
         status: TestStatus.Success,
         timeSpentMs: elapsed
       };

package/dist/templates/coverage-preload.ts

@@ -4,7 +4,7 @@
  * Note: This is a template file with a placeholder import that gets replaced at runtime
  */
 
-import { beforeEach, afterEach, afterAll } from 'bun:test';
+import { beforeEach, afterEach, afterAll, describe, test, it } from 'bun:test';
 import {
     getPreloadConfig,
     shouldCollectCoverage as shouldCollect,
@@ -17,6 +17,14 @@ import {
     type StrykerNamespace
 } from '__PRELOAD_LOGIC_PATH__';
 
+// Patch .concurrent() to be regular sequential execution
+// This ensures accurate coverage tracking during mutation testing
+// Users can still use .concurrent() for faster normal test runs
+// Use Object.defineProperty to override readonly properties
+Object.defineProperty(describe, 'concurrent', { value: describe, writable: true, configurable: true });
+Object.defineProperty(test, 'concurrent', { value: test, writable: true, configurable: true });
+Object.defineProperty(it, 'concurrent', { value: it, writable: true, configurable: true });
+
 interface StrykerGlobal {
     [key: string]:       unknown
     __stryker__?:        StrykerNamespace

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hughescr/stryker-bun-runner",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Stryker test runner plugin for Bun with perTest coverage support",
   "keywords": [
     "stryker",