root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): fix test failures from recent feature additions

Browse Source 
- Add ChatStore mock to all dashboard route tests that mock @fusion/core,
  since server.ts now instantiates ChatStore(store.getFusionDir(), ...)
- Add getFusionDir to createMockStore in server.test.ts
- Gate AI session cleanup scheduling behind shouldScheduleAiSessionCleanup()
  (returns false in test env) to prevent open handle warnings
- Fix desktop tests: DASHBOARD_URL is now exported as a function alias,
  update assertions to call DASHBOARD_URL() instead of using as string
- Add node:os mocks to system-metrics.test.ts for deterministic results
- Replace hardcoded maxWorkers=16 with availableParallelism()-based
  calculation in all vitest configs to prevent OOM on 2-core CI runners
- Add --workspace-concurrency=2 to pnpm test commands
- Fix TaskCard tests: update mission badge title assertions to full titles
- Remove unused /api/mesh/state route
- Fix plugin-auto-label: add isError field, async onTaskCreated, "tests" keyword
- Fix plugin-ci-status: add module-level logger, tighten test assertions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
gsxdsm
2026-04-10 08:01:03 -07:00
parent 71cad2ff53
commit 9830ea96c6
29 changed files with 214 additions and 121 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/desktop/src/__tests__/main-integration.test.ts
View File

