Revert "perf(test): parallelize harder — unlock worker count, split build-output, bump workspace concurrency"
This reverts commit 972ad0f9fa55dc1419660aba4758a9d3c325919a.
This commit is contained in:
@@ -22,10 +22,9 @@
|
|||||||
"verify:workspace": "pnpm lint && pnpm test && pnpm build",
|
"verify:workspace": "pnpm lint && pnpm test && pnpm build",
|
||||||
"build:exe": "pnpm build && pnpm --filter @runfusion/fusion build:exe",
|
"build:exe": "pnpm build && pnpm --filter @runfusion/fusion build:exe",
|
||||||
"build:exe:all": "pnpm build && pnpm --filter @runfusion/fusion build:exe:all",
|
"build:exe:all": "pnpm build && pnpm --filter @runfusion/fusion build:exe:all",
|
||||||
"test": "pnpm -r --workspace-concurrency=4 test",
|
"test": "VITEST_MAX_WORKERS=4 pnpm -r --workspace-concurrency=2 test",
|
||||||
"test:serial": "pnpm -r --workspace-concurrency=1 test",
|
"test:serial": "pnpm -r --workspace-concurrency=1 test",
|
||||||
"test:fast": "pnpm -r --workspace-concurrency=4 test",
|
"test:fast": "VITEST_MAX_WORKERS=4 pnpm -r --workspace-concurrency=2 test",
|
||||||
"test:build": "pnpm --filter @fusion/dashboard test:build",
|
|
||||||
"test:isolated": "node scripts/check-test-isolation.mjs --before && pnpm test && node scripts/check-test-isolation.mjs",
|
"test:isolated": "node scripts/check-test-isolation.mjs --before && pnpm test && node scripts/check-test-isolation.mjs",
|
||||||
"test:check-isolation": "node scripts/check-test-isolation.mjs",
|
"test:check-isolation": "node scripts/check-test-isolation.mjs",
|
||||||
"test:coverage": "pnpm -r --workspace-concurrency=1 exec vitest run --silent=passed-only --reporter=dot --coverage",
|
"test:coverage": "pnpm -r --workspace-concurrency=1 exec vitest run --silent=passed-only --reporter=dot --coverage",
|
||||||
|
|||||||
@@ -1,12 +1,9 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { resolve } from "node:path";
|
import { resolve } from "node:path";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
// Use all-but-one core by default. Override with VITEST_MAX_WORKERS for
|
const defaultMaxWorkers = 2;
|
||||||
// constrained environments (CI runners, laptops on battery, etc.).
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,10 +1,9 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { resolve } from "node:path";
|
import { resolve } from "node:path";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -37,8 +37,7 @@
|
|||||||
"dev": "pnpm build && pnpm typecheck && pnpm dev:serve",
|
"dev": "pnpm build && pnpm typecheck && pnpm dev:serve",
|
||||||
"dev:serve": "vite dev",
|
"dev:serve": "vite dev",
|
||||||
"postinstall": "chmod +x node_modules/.pnpm/node-pty*/node_modules/node-pty/prebuilds/darwin-*/spawn-helper node_modules/.pnpm/node-pty*/node_modules/node-pty/prebuilds/darwin-*/*.node 2>/dev/null || true",
|
"postinstall": "chmod +x node_modules/.pnpm/node-pty*/node_modules/node-pty/prebuilds/darwin-*/spawn-helper node_modules/.pnpm/node-pty*/node_modules/node-pty/prebuilds/darwin-*/*.node 2>/dev/null || true",
|
||||||
"test": "vitest run --silent=passed-only --reporter=dot --exclude '**/build-output.test.ts'",
|
"test": "vitest run --silent=passed-only --reporter=dot",
|
||||||
"test:build": "vitest run --silent=passed-only --reporter=dot app/__tests__/build-output.test.ts",
|
|
||||||
"typecheck": "tsc --noEmit && tsc --noEmit -p tsconfig.app.json"
|
"typecheck": "tsc --noEmit && tsc --noEmit -p tsconfig.app.json"
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
|
|||||||
@@ -1,11 +1,10 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import react from "@vitejs/plugin-react";
|
import react from "@vitejs/plugin-react";
|
||||||
import { resolve } from "node:path";
|
import { resolve } from "node:path";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,9 +1,8 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,10 +1,9 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { resolve } from "node:path";
|
import { resolve } from "node:path";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,9 +1,8 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,9 +1,7 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? "2", 10);
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : 2));
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
|
|
||||||
export default defineConfig({
|
export default defineConfig({
|
||||||
|
|||||||
@@ -1,10 +1,9 @@
|
|||||||
import { defineConfig } from "vitest/config";
|
import { defineConfig } from "vitest/config";
|
||||||
import { fileURLToPath } from "node:url";
|
import { fileURLToPath } from "node:url";
|
||||||
import { cpus } from "node:os";
|
|
||||||
|
|
||||||
const defaultMaxWorkers = Math.max(1, cpus().length - 1);
|
const defaultMaxWorkers = 2;
|
||||||
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
const requestedMaxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
|
||||||
const maxWorkers = Math.max(1, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers);
|
const maxWorkers = Math.max(1, Math.min(4, Number.isFinite(requestedMaxWorkers) ? requestedMaxWorkers : defaultMaxWorkers));
|
||||||
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
process.env.VITEST_MAX_WORKERS = String(maxWorkers);
|
||||||
const coreSourceEntry = fileURLToPath(new URL("../core/src/index.ts", import.meta.url));
|
const coreSourceEntry = fileURLToPath(new URL("../core/src/index.ts", import.meta.url));
|
||||||
const testUtilsEntry = fileURLToPath(new URL("../core/src/__test-utils__/workspace.ts", import.meta.url));
|
const testUtilsEntry = fileURLToPath(new URL("../core/src/__test-utils__/workspace.ts", import.meta.url));
|
||||||
|
|||||||
Reference in New Issue
Block a user