@libredb/studio 0.9.7 → 0.9.13

package/tests/unit/schema-diff/migration-generator.test.ts

@@ -1,513 +0,0 @@
-import { describe, test, expect } from 'bun:test';
-import { generateMigrationSQL } from '@/lib/schema-diff/migration-generator';
-import type { SchemaDiff } from '@/lib/schema-diff/types';
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-const noChangesDiff: SchemaDiff = {
-  tables: [],
-  summary: { added: 0, removed: 0, modified: 0 },
-  hasChanges: false,
-};
-
-function makeAddedTableDiff(): SchemaDiff {
-  return {
-    tables: [{
-      action: 'added',
-      tableName: 'users',
-      columns: [
-        { action: 'added', columnName: 'id', targetType: 'integer', targetNullable: false, targetIsPrimary: true, changes: ['Added column "id"'] },
-        { action: 'added', columnName: 'name', targetType: 'varchar(255)', targetNullable: false, targetIsPrimary: false, changes: ['Added column "name"'] },
-        { action: 'added', columnName: 'email', targetType: 'varchar(255)', targetNullable: true, targetIsPrimary: false, changes: ['Added column "email"'] },
-      ],
-      indexes: [
-        { action: 'added', indexName: 'idx_users_email', targetColumns: ['email'], targetUnique: true, changes: ['Added index'] },
-      ],
-      foreignKeys: [],
-    }],
-    summary: { added: 1, removed: 0, modified: 0 },
-    hasChanges: true,
-  };
-}
-
-function makeDroppedTableDiff(): SchemaDiff {
-  return {
-    tables: [{
-      action: 'removed',
-      tableName: 'legacy',
-      columns: [
-        { action: 'removed', columnName: 'id', sourceType: 'integer', changes: ['Removed column'] },
-      ],
-      indexes: [],
-      foreignKeys: [],
-    }],
-    summary: { added: 0, removed: 1, modified: 0 },
-    hasChanges: true,
-  };
-}
-
-function makeModifiedTableDiff(): SchemaDiff {
-  return {
-    tables: [{
-      action: 'modified',
-      tableName: 'users',
-      columns: [
-        { action: 'added', columnName: 'phone', targetType: 'varchar(20)', targetNullable: true, changes: ['Added column "phone"'] },
-        { action: 'removed', columnName: 'legacy_col', sourceType: 'text', changes: ['Removed column'] },
-        {
-          action: 'modified', columnName: 'name',
-          sourceType: 'varchar(100)', targetType: 'varchar(255)',
-          sourceNullable: true, targetNullable: false,
-          sourceDefault: undefined, targetDefault: "'unknown'",
-          changes: ['Type changed', 'Nullable changed', 'Default changed'],
-        },
-      ],
-      indexes: [
-        { action: 'added', indexName: 'idx_phone', targetColumns: ['phone'], targetUnique: false, changes: ['Added index'] },
-        { action: 'removed', indexName: 'idx_legacy', sourceColumns: ['legacy_col'], changes: ['Removed index'] },
-      ],
-      foreignKeys: [
-        { action: 'added', columnName: 'dept_id', targetReferencedTable: 'departments', targetReferencedColumn: 'id', changes: ['Added FK'] },
-        { action: 'removed', columnName: 'old_ref', sourceReferencedTable: 'old_table', sourceReferencedColumn: 'id', changes: ['Removed FK'] },
-      ],
-    }],
-    summary: { added: 0, removed: 0, modified: 1 },
-    hasChanges: true,
-  };
-}
-
-// ============================================================================
-// No changes
-// ============================================================================
-
-describe('generateMigrationSQL: no changes', () => {
-  test('returns comment when no changes', () => {
-    const sql = generateMigrationSQL(noChangesDiff, 'postgres');
-    expect(sql).toBe('-- No schema changes detected.');
-  });
-});
-
-// ============================================================================
-// CREATE TABLE
-// ============================================================================
-
-describe('generateMigrationSQL: CREATE TABLE', () => {
-  test('postgres uses double-quoted identifiers', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'postgres');
-    expect(sql).toContain('CREATE TABLE "users"');
-    expect(sql).toContain('"id" integer NOT NULL');
-    expect(sql).toContain('"name" varchar(255) NOT NULL');
-    expect(sql).toContain('PRIMARY KEY ("id")');
-    expect(sql).toContain('CREATE UNIQUE INDEX "idx_users_email" ON "users" ("email")');
-    expect(sql).toContain('BEGIN;');
-    expect(sql).toContain('COMMIT;');
-  });
-
-  test('mysql uses backtick identifiers', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'mysql');
-    expect(sql).toContain('CREATE TABLE `users`');
-    expect(sql).toContain('`id` integer NOT NULL');
-    expect(sql).toContain('BEGIN;');
-    expect(sql).toContain('COMMIT;');
-  });
-
-  test('sqlite uses double-quoted identifiers and no BEGIN/COMMIT', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'sqlite');
-    expect(sql).toContain('CREATE TABLE "users"');
-    expect(sql).not.toContain('BEGIN;');
-    expect(sql).not.toContain('COMMIT;');
-  });
-
-  test('mssql uses bracket identifiers', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'mssql');
-    expect(sql).toContain('CREATE TABLE [users]');
-    expect(sql).toContain('[id] integer NOT NULL');
-    expect(sql).toContain('BEGIN;');
-  });
-
-  test('oracle uses double-quoted identifiers', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'oracle');
-    expect(sql).toContain('CREATE TABLE "users"');
-    expect(sql).toContain('BEGIN;');
-  });
-});
-
-// ============================================================================
-// DROP TABLE
-// ============================================================================
-
-describe('generateMigrationSQL: DROP TABLE', () => {
-  test('generates DROP TABLE IF EXISTS', () => {
-    const sql = generateMigrationSQL(makeDroppedTableDiff(), 'postgres');
-    expect(sql).toContain('DROP TABLE IF EXISTS "legacy"');
-  });
-
-  test('mysql uses backtick for DROP', () => {
-    const sql = generateMigrationSQL(makeDroppedTableDiff(), 'mysql');
-    expect(sql).toContain('DROP TABLE IF EXISTS `legacy`');
-  });
-});
-
-// ============================================================================
-// ALTER TABLE
-// ============================================================================
-
-describe('generateMigrationSQL: ALTER TABLE', () => {
-  test('postgres: ADD COLUMN, DROP COLUMN, ALTER COLUMN TYPE/NULL/DEFAULT', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'postgres');
-    expect(sql).toContain('ALTER TABLE "users" ADD COLUMN "phone" varchar(20)');
-    expect(sql).toContain('ALTER TABLE "users" DROP COLUMN "legacy_col"');
-    expect(sql).toContain('ALTER TABLE "users" ALTER COLUMN "name" TYPE varchar(255)');
-    expect(sql).toContain('ALTER TABLE "users" ALTER COLUMN "name" SET NOT NULL');
-    expect(sql).toContain("ALTER TABLE \"users\" ALTER COLUMN \"name\" SET DEFAULT 'unknown'");
-  });
-
-  test('mysql: MODIFY COLUMN syntax', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'mysql');
-    expect(sql).toContain('ALTER TABLE `users` MODIFY COLUMN `name` varchar(255) NOT NULL');
-  });
-
-  test('sqlite: column drop produces comment', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'sqlite');
-    expect(sql).toContain('-- SQLite: Cannot drop column "legacy_col" directly');
-    expect(sql).toContain('-- SQLite: Cannot alter column "name" type directly');
-  });
-
-  test('mssql: ALTER COLUMN syntax', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'mssql');
-    expect(sql).toContain('ALTER TABLE [users] ALTER COLUMN [name] varchar(255) NOT NULL');
-    expect(sql).toContain("ALTER TABLE [users] ADD DEFAULT 'unknown' FOR [name]");
-  });
-
-  test('oracle: MODIFY() syntax', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'oracle');
-    expect(sql).toContain('ALTER TABLE "users" MODIFY ("name"');
-  });
-});
-
-// ============================================================================
-// Index operations in ALTER TABLE
-// ============================================================================
-
-describe('generateMigrationSQL: indexes in ALTER', () => {
-  test('CREATE INDEX for added index', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'postgres');
-    expect(sql).toContain('CREATE INDEX "idx_phone" ON "users" ("phone")');
-  });
-
-  test('CREATE UNIQUE INDEX for added unique index', () => {
-    const sql = generateMigrationSQL(makeAddedTableDiff(), 'postgres');
-    expect(sql).toContain('CREATE UNIQUE INDEX');
-  });
-
-  test('DROP INDEX for removed index (postgres)', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'postgres');
-    expect(sql).toContain('DROP INDEX IF EXISTS "idx_legacy"');
-  });
-
-  test('DROP INDEX ON for removed index (mysql)', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'mysql');
-    expect(sql).toContain('DROP INDEX `idx_legacy` ON `users`');
-  });
-});
-
-// ============================================================================
-// Foreign key operations in ALTER TABLE
-// ============================================================================
-
-describe('generateMigrationSQL: foreign keys in ALTER', () => {
-  test('ADD CONSTRAINT for added FK', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'postgres');
-    expect(sql).toContain('ADD CONSTRAINT "fk_users_dept_id" FOREIGN KEY ("dept_id") REFERENCES "departments"("id")');
-  });
-
-  test('DROP CONSTRAINT for removed FK (postgres)', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'postgres');
-    expect(sql).toContain('DROP CONSTRAINT IF EXISTS "fk_users_old_ref"');
-  });
-
-  test('DROP FOREIGN KEY for removed FK (mysql)', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'mysql');
-    expect(sql).toContain('DROP FOREIGN KEY `fk_users_old_ref`');
-  });
-
-  test('SQLite FK drop produces comment', () => {
-    const sql = generateMigrationSQL(makeModifiedTableDiff(), 'sqlite');
-    expect(sql).toContain('-- SQLite: Cannot drop foreign key directly');
-  });
-});
-
-// ============================================================================
-// Column def edge cases
-// ============================================================================
-
-describe('generateMigrationSQL: column def edge cases', () => {
-  test('column def uses sourceType fallback when targetType undefined', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'added',
-        tableName: 'test_table',
-        columns: [
-          { action: 'added', columnName: 'col1', sourceType: 'integer', targetType: undefined as unknown as string, targetNullable: true, changes: ['Added'] },
-        ],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 1, removed: 0, modified: 0 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('"col1" integer');
-  });
-
-  test('column def uses TEXT fallback when both types undefined', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'added',
-        tableName: 'test_table',
-        columns: [
-          { action: 'added', columnName: 'col1', targetNullable: true, changes: ['Added'] },
-        ],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 1, removed: 0, modified: 0 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('"col1" TEXT');
-  });
-
-  test('column def includes DEFAULT clause', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'added',
-        tableName: 'test_table',
-        columns: [
-          { action: 'added', columnName: 'status', targetType: 'varchar(20)', targetNullable: true, targetDefault: "'active'", changes: ['Added'] },
-        ],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 1, removed: 0, modified: 0 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain("\"status\" varchar(20) DEFAULT 'active'");
-  });
-
-  test('CREATE TABLE without PRIMARY KEY', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'added',
-        tableName: 'logs',
-        columns: [
-          { action: 'added', columnName: 'message', targetType: 'text', targetNullable: true, targetIsPrimary: false, changes: ['Added'] },
-          { action: 'added', columnName: 'level', targetType: 'varchar(10)', targetNullable: true, targetIsPrimary: false, changes: ['Added'] },
-        ],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 1, removed: 0, modified: 0 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('CREATE TABLE "logs"');
-    expect(sql).not.toContain('PRIMARY KEY');
-  });
-});
-
-// ============================================================================
-// PostgreSQL-specific ALTER edge cases
-// ============================================================================
-
-describe('generateMigrationSQL: PostgreSQL ALTER edge cases', () => {
-  test('nullable-only change (no type change) generates SET NOT NULL', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'email',
-          sourceType: 'varchar(255)', targetType: 'varchar(255)',
-          sourceNullable: true, targetNullable: false,
-          changes: ['Nullable changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).not.toContain('TYPE');
-    expect(sql).toContain('SET NOT NULL');
-  });
-
-  test('DROP NOT NULL (targetNullable=true)', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'email',
-          sourceType: 'varchar(255)', targetType: 'varchar(255)',
-          sourceNullable: false, targetNullable: true,
-          changes: ['Nullable changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('DROP NOT NULL');
-  });
-
-  test('DROP DEFAULT (targetDefault falsy)', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'status',
-          sourceType: 'varchar(20)', targetType: 'varchar(20)',
-          sourceDefault: "'active'", targetDefault: undefined,
-          changes: ['Default changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('DROP DEFAULT');
-  });
-
-  test('type-only change (no nullable/default changes)', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'age',
-          sourceType: 'smallint', targetType: 'integer',
-          sourceNullable: true, targetNullable: true,
-          changes: ['Type changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('TYPE integer');
-    expect(sql).not.toContain('NOT NULL');
-    expect(sql).not.toContain('DEFAULT');
-  });
-});
-
-// ============================================================================
-// MSSQL + Oracle ALTER edge cases
-// ============================================================================
-
-describe('generateMigrationSQL: MSSQL/Oracle ALTER edge cases', () => {
-  test('MSSQL modified column without default change (no ADD DEFAULT)', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'name',
-          sourceType: 'varchar(100)', targetType: 'varchar(255)',
-          sourceDefault: undefined, targetDefault: undefined,
-          changes: ['Type changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'mssql');
-    expect(sql).toContain('ALTER COLUMN [name] varchar(255)');
-    expect(sql).not.toContain('ADD DEFAULT');
-  });
-
-  test('MSSQL bracket escaping with ] in table name', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'removed',
-        tableName: 'table]name',
-        columns: [],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 1, modified: 0 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'mssql');
-    expect(sql).toContain('[table]]name]');
-  });
-
-  test('Oracle MODIFY with default value', () => {
-    const diff: SchemaDiff = {
-      tables: [{
-        action: 'modified',
-        tableName: 'users',
-        columns: [{
-          action: 'modified', columnName: 'status',
-          sourceType: 'VARCHAR2(10)', targetType: 'VARCHAR2(50)',
-          targetDefault: "'active'",
-          changes: ['Type changed', 'Default changed'],
-        }],
-        indexes: [],
-        foreignKeys: [],
-      }],
-      summary: { added: 0, removed: 0, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'oracle');
-    expect(sql).toContain('MODIFY');
-    expect(sql).toContain("DEFAULT 'active'");
-  });
-});
-
-// ============================================================================
-// Multi-table diff
-// ============================================================================
-
-describe('generateMigrationSQL: multi-table batch', () => {
-  test('handles added + removed + modified in one batch', () => {
-    const diff: SchemaDiff = {
-      tables: [
-        {
-          action: 'removed', tableName: 'legacy',
-          columns: [{ action: 'removed', columnName: 'id', sourceType: 'int', changes: [] }],
-          indexes: [], foreignKeys: [],
-        },
-        {
-          action: 'added', tableName: 'new_table',
-          columns: [{ action: 'added', columnName: 'id', targetType: 'integer', targetNullable: false, targetIsPrimary: true, changes: [] }],
-          indexes: [], foreignKeys: [],
-        },
-        {
-          action: 'modified', tableName: 'users',
-          columns: [{ action: 'added', columnName: 'phone', targetType: 'varchar(20)', targetNullable: true, changes: [] }],
-          indexes: [], foreignKeys: [],
-        },
-      ],
-      summary: { added: 1, removed: 1, modified: 1 },
-      hasChanges: true,
-    };
-    const sql = generateMigrationSQL(diff, 'postgres');
-    expect(sql).toContain('DROP TABLE IF EXISTS "legacy"');
-    expect(sql).toContain('CREATE TABLE "new_table"');
-    expect(sql).toContain('ALTER TABLE "users" ADD COLUMN "phone"');
-  });
-});

