ComfyUI-Lora-Manager/tests/frontend/managers/settingsManager.sidecarStorage.test.js

import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';

vi.mock('../../../static/js/managers/ModalManager.js', () => ({
    modalManager: {
        closeModal: vi.fn(),
    },
}));

vi.mock('../../../static/js/utils/uiHelpers.js', () => ({
    showToast: vi.fn(),
}));

vi.mock('../../../static/js/state/index.js', () => {
    const settings = {};
    return {
        state: {
            global: {
                settings,
            },
            loadingManager: {
                showSimpleLoading: vi.fn(),
                hide: vi.fn(),
            },
        },
        createDefaultSettings: () => ({
            language: 'en',
            sidecar_storage_mode: 'alongside',
            sidecar_storage_path: '',
        }),
    };
});

vi.mock('../../../static/js/api/modelApiFactory.js', () => ({
    resetAndReload: vi.fn(),
    getModelApiClient: vi.fn(),
}));

vi.mock('../../../static/js/utils/constants.js', () => ({
    DOWNLOAD_PATH_TEMPLATES: {},
    DEFAULT_PATH_TEMPLATES: {},
    MAPPABLE_BASE_MODELS: [],
    PATH_TEMPLATE_PLACEHOLDERS: {},
    FILENAME_TEMPLATE_PLACEHOLDERS: [],
    DEFAULT_FILENAME_TEMPLATES: { lora: '', checkpoint: '', embedding: '' },
    DEFAULT_PRIORITY_TAG_CONFIG: {
        lora: 'character, style',
        checkpoint: 'base, guide',
        embedding: 'hint',
    },
    getMappableBaseModelsDynamic: () => [],
}));

vi.mock('../../../static/js/utils/i18nHelpers.js', () => ({
    translate: (_key, _params, fallback) => fallback ?? '',
}));

vi.mock('../../../static/js/i18n/index.js', () => ({
    i18n: {
        getCurrentLocale: () => 'en',
        setLanguage: vi.fn().mockResolvedValue(),
    },
}));

vi.mock('../../../static/js/components/shared/ModelCard.js', () => ({
    configureModelCardVideo: vi.fn(),
}));

import { SettingsManager } from '../../../static/js/managers/SettingsManager.js';
import { showToast } from '../../../static/js/utils/uiHelpers.js';
import { resetAndReload } from '../../../static/js/api/modelApiFactory.js';
import { state } from '../../../static/js/state/index.js';

const createManager = () => {
    state.global.settings = {};
    const initSettingsSpy = vi
        .spyOn(SettingsManager.prototype, 'initializeSettings')
        .mockResolvedValue();
    const initializeSpy = vi
        .spyOn(SettingsManager.prototype, 'initialize')
        .mockImplementation(() => {});

    const manager = new SettingsManager();

    initSettingsSpy.mockRestore();
    initializeSpy.mockRestore();

    return manager;
};

const appendSidecarControls = () => {
    const select = document.createElement('select');
    select.id = 'sidecarStorageMode';
    ['alongside', 'centralized'].forEach((value) => {
        const option = document.createElement('option');
        option.value = value;
        select.appendChild(option);
    });

    const pathSetting = document.createElement('div');
    pathSetting.id = 'sidecarStoragePathSetting';
    pathSetting.style.display = 'none';

    const pathInput = document.createElement('input');
    pathInput.id = 'sidecarStoragePath';

    const migrateBtn = document.createElement('button');
    migrateBtn.id = 'migrateSidecarsBtn';

    document.body.append(select, pathSetting, pathInput, migrateBtn);
    return { select, pathSetting, pathInput, migrateBtn };
};

const appendMigrationModal = () => {
    const modal = document.createElement('div');
    modal.id = 'sidecarMigrationConfirmModal';
    modal.innerHTML = `
        <h2 data-role="title"></h2>
        <p data-role="message"></p>
        <p data-role="destination" style="display:none"></p>
        <p data-role="default-location-hint" style="display:none"></p>
        <button data-action="confirm-sidecar-migration"></button>
        <button data-action="cancel-sidecar-migration"></button>`;
    document.body.appendChild(modal);
    return modal;
};

