diff --git a/app/resources/js/views/admin/AdminSystemView.vue b/app/resources/js/views/admin/AdminSystemView.vue index 6db15975..846545b8 100644 --- a/app/resources/js/views/admin/AdminSystemView.vue +++ b/app/resources/js/views/admin/AdminSystemView.vue @@ -5,10 +5,8 @@ * Глобальные настройки SaaS-уровня (system_settings по schema v8.7 §10): * лимиты квот, тарифные планы, фичефлаги, fallback supplier_id. * - * MVP — display + read-only edit-режим. Backend `/api/admin/system-settings` - * + edit-flow подключаются отдельным коммитом. + * Display + edit-режим. Данные с backend GET /api/admin/system-settings. */ -import { ADMIN_SYSTEM_SETTINGS } from '../../composables/mockAdmin'; import type { AdminSystemSetting } from '../../composables/mockAdmin'; import * as adminApi from '../../api/admin'; import type { SystemSetting as ApiSystemSetting } from '../../api/admin'; @@ -21,13 +19,10 @@ const loading = ref(false); const fetchError = ref(null); /** - * Settings-state. Инициируется mock-данными (fallback если backend недоступен), - * на mount — replace через `adminApi.listSystemSettings()`. - * - * Type-narrowing: AdminSystemSetting (mock) vs ApiSystemSetting различаются - * только origin (mock vs БД), shape совместим — оба `{key, value, type, ...}`. + * Settings-state. Наполняется на mount через `adminApi.listSystemSettings()`. + * До загрузки и при ошибке — пустой; ошибка показывается через fetchError-banner. */ -const settingsState = reactive([...ADMIN_SYSTEM_SETTINGS]); +const settingsState = reactive([]); async function loadSettings() { loading.value = true; @@ -37,8 +32,8 @@ async function loadSettings() { // Replace всё содержимое сохранив reactive-ref. settingsState.splice(0, settingsState.length, ...(fromApi as unknown as AdminSystemSetting[])); } catch (err) { - // На fail оставляем mock (не очищаем UI). Показываем error-banner. - fetchError.value = extractErrorMessage(err, 'Не удалось загрузить настройки с сервера. Показаны mock-данные.'); + // На fail — settingsState пустой, показываем error-banner. + fetchError.value = extractErrorMessage(err, 'Не удалось загрузить настройки с сервера. Попробуйте обновить.'); } finally { loading.value = false; } diff --git a/app/resources/js/views/admin/AdminTenantsView.vue b/app/resources/js/views/admin/AdminTenantsView.vue index 3820f3c7..978611d9 100644 --- a/app/resources/js/views/admin/AdminTenantsView.vue +++ b/app/resources/js/views/admin/AdminTenantsView.vue @@ -15,7 +15,7 @@ */ import { computed, onMounted, reactive, ref } from 'vue'; import { useRouter } from 'vue-router'; -import { MOCK_STATS, MOCK_TENANTS, type AdminTenant, type TenantStatus } from '../../composables/mockTenants'; +import { type AdminTenant, type TenantStatus } from '../../composables/mockTenants'; import { mapApiAdminTenant } from '../../composables/adminTenantsMapper'; import { usePolling } from '../../composables/usePolling'; import * as adminApi from '../../api/admin'; @@ -29,8 +29,8 @@ import TenantsTable from '../../components/admin/tenants/TenantsTable.vue'; const router = useRouter(); -const tenantsState = reactive(MOCK_TENANTS.map((t) => ({ ...t }))); -const stats = reactive({ ...MOCK_STATS }); +const tenantsState = reactive([]); +const stats = reactive({ total: 0, active: 0, trial: 0, overdue: 0, monthlyRevenueRub: 0 }); const loading = ref(false); const fetchError = ref(false); @@ -123,7 +123,7 @@ const filteredTenants = computed(() => { class="mt-3" data-testid="fetch-error-alert" > - Backend недоступен — показаны mock-данные. + Не удалось загрузить тенантов. Попробуйте обновить. { }); await router.push('/admin/system'); await router.isReady(); - return mount(AdminSystemView, { + const wrapper = mount(AdminSystemView, { global: { plugins: [createVuetify(), router], stubs: { SystemSettingEditDialog: true }, }, }); + await flushPromises(); + return wrapper; }; describe('AdminSystemView.vue', () => { @@ -120,15 +122,15 @@ describe('AdminSystemView.vue', () => { expect(adminApi.listSystemSettings).toHaveBeenCalledTimes(1); }); - it('при сетевой ошибке показывает warning-banner + сохраняет mock-данные', async () => { + it('при сетевой ошибке показывает warning-banner + settingsState пустой', async () => { vi.mocked(adminApi.listSystemSettings).mockRejectedValueOnce(new Error('Network down')); const wrapper = await mountView(); await flushPromises(); const banner = wrapper.find('[data-testid="fetch-error-alert"]'); expect(banner.exists()).toBe(true); - // Mock-настройки остались (fallback) + // Пустой при ошибке — без mock-fallback const rows = wrapper.findAll('[data-testid="setting-row"]'); - expect(rows.length).toBe(7); + expect(rows.length).toBe(0); }); it('onSettingUpdated обновляет value и updated_at в settingsState', async () => { diff --git a/app/tests/Frontend/AdminTenantsView.spec.ts b/app/tests/Frontend/AdminTenantsView.spec.ts index 0adb39b4..beb0420b 100644 --- a/app/tests/Frontend/AdminTenantsView.spec.ts +++ b/app/tests/Frontend/AdminTenantsView.spec.ts @@ -1,12 +1,33 @@ -import { describe, it, expect } from 'vitest'; -import { mount } from '@vue/test-utils'; +import { describe, it, expect, vi, beforeEach } from 'vitest'; +import { mount, flushPromises } from '@vue/test-utils'; import { createVuetify } from 'vuetify'; import { createRouter, createMemoryHistory } from 'vue-router'; import AdminTenantsView from '../../resources/js/views/admin/AdminTenantsView.vue'; -import { MOCK_STATS, MOCK_TENANTS } from '../../resources/js/composables/mockTenants'; +import { MOCK_STATS, MOCK_TENANTS, type AdminTenant } from '../../resources/js/composables/mockTenants'; + +// Мокаем api/admin: listAdminTenants возвращает пустой ответ — +// smoke-тесты затем seed'ят tenantsState/stats напрямую через vm (defineExpose). +vi.mock('../../resources/js/api/admin', async (importOriginal) => { + const orig = await importOriginal(); + return { + ...orig, + listAdminTenants: vi.fn().mockResolvedValue({ + tenants: [], + total: 0, + limit: 100, + offset: 0, + stats: { total: 0, active: 0, trial: 0, overdue: 0 }, + }), + }; +}); + +beforeEach(() => { + vi.clearAllMocks(); +}); describe('AdminTenantsView.vue', () => { - const factory = () => { + /** Монтирует view, ждёт mount-цикл, затем seed'ит state фикстурами. */ + const factory = async () => { // useRouter() в AdminTenantsView требует router-context в тестах. const router = createRouter({ history: createMemoryHistory(), @@ -15,22 +36,34 @@ describe('AdminTenantsView.vue', () => { { path: '/admin/tenants/:code', name: 'admin-tenant-detail', component: { template: '
' } }, ], }); - return mount(AdminTenantsView, { + await router.push('/admin/tenants'); + await router.isReady(); + const wrapper = mount(AdminTenantsView, { global: { plugins: [createVuetify(), router], // ImpersonationDialog stubим — внутри использует api/admin axios. stubs: { ImpersonationDialog: true }, }, }); + await flushPromises(); + // Seed state напрямую через defineExpose — имитирует успешную загрузку с теми же фикстурами. + const vm = wrapper.vm as unknown as { + tenantsState: AdminTenant[]; + stats: typeof MOCK_STATS; + }; + vm.tenantsState.splice(0, vm.tenantsState.length, ...MOCK_TENANTS.map((t) => ({ ...t }))); + Object.assign(vm.stats, MOCK_STATS); + await wrapper.vm.$nextTick(); + return wrapper; }; - it('монтируется и содержит заголовок «Тенанты»', () => { - const wrapper = factory(); + it('монтируется и содержит заголовок «Тенанты»', async () => { + const wrapper = await factory(); expect(wrapper.find('h1').text()).toBe('Тенанты'); }); - it('показывает 5 stats: всего/активны/trial/просрочка/выручка', () => { - const wrapper = factory(); + it('показывает 5 stats: всего/активны/trial/просрочка/выручка', async () => { + const wrapper = await factory(); const text = wrapper.text(); expect(text).toContain(`${MOCK_STATS.total}`); // 142 expect(text).toContain('всего'); @@ -45,22 +78,22 @@ describe('AdminTenantsView.vue', () => { expect(text).toMatch(/1\s+248\s+600\s*₽/); }); - it('таблица содержит 7 колонок (Тенант/Статус/Тариф/Баланс/Желаем×факт/MRR/Активность)', () => { - const wrapper = factory(); + it('таблица содержит 7 колонок (Тенант/Статус/Тариф/Баланс/Желаем×факт/MRR/Активность)', async () => { + const wrapper = await factory(); const headers = wrapper.findAll('thead th').map((h) => h.text()); ['Тенант', 'Статус', 'Тариф', 'Баланс', 'Желаем×факт', 'MRR', 'Активность'].forEach((label) => { expect(headers.some((h) => h.includes(label))).toBe(true); }); }); - it('рендерит все 7 mock-tenants', () => { - const wrapper = factory(); + it('рендерит все 7 mock-tenants', async () => { + const wrapper = await factory(); const rows = wrapper.findAll('tbody tr'); expect(rows.length).toBe(MOCK_TENANTS.length); }); - it('первая строка — Окна Москва ООО + ИНН + Активен + Команда', () => { - const wrapper = factory(); + it('первая строка — Окна Москва ООО + ИНН + Активен + Команда', async () => { + const wrapper = await factory(); const text = wrapper.text(); expect(text).toContain('Окна Москва ООО'); expect(text).toContain('ИНН 7724444444'); @@ -68,37 +101,37 @@ describe('AdminTenantsView.vue', () => { expect(text).toContain('Команда'); }); - it('overdue-тенант (Двери Премиум) показывает «Просрочка 3 дня» + отрицательный баланс', () => { - const wrapper = factory(); + it('overdue-тенант (Двери Премиум) показывает «Просрочка 3 дня» + отрицательный баланс', async () => { + const wrapper = await factory(); const text = wrapper.text(); expect(text).toContain('Двери Премиум'); expect(text).toContain('Просрочка 3 дня'); expect(text).toMatch(/−1\s+200/); // -1200 без 0 ₽ }); - it('trial-тенант (Ремонт под ключ) показывает «Trial · 4 дня» + MRR=—', () => { - const wrapper = factory(); + it('trial-тенант (Ремонт под ключ) показывает «Trial · 4 дня» + MRR=—', async () => { + const wrapper = await factory(); const text = wrapper.text(); expect(text).toContain('Ремонт под ключ'); expect(text).toContain('Trial · 4 дня'); }); - it('suspended-тенант (Оконные системы РФ) показывает «Приостановлен»', () => { - const wrapper = factory(); + it('suspended-тенант (Оконные системы РФ) показывает «Приостановлен»', async () => { + const wrapper = await factory(); const text = wrapper.text(); expect(text).toContain('Оконные системы РФ'); expect(text).toContain('Приостановлен'); }); - it('содержит search-input с placeholder «ИНН, юр. лицо, email админа…»', () => { - const wrapper = factory(); + it('содержит search-input с placeholder «ИНН, юр. лицо, email админа…»', async () => { + const wrapper = await factory(); const input = wrapper.find('input[type="text"]'); expect(input.exists()).toBe(true); expect(input.attributes('placeholder')).toContain('ИНН'); }); it('фильтр по search оставляет только matching-tenants', async () => { - const wrapper = factory(); + const wrapper = await factory(); const input = wrapper.find('input[type="text"]'); await input.setValue('Натяжные'); await wrapper.vm.$nextTick(); @@ -107,15 +140,15 @@ describe('AdminTenantsView.vue', () => { expect(rows[0].text()).toContain('Натяжные потолки СПб'); }); - it('содержит Экспорт-кнопку и фильтры Статус/Тариф', () => { - const wrapper = factory(); + it('содержит Экспорт-кнопку и фильтры Статус/Тариф', async () => { + const wrapper = await factory(); expect(wrapper.text()).toContain('Экспорт'); expect(wrapper.find('[data-testid="filter-statuses"]').exists()).toBe(true); expect(wrapper.find('[data-testid="filter-tariffs"]').exists()).toBe(true); }); it('фильтр по статусу «overdue» оставляет только просроченных', async () => { - const wrapper = factory(); + const wrapper = await factory(); const vm = wrapper.vm as unknown as { filterStatuses: string[] }; vm.filterStatuses = ['overdue']; await wrapper.vm.$nextTick(); @@ -125,7 +158,7 @@ describe('AdminTenantsView.vue', () => { }); it('фильтр по тарифу «Pro» оставляет 1 row', async () => { - const wrapper = factory(); + const wrapper = await factory(); const vm = wrapper.vm as unknown as { filterTariffs: string[] }; vm.filterTariffs = ['Pro']; await wrapper.vm.$nextTick(); @@ -135,7 +168,7 @@ describe('AdminTenantsView.vue', () => { }); it('clearFilters сбрасывает оба фильтра + кнопка «Сбросить» появляется только когда фильтры активны', async () => { - const wrapper = factory(); + const wrapper = await factory(); const vm = wrapper.vm as unknown as { filterStatuses: string[]; filterTariffs: string[]; @@ -152,8 +185,8 @@ describe('AdminTenantsView.vue', () => { expect(vm.filterTariffs).toEqual([]); }); - it('каждая строка имеет impersonate-кнопку (mdi-account-switch) с уникальным data-testid', () => { - const wrapper = factory(); + it('каждая строка имеет impersonate-кнопку (mdi-account-switch) с уникальным data-testid', async () => { + const wrapper = await factory(); // Все 7 mock-tenants должны иметь кнопку MOCK_TENANTS.forEach((t) => { const btn = wrapper.find(`[data-testid="impersonate-btn-${t.id}"]`); @@ -161,8 +194,8 @@ describe('AdminTenantsView.vue', () => { }); }); - it('impersonate-кнопка disabled для suspended-тенанта (Оконные системы РФ id=105)', () => { - const wrapper = factory(); + it('impersonate-кнопка disabled для suspended-тенанта (Оконные системы РФ id=105)', async () => { + const wrapper = await factory(); const suspendedBtn = wrapper.find('[data-testid="impersonate-btn-105"]'); expect(suspendedBtn.exists()).toBe(true); // v-btn disabled-state — атрибут disabled на DOM-элементе @@ -170,7 +203,7 @@ describe('AdminTenantsView.vue', () => { }); it('click на impersonate-кнопке открывает ImpersonationDialog с правильным tenant', async () => { - const wrapper = factory(); + const wrapper = await factory(); // До click — диалог закрыт (modelValue=false) const dialogStub = wrapper.findComponent({ name: 'ImpersonationDialog' }); expect(dialogStub.exists()).toBe(true); @@ -186,4 +219,28 @@ describe('AdminTenantsView.vue', () => { expect(dialogStub.props('tenant')).toMatchObject({ id: 42, name: 'Окна Москва ООО' }); expect(dialogStub.props('requestedBy')).toBe(1); }); + + it('API reject → tenantsState пустой + fetch-error-alert виден', async () => { + const adminApi = await import('../../resources/js/api/admin'); + vi.mocked(adminApi.listAdminTenants).mockRejectedValueOnce(new Error('Network error')); + + const router = createRouter({ + history: createMemoryHistory(), + routes: [ + { path: '/admin/tenants', name: 'admin-tenants', component: AdminTenantsView }, + { path: '/admin/tenants/:code', name: 'admin-tenant-detail', component: { template: '
' } }, + ], + }); + await router.push('/admin/tenants'); + await router.isReady(); + const wrapper = mount(AdminTenantsView, { + global: { plugins: [createVuetify(), router], stubs: { ImpersonationDialog: true } }, + }); + await flushPromises(); + + const vm = wrapper.vm as unknown as { fetchError: boolean; tenantsState: unknown[] }; + expect(vm.fetchError).toBe(true); + expect(vm.tenantsState.length).toBe(0); + expect(wrapper.find('[data-testid="fetch-error-alert"]').exists()).toBe(true); + }); }); diff --git a/app/tests/Frontend/AdminTenantsViewApi.spec.ts b/app/tests/Frontend/AdminTenantsViewApi.spec.ts index 5156f95e..e7b5998b 100644 --- a/app/tests/Frontend/AdminTenantsViewApi.spec.ts +++ b/app/tests/Frontend/AdminTenantsViewApi.spec.ts @@ -103,7 +103,7 @@ describe('AdminTenantsView ↔ GET /api/admin/tenants integration', () => { expect(vm.stats.trial).toBe(1); }); - it('reject → fetchError=true + alert виден + MOCK_TENANTS остаётся', async () => { + it('reject → fetchError=true + alert виден + tenantsState пустой', async () => { vi.mocked(adminApi.listAdminTenants).mockRejectedValueOnce(new Error('500')); const wrapper = await mountView(); @@ -111,7 +111,7 @@ describe('AdminTenantsView ↔ GET /api/admin/tenants integration', () => { const vm = wrapper.vm as unknown as { fetchError: boolean; tenantsState: unknown[] }; expect(vm.fetchError).toBe(true); - expect(vm.tenantsState.length).toBeGreaterThan(0); // mock-fallback + expect(vm.tenantsState.length).toBe(0); // пустой при ошибке, не mock-fallback expect(wrapper.find('[data-testid="fetch-error-alert"]').exists()).toBe(true); });