package/tests/unit/seed/config-loader.test.ts

@@ -1,73 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'bun:test';
-import path from 'path';
-import { loadConfig, resetCache } from '@/lib/seed/config-loader';
-
-const FIXTURES = path.resolve(__dirname, '../../fixtures/seed-connections');
-
-describe('config-loader', () => {
-  beforeEach(() => {
-    resetCache();
-  });
-
-  afterEach(() => {
-    delete process.env.SEED_CONFIG_PATH;
-    delete process.env.SEED_CACHE_TTL_MS;
-  });
-
-  it('loads and parses valid YAML config', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'valid-config.yaml');
-    const config = await loadConfig();
-    expect(config).not.toBeNull();
-    expect(config!.version).toBe('1');
-    expect(config!.connections).toHaveLength(4);
-    expect(config!.connections[0].id).toBe('test-postgres');
-  });
-
-  it('loads and parses valid JSON config', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'valid-config.json');
-    const config = await loadConfig();
-    expect(config).not.toBeNull();
-    expect(config!.version).toBe('1');
-    expect(config!.connections).toHaveLength(1);
-  });
-
-  it('returns null when config file does not exist', async () => {
-    process.env.SEED_CONFIG_PATH = '/nonexistent/path/config.yaml';
-    const config = await loadConfig();
-    expect(config).toBeNull();
-  });
-
-  it('throws on invalid YAML (validation fails)', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'invalid-config.yaml');
-    await expect(loadConfig()).rejects.toThrow();
-  });
-
-  it('uses default path when SEED_CONFIG_PATH not set', async () => {
-    const config = await loadConfig();
-    expect(config).toBeNull();
-  });
-
-  it('caches result within TTL', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'valid-config.yaml');
-    process.env.SEED_CACHE_TTL_MS = '60000';
-    const config1 = await loadConfig();
-    const config2 = await loadConfig();
-    expect(config1).toBe(config2);
-  });
-
-  it('reloads after cache reset', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'valid-config.yaml');
-    const config1 = await loadConfig();
-    resetCache();
-    const config2 = await loadConfig();
-    expect(config1).not.toBe(config2);
-    expect(config1!.connections).toHaveLength(config2!.connections.length);
-  });
-
-  it('loads minimal config with only required fields', async () => {
-    process.env.SEED_CONFIG_PATH = path.join(FIXTURES, 'minimal-config.yaml');
-    const config = await loadConfig();
-    expect(config).not.toBeNull();
-    expect(config!.connections).toHaveLength(1);
-  });
-});

