perf(test): reduce hotspot reload churn (#59033)

This commit is contained in:
Vincent Koc
2026-04-01 22:19:19 +09:00
committed by GitHub
parent adb961e056
commit 7a7549f12f
8 changed files with 29 additions and 19 deletions

View File

@@ -1,4 +1,4 @@
import { beforeEach, describe, expect, it, vi } from "vitest";
import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
const mocks = vi.hoisted(() => ({
ensureBrowserControlAuth: vi.fn(async () => ({ generatedToken: false })),
@@ -45,12 +45,14 @@ vi.mock("./runtime-lifecycle.js", () => ({
let startBrowserControlServiceFromConfig: typeof import("../control-service.js").startBrowserControlServiceFromConfig;
describe("startBrowserControlServiceFromConfig", () => {
beforeEach(async () => {
beforeAll(async () => {
({ startBrowserControlServiceFromConfig } = await import("../control-service.js"));
});
beforeEach(() => {
mocks.ensureBrowserControlAuth.mockClear();
mocks.createBrowserRuntimeState.mockClear();
mocks.loadConfig.mockClear();
vi.resetModules();
({ startBrowserControlServiceFromConfig } = await import("../control-service.js"));
});
it("does not start the default service when the browser plugin is disabled", async () => {

View File

@@ -58,7 +58,6 @@ async function createWorkProfileWithConfig(params: {
describe("BrowserProfilesService", () => {
beforeAll(async () => {
vi.resetModules();
({ resolveBrowserConfig } = await import("./config.js"));
({ createBrowserProfilesService } = await import("./profiles-service.js"));
});

View File

@@ -1,5 +1,5 @@
import fs from "node:fs";
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
import type { BrowserServerState } from "./server-context.js";
vi.mock("./chrome-mcp.js", () => ({
@@ -62,12 +62,15 @@ afterEach(() => {
vi.clearAllMocks();
});
beforeEach(async () => {
vi.resetModules();
beforeAll(async () => {
({ createBrowserRouteContext } = await import("./server-context.js"));
chromeMcp = await import("./chrome-mcp.js");
});
beforeEach(() => {
vi.clearAllMocks();
});
describe("browser server-context existing-session profile", () => {
it("routes tab operations through the Chrome MCP backend", async () => {
fs.mkdirSync("/tmp/brave-profile", { recursive: true });

View File

@@ -1,4 +1,4 @@
import { beforeEach, describe, expect, it, vi } from "vitest";
import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
const { stopOpenClawChromeMock } = vi.hoisted(() => ({
stopOpenClawChromeMock: vi.fn(async () => {}),
@@ -21,10 +21,12 @@ vi.mock("./server-context.js", () => ({
let ensureExtensionRelayForProfiles: typeof import("./server-lifecycle.js").ensureExtensionRelayForProfiles;
let stopKnownBrowserProfiles: typeof import("./server-lifecycle.js").stopKnownBrowserProfiles;
beforeEach(async () => {
vi.resetModules();
beforeAll(async () => {
({ ensureExtensionRelayForProfiles, stopKnownBrowserProfiles } =
await import("./server-lifecycle.js"));
});
beforeEach(() => {
createBrowserRouteContextMock.mockClear();
listKnownProfileNamesMock.mockClear();
stopOpenClawChromeMock.mockClear();