nebula-cms 0.1.3 → 0.1.4

package/tests/client/js/drafts/ops.test.ts

@@ -1,658 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-//////////////////////////////
-// Module-level mocks
-//
-// ops.svelte.ts imports from ./storage, ../utils/stable-stringify,
-// ../editor/editor.svelte, and ../state/state.svelte. All of these
-// must be mocked before the module under test is imported so that
-// side-effects (like the registerDirtyChecker call in editor.svelte.ts)
-// do not run against uninitialized browser globals.
-//////////////////////////////
-
-vi.mock('../../../../src/client/js/drafts/storage', () => ({
-  saveDraft: vi.fn(async () => undefined),
-  loadDraft: vi.fn(async () => null),
-  deleteDraft: vi.fn(async () => undefined),
-}));
-
-vi.mock('../../../../src/client/js/utils/stable-stringify', () => ({
-  stableStringify: vi.fn((v: unknown) => JSON.stringify(v)),
-}));
-
-vi.mock('../../../../src/client/js/editor/editor.svelte', () => ({
-  applyEditorState: vi.fn(),
-  _getDraftState: vi.fn(() => ({
-    saving: false,
-    draftId: null,
-    isNewDraft: false,
-    snapshot: null,
-    currentCollection: 'posts',
-    draftCreatedAt: null,
-    lastSavedFormData: '{"title":"Hello"}',
-    lastSavedBody: 'Original body',
-    formData: { title: 'Hello' },
-    body: 'Current body',
-    filename: 'post.md',
-    originalFilename: 'post.md',
-    dirty: false,
-  })),
-  _setDraftState: vi.fn(),
-}));
-
-// storageClient is a direct object export (not a getter function), so the
-// mock uses a hoisted ref that tests swap between null and a fake client.
-const { mockStorageClientRef } = vi.hoisted(() => ({
-  mockStorageClientRef: { current: null as any },
-}));
-vi.mock('../../../../src/client/js/state/state.svelte', () => ({
-  get storageClient() {
-    return mockStorageClientRef.current;
-  },
-}));
-
-vi.mock('../../../../src/client/js/utils/file-types', () => ({
-  getFileCategory: vi.fn(() => 'frontmatter'),
-  getDataFormat: vi.fn(() => null),
-}));
-
-vi.mock('js-yaml', () => ({
-  dump: vi.fn((data: unknown) => 'title: Hello\n'),
-}));
-
-vi.mock('smol-toml', () => ({
-  stringify: vi.fn((data: unknown) => 'title = "Hello"\n'),
-}));
-
-import {
-  saveDraft as persistDraft,
-  loadDraft as fetchDraft,
-  deleteDraft as removeDraft,
-} from '../../../../src/client/js/drafts/storage';
-import {
-  applyEditorState,
-  _getDraftState,
-  _setDraftState,
-} from '../../../../src/client/js/editor/editor.svelte';
-// storageClient import removed — tests configure mockStorageClientRef.current directly
-import { stableStringify } from '../../../../src/client/js/utils/stable-stringify';
-import {
-  getFileCategory,
-  getDataFormat,
-} from '../../../../src/client/js/utils/file-types';
-import { dump } from 'js-yaml';
-import { stringify as tomlStringify } from 'smol-toml';
-
-import { makeDraft as _makeDraft } from './fixtures';
-import type { Draft } from '../../../../src/client/js/drafts/storage';
-
-// Ops tests use different defaults than the shared fixture. This wrapper
-// applies ops-specific overrides so callers don't repeat them everywhere.
-const opsDefaults: Partial<Draft> = {
-  filename: 'post.md',
-  formData: { title: 'Hello' },
-  body: 'Draft body',
-  snapshot: '{"body":"orig","formData":{"title":"Orig"}}',
-};
-
-/**
- * Creates a Draft fixture with ops-test-specific defaults layered on top of the shared fixture.
- * @param {Partial<Draft>} overrides - Per-test field overrides
- * @return {Draft} A complete Draft object
- */
-function makeDraft(overrides: Partial<Draft> = {}): Draft {
-  return _makeDraft({ ...opsDefaults, ...overrides });
-}
-
-/**
- * Returns a draft state fixture representing the default editor state shape.
- * @param {object} overrides - Optional field overrides
- * @return {ReturnType<typeof _getDraftState>} The draft state snapshot
- */
-function makeEditorState(
-  overrides: Partial<ReturnType<typeof _getDraftState>> = {},
-): ReturnType<typeof _getDraftState> {
-  return {
-    saving: false,
-    draftId: null,
-    isNewDraft: false,
-    snapshot: null,
-    currentCollection: 'posts',
-    draftCreatedAt: null,
-    lastSavedFormData: '{"title":"Hello"}',
-    lastSavedBody: 'Original body',
-    formData: { title: 'Hello' },
-    body: 'Current body',
-    filename: 'post.md',
-    originalFilename: 'post.md',
-    dirty: false,
-    ...overrides,
-  };
-}
-
-//////////////////////////////
-// loadDraftById
-//////////////////////////////
-
-describe('loadDraftById', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('applies draft state from IDB when draft is found', async () => {
-    const draft = makeDraft({ id: 'ldi-01' });
-    vi.mocked(fetchDraft).mockResolvedValue(draft);
-
-    const { loadDraftById } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await loadDraftById('ldi-01', 'posts');
-
-    expect(applyEditorState).toHaveBeenCalledWith(
-      expect.objectContaining({
-        body: draft.body,
-        formData: draft.formData,
-        filename: draft.filename,
-        draftId: draft.id,
-        isNewDraft: draft.isNew,
-        snapshot: draft.snapshot,
-        collection: 'posts',
-        bodyLoaded: true,
-      }),
-      true,
-    );
-  });
-
-  it('applies fallback empty state when draft is not found', async () => {
-    vi.mocked(fetchDraft).mockResolvedValue(null);
-
-    const { loadDraftById } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await loadDraftById('ghost-id', 'posts');
-
-    expect(applyEditorState).toHaveBeenCalledWith(
-      expect.objectContaining({
-        body: '',
-        formData: {},
-        bodyLoaded: true,
-        draftId: 'ghost-id',
-        isNewDraft: true,
-        collection: 'posts',
-      }),
-      true,
-    );
-  });
-
-  it('fetches the draft by the given ID', async () => {
-    vi.mocked(fetchDraft).mockResolvedValue(null);
-
-    const { loadDraftById } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await loadDraftById('specific-id', 'posts');
-
-    expect(fetchDraft).toHaveBeenCalledWith('specific-id');
-  });
-});
-
-//////////////////////////////
-// saveDraftToIDB
-//////////////////////////////
-
-describe('saveDraftToIDB', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('calls persistDraft with data from the current editor state', async () => {
-    const editorState = makeEditorState({
-      draftId: 'existing-draft-id',
-      draftCreatedAt: '2026-01-01T00:00:00.000Z',
-    });
-    vi.mocked(_getDraftState).mockReturnValue(editorState);
-
-    const { saveDraftToIDB } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveDraftToIDB();
-
-    expect(persistDraft).toHaveBeenCalledWith(
-      expect.objectContaining({
-        id: 'existing-draft-id',
-        collection: 'posts',
-        filename: 'post.md',
-        body: 'Current body',
-        formData: { title: 'Hello' },
-      }),
-    );
-  });
-
-  it('generates a UUID when draftId is null (first save)', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: null,
-        draftCreatedAt: null,
-        isNewDraft: true,
-      }),
-    );
-
-    const { saveDraftToIDB } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveDraftToIDB();
-
-    // A UUID must have been provided — check that persistDraft received a non-null id
-    const call = vi.mocked(persistDraft).mock.calls[0][0];
-    expect(call.id).toBeTruthy();
-    expect(typeof call.id).toBe('string');
-  });
-
-  it('captures a snapshot for live content on first save', async () => {
-    // isNewDraft=false means editing an existing file — snapshot should be set
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: null,
-        draftCreatedAt: null,
-        isNewDraft: false,
-      }),
-    );
-    vi.mocked(stableStringify).mockReturnValue('{"snapshot":"value"}');
-
-    const { saveDraftToIDB } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveDraftToIDB();
-
-    expect(stableStringify).toHaveBeenCalled();
-    const call = vi.mocked(persistDraft).mock.calls[0][0];
-    expect(call.snapshot).toBe('{"snapshot":"value"}');
-  });
-
-  it('sets saving to true then false regardless of outcome', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'sd-01',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { saveDraftToIDB } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveDraftToIDB();
-
-    const calls = vi.mocked(_setDraftState).mock.calls;
-    const savingTrue = calls.some((c) => c[0].saving === true);
-    const savingFalse = calls.some((c) => c[0].saving === false);
-    expect(savingTrue).toBe(true);
-    expect(savingFalse).toBe(true);
-  });
-
-  it('clears dirty flag after successful save', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'sd-02',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { saveDraftToIDB } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveDraftToIDB();
-
-    const calls = vi.mocked(_setDraftState).mock.calls;
-    const dirtyCall = calls.find((c) => 'dirty' in c[0]);
-    expect(dirtyCall?.[0].dirty).toBe(false);
-  });
-});
-
-//////////////////////////////
-// saveFile (alias)
-//////////////////////////////
-
-describe('saveFile', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('delegates to saveDraftToIDB and calls persistDraft', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'sf-01',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { saveFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await saveFile();
-
-    expect(persistDraft).toHaveBeenCalled();
-  });
-});
-
-//////////////////////////////
-// publishFile
-//////////////////////////////
-
-describe('publishFile', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('writes the file via the storage client', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'pf-02',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-        formData: { title: 'Hello' },
-        body: 'The body',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'posts',
-      'post.md',
-      expect.stringContaining('The body'),
-    );
-  });
-
-  it('deletes the draft from IDB after a successful write', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'pf-03',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    expect(removeDraft).toHaveBeenCalledWith('pf-03');
-  });
-
-  it('clears draftId in state after publish', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'pf-04',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    const calls = vi.mocked(_setDraftState).mock.calls;
-    const clearCall = calls.find((c) => c[0].draftId === null);
-    expect(clearCall).toBeDefined();
-  });
-
-  it('sets saving to true then false', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        draftId: 'pf-05',
-        draftCreatedAt: '2026-01-01T00:00:00.000Z',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    const calls = vi.mocked(_setDraftState).mock.calls;
-    expect(calls.some((c) => c[0].saving === true)).toBe(true);
-    expect(calls.some((c) => c[0].saving === false)).toBe(true);
-  });
-
-  it('skips draft deletion when draftId is null', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({ draftId: null }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    expect(removeDraft).not.toHaveBeenCalled();
-  });
-});
-
-//////////////////////////////
-// deleteCurrentDraft
-//////////////////////////////
-
-describe('deleteCurrentDraft', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('calls removeDraft with the current draftId', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({ draftId: 'dcd-01' }),
-    );
-
-    const { deleteCurrentDraft } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await deleteCurrentDraft();
-
-    expect(removeDraft).toHaveBeenCalledWith('dcd-01');
-  });
-
-  it('skips removeDraft when draftId is null', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({ draftId: null }),
-    );
-
-    const { deleteCurrentDraft } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await deleteCurrentDraft();
-
-    expect(removeDraft).not.toHaveBeenCalled();
-  });
-
-  it('resets draft state fields to null/false', async () => {
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({ draftId: 'dcd-02' }),
-    );
-
-    const { deleteCurrentDraft } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await deleteCurrentDraft();
-
-    expect(_setDraftState).toHaveBeenCalledWith(
-      expect.objectContaining({
-        draftId: null,
-        isNewDraft: false,
-        snapshot: null,
-        draftCreatedAt: null,
-      }),
-    );
-  });
-});
-
-//////////////////////////////
-// publishFile — multi-format serialization
-//////////////////////////////
-
-describe('publishFile multi-format serialization', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('serializes JSON data files as formatted JSON', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(getFileCategory).mockReturnValue('data');
-    vi.mocked(getDataFormat).mockReturnValue('json');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { name: 'Alice', age: 30 },
-        body: '',
-        filename: 'author.json',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('data', 'author.json');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'data',
-      'author.json',
-      expect.any(String),
-    );
-    const content = writeFile.mock.calls[0][2];
-    // JSON output should be indented (multi-line) and end with a newline
-    expect(content).toContain('\n');
-    expect(content).toMatch(/^\{/);
-    expect(content.endsWith('\n')).toBe(true);
-    // No frontmatter delimiters
-    expect(content).not.toContain('---');
-  });
-
-  it('serializes YAML data files as plain YAML without frontmatter delimiters', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(getFileCategory).mockReturnValue('data');
-    vi.mocked(getDataFormat).mockReturnValue('yaml');
-    vi.mocked(dump).mockReturnValue('title: Hello\n');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { title: 'Hello' },
-        body: '',
-        filename: 'config.yml',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('data', 'config.yml');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'data',
-      'config.yml',
-      expect.any(String),
-    );
-    const content = writeFile.mock.calls[0][2];
-    // Plain YAML without frontmatter delimiters
-    expect(content).not.toContain('---');
-    expect(dump).toHaveBeenCalled();
-  });
-
-  it('serializes TOML data files', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(getFileCategory).mockReturnValue('data');
-    vi.mocked(getDataFormat).mockReturnValue('toml');
-    vi.mocked(tomlStringify).mockReturnValue('title = "Hello"\n');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { title: 'Hello' },
-        body: '',
-        filename: 'config.toml',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('data', 'config.toml');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'data',
-      'config.toml',
-      expect.any(String),
-    );
-    const content = writeFile.mock.calls[0][2];
-    expect(content).not.toContain('---');
-    expect(tomlStringify).toHaveBeenCalled();
-  });
-
-  it('serializes frontmatter files with --- delimiters', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile };
-    vi.mocked(getFileCategory).mockReturnValue('frontmatter');
-    vi.mocked(getDataFormat).mockReturnValue(null);
-    vi.mocked(dump).mockReturnValue('title: Hello\n');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { title: 'Hello' },
-        body: 'The body',
-        filename: 'post.md',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'posts',
-      'post.md',
-      expect.any(String),
-    );
-    const content = writeFile.mock.calls[0][2];
-    expect(content).toMatch(/^---\n/);
-    expect(content).toContain('---\n\n');
-    expect(content).toContain('The body');
-  });
-
-  it('deletes old file on type conversion when originalFilename differs', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    const deleteFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile, deleteFile };
-    vi.mocked(getFileCategory).mockReturnValue('frontmatter');
-    vi.mocked(getDataFormat).mockReturnValue(null);
-    vi.mocked(dump).mockReturnValue('title: Hello\n');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { title: 'Hello' },
-        body: 'The body',
-        filename: 'post.mdx',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.mdx', 'post.md');
-
-    expect(writeFile).toHaveBeenCalledWith(
-      'posts',
-      'post.mdx',
-      expect.any(String),
-    );
-    expect(deleteFile).toHaveBeenCalledWith('posts', 'post.md');
-  });
-
-  it('does not delete when originalFilename matches filename', async () => {
-    const writeFile = vi.fn(async () => undefined);
-    const deleteFile = vi.fn(async () => undefined);
-    mockStorageClientRef.current = { writeFile, deleteFile };
-    vi.mocked(getFileCategory).mockReturnValue('frontmatter');
-    vi.mocked(getDataFormat).mockReturnValue(null);
-    vi.mocked(dump).mockReturnValue('title: Hello\n');
-    vi.mocked(_getDraftState).mockReturnValue(
-      makeEditorState({
-        formData: { title: 'Hello' },
-        body: 'The body',
-        filename: 'post.md',
-      }),
-    );
-
-    const { publishFile } =
-      await import('../../../../src/client/js/drafts/ops.svelte');
-    await publishFile('posts', 'post.md', 'post.md');
-
-    expect(writeFile).toHaveBeenCalled();
-    expect(deleteFile).not.toHaveBeenCalled();
-  });
-});