mirror of
https://github.com/tiennm99/ccs.git
synced 2026-10-12 22:14:22 +00:00
perf(test-bucket): only serialize slow bucket, parallelize fast
Reviewer noted `--max-concurrency=1` was applied to both buckets. Slow bucket needs it (subprocesses, ports, shared state → flaky in parallel) but fast bucket was unnecessarily capped. Remove the flag for fast; keep for slow. Verified: test:fast still green (2510 pass, 14.4s locally).
This commit is contained in:
1 parent
eab2b48f2c
commit
6b5c74aa4b
1 file changed
+12
-9
@@ -125,15 +125,18 @@ function runBucket(name) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
const result = spawnSync(
|
// Slow bucket forces sequential execution because it spawns subprocesses,
|
||||||
'bun',
|
// binds ports, and touches shared state — parallelism causes flakes.
|
||||||
['test', '--max-concurrency=1', ...selected],
|
// Fast bucket keeps bun's default parallelism for speed.
|
||||||
{
|
const bunArgs = name === 'slow'
|
||||||
cwd: rootDir,
|
? ['test', '--max-concurrency=1', ...selected]
|
||||||
stdio: 'inherit',
|
: ['test', ...selected];
|
||||||
shell: process.platform === 'win32',
|
|
||||||
},
|
const result = spawnSync('bun', bunArgs, {
|
||||||
);
|
cwd: rootDir,
|
||||||
|
stdio: 'inherit',
|
||||||
|
shell: process.platform === 'win32',
|
||||||
|
});
|
||||||
|
|
||||||
return result.status ?? 1;
|
return result.status ?? 1;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in new issue
Block a user