Add tests and update messages for autosave functionality

tests/domain/editor-store.test.ts

@@ -92,7 +92,7 @@ describe("EditorStore", () => {
     });
     expect(writerStore.saveDraft()).toEqual({
       status: "saved",
-      message: "Local draft saved."
+      message: "Autosave updated."
     });
 
     const readerStore = createEditorStore({
@@ -102,7 +102,7 @@ describe("EditorStore", () => {
 
     expect(readerStore.loadDraft()).toMatchObject({
       status: "loaded",
-      message: "Local draft loaded."
+      message: "Recovered latest autosave."
     });
     expect(readerStore.getState().document.name).toBe("Draft Scene");
     expect(readerStore.getState().viewportLayoutMode).toBe("quad");

tests/unit/app-project-persistence.integration.test.tsx

@@ -0,0 +1,141 @@
+import { fireEvent, render, screen, waitFor } from "@testing-library/react";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+const { MockViewportHost, viewportHostInstances } = vi.hoisted(() => {
+  const viewportHostInstances: Array<{
+    setTransformCommitHandler: ReturnType<typeof vi.fn>;
+    mount: ReturnType<typeof vi.fn>;
+    dispose: ReturnType<typeof vi.fn>;
+    updateWorld: ReturnType<typeof vi.fn>;
+    updateAssets: ReturnType<typeof vi.fn>;
+    updateDocument: ReturnType<typeof vi.fn>;
+    setPanelId: ReturnType<typeof vi.fn>;
+    setViewMode: ReturnType<typeof vi.fn>;
+    setDisplayMode: ReturnType<typeof vi.fn>;
+    setCameraState: ReturnType<typeof vi.fn>;
+    setBrushSelectionChangeHandler: ReturnType<typeof vi.fn>;
+    setCameraStateChangeHandler: ReturnType<typeof vi.fn>;
+    setCreationPreviewChangeHandler: ReturnType<typeof vi.fn>;
+    setCreationCommitHandler: ReturnType<typeof vi.fn>;
+    setTransformSessionChangeHandler: ReturnType<typeof vi.fn>;
+    setTransformCancelHandler: ReturnType<typeof vi.fn>;
+    setWhiteboxHoverLabelChangeHandler: ReturnType<typeof vi.fn>;
+    setWhiteboxSelectionMode: ReturnType<typeof vi.fn>;
+    setWhiteboxSnapSettings: ReturnType<typeof vi.fn>;
+    setToolMode: ReturnType<typeof vi.fn>;
+    setCreationPreview: ReturnType<typeof vi.fn>;
+    setTransformSession: ReturnType<typeof vi.fn>;
+    focusSelection: ReturnType<typeof vi.fn>;
+  }> = [];
+
+  class MockViewportHost {
+    setPanelId = vi.fn();
+    mount = vi.fn();
+    dispose = vi.fn();
+    updateWorld = vi.fn();
+    updateAssets = vi.fn();
+    updateDocument = vi.fn();
+    setViewMode = vi.fn();
+    setDisplayMode = vi.fn();
+    setCameraState = vi.fn();
+    setBrushSelectionChangeHandler = vi.fn();
+    setCameraStateChangeHandler = vi.fn();
+    setCreationPreviewChangeHandler = vi.fn();
+    setCreationCommitHandler = vi.fn();
+    setTransformSessionChangeHandler = vi.fn();
+    setTransformCommitHandler = vi.fn();
+    setTransformCancelHandler = vi.fn();
+    setWhiteboxHoverLabelChangeHandler = vi.fn();
+    setWhiteboxSelectionMode = vi.fn();
+    setWhiteboxSnapSettings = vi.fn();
+    setToolMode = vi.fn();
+    setCreationPreview = vi.fn();
+    setTransformSession = vi.fn();
+    focusSelection = vi.fn();
+
+    constructor() {
+      viewportHostInstances.push(this);
+    }
+  }
+
+  return {
+    MockViewportHost,
+    viewportHostInstances
+  };
+});
+
+const saveProjectPackageMock = vi.fn(async () => new Uint8Array([1, 2, 3]));
+const loadProjectPackageMock = vi.fn();
+
+vi.mock("../../src/viewport-three/viewport-host", () => ({
+  ViewportHost: MockViewportHost
+}));
+
+vi.mock("../../src/assets/project-asset-storage", () => ({
+  getBrowserProjectAssetStorageAccess: vi.fn(async () => ({
+    storage: null,
+    diagnostic: null
+  }))
+}));
+
+vi.mock("../../src/serialization/project-package", () => ({
+  PROJECT_PACKAGE_FILE_EXTENSION: ".we3d",
+  saveProjectPackage: saveProjectPackageMock,
+  loadProjectPackage: loadProjectPackageMock
+}));
+
+import { App } from "../../src/app/App";
+import { createEditorStore } from "../../src/app/editor-store";
+
+describe("App project persistence controls", () => {
+  beforeEach(() => {
+    viewportHostInstances.length = 0;
+    saveProjectPackageMock.mockClear();
+    loadProjectPackageMock.mockClear();
+    vi.spyOn(HTMLCanvasElement.prototype, "getContext").mockImplementation(() => ({}) as never);
+    vi.stubGlobal("URL", {
+      createObjectURL: vi.fn(() => "blob:project"),
+      revokeObjectURL: vi.fn()
+    });
+    vi.spyOn(HTMLAnchorElement.prototype, "click").mockImplementation(() => undefined);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+    vi.unstubAllGlobals();
+  });
+
+  it("shows Save Project and Load Project instead of the old draft/json actions", async () => {
+    render(<App store={createEditorStore()} />);
+
+    await waitFor(() => {
+      expect(viewportHostInstances.length).toBeGreaterThan(0);
+    });
+
+    expect(screen.getByRole("button", { name: "Save Project" })).toBeInTheDocument();
+    expect(screen.getByRole("button", { name: "Load Project" })).toBeInTheDocument();
+    expect(screen.queryByRole("button", { name: "Save Draft" })).not.toBeInTheDocument();
+    expect(screen.queryByRole("button", { name: "Load Draft" })).not.toBeInTheDocument();
+    expect(screen.queryByRole("button", { name: "Export JSON" })).not.toBeInTheDocument();
+    expect(screen.queryByRole("button", { name: "Import JSON" })).not.toBeInTheDocument();
+  });
+
+  it("invokes Save Project when Cmd/Ctrl+S is pressed", async () => {
+    const store = createEditorStore();
+
+    render(<App store={store} />);
+
+    await waitFor(() => {
+      expect(viewportHostInstances.length).toBeGreaterThan(0);
+    });
+
+    fireEvent.keyDown(window, {
+      code: "KeyS",
+      ctrlKey: true
+    });
+
+    await waitFor(() => {
+      expect(saveProjectPackageMock).toHaveBeenCalledWith(store.getState().document, null);
+    });
+  });
+});

tests/unit/editor-autosave.test.ts

@@ -0,0 +1,77 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+import { EditorAutosaveController } from "../../src/serialization/editor-autosave";
+
+describe("EditorAutosaveController", () => {
+  beforeEach(() => {
+    vi.useFakeTimers();
+  });
+
+  afterEach(() => {
+    vi.useRealTimers();
+  });
+
+  it("debounces repeated autosave schedules into one save", () => {
+    const saveDraft = vi.fn(() => ({
+      status: "saved" as const,
+      message: "Autosave updated."
+    }));
+    const autosave = new EditorAutosaveController({
+      debounceMs: 200,
+      saveDraft
+    });
+
+    autosave.schedule();
+    autosave.schedule();
+    autosave.schedule();
+
+    vi.advanceTimersByTime(199);
+    expect(saveDraft).not.toHaveBeenCalled();
+
+    vi.advanceTimersByTime(1);
+    expect(saveDraft).toHaveBeenCalledTimes(1);
+  });
+
+  it("flushes a pending autosave immediately", () => {
+    const saveDraft = vi.fn(() => ({
+      status: "saved" as const,
+      message: "Autosave updated."
+    }));
+    const autosave = new EditorAutosaveController({
+      debounceMs: 200,
+      saveDraft
+    });
+
+    autosave.schedule();
+    const flushResult = autosave.flush();
+
+    expect(flushResult).toEqual({
+      status: "saved",
+      message: "Autosave updated."
+    });
+    expect(saveDraft).toHaveBeenCalledTimes(1);
+
+    vi.advanceTimersByTime(200);
+    expect(saveDraft).toHaveBeenCalledTimes(1);
+  });
+
+  it("reports autosave failures through the completion callback", () => {
+    const onComplete = vi.fn();
+    const autosave = new EditorAutosaveController({
+      debounceMs: 100,
+      onComplete,
+      saveDraft: () => ({
+        status: "error" as const,
+        message: "Autosave could not be saved. quota exceeded"
+      })
+    });
+
+    autosave.schedule();
+    vi.advanceTimersByTime(100);
+
+    expect(onComplete).toHaveBeenCalledWith({
+      status: "error",
+      message: "Autosave could not be saved. quota exceeded"
+    });
+  });
+});