package/tests/unit/seed/connection-filter.test.ts

@@ -1,91 +0,0 @@
-import { describe, it, expect } from 'bun:test';
-import { filterByRoles, mergeDefaults } from '@/lib/seed/connection-filter';
-import type { SeedConnection, SeedDefaults } from '@/lib/seed/types';
-
-const baseConn: SeedConnection = {
-  id: 'test',
-  name: 'Test',
-  type: 'postgres',
-  host: 'localhost',
-  roles: ['*'],
-};
-
-describe('mergeDefaults', () => {
-  it('applies defaults when connection fields are missing', () => {
-    const defaults: SeedDefaults = { managed: true, environment: 'production' };
-    const merged = mergeDefaults({ ...baseConn }, defaults);
-    expect(merged.managed).toBe(true);
-    expect(merged.environment).toBe('production');
-  });
-
-  it('connection-level values override defaults', () => {
-    const defaults: SeedDefaults = { managed: true, environment: 'production' };
-    const merged = mergeDefaults({ ...baseConn, managed: false, environment: 'staging' }, defaults);
-    expect(merged.managed).toBe(false);
-    expect(merged.environment).toBe('staging');
-  });
-
-  it('returns connection unchanged when no defaults', () => {
-    const merged = mergeDefaults({ ...baseConn, managed: true }, undefined);
-    expect(merged.managed).toBe(true);
-  });
-
-  it('merges ssl defaults', () => {
-    const defaults: SeedDefaults = { ssl: { mode: 'require', rejectUnauthorized: true } };
-    const merged = mergeDefaults({ ...baseConn }, defaults);
-    expect(merged.ssl).toEqual({ mode: 'require', rejectUnauthorized: true });
-  });
-
-  it('connection ssl overrides default ssl', () => {
-    const defaults: SeedDefaults = { ssl: { mode: 'require' } };
-    const merged = mergeDefaults({ ...baseConn, ssl: { mode: 'disable' } }, defaults);
-    expect(merged.ssl?.mode).toBe('disable');
-  });
-});
-
-describe('filterByRoles', () => {
-  it('includes connections with wildcard role', () => {
-    const result = filterByRoles([{ ...baseConn, roles: ['*'] }], ['user']);
-    expect(result).toHaveLength(1);
-  });
-
-  it('includes connections matching user role', () => {
-    const result = filterByRoles([{ ...baseConn, roles: ['admin'] }], ['admin']);
-    expect(result).toHaveLength(1);
-  });
-
-  it('excludes connections not matching user role', () => {
-    const result = filterByRoles([{ ...baseConn, roles: ['admin'] }], ['user']);
-    expect(result).toHaveLength(0);
-  });
-
-  it('handles multi-role connections', () => {
-    const result = filterByRoles([{ ...baseConn, roles: ['admin', 'user'] }], ['user']);
-    expect(result).toHaveLength(1);
-  });
-
-  it('maps SeedConnection to ManagedConnection correctly', () => {
-    const result = filterByRoles([{
-      ...baseConn, id: 'my-pg', managed: true, color: '#FF0000', group: 'Backend',
-    }], ['admin']);
-    expect(result[0].seedId).toBe('my-pg');
-    expect(result[0].id).toBe('seed:my-pg');
-    expect(result[0].managed).toBe(true);
-    expect(result[0].color).toBe('#FF0000');
-    expect(result[0].group).toBe('Backend');
-    expect(result[0].createdAt).toBeInstanceOf(Date);
-  });
-
-  it('defaults managed to true when not specified', () => {
-    const result = filterByRoles([{ ...baseConn }], ['admin']);
-    expect(result[0].managed).toBe(true);
-  });
-
-  it('returns empty array when no connections match', () => {
-    const result = filterByRoles([
-      { ...baseConn, roles: ['admin'] },
-      { ...baseConn, id: 'other', roles: ['admin'] },
-    ], ['user']);
-    expect(result).toHaveLength(0);
-  });
-});