nebula-cms 0.1.3 → 0.1.5

package/tests/e2e/publishing.test.ts

@@ -1,345 +0,0 @@
-import { describe, it, expect, vi, afterEach, beforeEach } from 'vitest';
-import { render, cleanup, fireEvent } from '@testing-library/svelte';
-import {
-  resetMocks,
-  configureFileOpen,
-  configureDraftOpen,
-} from './helpers/test-app';
-
-/*
-//////////////////////////////
-// Hoisted mocks
-//////////////////////////////
-*/
-
-const mocks = vi.hoisted(() => ({
-  mockBackendReady: vi.fn(() => false),
-  mockRoute: vi.fn(() => ({ view: 'home' as const })),
-  mockCollections: vi.fn(() => [] as string[]),
-  mockContentList: vi.fn(
-    () => [] as Array<{ filename: string; data: Record<string, unknown> }>,
-  ),
-  mockLoading: vi.fn(() => false),
-  mockError: vi.fn(() => null as string | null),
-  mockDrafts: vi.fn(() => []),
-  mockOutdatedMap: vi.fn(() => ({}) as Record<string, boolean>),
-  mockActiveTab: vi.fn(() => 'metadata'),
-  mockGetEditorFile: vi.fn(() => null),
-  mockSchema: vi.fn(() => null),
-  mockCollectionHasDates: vi.fn(() => false),
-  mockComputePublishDisabled: vi.fn(() => false),
-}));
-
-// Handler mocks for publish assertions
-const handlers = vi.hoisted(() => ({
-  mockHandlePublish: vi.fn(async () => ({ status: 'ok' as const })),
-}));
-
-// Tracks which dialog is active so dialogs.active reflects it
-const dialogState = vi.hoisted(() => ({
-  active: null as string | null,
-}));
-
-/*
-//////////////////////////////
-// Module mocks
-//////////////////////////////
-*/
-
-vi.mock('virtual:nebula/collections', () => ({
-  default: {
-    pages: '/collections/pages.schema.json',
-    posts: '/collections/posts.schema.json',
-  },
-}));
-vi.mock('virtual:nebula/config', () => ({
-  default: { basePath: '/admin', collectionsPath: '/collections' },
-}));
-vi.mock('../../src/client/js/state/state.svelte', () => ({
-  backend: {
-    get type() {
-      return null;
-    },
-    get ready() {
-      return mocks.mockBackendReady();
-    },
-    get permission() {
-      return 'denied';
-    },
-  },
-  content: {
-    get list() {
-      return mocks.mockContentList();
-    },
-    get loading() {
-      return mocks.mockLoading();
-    },
-    get error() {
-      return mocks.mockError();
-    },
-  },
-  get collections() {
-    return mocks.mockCollections();
-  },
-  storageClient: null,
-  drafts: {
-    get all() {
-      return mocks.mockDrafts();
-    },
-    get outdated() {
-      return mocks.mockOutdatedMap();
-    },
-  },
-  restoreBackend: vi.fn(async () => {}),
-  loadCollection: vi.fn(),
-  reloadCollection: vi.fn(),
-  disconnect: vi.fn(),
-  refreshDrafts: vi.fn(async () => {}),
-  updateContentItem: vi.fn(),
-  pickDirectory: vi.fn(),
-  requestPermission: vi.fn(),
-  connectGitHub: vi.fn(async () => {}),
-}));
-vi.mock('../../src/client/js/state/router.svelte', () => ({
-  initRouter: vi.fn(),
-  nav: {
-    get route() {
-      return mocks.mockRoute();
-    },
-  },
-  navigate: vi.fn(),
-  registerDirtyChecker: vi.fn(),
-  adminPath: vi.fn((...segments) =>
-    segments.length === 0 ? '/admin' : '/admin/' + segments.join('/'),
-  ),
-}));
-vi.mock('../../src/client/js/state/schema.svelte', () => ({
-  fetchSchema: vi.fn(async () => {}),
-  schema: {
-    get active() {
-      return mocks.mockSchema();
-    },
-  },
-  clearSchema: vi.fn(),
-  prefetchAllSchemas: vi.fn(async () => {}),
-  collectionHasDates: mocks.mockCollectionHasDates,
-  getCollectionTitle: vi.fn(() => null),
-  getCollectionDescription: vi.fn(() => null),
-}));
-vi.mock('../../src/client/js/editor/editor.svelte', () => ({
-  preloadFile: vi.fn(async () => {}),
-  loadFileBody: vi.fn(async () => {}),
-  clearEditor: vi.fn(),
-  editor: {
-    get tab() {
-      return mocks.mockActiveTab();
-    },
-    get data() {
-      return {};
-    },
-    get originalFilename() {
-      return '';
-    },
-  },
-  setActiveTab: vi.fn(),
-  getEditorFile: mocks.mockGetEditorFile,
-  loadDraftById: vi.fn(async () => {}),
-  setFilename: vi.fn(),
-  updateBody: vi.fn(),
-  updateFormField: vi.fn(),
-  saveDraftToIDB: vi.fn(async () => {}),
-  saveFile: vi.fn(async () => {}),
-  publishFile: vi.fn(async () => {}),
-  deleteCurrentDraft: vi.fn(async () => {}),
-  applyEditorState: vi.fn(),
-  _getDraftState: vi.fn(() => ({})),
-  _setDraftState: vi.fn(),
-  changeFileFormat: vi.fn(),
-  setDefaultFormat: vi.fn(),
-}));
-vi.mock('../../src/client/js/handlers/admin', async (importOriginal) => {
-  const actual =
-    await importOriginal<typeof import('../../src/client/js/handlers/admin')>();
-  return {
-    ...actual,
-    handleSave: vi.fn(async () => {}),
-    handlePublish: handlers.mockHandlePublish,
-    handleDeleteDraft: vi.fn(async () => {}),
-    handleFilenameConfirm: vi.fn(async () => {}),
-    computePublishDisabled: mocks.mockComputePublishDisabled,
-    // Override buildCollectionItems to read from mockCollections — the
-    // module-level getter for `collections` is not a live binding in
-    // vitest's browser mode, so the real function would see an empty array.
-    buildCollectionItems: () =>
-      mocks.mockCollections().map((name: string) => ({
-        label: name.charAt(0).toUpperCase() + name.slice(1),
-        href: '/admin/' + name,
-      })),
-  };
-});
-vi.mock('../../src/client/js/utils/sort', () => ({
-  toSortDate: vi.fn(() => undefined),
-  readSortMode: vi.fn(() => 'alpha'),
-  writeSortMode: vi.fn(),
-  createComparator: vi.fn(() => () => 0),
-  SORT_MODES: {
-    alpha: { icon: 'sort_by_alpha', label: 'Alphabetical' },
-    'date-asc': { icon: 'hourglass_arrow_down', label: 'Oldest first' },
-    'date-desc': { icon: 'hourglass_arrow_up', label: 'Newest first' },
-  },
-  SORT_ORDER: ['alpha', 'date-asc', 'date-desc'],
-}));
-vi.mock('../../src/client/js/drafts/storage', () => ({
-  saveDraft: vi.fn(async () => {}),
-  getDraftByFile: vi.fn(async () => null),
-  loadDrafts: vi.fn(async () => []),
-  loadDraft: vi.fn(async () => null),
-  deleteDraft: vi.fn(async () => {}),
-}));
-vi.mock('../../src/client/js/utils/schema-utils', () => ({
-  extractTabs: vi.fn(() => []),
-  getFieldsForTab: vi.fn(() => []),
-  resolveFieldType: vi.fn(() => ({ kind: 'string' })),
-  createDefaultValue: vi.fn(() => ''),
-  getByPath: vi.fn(),
-  setByPath: vi.fn(),
-  isReadOnly: vi.fn(() => false),
-  isNullable: vi.fn(() => false),
-  getProperties: vi.fn(
-    (schema: Record<string, unknown>) => schema['properties'],
-  ),
-  getRequiredFields: vi.fn((schema: Record<string, unknown>) =>
-    Array.isArray(schema['required']) ? schema['required'] : [],
-  ),
-  getLabel: vi.fn((schema: Record<string, unknown>, name: string) =>
-    typeof schema['title'] === 'string' ? schema['title'] : name,
-  ),
-}));
-vi.mock('../../src/client/js/drafts/merge.svelte', () => ({
-  drafts: {
-    get all() {
-      return mocks.mockDrafts();
-    },
-    get outdated() {
-      return mocks.mockOutdatedMap();
-    },
-  },
-  mergeDrafts: vi.fn(async () => {}),
-  refreshDrafts: vi.fn(async () => {}),
-  resetDraftMerge: vi.fn(),
-}));
-
-vi.mock('../../src/client/js/state/theme.svelte', () => ({
-  initTheme: vi.fn(() => () => {}),
-  cycleTheme: vi.fn(),
-  theme: { resolved: 'dark', icon: 'brightness_auto', label: 'Auto' },
-}));
-vi.mock('../../src/client/js/state/dialogs.svelte', () => ({
-  dialog: {
-    get active() {
-      return dialogState.active;
-    },
-    open: vi.fn((type: string) => {
-      dialogState.active = type;
-    }),
-    close: vi.fn(() => {
-      dialogState.active = null;
-    }),
-  },
-}));
-
-import Admin from '../../src/client/Admin.svelte';
-import { dialog } from '../../src/client/js/state/dialogs.svelte';
-
-afterEach(() => cleanup());
-beforeEach(() => {
-  resetMocks(mocks);
-  handlers.mockHandlePublish.mockClear();
-  handlers.mockHandlePublish.mockResolvedValue({ status: 'ok' });
-  dialogState.active = null;
-});
-
-describe('Publishing', () => {
-  it('calls handlePublish when publish button is clicked', async () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Content',
-      formData: { title: 'Hello World' },
-    });
-
-    const { container } = render(Admin);
-
-    const publishBtn = container.querySelector('.editor-area .btn--primary');
-    expect(publishBtn).not.toBeNull();
-
-    if (publishBtn) await fireEvent.click(publishBtn);
-    expect(handlers.mockHandlePublish).toHaveBeenCalled();
-  });
-
-  it('shows filename dialog when publish returns needs-filename', async () => {
-    configureDraftOpen(mocks, 'posts', 'draft-abc', {
-      body: 'New content',
-      formData: { title: 'New Post' },
-    });
-    handlers.mockHandlePublish.mockResolvedValue({ status: 'needs-filename' });
-
-    const { container } = render(Admin);
-
-    const publishBtn = container.querySelector('.editor-area .btn--primary');
-
-    if (publishBtn) await fireEvent.click(publishBtn);
-
-    // EditorToolbar calls showFilenameDialog when handlePublish returns 'needs-filename'
-    expect(dialog.open).toHaveBeenCalledWith('filename');
-  });
-
-  it('disables publish button when computePublishDisabled returns true', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Content',
-      formData: { title: 'Hello World' },
-    });
-    mocks.mockComputePublishDisabled.mockReturnValue(true);
-
-    const { container } = render(Admin);
-
-    const publishBtn = container.querySelector(
-      '.editor-area .btn--primary',
-    ) as HTMLButtonElement;
-    expect(publishBtn).not.toBeNull();
-    expect(publishBtn?.disabled).toBe(true);
-  });
-
-  it('enables publish button when computePublishDisabled returns false', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Content',
-      formData: { title: 'Hello World' },
-    });
-    mocks.mockComputePublishDisabled.mockReturnValue(false);
-
-    const { container } = render(Admin);
-
-    const publishBtn = container.querySelector(
-      '.editor-area .btn--primary',
-    ) as HTMLButtonElement;
-    expect(publishBtn?.disabled).toBe(false);
-  });
-
-  it('shows delete draft button when editing a draft', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Content',
-      formData: { title: 'Hello World' },
-      draftId: 'draft-1',
-    });
-
-    const { container } = render(Admin);
-
-    const deleteBtn = container.querySelector(
-      '.editor-area .btn--danger-outline',
-    );
-    expect(deleteBtn).not.toBeNull();
-    expect(deleteBtn?.textContent?.trim()).toContain('Delete Draft');
-  });
-});

