liderra/portal
1
0
Fork 0
Code Issues Pull Requests Actions 63 Packages Projects Releases Wiki Activity

fix(admin): I3 — убрать mock fallback в System/Tenants

Browse Source
This commit is contained in:
Дмитрий
2026-05-17 06:39:06 +03:00
parent aed9475c26
commit 07d2e9705e
5 changed files with 109 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/resources/js/views/admin/AdminSystemView.vue
+6 -11
View File
@@ -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<string | null>(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<AdminSystemSetting[]>([...ADMIN_SYSTEM_SETTINGS]);
const settingsState = reactive<AdminSystemSetting[]>([]);
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;
}
app/resources/js/views/admin/AdminTenantsView.vue
+4 -4
View File
@@ -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<AdminTenant[]>(MOCK_TENANTS.map((t) => ({ ...t })));
const stats = reactive({ ...MOCK_STATS });
const tenantsState = reactive<AdminTenant[]>([]);
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<AdminTenant[]>(() => {
class="mt-3"
data-testid="fetch-error-alert"
>
Backend недоступен — показаны mock-данные.
Не удалось загрузить тенантов. Попробуйте обновить.
</v-alert>
<TenantsFilters
app/tests/Frontend/AdminSystemView.spec.ts
+6 -4
View File
@@ -26,12 +26,14 @@ const mountView = async () => {
});
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 () => {
app/tests/Frontend/AdminTenantsView.spec.ts
+91 -34
View File
@@ -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<typeof import('../../resources/js/api/admin')>();
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: '<div />' } },
],
});
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: '<div />' } },
],
});
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);
});
});
app/tests/Frontend/AdminTenantsViewApi.spec.ts
+2 -2
View File
@@ -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);
});