const mockFetchOk = (payload = { success: true }) => {
    global.fetch = vi.fn().mockResolvedValue({
        ok: true,
        json: () => Promise.resolve(payload),
    });
};

// Fetch mock where the GET /api/lm/settings refresh reports ``resolvedRoot``
// as the new resolved sidecar root; every other call succeeds generically.
const mockFetchWithResolvedRoot = (resolvedRoot) => {
    global.fetch = vi.fn((url, options) => {
        if (url === '/api/lm/settings' && !options) {
            return Promise.resolve({
                ok: true,
                json: () => Promise.resolve({
                    success: true,
                    settings: { sidecar_storage_root: resolvedRoot },
                }),
            });
        }
        return Promise.resolve({
            ok: true,
            json: () => Promise.resolve({ success: true }),
        });
    });
};

beforeEach(() => {
    document.body.innerHTML = '';
    vi.clearAllMocks();
});

afterEach(() => {
    delete global.fetch;
});

describe('SettingsManager sidecar storage', () => {
    describe('loadSidecarStorageSettings', () => {
        it('loads alongside mode and hides the centralized path input', () => {
            const manager = createManager();
            const { select, pathSetting, pathInput } = appendSidecarControls();
            state.global.settings = { sidecar_storage_mode: 'alongside', sidecar_storage_path: '/data/sidecars' };

            manager.loadSidecarStorageSettings();

            expect(select.value).toBe('alongside');
            expect(pathInput.value).toBe('/data/sidecars');
            expect(pathSetting.style.display).toBe('none');
            expect(manager._loadedSidecarStorageMode).toBe('alongside');
        });

        it('loads centralized mode and shows the path input', () => {
            const manager = createManager();
            const { select, pathSetting } = appendSidecarControls();
            state.global.settings = { sidecar_storage_mode: 'centralized' };

            manager.loadSidecarStorageSettings();

            expect(select.value).toBe('centralized');
            expect(pathSetting.style.display).toBe('block');
            expect(manager._loadedSidecarStorageMode).toBe('centralized');
        });

        it('falls back to alongside for unknown stored modes', () => {
            const manager = createManager();
            const { select } = appendSidecarControls();
            state.global.settings = { sidecar_storage_mode: 'bogus' };

            manager.loadSidecarStorageSettings();

            expect(select.value).toBe('alongside');
        });
    });

    describe('handleSidecarStorageModeChange', () => {
        it('does not prompt for migration when the mode is unchanged', async () => {
            const manager = createManager();
            const { select } = appendSidecarControls();
            appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'alongside' };
            manager._loadedSidecarStorageMode = 'alongside';
            select.value = 'alongside';
            mockFetchOk();

            await manager.handleSidecarStorageModeChange();

            expect(global.fetch).not.toHaveBeenCalledWith(
                '/api/lm/sidecars/migrate',
                expect.anything()
            );
            expect(showToast).toHaveBeenCalledWith(
                'toast.settings.settingsUpdated',
                expect.anything(),
                'success'
            );
        });

        it('migrates to centralized after the user confirms the prompt', async () => {
            const manager = createManager();
            const { select, pathSetting } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'alongside' };
            manager._loadedSidecarStorageMode = 'alongside';
            select.value = 'centralized';
            mockFetchOk();

            const changePromise = manager.handleSidecarStorageModeChange();

            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await changePromise;

            expect(state.global.settings.sidecar_storage_mode).toBe('centralized');
            expect(pathSetting.style.display).toBe('block');
            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/migrate', expect.objectContaining({
                method: 'POST',
                body: JSON.stringify({ direction: 'to_centralized', force: true }),
            }));
            // The summary modal stacks above the settings modal; the reload
            // only happens once the user dismisses it.
            const summaryModal = document.getElementById('sidecarMigrationSummaryModal');
            expect(summaryModal).not.toBeNull();
            expect(resetAndReload).not.toHaveBeenCalled();
            expect(modal.classList.contains('show')).toBe(false);

            summaryModal.querySelector('[data-action="close-modal"]').click();
            expect(resetAndReload).toHaveBeenCalledWith(true);
        });

        it('names the resolved destination in the confirm dialog', async () => {
            const manager = createManager();
            const { select } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'alongside',
                sidecar_storage_root: '/data/sidecars',
            };
            manager._loadedSidecarStorageMode = 'alongside';
            select.value = 'centralized';
            mockFetchOk();

            const changePromise = manager.handleSidecarStorageModeChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));

            const destination = modal.querySelector('[data-role="destination"]');
            expect(destination.textContent).toContain('/data/sidecars');
            expect(destination.style.display).toBe('block');

            modal.querySelector('[data-action="cancel-sidecar-migration"]').click();
            await changePromise;
        });

        it('hints at setting a custom folder first when migrating to the default location', async () => {
            const manager = createManager();
            const { select } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'alongside',
                sidecar_storage_root: '/settings/sidecars',
                sidecar_storage_root_is_default: true,
            };
            manager._loadedSidecarStorageMode = 'alongside';
            select.value = 'centralized';
            mockFetchOk();

            const changePromise = manager.handleSidecarStorageModeChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));

            const hint = modal.querySelector('[data-role="default-location-hint"]');
            expect(hint.style.display).toBe('block');
            expect(hint.textContent).toContain('default location');

            modal.querySelector('[data-action="cancel-sidecar-migration"]').click();
            await changePromise;
        });

        it('hides the default-location hint for custom roots and other directions', async () => {
            const manager = createManager();
            const { select } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'alongside',
                sidecar_storage_root: '/data/sidecars',
                sidecar_storage_root_is_default: false,
            };
            manager._loadedSidecarStorageMode = 'alongside';
            select.value = 'centralized';
            mockFetchOk();

            const changePromise = manager.handleSidecarStorageModeChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));

            const hint = modal.querySelector('[data-role="default-location-hint"]');
            expect(hint.style.display).toBe('none');

            modal.querySelector('[data-action="cancel-sidecar-migration"]').click();
            await changePromise;
        });

        it('shows a deferred notice and skips migration when the user cancels', async () => {            const manager = createManager();
            const { select } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'centralized' };
            manager._loadedSidecarStorageMode = 'centralized';
            select.value = 'alongside';
            mockFetchOk();

            const changePromise = manager.handleSidecarStorageModeChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="cancel-sidecar-migration"]').click();
            await changePromise;

            const migrateCalls = global.fetch.mock.calls.filter(([url]) => url === '/api/lm/sidecars/migrate');
            expect(migrateCalls).toHaveLength(0);
            expect(showToast).toHaveBeenCalledWith('settings.sidecarStorage.migrationDeferred', {}, 'info');
        });
    });

    describe('confirmAndMigrateSidecars', () => {
        it('derives the migration direction from the saved mode', async () => {
            const manager = createManager();
            appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'centralized' };
            mockFetchOk();

            const confirmPromise = manager.confirmAndMigrateSidecars();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await confirmPromise;

            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/migrate', expect.objectContaining({
                body: JSON.stringify({ direction: 'to_centralized', force: true }),
            }));
        });

        // The button opens this dialog directly, so the message must not
        // describe a mode change or point back at the button itself.
        it('uses the button-triggered wording without the mode-change backreference', async () => {
            const manager = createManager();
            appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'centralized' };
            mockFetchOk();

            const confirmPromise = manager.confirmAndMigrateSidecars();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));

            const message = modal.querySelector('[data-role="message"]').textContent;
            expect(message).toBe('Move existing .metadata.json sidecars and preview images into the centralized storage directory now?');
            expect(message).not.toContain('later');
            expect(message).not.toContain('Migrate Sidecars Now');

            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await confirmPromise;
        });

        it('uses the button-triggered wording when migrating back alongside', async () => {
            const manager = createManager();
            appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = { sidecar_storage_mode: 'alongside' };
            mockFetchOk();

            const confirmPromise = manager.confirmAndMigrateSidecars();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));

            const message = modal.querySelector('[data-role="message"]').textContent;
            expect(message).toBe('Move existing .metadata.json sidecars and preview images back next to their model files now?');

            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await confirmPromise;
        });
    });

    describe('migrateSidecars', () => {
        it('surfaces backend failures as an error toast', async () => {
            const manager = createManager();
            const { migrateBtn } = appendSidecarControls();
            mockFetchOk({ success: false, error: 'disk full' });

            await manager.migrateSidecars('to_alongside');

            expect(showToast).toHaveBeenCalledWith(
                'settings.sidecarStorage.migrateFailed',
                { message: 'disk full' },
                'error'
            );
            expect(resetAndReload).not.toHaveBeenCalled();
            expect(migrateBtn.disabled).toBe(false);
        });
    });

    describe('handleSidecarStoragePathChange', () => {
        it('offers root relocation when the path changes in centralized mode', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'centralized',
                sidecar_storage_path: '/old/root',
                sidecar_storage_root: '/old/root',
            };
            manager._loadedSidecarStoragePath = '/old/root';
            manager._loadedSidecarStorageRoot = '/old/root';
            pathInput.value = '/new/root';
            mockFetchWithResolvedRoot('/new/root');

            const changePromise = manager.handleSidecarStoragePathChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await changePromise;

            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/migrate', expect.objectContaining({
                body: JSON.stringify({ direction: 'relocate_root', force: true, old_root: '/old/root' }),
            }));
            expect(manager._loadedSidecarStoragePath).toBe('/new/root');
            expect(manager._loadedSidecarStorageRoot).toBe('/new/root');
        });

        it('offers relocation from the default root when no custom path was set (#1142)', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            const modal = appendMigrationModal();
            // No custom path: the raw setting is empty, but sidecars live in
            // the resolved default root.
            state.global.settings = {
                sidecar_storage_mode: 'centralized',
                sidecar_storage_path: '',
                sidecar_storage_root: '/settings/sidecars',
            };
            manager._loadedSidecarStoragePath = '';
            manager._loadedSidecarStorageRoot = '/settings/sidecars';
            pathInput.value = '/custom/dir';
            mockFetchWithResolvedRoot('/custom/dir');

            const changePromise = manager.handleSidecarStoragePathChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await changePromise;

            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/migrate', expect.objectContaining({
                body: JSON.stringify({ direction: 'relocate_root', force: true, old_root: '/settings/sidecars' }),
            }));
        });

        it('offers relocation back to the default root when the path is cleared', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'centralized',
                sidecar_storage_path: '/custom/dir',
                sidecar_storage_root: '/custom/dir',
            };
            manager._loadedSidecarStoragePath = '/custom/dir';
            manager._loadedSidecarStorageRoot = '/custom/dir';
            pathInput.value = '';
            mockFetchWithResolvedRoot('/settings/sidecars');

            const changePromise = manager.handleSidecarStoragePathChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="confirm-sidecar-migration"]').click();
            await changePromise;

            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/migrate', expect.objectContaining({
                body: JSON.stringify({ direction: 'relocate_root', force: true, old_root: '/custom/dir' }),
            }));
        });

        it('does not prompt when the resolved root is unchanged', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'centralized',
                sidecar_storage_path: '/old/root',
                sidecar_storage_root: '/old/root',
            };
            manager._loadedSidecarStoragePath = '/old/root';
            manager._loadedSidecarStorageRoot = '/old/root';
            // Whitespace-only edit: saveInputSetting no-ops, root stays put.
            pathInput.value = '/old/root';
            mockFetchWithResolvedRoot('/old/root');

            await manager.handleSidecarStoragePathChange();

            const migrateCalls = global.fetch.mock.calls.filter(([url]) => url === '/api/lm/sidecars/migrate');
            expect(migrateCalls).toHaveLength(0);
        });

        it('does not prompt when the path changes in alongside mode', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'alongside',
                sidecar_storage_path: '/old/root',
                sidecar_storage_root: '/old/root',
            };
            manager._loadedSidecarStoragePath = '/old/root';
            manager._loadedSidecarStorageRoot = '/old/root';
            pathInput.value = '/new/root';
            mockFetchWithResolvedRoot('/new/root');

            await manager.handleSidecarStoragePathChange();

            const migrateCalls = global.fetch.mock.calls.filter(([url]) => url === '/api/lm/sidecars/migrate');
            expect(migrateCalls).toHaveLength(0);
        });

        it('shows a deferred notice when relocation is cancelled', async () => {
            const manager = createManager();
            const { pathInput } = appendSidecarControls();
            const modal = appendMigrationModal();
            state.global.settings = {
                sidecar_storage_mode: 'centralized',
                sidecar_storage_path: '/old/root',
                sidecar_storage_root: '/old/root',
            };
            manager._loadedSidecarStoragePath = '/old/root';
            manager._loadedSidecarStorageRoot = '/old/root';
            pathInput.value = '/new/root';
            mockFetchWithResolvedRoot('/new/root');

            const changePromise = manager.handleSidecarStoragePathChange();
            await vi.waitFor(() => expect(modal.classList.contains('show')).toBe(true));
            modal.querySelector('[data-action="cancel-sidecar-migration"]').click();
            await changePromise;

            const migrateCalls = global.fetch.mock.calls.filter(([url]) => url === '/api/lm/sidecars/migrate');
            expect(migrateCalls).toHaveLength(0);
            expect(showToast).toHaveBeenCalledWith('settings.sidecarStorage.migrationDeferred', {}, 'info');
        });
    });

    describe('renderSidecarStorageInfo', () => {
        const appendStorageInfoElements = () => {
            const resolved = document.createElement('code');
            resolved.id = 'sidecarStorageResolvedPath';
            const warning = document.createElement('div');
            warning.id = 'sidecarStorageRepoWarning';
            warning.style.display = 'none';
            document.body.append(resolved, warning);
            return { resolved, warning };
        };

        it('shows the resolved root and the repo warning when inside the install folder', () => {
            const manager = createManager();
            const { resolved, warning } = appendStorageInfoElements();
            state.global.settings = {
                sidecar_storage_root: '/repo/ComfyUI-Lora-Manager/sidecars',
                sidecar_storage_root_in_repo: true,
            };

            manager.renderSidecarStorageInfo();

            expect(resolved.textContent).toBe('/repo/ComfyUI-Lora-Manager/sidecars');
            expect(warning.style.display).toBe('block');
        });

        it('hides the repo warning when the root lives outside the install folder', () => {
            const manager = createManager();
            const { resolved, warning } = appendStorageInfoElements();
            state.global.settings = {
                sidecar_storage_root: '/data/sidecars',
                sidecar_storage_root_in_repo: false,
            };

            manager.renderSidecarStorageInfo();

            expect(resolved.textContent).toBe('/data/sidecars');
            expect(warning.style.display).toBe('none');
        });
    });

    describe('openSidecarStorageLocation', () => {
        it('posts to the open-location endpoint', async () => {
            const manager = createManager();
            mockFetchOk({ success: true });

            await manager.openSidecarStorageLocation();

            expect(global.fetch).toHaveBeenCalledWith('/api/lm/sidecars/open-location', { method: 'POST' });
            expect(showToast).toHaveBeenCalledWith('settings.sidecarStorage.openLocationSuccess', {}, 'success');
        });

        it('copies the path to the clipboard in clipboard mode', async () => {
            const manager = createManager();
            mockFetchOk({ success: true, mode: 'clipboard', path: '/data/sidecars' });
            const writeText = vi.fn().mockResolvedValue();
            Object.defineProperty(navigator, 'clipboard', {
                value: { writeText },
                configurable: true,
            });

            await manager.openSidecarStorageLocation();

            expect(writeText).toHaveBeenCalledWith('/data/sidecars');
            expect(showToast).toHaveBeenCalledWith(
                'settings.sidecarStorage.openLocationCopied',
                { path: '/data/sidecars' },
                'success'
            );
        });
    });

    describe('showSidecarMigrationResult', () => {
        const baseResult = {
            success: true,
            direction: 'to_centralized',
            moved: 12,
            models_moved: 5,
            models_total: 6,
            skipped: 1,
            conflicts: 2,
            errors: [],
            error_count: 0,
            sidecar_root: '/data/sidecars',
        };

        it('renders stat cards and location, reloads only when closed', async () => {
            const manager = createManager();
            mockFetchOk({ success: true });

            manager.showSidecarMigrationResult(baseResult);

            const modal = document.getElementById('sidecarMigrationSummaryModal');
            expect(modal).not.toBeNull();
            const statValues = [...modal.querySelectorAll('.stat-card-value')].map((el) => el.textContent);
            expect(statValues).toEqual(['12', '5', '1', '2']);
            expect(modal.querySelector('.sidecar-migration-location').textContent).toContain('/data/sidecars');
            expect(modal.querySelector('[data-action="open-sidecar-location"]')).not.toBeNull();
            expect(modal.querySelector('.refresh-success-message')).not.toBeNull();
            expect(resetAndReload).not.toHaveBeenCalled();

            // "Open Folder" keeps the summary modal open.
            modal.querySelector('[data-action="open-sidecar-location"]').click();
            await vi.waitFor(() => expect(global.fetch).toHaveBeenCalledWith(
                '/api/lm/sidecars/open-location',
                { method: 'POST' }
            ));
            expect(document.getElementById('sidecarMigrationSummaryModal')).not.toBeNull();

            modal.querySelector('[data-action="close-modal"]').click();
            expect(document.getElementById('sidecarMigrationSummaryModal')).toBeNull();
            expect(resetAndReload).toHaveBeenCalledWith(true);
        });

        it('hides the location line and open button when migrating back alongside', () => {
            const manager = createManager();

            manager.showSidecarMigrationResult({ ...baseResult, direction: 'to_alongside' });

            const modal = document.getElementById('sidecarMigrationSummaryModal');
            expect(modal.querySelector('.sidecar-migration-location')).toBeNull();
            expect(modal.querySelector('[data-action="open-sidecar-location"]')).toBeNull();
        });

        it('renders the failure table when errors occurred', () => {
            const manager = createManager();

            manager.showSidecarMigrationResult({
                ...baseResult,
                errors: [{ model: 'broken.safetensors', error: 'permission denied' }],
                error_count: 1,
            });

            const modal = document.getElementById('sidecarMigrationSummaryModal');
            expect(modal.querySelector('.summary-header').classList.contains('warning')).toBe(true);
            expect(modal.querySelector('.refresh-success-message')).toBeNull();
            const rows = modal.querySelectorAll('.failure-table tbody tr');
            expect(rows).toHaveLength(1);
            expect(rows[0].textContent).toContain('broken.safetensors');
            expect(rows[0].textContent).toContain('permission denied');
            const statValues = [...modal.querySelectorAll('.stat-card-value')].map((el) => el.textContent);
            expect(statValues).toContain('1');
        });

        it('closes on ESC without leaking the keydown to the settings modal', () => {
            const manager = createManager();
            const underlyingHandler = vi.fn();
            document.addEventListener('keydown', underlyingHandler);

            manager.showSidecarMigrationResult(baseResult);

            document.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape', bubbles: true }));

            expect(document.getElementById('sidecarMigrationSummaryModal')).toBeNull();
            expect(resetAndReload).toHaveBeenCalledWith(true);
            expect(underlyingHandler).not.toHaveBeenCalled();
            document.removeEventListener('keydown', underlyingHandler);
        });
    });
});