/** * @vitest-environment jsdom */ import { cleanup, render, screen, waitFor } from "@testing-library/react"; import userEvent from "@testing-library/user-event"; import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import { apiClient } from "../api/client"; vi.mock("../api/client", () => ({ apiClient: { getAuthHeader: vi.fn(), }, ApiClientError: class ApiClientError extends Error { constructor( message: string, public status: number, public code?: string, ) { super(message); this.name = "ApiClientError"; } }, })); // Import after mock is set up import { EditNoteTypeModal } from "./EditNoteTypeModal"; // Mock fetch globally const mockFetch = vi.fn(); global.fetch = mockFetch; describe("EditNoteTypeModal", () => { const mockNoteType = { id: "note-type-123", name: "Basic", frontTemplate: "{{Front}}", backTemplate: "{{Back}}", isReversible: false, }; const defaultProps = { isOpen: true, noteType: mockNoteType, onClose: vi.fn(), onNoteTypeUpdated: vi.fn(), }; beforeEach(() => { vi.clearAllMocks(); vi.mocked(apiClient.getAuthHeader).mockReturnValue({ Authorization: "Bearer access-token", }); }); afterEach(() => { cleanup(); vi.restoreAllMocks(); }); it("does not render when closed", () => { render(); expect(screen.queryByRole("dialog")).toBeNull(); }); it("does not render when noteType is null", () => { render(); expect(screen.queryByRole("dialog")).toBeNull(); }); it("renders modal when open with noteType", () => { render(); expect(screen.getByRole("dialog")).toBeDefined(); expect( screen.getByRole("heading", { name: "Edit Note Type" }), ).toBeDefined(); expect(screen.getByLabelText("Name")).toBeDefined(); expect(screen.getByLabelText("Front Template")).toBeDefined(); expect(screen.getByLabelText("Back Template")).toBeDefined(); expect(screen.getByLabelText("Create reversed cards")).toBeDefined(); expect(screen.getByRole("button", { name: "Cancel" })).toBeDefined(); expect(screen.getByRole("button", { name: "Save Changes" })).toBeDefined(); }); it("populates form with noteType data", () => { render(); expect(screen.getByLabelText("Name")).toHaveProperty("value", "Basic"); expect(screen.getByLabelText("Front Template")).toHaveProperty( "value", "{{Front}}", ); expect(screen.getByLabelText("Back Template")).toHaveProperty( "value", "{{Back}}", ); expect(screen.getByLabelText("Create reversed cards")).toHaveProperty( "checked", false, ); }); it("populates form with reversible noteType", () => { const reversibleNoteType = { ...mockNoteType, isReversible: true, }; render( , ); expect(screen.getByLabelText("Create reversed cards")).toHaveProperty( "checked", true, ); }); it("disables save button when name is empty", async () => { const user = userEvent.setup(); render(); const nameInput = screen.getByLabelText("Name"); await user.clear(nameInput); const saveButton = screen.getByRole("button", { name: "Save Changes" }); expect(saveButton).toHaveProperty("disabled", true); }); it("calls onClose when Cancel is clicked", async () => { const user = userEvent.setup(); const onClose = vi.fn(); render(); await user.click(screen.getByRole("button", { name: "Cancel" })); expect(onClose).toHaveBeenCalledTimes(1); }); it("calls onClose when clicking outside the modal", async () => { const user = userEvent.setup(); const onClose = vi.fn(); render(); const dialog = screen.getByRole("dialog"); await user.click(dialog); expect(onClose).toHaveBeenCalledTimes(1); }); it("updates noteType when Save Changes is clicked", async () => { const user = userEvent.setup(); const onClose = vi.fn(); const onNoteTypeUpdated = vi.fn(); mockFetch.mockResolvedValue({ ok: true, json: async () => ({ noteType: { ...mockNoteType, name: "Updated Basic", isReversible: true, }, }), }); render( , ); // Update fields const nameInput = screen.getByLabelText("Name"); await user.clear(nameInput); await user.type(nameInput, "Updated Basic"); await user.click(screen.getByLabelText("Create reversed cards")); // Save await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(mockFetch).toHaveBeenCalledWith( "/api/note-types/note-type-123", { method: "PUT", headers: { "Content-Type": "application/json", Authorization: "Bearer access-token", }, body: JSON.stringify({ name: "Updated Basic", frontTemplate: "{{Front}}", backTemplate: "{{Back}}", isReversible: true, }), }, ); }); expect(onNoteTypeUpdated).toHaveBeenCalledTimes(1); expect(onClose).toHaveBeenCalledTimes(1); }); it("trims whitespace from text fields", async () => { const user = userEvent.setup(); mockFetch.mockResolvedValue({ ok: true, json: async () => ({ noteType: mockNoteType }), }); render(); const nameInput = screen.getByLabelText("Name"); await user.clear(nameInput); await user.type(nameInput, " Updated Basic "); await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(mockFetch).toHaveBeenCalledWith( "/api/note-types/note-type-123", expect.objectContaining({ body: expect.stringContaining('"name":"Updated Basic"'), }), ); }); }); it("shows loading state during submission", async () => { const user = userEvent.setup(); mockFetch.mockImplementation(() => new Promise(() => {})); // Never resolves render(); await user.click(screen.getByRole("button", { name: "Save Changes" })); expect(screen.getByRole("button", { name: "Saving..." })).toBeDefined(); expect(screen.getByRole("button", { name: "Saving..." })).toHaveProperty( "disabled", true, ); expect(screen.getByRole("button", { name: "Cancel" })).toHaveProperty( "disabled", true, ); expect(screen.getByLabelText("Name")).toHaveProperty("disabled", true); }); it("displays API error message", async () => { const user = userEvent.setup(); mockFetch.mockResolvedValue({ ok: false, status: 404, json: async () => ({ error: "Note type not found" }), }); render(); await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(screen.getByRole("alert").textContent).toContain( "Note type not found", ); }); }); it("displays generic error on unexpected failure", async () => { const user = userEvent.setup(); mockFetch.mockRejectedValue(new Error("Network error")); render(); await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(screen.getByRole("alert").textContent).toContain( "Failed to update note type. Please try again.", ); }); }); it("displays error when not authenticated", async () => { const user = userEvent.setup(); vi.mocked(apiClient.getAuthHeader).mockReturnValue(undefined); render(); await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(screen.getByRole("alert").textContent).toContain( "Not authenticated", ); }); }); it("updates form when noteType prop changes", () => { const { rerender } = render(); expect(screen.getByLabelText("Name")).toHaveProperty("value", "Basic"); const newNoteType = { id: "note-type-456", name: "Another Note Type", frontTemplate: "Q: {{Front}}", backTemplate: "A: {{Back}}", isReversible: true, }; rerender(); expect(screen.getByLabelText("Name")).toHaveProperty( "value", "Another Note Type", ); expect(screen.getByLabelText("Front Template")).toHaveProperty( "value", "Q: {{Front}}", ); expect(screen.getByLabelText("Back Template")).toHaveProperty( "value", "A: {{Back}}", ); expect(screen.getByLabelText("Create reversed cards")).toHaveProperty( "checked", true, ); }); it("clears error when modal is closed", async () => { const user = userEvent.setup(); const onClose = vi.fn(); mockFetch.mockResolvedValue({ ok: false, status: 404, json: async () => ({ error: "Some error" }), }); const { rerender } = render( , ); // Trigger error await user.click(screen.getByRole("button", { name: "Save Changes" })); await waitFor(() => { expect(screen.getByRole("alert")).toBeDefined(); }); // Close and reopen the modal await user.click(screen.getByRole("button", { name: "Cancel" })); rerender(); // Error should be cleared expect(screen.queryByRole("alert")).toBeNull(); }); });