@@ -160,7 +160,7 @@ vi.mock("../renderer.js", () => ({
getRendererFilePath: vi.fn(() => "/path/to/dist/client/index.html"),
isUrlRenderer: vi.fn(() => false),
IS_DEVELOPMENT: false,
DASHBOARD_URL: "file:///path/to/dist/client/index.html",
DASHBOARD_URL: vi.fn(() => "file:///path/to/dist/client/index.html"),
}));
async function importMainModule() {

2
packages/desktop/src/__tests__/main.integration.test.ts
View File

@@ -83,7 +83,7 @@ vi.mock("../renderer.js", () => ({
getRendererFilePath: vi.fn(() => "/path/to/dist/client/index.html"),
isUrlRenderer: vi.fn(() => false),
IS_DEVELOPMENT: false,
DASHBOARD_URL: "file:///path/to/dist/client/index.html",
DASHBOARD_URL: vi.fn(() => "file:///path/to/dist/client/index.html"),
}));
describe("main module integration", () => {

53
packages/desktop/src/__tests__/main.test.ts
View File

@@ -52,6 +52,11 @@ const mocks = vi.hoisted(() => {
openExternal: vi.fn(() => Promise.resolve()),
};
const isDevelopmentMode = vi.fn(() => false);
const getRendererUrl = vi.fn(() => "file:///path/to/dist/client/index.html");
const getRendererFilePath = vi.fn(() => "/path/to/dist/client/index.html");
const isUrlRenderer = vi.fn(() => false);
return {
app,
BrowserWindow,
@@ -62,6 +67,10 @@ const mocks = vi.hoisted(() => {
nativeImage,
shell,
browserWindowInstance,
isDevelopmentMode,
getRendererUrl,
getRendererFilePath,
isUrlRenderer,
};
});
@@ -77,12 +86,12 @@ vi.mock("electron", () => ({
// Mock renderer module
vi.mock("../renderer.js", () => ({
isDevelopmentMode: vi.fn(() => false),
getRendererUrl: vi.fn(() => "file:///path/to/dist/client/index.html"),
getRendererFilePath: vi.fn(() => "/path/to/dist/client/index.html"),
isUrlRenderer: vi.fn(() => false),
isDevelopmentMode: mocks.isDevelopmentMode,
getRendererUrl: mocks.getRendererUrl,
getRendererFilePath: mocks.getRendererFilePath,
isUrlRenderer: mocks.isUrlRenderer,
IS_DEVELOPMENT: false,
DASHBOARD_URL: "file:///path/to/dist/client/index.html",
DASHBOARD_URL: mocks.getRendererUrl,
}));
async function importMainModule() {
@@ -107,10 +116,10 @@ describe("main process", () => {
process.env.NODE_ENV = originalNodeEnv;
}
// Ensure we're in production mode for these tests
vi.mocked(require("../renderer.js")).isDevelopmentMode.mockReturnValue(false);
vi.mocked(require("../renderer.js")).getRendererUrl.mockReturnValue("file:///path/to/dist/client/index.html");
vi.mocked(require("../renderer.js")).getRendererFilePath.mockReturnValue("/path/to/dist/client/index.html");
vi.mocked(require("../renderer.js")).isUrlRenderer.mockReturnValue(false);
mocks.isDevelopmentMode.mockReturnValue(false);
mocks.getRendererUrl.mockReturnValue("file:///path/to/dist/client/index.html");
mocks.getRendererFilePath.mockReturnValue("/path/to/dist/client/index.html");
mocks.isUrlRenderer.mockReturnValue(false);
});
it("DASHBOARD_URL defaults to local file URL in production mode", async () => {
@@ -118,21 +127,21 @@ describe("main process", () => {
const { DASHBOARD_URL } = await importMainModule();
expect(DASHBOARD_URL.startsWith("file://")).toBe(true);
expect(DASHBOARD_URL).toContain("/client/index.html");
expect(DASHBOARD_URL()).toMatch(/^file:\/\//);
expect(DASHBOARD_URL()).toContain("/client/index.html");
});
it("DASHBOARD_URL uses env override in development mode", async () => {
process.env.FUSION_DASHBOARD_URL = "http://localhost:5050";
// Mock development mode to use the env var
vi.mocked(require("../renderer.js")).isDevelopmentMode.mockReturnValue(true);
vi.mocked(require("../renderer.js")).getRendererUrl.mockReturnValue("http://localhost:5050");
vi.mocked(require("../renderer.js")).getRendererFilePath.mockReturnValue("");
vi.mocked(require("../renderer.js")).isUrlRenderer.mockReturnValue(true);
mocks.isDevelopmentMode.mockReturnValue(true);
mocks.getRendererUrl.mockReturnValue("http://localhost:5050");
mocks.getRendererFilePath.mockReturnValue("");
mocks.isUrlRenderer.mockReturnValue(true);
const { DASHBOARD_URL } = await importMainModule();
expect(DASHBOARD_URL).toBe("http://localhost:5050");
expect(DASHBOARD_URL()).toBe("http://localhost:5050");
});
it("createMainWindow creates BrowserWindow with secure preferences", async () => {
@@ -157,9 +166,9 @@ describe("main process", () => {
});
it("createMainWindow loads the renderer URL in URL mode", async () => {
vi.mocked(require("../renderer.js")).isUrlRenderer.mockReturnValue(true);
vi.mocked(require("../renderer.js")).getRendererUrl.mockReturnValue("http://localhost:3000/index.html");
vi.mocked(require("../renderer.js")).getRendererFilePath.mockReturnValue("");
mocks.isUrlRenderer.mockReturnValue(true);
mocks.getRendererUrl.mockReturnValue("http://localhost:3000/index.html");
mocks.getRendererFilePath.mockReturnValue("");
const { createMainWindow } = await importMainModule();
@@ -170,9 +179,9 @@ describe("main process", () => {
});
it("createMainWindow loads the renderer file in file mode (production)", async () => {
vi.mocked(require("../renderer.js")).isUrlRenderer.mockReturnValue(false);
vi.mocked(require("../renderer.js")).getRendererUrl.mockReturnValue("file:///path/to/dist/client/index.html");
vi.mocked(require("../renderer.js")).getRendererFilePath.mockReturnValue("/path/to/dist/client/index.html");
mocks.isUrlRenderer.mockReturnValue(false);
mocks.getRendererUrl.mockReturnValue("file:///path/to/dist/client/index.html");
mocks.getRendererFilePath.mockReturnValue("/path/to/dist/client/index.html");
const { createMainWindow } = await importMainModule();

4
packages/desktop/vitest.config.ts
View File

@@ -1,6 +1,8 @@
import { defineConfig } from "vitest/config";
import { availableParallelism } from "node:os";
const maxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? "16", 10);
const defaultMaxWorkers = Math.max(1, Math.min(4, Math.ceil(availableParallelism() / 8)));
const maxWorkers = Number.parseInt(process.env.VITEST_MAX_WORKERS ?? String(defaultMaxWorkers), 10);
export default defineConfig({
test: {