@libredb/studio 0.9.7 → 0.9.12

package/tests/isolated/factory-singleton.test.ts

@@ -1,150 +0,0 @@
-/**
- * Factory singleton tests — isolated process required.
- * Mocks provider modules to test getStorageProvider() and closeStorageProvider()
- * without real database connections.
- */
-import { describe, test, expect, beforeEach, mock } from 'bun:test';
-
-// ── Mock providers ──────────────────────────────────────────────────────────
-
-const mockInitialize = mock(async () => {});
-const mockClose = mock(async () => {});
-const mockGetAllData = mock(async () => ({}));
-
-function makeMockProvider() {
-  return {
-    initialize: mockInitialize,
-    close: mockClose,
-    getAllData: mockGetAllData,
-    getCollection: mock(async () => null),
-    setCollection: mock(async () => {}),
-    mergeData: mock(async () => {}),
-    isHealthy: mock(async () => true),
-  };
-}
-
-const mockSQLiteInstance = makeMockProvider();
-const mockPostgresInstance = makeMockProvider();
-
-mock.module('@/lib/storage/providers/sqlite', () => ({
-  SQLiteStorageProvider: mock(() => mockSQLiteInstance),
-}));
-
-mock.module('@/lib/storage/providers/postgres', () => ({
-  PostgresStorageProvider: mock(() => mockPostgresInstance),
-}));
-
-// Import factory AFTER mocking providers
-import {
-  getStorageProvider,
-  closeStorageProvider,
-  getStorageProviderType,
-} from '@/lib/storage/factory';
-
-// ── Tests ───────────────────────────────────────────────────────────────────
-
-describe('factory: getStorageProvider', () => {
-  beforeEach(async () => {
-    // Reset singleton state between tests
-    await closeStorageProvider();
-    mockInitialize.mockClear();
-    mockClose.mockClear();
-    delete process.env.STORAGE_PROVIDER;
-  });
-
-  test('returns null when STORAGE_PROVIDER is local', async () => {
-    process.env.STORAGE_PROVIDER = 'local';
-    const provider = await getStorageProvider();
-    expect(provider).toBeNull();
-  });
-
-  test('returns null when STORAGE_PROVIDER is not set', async () => {
-    const provider = await getStorageProvider();
-    expect(provider).toBeNull();
-  });
-
-  test('creates SQLite provider when STORAGE_PROVIDER=sqlite', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    const provider = await getStorageProvider();
-
-    expect(provider).not.toBeNull();
-    expect(mockInitialize).toHaveBeenCalledTimes(1);
-  });
-
-  test('creates Postgres provider when STORAGE_PROVIDER=postgres', async () => {
-    process.env.STORAGE_PROVIDER = 'postgres';
-    const provider = await getStorageProvider();
-
-    expect(provider).not.toBeNull();
-    expect(mockInitialize).toHaveBeenCalledTimes(1);
-  });
-
-  test('returns same instance on second call (singleton)', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    const first = await getStorageProvider();
-    const second = await getStorageProvider();
-
-    expect(first).toBe(second);
-    // initialize called only once, not twice
-    expect(mockInitialize).toHaveBeenCalledTimes(1);
-  });
-
-  test('calls initialize() on first creation', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    await getStorageProvider();
-
-    expect(mockInitialize).toHaveBeenCalledTimes(1);
-  });
-
-  test('propagates error when initialize() throws', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    mockInitialize.mockRejectedValueOnce(new Error('DB init failed'));
-
-    await expect(getStorageProvider()).rejects.toThrow('DB init failed');
-  });
-});
-
-describe('factory: closeStorageProvider', () => {
-  beforeEach(async () => {
-    await closeStorageProvider();
-    mockInitialize.mockClear();
-    mockClose.mockClear();
-    delete process.env.STORAGE_PROVIDER;
-  });
-
-  test('closes and resets singleton', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    await getStorageProvider();
-
-    await closeStorageProvider();
-    expect(mockClose).toHaveBeenCalledTimes(1);
-  });
-
-  test('creates new instance after close + re-get', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    await getStorageProvider();
-    await closeStorageProvider();
-
-    mockInitialize.mockClear();
-    const provider = await getStorageProvider();
-
-    expect(provider).not.toBeNull();
-    // New initialize call — fresh instance
-    expect(mockInitialize).toHaveBeenCalledTimes(1);
-  });
-
-  test('does not throw when called without active provider', async () => {
-    await expect(closeStorageProvider()).resolves.toBeUndefined();
-    expect(mockClose).not.toHaveBeenCalled();
-  });
-
-  test('double close does not throw', async () => {
-    process.env.STORAGE_PROVIDER = 'sqlite';
-    await getStorageProvider();
-
-    await closeStorageProvider();
-    await expect(closeStorageProvider()).resolves.toBeUndefined();
-    // close called only once (second call has no provider)
-    expect(mockClose).toHaveBeenCalledTimes(1);
-  });
-});

