mirror of
https://github.com/multica-ai/multica.git
synced 2026-07-25 20:15:37 +02:00
* fix(desktop): contain renderer crashes Co-authored-by: multica-agent <github@multica.ai> * fix(desktop): filter renderer exit prompts Co-authored-by: multica-agent <github@multica.ai> * refactor(desktop): drop redundant page-level ErrorBoundary on issue detail The whole-page <ErrorBoundary> wrapper duplicated the new route-level errorElement (DesktopRouteErrorPage). Let render errors bubble to the root route boundary so all detail routes are contained the same way. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * feat(desktop): add Close tab escape to route error page Reload tab recreates the same crashing path and Go to issues is a dead end when the issues route itself crashed. Add a Close tab action that destroys the crashing router entirely and falls back to a sibling tab (or a reseeded default), the only always-safe escape regardless of which route crashed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: multica-agent <github@multica.ai> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
97 lines
3.2 KiB
TypeScript
97 lines
3.2 KiB
TypeScript
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
|
import { render, screen } from "@testing-library/react";
|
|
import { forwardRef, useImperativeHandle } from "react";
|
|
|
|
let storedDraftMessage = "saved draft";
|
|
|
|
vi.mock("react-i18next", () => ({
|
|
useTranslation: () => ({ t: (key: string) => key, i18n: { changeLanguage: vi.fn() } }),
|
|
Trans: ({ children }: { children: any }) => children,
|
|
initReactI18next: { type: "3rdParty", init: vi.fn() },
|
|
}));
|
|
|
|
vi.mock("../i18n", () => ({
|
|
useT: () => ({
|
|
t: (selector: (resources: any) => string) =>
|
|
selector({
|
|
feedback: {
|
|
title: "Feedback",
|
|
github_hint_prefix: "Prefer GitHub? ",
|
|
github_hint_link: "Open an issue",
|
|
placeholder: "Tell us what happened",
|
|
toast_uploading: "Uploading",
|
|
toast_too_long: "Too long",
|
|
toast_sent: "Sent",
|
|
toast_failed: "Failed",
|
|
sending: "Sending",
|
|
send: "Send",
|
|
},
|
|
}),
|
|
}),
|
|
}));
|
|
|
|
vi.mock("@multica/core/paths", () => ({ useCurrentWorkspace: () => ({ id: "ws1" }) }));
|
|
vi.mock("@multica/core/hooks/use-file-upload", () => ({
|
|
useFileUpload: () => ({ uploadWithToast: vi.fn() }),
|
|
}));
|
|
vi.mock("@multica/core/api", () => ({ api: {} }));
|
|
vi.mock("@multica/core/analytics", () => ({ captureFeedbackOpened: vi.fn() }));
|
|
vi.mock("sonner", () => ({ toast: { info: vi.fn(), error: vi.fn(), success: vi.fn() } }));
|
|
vi.mock("@multica/core/platform", () => ({
|
|
formatShortcut: () => "⌘↵",
|
|
modKey: "mod",
|
|
enterKey: "enter",
|
|
}));
|
|
vi.mock("@multica/core/feedback", () => ({
|
|
useCreateFeedback: () => ({ isPending: false, mutateAsync: vi.fn() }),
|
|
useFeedbackDraftStore: (selector: any) =>
|
|
selector({ draft: { message: storedDraftMessage }, setDraft: vi.fn(), clearDraft: vi.fn() }),
|
|
}));
|
|
vi.mock("../editor", () => {
|
|
const ContentEditor = forwardRef(({ defaultValue }: any, ref) => {
|
|
useImperativeHandle(ref, () => ({
|
|
hasActiveUploads: () => false,
|
|
getMarkdown: () => defaultValue,
|
|
uploadFile: vi.fn(),
|
|
}));
|
|
return <textarea aria-label="feedback editor" defaultValue={defaultValue} />;
|
|
});
|
|
ContentEditor.displayName = "MockContentEditor";
|
|
return {
|
|
ContentEditor,
|
|
useFileDropZone: () => ({ isDragOver: false, dropZoneProps: {} }),
|
|
FileDropOverlay: () => null,
|
|
FileUploadButton: () => <button type="button">Upload</button>,
|
|
};
|
|
});
|
|
|
|
import { FeedbackModal } from "./feedback";
|
|
|
|
describe("FeedbackModal", () => {
|
|
beforeEach(() => {
|
|
vi.spyOn(console, "error").mockImplementation(() => {});
|
|
});
|
|
|
|
afterEach(() => {
|
|
vi.restoreAllMocks();
|
|
});
|
|
|
|
it("uses a crash-report initialMessage when there is no saved draft", () => {
|
|
storedDraftMessage = "";
|
|
|
|
render(<FeedbackModal onClose={vi.fn()} initialMessage="kind: desktop_route_error" />);
|
|
|
|
expect(screen.getByLabelText("feedback editor")).toHaveValue("kind: desktop_route_error");
|
|
});
|
|
|
|
it("does not overwrite an existing feedback draft when crash report context is provided", () => {
|
|
storedDraftMessage = "saved draft";
|
|
|
|
render(<FeedbackModal onClose={vi.fn()} initialMessage="kind: desktop_route_error" />);
|
|
|
|
expect(screen.getByLabelText("feedback editor")).toHaveValue(
|
|
"saved draft\n\n---\n\nkind: desktop_route_error",
|
|
);
|
|
});
|
|
});
|