package/tests/e2e/unsaved-changes.test.ts

@@ -1,317 +0,0 @@
-import { describe, it, expect, vi, afterEach, beforeEach } from 'vitest';
-import { render, cleanup } from '@testing-library/svelte';
-import { resetMocks, configureFileOpen } from './helpers/test-app';
-
-/*
-//////////////////////////////
-// Hoisted mocks
-//////////////////////////////
-*/
-
-const mocks = vi.hoisted(() => ({
-  mockBackendReady: vi.fn(() => false),
-  mockRoute: vi.fn(() => ({ view: 'home' as const })),
-  mockCollections: vi.fn(() => [] as string[]),
-  mockContentList: vi.fn(
-    () => [] as Array<{ filename: string; data: Record<string, unknown> }>,
-  ),
-  mockLoading: vi.fn(() => false),
-  mockError: vi.fn(() => null as string | null),
-  mockDrafts: vi.fn(() => []),
-  mockOutdatedMap: vi.fn(() => ({}) as Record<string, boolean>),
-  mockActiveTab: vi.fn(() => 'metadata'),
-  mockGetEditorFile: vi.fn(() => null),
-  mockSchema: vi.fn(() => null),
-  mockCollectionHasDates: vi.fn(() => false),
-  mockComputePublishDisabled: vi.fn(() => false),
-}));
-
-/*
-//////////////////////////////
-// Module mocks
-//////////////////////////////
-*/
-
-vi.mock('virtual:nebula/collections', () => ({
-  default: {
-    pages: '/collections/pages.schema.json',
-    posts: '/collections/posts.schema.json',
-  },
-}));
-vi.mock('virtual:nebula/config', () => ({
-  default: { basePath: '/admin', collectionsPath: '/collections' },
-}));
-vi.mock('../../src/client/js/state/state.svelte', () => ({
-  backend: {
-    get type() {
-      return null;
-    },
-    get ready() {
-      return mocks.mockBackendReady();
-    },
-    get permission() {
-      return 'denied';
-    },
-  },
-  content: {
-    get list() {
-      return mocks.mockContentList();
-    },
-    get loading() {
-      return mocks.mockLoading();
-    },
-    get error() {
-      return mocks.mockError();
-    },
-  },
-  get collections() {
-    return mocks.mockCollections();
-  },
-  storageClient: null,
-  drafts: {
-    get all() {
-      return mocks.mockDrafts();
-    },
-    get outdated() {
-      return mocks.mockOutdatedMap();
-    },
-  },
-  restoreBackend: vi.fn(async () => {}),
-  loadCollection: vi.fn(),
-  reloadCollection: vi.fn(),
-  disconnect: vi.fn(),
-  refreshDrafts: vi.fn(async () => {}),
-  updateContentItem: vi.fn(),
-  pickDirectory: vi.fn(),
-  requestPermission: vi.fn(),
-  connectGitHub: vi.fn(async () => {}),
-}));
-vi.mock('../../src/client/js/state/router.svelte', () => ({
-  initRouter: vi.fn(),
-  nav: {
-    get route() {
-      return mocks.mockRoute();
-    },
-  },
-  navigate: vi.fn(),
-  registerDirtyChecker: vi.fn(),
-  adminPath: vi.fn((...segments) =>
-    segments.length === 0 ? '/admin' : '/admin/' + segments.join('/'),
-  ),
-}));
-vi.mock('../../src/client/js/state/schema.svelte', () => ({
-  fetchSchema: vi.fn(async () => {}),
-  schema: {
-    get active() {
-      return mocks.mockSchema();
-    },
-  },
-  clearSchema: vi.fn(),
-  prefetchAllSchemas: vi.fn(async () => {}),
-  collectionHasDates: mocks.mockCollectionHasDates,
-  getCollectionTitle: vi.fn(() => null),
-  getCollectionDescription: vi.fn(() => null),
-}));
-vi.mock('../../src/client/js/editor/editor.svelte', () => ({
-  preloadFile: vi.fn(async () => {}),
-  loadFileBody: vi.fn(async () => {}),
-  clearEditor: vi.fn(),
-  editor: {
-    get tab() {
-      return mocks.mockActiveTab();
-    },
-    get data() {
-      return {};
-    },
-    get originalFilename() {
-      return '';
-    },
-  },
-  setActiveTab: vi.fn(),
-  getEditorFile: mocks.mockGetEditorFile,
-  loadDraftById: vi.fn(async () => {}),
-  setFilename: vi.fn(),
-  updateBody: vi.fn(),
-  updateFormField: vi.fn(),
-  saveDraftToIDB: vi.fn(async () => {}),
-  saveFile: vi.fn(async () => {}),
-  publishFile: vi.fn(async () => {}),
-  deleteCurrentDraft: vi.fn(async () => {}),
-  applyEditorState: vi.fn(),
-  _getDraftState: vi.fn(() => ({})),
-  _setDraftState: vi.fn(),
-  changeFileFormat: vi.fn(),
-  setDefaultFormat: vi.fn(),
-}));
-vi.mock('../../src/client/js/handlers/admin', async (importOriginal) => {
-  const actual =
-    await importOriginal<typeof import('../../src/client/js/handlers/admin')>();
-  return {
-    ...actual,
-    handleSave: vi.fn(async () => {}),
-    handlePublish: vi.fn(async () => ({ status: 'ok' })),
-    handleDeleteDraft: vi.fn(async () => {}),
-    handleFilenameConfirm: vi.fn(async () => {}),
-    computePublishDisabled: mocks.mockComputePublishDisabled,
-    // Override buildCollectionItems to read from mockCollections — the
-    // module-level getter for `collections` is not a live binding in
-    // vitest's browser mode, so the real function would see an empty array.
-    buildCollectionItems: () =>
-      mocks.mockCollections().map((name: string) => ({
-        label: name.charAt(0).toUpperCase() + name.slice(1),
-        href: '/admin/' + name,
-      })),
-  };
-});
-vi.mock('../../src/client/js/utils/sort', () => ({
-  toSortDate: vi.fn(() => undefined),
-  readSortMode: vi.fn(() => 'alpha'),
-  writeSortMode: vi.fn(),
-  createComparator: vi.fn(() => () => 0),
-  SORT_MODES: {
-    alpha: { icon: 'sort_by_alpha', label: 'Alphabetical' },
-    'date-asc': { icon: 'hourglass_arrow_down', label: 'Oldest first' },
-    'date-desc': { icon: 'hourglass_arrow_up', label: 'Newest first' },
-  },
-  SORT_ORDER: ['alpha', 'date-asc', 'date-desc'],
-}));
-vi.mock('../../src/client/js/drafts/storage', () => ({
-  saveDraft: vi.fn(async () => {}),
-  getDraftByFile: vi.fn(async () => null),
-  loadDrafts: vi.fn(async () => []),
-  loadDraft: vi.fn(async () => null),
-  deleteDraft: vi.fn(async () => {}),
-}));
-vi.mock('../../src/client/js/utils/schema-utils', () => ({
-  extractTabs: vi.fn(() => []),
-  getFieldsForTab: vi.fn(() => []),
-  resolveFieldType: vi.fn(() => ({ kind: 'string' })),
-  createDefaultValue: vi.fn(() => ''),
-  getByPath: vi.fn(),
-  setByPath: vi.fn(),
-  isReadOnly: vi.fn(() => false),
-  isNullable: vi.fn(() => false),
-  getProperties: vi.fn(
-    (schema: Record<string, unknown>) => schema['properties'],
-  ),
-  getRequiredFields: vi.fn((schema: Record<string, unknown>) =>
-    Array.isArray(schema['required']) ? schema['required'] : [],
-  ),
-  getLabel: vi.fn((schema: Record<string, unknown>, name: string) =>
-    typeof schema['title'] === 'string' ? schema['title'] : name,
-  ),
-}));
-vi.mock('../../src/client/js/drafts/merge.svelte', () => ({
-  drafts: {
-    get all() {
-      return mocks.mockDrafts();
-    },
-    get outdated() {
-      return mocks.mockOutdatedMap();
-    },
-  },
-  mergeDrafts: vi.fn(async () => {}),
-  refreshDrafts: vi.fn(async () => {}),
-  resetDraftMerge: vi.fn(),
-}));
-
-vi.mock('../../src/client/js/state/theme.svelte', () => ({
-  initTheme: vi.fn(() => () => {}),
-  cycleTheme: vi.fn(),
-  theme: { resolved: 'dark', icon: 'brightness_auto', label: 'Auto' },
-}));
-vi.mock('../../src/client/js/state/dialogs.svelte', () => ({
-  dialog: {
-    get active() {
-      return null;
-    },
-    open: vi.fn(),
-    close: vi.fn(),
-  },
-}));
-
-import Admin from '../../src/client/Admin.svelte';
-
-afterEach(() => cleanup());
-beforeEach(() => resetMocks(mocks));
-
-describe('Unsaved Changes', () => {
-  it('shows dirty indicator for a file with unsaved changes', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Modified body',
-      formData: { title: 'Hello World' },
-      dirty: true,
-    });
-
-    const { container } = render(Admin);
-
-    // The bullet indicator gets the --visible class when dirty
-    const indicator = container.querySelector('.dirty-indicator--visible');
-    expect(indicator).not.toBeNull();
-  });
-
-  it('does not show dirty indicator for a clean file', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Original body',
-      formData: { title: 'Hello World' },
-      dirty: false,
-    });
-
-    const { container } = render(Admin);
-
-    // Indicator element exists but without --visible class
-    const indicator = container.querySelector('.dirty-indicator--visible');
-    expect(indicator).toBeNull();
-  });
-
-  it('shows dirty indicator for a dirty draft', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Changed',
-      formData: { title: 'Hello World' },
-      dirty: true,
-      draftId: 'draft-1',
-    });
-
-    const { container } = render(Admin);
-
-    const indicator = container.querySelector('.dirty-indicator--visible');
-    expect(indicator).not.toBeNull();
-  });
-
-  it('renders editor area even when dirty state is true', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Some content',
-      formData: { title: 'Hello World' },
-      dirty: true,
-    });
-
-    const { container } = render(Admin);
-
-    expect(container.querySelector('.editor-area')).not.toBeNull();
-    // Toolbar is inside editor-area with class .toolbar (not .editor-toolbar)
-    expect(
-      container.querySelector('.editor-area .toolbar__title'),
-    ).not.toBeNull();
-  });
-
-  it('maintains three-column layout when dirty and file open', () => {
-    configureFileOpen(mocks, 'posts', 'hello-world', {
-      filename: 'hello-world.md',
-      body: 'Content',
-      formData: { title: 'Hello World' },
-      dirty: true,
-    });
-
-    const { container } = render(Admin);
-
-    // Both sidebars + editor area = 3 columns
-    const sidebars = container.querySelectorAll('.sidebar');
-    expect(sidebars.length).toBe(2);
-    expect(container.querySelector('.editor-area')).not.toBeNull();
-    expect(container.querySelector('.admin--file-open')).not.toBeNull();
-  });
-});

package/tests/setup.ts

@@ -1,2 +0,0 @@
-// Polyfills IndexedDB in Node.js for unit tests that exercise storage layer code.
-import 'fake-indexeddb/auto';