package/tests/isolated/use-storage-sync.test.ts

@@ -1,389 +0,0 @@
-import '../setup-dom';
-
-import { describe, test, expect, mock, beforeEach, afterEach } from 'bun:test';
-import { renderHook, waitFor, act, cleanup } from '@testing-library/react';
-import { mockGlobalFetch, restoreGlobalFetch } from '../helpers/mock-fetch';
-
-// ── Mock storage module ─────────────────────────────────────────────────────
-
-const mockStorage = {
-  getConnections: mock(() => [{ id: 'c1' }]),
-  getHistory: mock(() => []),
-  getSavedQueries: mock(() => []),
-  getSchemaSnapshots: mock(() => []),
-  getSavedCharts: mock(() => []),
-  getActiveConnectionId: mock(() => null),
-  getAuditLog: mock(() => []),
-  getMaskingConfig: mock(() => ({ enabled: true, patterns: [], roleSettings: { admin: { canToggle: true, canReveal: true }, user: { canToggle: false, canReveal: false } } })),
-  getThresholdConfig: mock(() => []),
-};
-
-mock.module('@/lib/storage', () => ({
-  storage: mockStorage,
-  STORAGE_COLLECTIONS: [
-    'connections', 'history', 'saved_queries', 'schema_snapshots',
-    'saved_charts', 'active_connection_id', 'audit_log',
-    'masking_config', 'threshold_config',
-  ],
-}));
-
-import { useStorageSync } from '@/hooks/use-storage-sync';
-
-// ── Helpers ─────────────────────────────────────────────────────────────────
-
-function setupLocalMode() {
-  return mockGlobalFetch({
-    '/api/storage/config': { ok: true, status: 200, json: { provider: 'local', serverMode: false } },
-  });
-}
-
-function setupServerMode(extraRoutes: Record<string, unknown> = {}) {
-  return mockGlobalFetch({
-    '/api/storage/config': { ok: true, status: 200, json: { provider: 'postgres', serverMode: true } },
-    '/api/storage/migrate': { ok: true, status: 200, json: { ok: true, migrated: ['connections'] } },
-    '/api/storage': { ok: true, status: 200, json: { connections: [{ id: 'server-c1' }] } },
-    ...extraRoutes,
-  });
-}
-
-// ── Tests ───────────────────────────────────────────────────────────────────
-
-describe('useStorageSync', () => {
-  beforeEach(() => {
-    localStorage.clear();
-    Object.values(mockStorage).forEach((fn) => fn.mockClear());
-  });
-
-  afterEach(() => {
-    restoreGlobalFetch();
-    cleanup();
-  });
-
-  // ── Mode discovery ──────────────────────────────────────────────────────
-
-  describe('mode discovery', () => {
-    test('starts with isServerMode=false', () => {
-      setupLocalMode();
-      const { result } = renderHook(() => useStorageSync());
-      expect(result.current.isServerMode).toBe(false);
-    });
-
-    test('stays in local mode when config returns serverMode=false', async () => {
-      setupLocalMode();
-      const { result } = renderHook(() => useStorageSync());
-
-      // Wait for config fetch to resolve
-      await waitFor(() => {
-        expect(result.current.isSyncing).toBe(false);
-      });
-
-      expect(result.current.isServerMode).toBe(false);
-    });
-
-    test('switches to server mode when config returns serverMode=true', async () => {
-      setupServerMode();
-      localStorage.setItem('libredb_server_migrated', 'true'); // Skip migration
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-    });
-
-    test('stays in local mode when config fetch fails', async () => {
-      mockGlobalFetch({
-        '/api/storage/config': { ok: false, status: 500, json: { error: 'Server error' } },
-      });
-
-      const { result } = renderHook(() => useStorageSync());
-
-      // Give it time to settle
-      await waitFor(() => {
-        expect(result.current.isSyncing).toBe(false);
-      });
-
-      expect(result.current.isServerMode).toBe(false);
-    });
-
-    test('stays in local mode when config fetch throws network error', async () => {
-      globalThis.fetch = mock(async () => {
-        throw new Error('Network error');
-      }) as unknown as typeof fetch;
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isSyncing).toBe(false);
-      });
-
-      expect(result.current.isServerMode).toBe(false);
-      expect(result.current.syncError).toBeNull();
-    });
-  });
-
-  // ── Migration ───────────────────────────────────────────────────────────
-
-  describe('migration', () => {
-    test('performs migration on first server-mode visit when localStorage has data', async () => {
-      // Seed localStorage with actual data so migration has something to send
-      localStorage.setItem('libredb_connections', JSON.stringify([{ id: 'test', name: 'Test DB' }]));
-      const fetchMock = setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // Migration flag should be set
-      expect(localStorage.getItem('libredb_server_migrated')).not.toBeNull();
-
-      // migrate endpoint was called
-      const calls = (fetchMock.mock.calls as unknown[][]).map((c) => {
-        const url = typeof c[0] === 'string' ? c[0] : '';
-        return new URL(url, 'http://localhost:3000').pathname;
-      });
-      expect(calls).toContain('/api/storage/migrate');
-    });
-
-    test('skips migration on fresh browser with empty localStorage', async () => {
-      const fetchMock = setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // Migration flag should still be set (to prevent future re-checks)
-      expect(localStorage.getItem('libredb_server_migrated')).not.toBeNull();
-
-      // migrate endpoint should NOT be called — no local data to migrate
-      const calls = (fetchMock.mock.calls as unknown[][]).map((c) => {
-        const url = typeof c[0] === 'string' ? c[0] : '';
-        return new URL(url, 'http://localhost:3000').pathname;
-      });
-      expect(calls).not.toContain('/api/storage/migrate');
-    });
-
-    test('skips migration when flag already set', async () => {
-      localStorage.setItem('libredb_server_migrated', '2026-01-01');
-      const fetchMock = setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // migrate endpoint should NOT be called
-      const calls = (fetchMock.mock.calls as unknown[][]).map((c) => {
-        const url = typeof c[0] === 'string' ? c[0] : '';
-        return new URL(url, 'http://localhost:3000').pathname;
-      });
-      expect(calls).not.toContain('/api/storage/migrate');
-    });
-
-    test('sets migration flag even when no data to migrate', async () => {
-      // All storage getters return empty
-      mockStorage.getConnections.mockReturnValue([]);
-      mockStorage.getActiveConnectionId.mockReturnValue(null);
-
-      setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      expect(localStorage.getItem('libredb_server_migrated')).not.toBeNull();
-    });
-  });
-
-  // ── Pull from server ──────────────────────────────────────────────────
-
-  describe('pull from server', () => {
-    test('pulls data from server on mount in server mode', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-      const fetchMock = setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // /api/storage was called for pull
-      const calls = (fetchMock.mock.calls as unknown[][]).map((c) => {
-        const url = typeof c[0] === 'string' ? c[0] : '';
-        return new URL(url, 'http://localhost:3000').pathname;
-      });
-      expect(calls).toContain('/api/storage');
-    });
-
-    test('writes server data to localStorage on pull', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-      setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.lastSyncedAt).not.toBeNull();
-      });
-
-      // Server returned connections: [{ id: 'server-c1' }]
-      const stored = localStorage.getItem('libredb_connections');
-      expect(stored).not.toBeNull();
-      expect(JSON.parse(stored!)).toEqual([{ id: 'server-c1' }]);
-    });
-
-    test('sets syncError on pull failure', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-      mockGlobalFetch({
-        '/api/storage/config': { ok: true, status: 200, json: { provider: 'postgres', serverMode: true } },
-        '/api/storage': { ok: false, status: 500, json: { error: 'DB error' } },
-      });
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // Pull failed but no syncError for non-ok response (graceful degradation)
-      // The hook just returns early without setting error for non-ok
-      expect(result.current.isSyncing).toBe(false);
-    });
-  });
-
-  // ── Push to server (debounced) ────────────────────────────────────────
-
-  describe('push to server', () => {
-    test('pushes collection to server on storage-change event', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-      const fetchMock = mockGlobalFetch({
-        '/api/storage/config': { ok: true, status: 200, json: { provider: 'postgres', serverMode: true } },
-        '/api/storage/migrate': { ok: true, status: 200, json: { ok: true, migrated: [] } },
-        '/api/storage': { ok: true, status: 200, json: {} },
-        '/api/storage/connections': { ok: true, status: 200, json: { ok: true } },
-      });
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // Dispatch storage change event
-      act(() => {
-        window.dispatchEvent(
-          new CustomEvent('libredb-storage-change', {
-            detail: { collection: 'connections', data: [{ id: 'c1' }] },
-          })
-        );
-      });
-
-      // Wait for debounce (500ms) + push
-      await waitFor(() => {
-        const calls = (fetchMock.mock.calls as unknown[][]).map((c) => {
-          const url = typeof c[0] === 'string' ? c[0] : '';
-          return new URL(url, 'http://localhost:3000').pathname;
-        });
-        return calls.includes('/api/storage/connections');
-      }, { timeout: 2000 });
-    });
-
-    test('sets syncError on push failure', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-
-      // Use a request handler that returns 500 specifically for PUT /connections
-      const fetchMock = mockGlobalFetch({
-        '/api/storage/config': { ok: true, status: 200, json: { provider: 'postgres', serverMode: true } },
-        '/api/storage/migrate': { ok: true, status: 200, json: { ok: true, migrated: [] } },
-        '/api/storage/connections': { ok: false, status: 500, json: { error: 'Write failed' } },
-        '/api/storage': { ok: true, status: 200, json: {} },
-      });
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.isServerMode).toBe(true);
-      });
-
-      // Ensure isSyncing is done before triggering push
-      await waitFor(() => {
-        expect(result.current.isSyncing).toBe(false);
-      });
-
-      act(() => {
-        window.dispatchEvent(
-          new CustomEvent('libredb-storage-change', {
-            detail: { collection: 'connections', data: [{ id: 'c1' }] },
-          })
-        );
-      });
-
-      // Wait for debounce (500ms) + push to complete and set syncError
-      await waitFor(() => {
-        expect(result.current.syncError).not.toBeNull();
-      }, { timeout: 3000 });
-    });
-  });
-
-  // ── Event listener lifecycle ──────────────────────────────────────────
-
-  describe('event listener lifecycle', () => {
-    test('does not listen for events in local mode', async () => {
-      setupLocalMode();
-      const spy = mock(() => {});
-      const origAdd = window.addEventListener.bind(window);
-      window.addEventListener = mock((...args: Parameters<typeof window.addEventListener>) => {
-        if (args[0] === 'libredb-storage-change') spy();
-        origAdd(...args);
-      }) as typeof window.addEventListener;
-
-      renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        // Give time for init to complete
-      });
-
-      // Event listener for storage change should not be added in local mode
-      expect(spy).not.toHaveBeenCalled();
-
-      window.addEventListener = origAdd;
-    });
-  });
-
-  // ── Initial state ─────────────────────────────────────────────────────
-
-  describe('initial state', () => {
-    test('returns correct initial state shape', () => {
-      setupLocalMode();
-      const { result } = renderHook(() => useStorageSync());
-
-      expect(result.current).toEqual({
-        isServerMode: false,
-        isSyncing: false,
-        isReady: false,
-        lastSyncedAt: null,
-        syncError: null,
-      });
-    });
-
-    test('updates lastSyncedAt after successful pull', async () => {
-      localStorage.setItem('libredb_server_migrated', 'true');
-      setupServerMode();
-
-      const { result } = renderHook(() => useStorageSync());
-
-      await waitFor(() => {
-        expect(result.current.lastSyncedAt).not.toBeNull();
-      });
-
-      expect(result.current.lastSyncedAt).toBeInstanceOf(Date);
-    });
-  });
-});