From 365ab2173cb8c9de5bf0e6e2a8472bc3104232fc Mon Sep 17 00:00:00 2001 From: Denozordec Date: Wed, 30 Sep 2026 12:05:28 +0700 Subject: [PATCH] =?UTF-8?q?fix(ui):=20=D0=B8=D0=B7=D0=BE=D0=BB=D0=B8=D1=80?= =?UTF-8?q?=D0=BE=D0=B2=D0=B0=D1=82=D1=8C=20=D0=B8=D1=81=D1=82=D0=BE=D1=87?= =?UTF-8?q?=D0=BD=D0=B8=D0=BA=D0=B8=20=D0=B4=D0=B0=D0=BD=D0=BD=D1=8B=D1=85?= =?UTF-8?q?=20Filters?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/(main)/filters/page.tsx | 32 ++++++++-- docs/ux-ui-plan.md | 2 +- docs/ux-ui-progress.md | 10 ++- docs/ux-ui-stage-9.md | 4 ++ playwright.config.ts | 2 +- playwright.filters-context.config.ts | 12 ++++ tests/e2e/filters-context.spec.ts | 93 ++++++++++++++++++++++++++++ 7 files changed, 146 insertions(+), 9 deletions(-) create mode 100644 playwright.filters-context.config.ts create mode 100644 tests/e2e/filters-context.spec.ts diff --git a/app/(main)/filters/page.tsx b/app/(main)/filters/page.tsx index 8bcd7370..38ae01bd 100644 --- a/app/(main)/filters/page.tsx +++ b/app/(main)/filters/page.tsx @@ -1263,8 +1263,13 @@ function makeApiFetch(backendUrl: string) { } } -export default function FiltersPage() { +function FiltersPage() { const { mode, backendUrl } = useDataSource() + const mounted = useRef(true) + useEffect(() => { + mounted.current = true + return () => { mounted.current = false } + }, []) const evo = useEvoBGP() /** Как на `/servers`: живые данные при режиме «Живые», без ожидания отдельного health-check */ const isLive = mode === "live" @@ -1309,25 +1314,29 @@ export default function FiltersPage() { const [loadAttempt, setLoadAttempt] = useState(0) const ensureGreTunnels = useCallback(async (serverId: string) => { - if (!isLive) return + if (!isLive || !mounted.current) return try { const d = await apiFetch<{ tunnels: GreTunnel[] }>( `/api/filters/gre-tunnels?serverId=${encodeURIComponent(serverId)}`, ) + if (!mounted.current) return setGreByServer(prev => ({ ...prev, [serverId]: d.tunnels })) } catch { + if (!mounted.current) return setGreByServer(prev => ({ ...prev, [serverId]: [] })) } }, [isLive, apiFetch]) const ensureRecursiveRoutes = useCallback(async (serverId: string) => { - if (!isLive) return + if (!isLive || !mounted.current) return try { const d = await apiFetch<{ routes: RecursiveRouteLite[] }>( `/api/recursive-routes?serverId=${encodeURIComponent(serverId)}`, ) + if (!mounted.current) return setRecRoutesByServer(prev => ({ ...prev, [serverId]: d.routes })) } catch { + if (!mounted.current) return setRecRoutesByServer(prev => ({ ...prev, [serverId]: [] })) } }, [isLive, apiFetch]) @@ -1423,7 +1432,7 @@ export default function FiltersPage() { }, [isLive, apiFetch, loadAttempt]) const loadLiveRules = useCallback(async (serverId: string) => { - if (!isLive || !serverId) return false + if (!isLive || !serverId || !mounted.current) return false const epoch = ++readEpoch.current setReadyServerId(null) setRulesReading(true) @@ -1521,6 +1530,7 @@ export default function FiltersPage() { nextRules: FilterRule[], source: "apply" | "copy" = "apply", ) => { + if (!mounted.current) return { ok: false, error: "Источник данных изменён" } if (applyLock.current || applyBusy || historyRestoring) return { ok: false, error: "Дождитесь завершения текущей операции" } if (isLive && (liveStale || liveLoadState !== "idle" || (serverId === selectedServerId && readyServerId !== serverId))) { const error = "Роутер недоступен — изменения заблокированы" @@ -1540,6 +1550,7 @@ export default function FiltersPage() { method: "PUT", body: JSON.stringify({ serverId, rules: nextRules, source }), }) + if (!mounted.current) return { ok: false, error: "Источник данных изменён" } const parsed = filtersMutationSchema.safeParse(payload) if (!parsed.success) throw new Error("Сервер не подтвердил применение правил") const res = parsed.data @@ -1547,6 +1558,7 @@ export default function FiltersPage() { toast.success("Правила применены на роутер") return { ok: true } } catch (err) { + if (!mounted.current) return { ok: false, error: "Источник данных изменён" } const error = err instanceof Error ? err.message : "Не удалось применить правила на роутер" toast.error("Не удалось применить правила на роутер", { description: error }) return { ok: false, error } @@ -1557,7 +1569,7 @@ export default function FiltersPage() { }, [isLive, apiFetch, applyBusy, historyRestoring, liveStale, liveLoadState, readyServerId, selectedServerId]) const refreshFromRouter = useCallback(async () => { - if (!isLive || !selectedServerId || applyLock.current || applyBusy || historyRestoring) return + if (!isLive || !selectedServerId || !mounted.current || applyLock.current || applyBusy || historyRestoring) return applyLock.current = true setApplyBusy(true) try { @@ -1575,7 +1587,7 @@ export default function FiltersPage() { useEffect(() => () => { historyEpoch.current += 1; readEpoch.current += 1 }, [selectedServerId, backendUrl, isLive]) const loadRevisions = useCallback(async () => { - if (!isLive || !selectedServerId) return + if (!isLive || !selectedServerId || !mounted.current) return const epoch = ++historyEpoch.current setHistoryLoading(true) setHistoryError(null) @@ -1606,6 +1618,7 @@ export default function FiltersPage() { `/api/filters/revisions/${encodeURIComponent(id)}/restore`, { method: "POST", body: JSON.stringify({ serverId: selectedServerId }) }, ) + if (!mounted.current) return const parsed = filtersMutationSchema.safeParse(payload) if (!parsed.success) throw new Error("Сервер не подтвердил восстановление") const res = parsed.data @@ -1614,9 +1627,11 @@ export default function FiltersPage() { } else { if (!await loadLiveRules(selectedServerId)) throw new Error("Версия применена, но актуальные правила не прочитаны — обновите данные") } + if (!mounted.current) return setLiveStale(false) setReadyServerId(selectedServerId) await loadRevisions() + if (!mounted.current) return toast.success("Версия применена на роутер") } finally { applyLock.current = false @@ -2048,3 +2063,8 @@ export default function FiltersPage() { ) } + +export default function FiltersSource() { + const { mode, backendUrl } = useDataSource() + return +} diff --git a/docs/ux-ui-plan.md b/docs/ux-ui-plan.md index c916115f..1433c3ba 100644 --- a/docs/ux-ui-plan.md +++ b/docs/ux-ui-plan.md @@ -88,7 +88,7 @@ flowchart LR Ближайший оставшийся объём: -1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. В 9.26 подтверждены включение/отключение address-list с общим замком переключений. В 9.27 удаление правил/address-list перенесено в диалог с подтверждением API и защитой повторов. В 9.28 создание/редактирование правил и address-list требуют подтверждения API, сохраняют ввод/ошибку и блокируют повторы/закрытие. В 9.29 перемещение Firewall требует ok=true, откатывает порядок при ошибке и использует общий замок. В 9.30 восстановление истории подключено к общему замку, ручное обновление и открытие истории заблокированы во время записи. В 9.31 добавлены схемы ответа Firewall/серверов, сохранение последних данных при ошибке и поколение чтения с отменой применения при начале записи. В 9.32 backend сообщает частичные сбои по устройству/семейству/таблице, UI сохраняет недоступные срезы и блокирует изменения неполного среза. В 9.33 состояние изолировано по backend/режиму; поздние операции старого экрана не запускают чтение и не показывают уведомления в новом контексте. В 9.34 показано время получения срезов Firewall: успешные обновляются, недоступные сохраняют прежнее время или остаются неизвестными. В 9.35 сохранение Filters ожидает ok=true, сохраняет ввод при ошибке и блокирует повторную отправку/закрытие; live-список меняется после подтверждения. В 9.36 копирование Filters ждёт подтверждения, сохраняет цель/правила/режим при ошибке и блокирует повторы/закрытие. В 9.37 удаление Filters перенесено в диалог с сохранением ошибки, подтверждением API и защитой повторов. В 9.38 перемещение Filters подтверждается API; поиск использует полный порядок для приоритета и границ, действия таблицы блокируются на время операции. В 9.39 обновление/восстановление Filters используют общий замок до чтения правил/истории; ошибка чтения сохраняет подтверждение, чужой ruleset и stale отклоняются. В 9.40 добавлены схемы правил/ответов Filters, ошибка первоначальной загрузки с повтором без подмены моками и сохранение последних данных. В 9.41 запись выбранного устройства требует успешного чтения live=true, старые ответы отклоняются поколением запросов при переключении/применении. Далее — изоляция состояния backend/режима и старых мутаций, актуальное чтение целевого набора копирования, схемы/ошибки шлюзов и свежесть Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми. +1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. В 9.26 подтверждены включение/отключение address-list с общим замком переключений. В 9.27 удаление правил/address-list перенесено в диалог с подтверждением API и защитой повторов. В 9.28 создание/редактирование правил и address-list требуют подтверждения API, сохраняют ввод/ошибку и блокируют повторы/закрытие. В 9.29 перемещение Firewall требует ok=true, откатывает порядок при ошибке и использует общий замок. В 9.30 восстановление истории подключено к общему замку, ручное обновление и открытие истории заблокированы во время записи. В 9.31 добавлены схемы ответа Firewall/серверов, сохранение последних данных при ошибке и поколение чтения с отменой применения при начале записи. В 9.32 backend сообщает частичные сбои по устройству/семейству/таблице, UI сохраняет недоступные срезы и блокирует изменения неполного среза. В 9.33 состояние изолировано по backend/режиму; поздние операции старого экрана не запускают чтение и не показывают уведомления в новом контексте. В 9.34 показано время получения срезов Firewall: успешные обновляются, недоступные сохраняют прежнее время или остаются неизвестными. В 9.35 сохранение Filters ожидает ok=true, сохраняет ввод при ошибке и блокирует повторную отправку/закрытие; live-список меняется после подтверждения. В 9.36 копирование Filters ждёт подтверждения, сохраняет цель/правила/режим при ошибке и блокирует повторы/закрытие. В 9.37 удаление Filters перенесено в диалог с сохранением ошибки, подтверждением API и защитой повторов. В 9.38 перемещение Filters подтверждается API; поиск использует полный порядок для приоритета и границ, действия таблицы блокируются на время операции. В 9.39 обновление/восстановление Filters используют общий замок до чтения правил/истории; ошибка чтения сохраняет подтверждение, чужой ruleset и stale отклоняются. В 9.40 добавлены схемы правил/ответов Filters, ошибка первоначальной загрузки с повтором без подмены моками и сохранение последних данных. В 9.41 запись выбранного устройства требует успешного чтения live=true, старые ответы отклоняются поколением запросов при переключении/применении. В 9.42 состояние Filters изолировано по backend/режиму, старые операции не уведомляют новый контекст. Далее — актуальное чтение целевого набора копирования, схемы/ошибки шлюзов и свежесть Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми. 2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения. 3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации. diff --git a/docs/ux-ui-progress.md b/docs/ux-ui-progress.md index 581e6502..616e06c9 100644 --- a/docs/ux-ui-progress.md +++ b/docs/ux-ui-progress.md @@ -25,12 +25,20 @@ | 6 | Навигация и дашборд | Завершён 2026-09-20 в границах локальной приёмки; ограничения ниже | | 7 | Серверы и мониторинг: детали, формы, URL-контекст, права | Завершён в границах локальной приёмки 2026-09-23; внешние ограничения ниже | | 8 | Трафик и статистика | Завершён 2026-09-28 в границах локальной приёмки; ограничения в блоке 8.13 | -| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.41 — IPsec/GRE/VXLAN, Firewall и операции Filters | +| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.42 — IPsec/GRE/VXLAN, Firewall и операции Filters | | 10 | Маршрутизация, диагностика, карта и терминал | Не начат | | 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат | | 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат | | 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат | +## Блок 9.42 — изоляция источника Filters (2026-09-30) + +- FiltersSource пересоздаёт состояние по ключу backend/режима. mounted-guard отменяет уведомления и продолжение старых мутаций; восстановление старого источника не запускает чтение правил/истории после ответа. Старые чтения уже отклоняются readEpoch/cleanup; загрузчики GRE/recursive также проверяют размонтирование перед применением результата. +- Контекстные проверки выполняются отдельно через playwright.filters-context.config.ts (порт 3101, NEXT_PUBLIC_BACKEND_URL=local); обычная конфигурация исключает filters-context.spec.ts. Это позволяет менять URL через настройки и проверять CORS HTTP-моки без реального backend. +- Прошли 26 E2E в двух финальных прогонах: шесть контекстных (поздний успех/ошибка чтения старого backend, поздний успех/403 применения после смены backend либо live/mock) и все 20 обычных Filters. Первый контекстный прогон остановлен после обнаружения неверного точного селектора ссылки с динамическим счётчиком; исправлен только селектор. Остановлено дерево только собственного Playwright-процесса, идентифицированного по точной командной строке. TypeScript прошёл, тесты/конфигурация lint-чистые; исходные 2 ошибки/13 предупреждений страницы сохранены. Diff-check прошёл. Последний frontend build — 9.40, backend — 9.32. +- Следующий шаг — актуальное чтение цели копирования перед append/replace; не использовать начальный кэш для объединения. Затем схемы/ошибки шлюзов и свежесть. Реальные RouterOS/JWT/БД не проверены; этап 9 открыт. +- Квота: 76% пятичасовой/12% недельной на старте, 79%/12% после проверок. Пользователь разрешил работу до конца лимитов, прежний резерв 25% для продолжения снят. Локальный коммит без push; чужие изменения сохранены. + ## Блок 9.41 — актуальное чтение выбранного устройства Filters (2026-09-30) - Запись выбранного устройства доступна после успешного чтения его ruleset с live=true и без stale. readyServerId должен совпадать с выбранным ID, до ответа действия readonly; статус объясняет ожидание. Исходный GET без serverId отдаёт кэш и сам не разрешает запись. Восстановление с подтверждёнными rules либо успешным повторным чтением устанавливает готовность устройства. diff --git a/docs/ux-ui-stage-9.md b/docs/ux-ui-stage-9.md index ed007606..c79783d6 100644 --- a/docs/ux-ui-stage-9.md +++ b/docs/ux-ui-stage-9.md @@ -205,3 +205,7 @@ Backend сообщает failures по срезу, неполный snapshot о ## Блок 9.41 — гонки чтения выбранного устройства Filters (2026-09-30) Запись выбранного устройства требует успешного чтения live=true; кэш остаётся readonly до ответа. Поколение запросов отменяет старый успех/ошибку при переключении, применении и cleanup. 20 E2E и TypeScript прошли, шесть профильных сценариев повторены после уточнения live; новых lint замечаний нет. Далее — изоляция состояния backend/режима и старых мутаций, затем актуальное чтение цели копирования. GRE/recursive и свежесть ещё открыты. Остаток пятичасовой квоты 25%; новый блок не начат. Frontend build — 9.40, backend — 9.32. + +## Блок 9.42 — источник Filters (2026-09-30) + +Состояние пересоздаётся по backend/режиму, старые записи не уведомляют новый экран и не запускают чтение. 26 E2E в двух финальных прогонах и TypeScript прошли; новых lint замечаний нет. Контекстные тесты — playwright.filters-context.config.ts. Далее — актуальный целевой ruleset копирования, затем шлюзы/свежесть. Пользователь разрешил продолжение за резерв 25%. Frontend build — 9.40, backend — 9.32. diff --git a/playwright.config.ts b/playwright.config.ts index 647248d4..932d3860 100644 --- a/playwright.config.ts +++ b/playwright.config.ts @@ -2,7 +2,7 @@ import { defineConfig } from "@playwright/test" export default defineConfig({ testDir: "./tests/e2e", - testIgnore: "**/firewall-context.spec.ts", + testIgnore: ["**/firewall-context.spec.ts", "**/filters-context.spec.ts"], fullyParallel: false, workers: 1, retries: 0, diff --git a/playwright.filters-context.config.ts b/playwright.filters-context.config.ts new file mode 100644 index 00000000..4e9c18cd --- /dev/null +++ b/playwright.filters-context.config.ts @@ -0,0 +1,12 @@ +import { defineConfig } from "@playwright/test" +import base from "./playwright.config" + +const server = base.webServer +if (!server || Array.isArray(server)) throw new Error("Ожидается один локальный сервер") +export default defineConfig({ + ...base, + use: { ...base.use, baseURL: "http://127.0.0.1:3101" }, + testIgnore: [], + testMatch: "filters-context.spec.ts", + webServer: { ...server, command: "node node_modules/next/dist/bin/next dev --hostname 127.0.0.1 --port 3101", url: "http://127.0.0.1:3101", env: { ...server.env, NEXT_PUBLIC_BACKEND_URL: "local" } }, +}) diff --git a/tests/e2e/filters-context.spec.ts b/tests/e2e/filters-context.spec.ts new file mode 100644 index 00000000..0988d0ea --- /dev/null +++ b/tests/e2e/filters-context.spec.ts @@ -0,0 +1,93 @@ +import { test, expect, type Page } from "@playwright/test" +const headers = { "access-control-allow-origin": "*", "access-control-allow-methods": "GET,POST,PUT,PATCH,DELETE,OPTIONS", "access-control-allow-headers": "Content-Type,Authorization" } +const makeRule = (port: string) => ({ id: "rule-1", community: port === "8101" ? "65001:201" : "65001:101", gateway: "", gatewayTunnelId: "", action: "blackhole", description: port === "8101" ? "Новый источник" : "Старый источник" }) +async function setup(page: Page) { + let oldReads = 0 + await page.route("**/health", route => route.fulfill({ headers, json: { ok: true } })) + await page.route("**/api/**", route => { + const url = new URL(route.request().url()), path = url.pathname + if (path === "/api/filters/rules" && url.port === "8000" && route.request().method() === "GET") oldReads++ + return route.fulfill({ headers, json: path === "/api/filters/rules" ? { rulesets: [{ serverId: "1", rules: [makeRule(url.port)] }], greTunnels: [], live: true } + : path === "/api/servers" ? [{ id: 1, name: "Роутер", host: "192.0.2.1", type: "exit-node", enabled: true, country: "RU", site: "TEST", asn: "", status: "online", latency: 1 }] + : path === "/api/auth/config" ? { required: false } + : path === "/api/filters/gre-tunnels" ? { tunnels: [] } + : path === "/api/recursive-routes" ? { routes: [] } : {} }) + }) + await page.goto("/settings") + await page.getByRole("link", { name: /^Фильтры/ }).click() + await expect(page.locator("tr[data-row-id]")).toHaveCount(1) + await expect(page.getByRole("button", { name: "Новое правило", exact: true })).toBeEnabled() + return () => oldReads +} +async function changeContext(page: Page, change: string) { + await page.goBack() + await expect(page).toHaveURL(/settings/) + if (change === "backend") { + const input = page.getByPlaceholder("http://localhost:8000") + await input.fill("http://localhost:8101") + await input.press("Enter") + } else await page.getByRole("button", { name: "Моковые", exact: true }).click() + await page.getByRole("link", { name: /^Фильтры/ }).click() + if (change === "backend") await expect(page.locator("tr[data-row-id]")).toContainText("Новый источник") + else await expect(page.locator('tr[data-row-id="rule-1"]')).toHaveCount(0) +} +async function flush(page: Page) { + await page.evaluate(() => new Promise(resolve => requestAnimationFrame(() => requestAnimationFrame(() => resolve())))) +} +for (const fails of [false, true]) { + test(`Filters: старое чтение при смене backend ${fails}`, async ({ page }) => { + await setup(page) + let release: (() => void) | undefined + const gate = new Promise(resolve => { release = resolve }) + let started = false + await page.route("**/api/filters/rules?serverId=1", async route => { + if (new URL(route.request().url()).port !== "8000") return route.fallback() + started = true + await gate + return route.fulfill(fails ? { headers, status: 503, json: { error: "Ошибка старого источника" } } : { headers, json: { rulesets: [{ serverId: "1", rules: [] }], greTunnels: [], live: true } }) + }) + await page.getByRole("button", { name: "Обновить", exact: true }).click() + await expect.poll(() => started).toBe(true) + await changeContext(page, "backend") + const response = page.waitForResponse(r => r.url().includes(":8000/api/filters/rules?serverId=1")) + release!() + await (await response).finished() + await flush(page) + await expect(page.locator("tr[data-row-id]")).toHaveCount(1) + await expect(page.locator("tr[data-row-id]")).toContainText("Новый источник") + await expect(page.getByRole("button", { name: "Новое правило", exact: true })).toBeEnabled() + await expect(page.getByText("Не удалось прочитать правила с роутера", { exact: true })).toHaveCount(0) + }) +} +for (const change of ["backend", "mode"]) { + for (const fails of [false, true]) { + test(`Filters: старое применение при смене ${change}/${fails}`, async ({ page }) => { + const oldReads = await setup(page) + let release: (() => void) | undefined + const gate = new Promise(resolve => { release = resolve }) + let started = false + await page.route("**/api/filters/rules", async route => { + if (route.request().method() === "OPTIONS") return route.fulfill({ headers, json: {} }) + if (route.request().method() !== "PUT") return route.fallback() + started = true + await gate + return route.fulfill(fails ? { headers, status: 403, json: { error: "Старое применение запрещено" } } : { headers, json: { ok: true, rules: [] } }) + }) + await page.locator("tr[data-row-id]").first().hover() + await page.getByRole("button", { name: "Удалить правило", exact: true }).click() + await page.getByRole("alertdialog").getByRole("button", { name: "Удалить", exact: true }).click() + await expect.poll(() => started).toBe(true) + const before = oldReads() + await changeContext(page, change) + const response = page.waitForResponse(r => r.url().includes(":8000/api/filters/rules") && r.request().method() === "PUT") + release!() + await (await response).finished() + await flush(page) + expect(oldReads()).toBe(before) + await expect(page.getByText("Правила применены на роутер", { exact: true })).toHaveCount(0) + await expect(page.getByText("Не удалось применить правила на роутер", { exact: true })).toHaveCount(0) + await expect(page.getByRole("alertdialog")).toHaveCount(0) + await expect(page.getByRole("button", { name: "Новое правило", exact: true })).toBeEnabled() + }) + } +}