From 75b9a0cfea4d0e5af3cb3ddd669a94cdde47629a Mon Sep 17 00:00:00 2001 From: claude-bot Date: Sun, 14 Jun 2026 01:47:11 +0300 Subject: [PATCH] analyst(ET): auto-commit from analyst run_id=2 --- .../ET-001/03-acceptance-criteria.md | 12 ++++++---- docs/work-items/ET-001/04-test-plan.yaml | 22 +++++++++++-------- .../ET-001/09-analyst-decision-required.md | 21 ++++++++++++++++++ 3 files changed, 42 insertions(+), 13 deletions(-) diff --git a/docs/work-items/ET-001/03-acceptance-criteria.md b/docs/work-items/ET-001/03-acceptance-criteria.md index a175210..f7b0fe0 100644 --- a/docs/work-items/ET-001/03-acceptance-criteria.md +++ b/docs/work-items/ET-001/03-acceptance-criteria.md @@ -61,11 +61,15 @@ AC-01 — **новая дельта** (подпись). ('1'/'0') и фактическая видимость слоёв POI не противоречат друг другу. ## AC-09 — Регрессия unit-тестов POI -- **Given** ветка с правкой подписи. +- **Given** ветка с правкой подписи (`index.html:88`) **и** синхронно + обновлённым ожиданием текста в `tests/unit/test_poi_toggle.py:54` + (`assert "Показывать POI" in html`). - **When** запускается `make test`. -- **Then** `tests/unit/poi_toggle.test.js` и - `tests/unit/test_poi_toggle.py` проходят (с обновлённым ожиданием - текста подписи, если оно проверяется). +- **Then** `tests/unit/test_poi_toggle.py` и + `tests/unit/poi_toggle.test.js` зелёные. Примечание: текст подписи + проверяет именно python-тест (строка 54); JS-тест проверяет только + поведение и подписи не касается. Без правки строки 54 `make test` + упадёт на ассерте `POI`. ## AC-10 — Без побочных эффектов на сервере (REQ-NF-02) - **Given** переключение чекбокса. diff --git a/docs/work-items/ET-001/04-test-plan.yaml b/docs/work-items/ET-001/04-test-plan.yaml index ddc4863..0dd3bc8 100644 --- a/docs/work-items/ET-001/04-test-plan.yaml +++ b/docs/work-items/ET-001/04-test-plan.yaml @@ -5,7 +5,7 @@ # Среда e2e/ui: https://openclaw.mva154.duckdns.org/enduro/ work_item: ET-001 -version: 2 +version: 3 relates_to: ET-002 related_acs: [AC-01, AC-02, AC-03, AC-04, AC-05, AC-06, AC-07, AC-08, AC-09, AC-10] @@ -76,18 +76,22 @@ tests: type: unit layer: frontend title: Подпись чекбокса равна «Показывать POI» (новая дельта) - target: src/web/index.html :: #poi-visible-cb label + target: tests/unit/test_poi_toggle.py:54 :: подпись #poi-visible-cb given: | - Разобранный DOM index.html (или тест из poi_toggle.test.js, - проверяющий текст подписи). + Существующий регресс-тест ET-002 test_poi_toggle.py строкой 54 + проверяет `assert "POI" in html`. Это и есть тест, + который фиксирует текст подписи (НЕ JS-тест poi_toggle.test.js — + тот проверяет только поведенческий блок). when: | - Прочитать textContent рядом с #poi-visible-cb. + Применена правка index.html:88 (Показывать POI) и + ожидание теста обновлено на `Показывать POI`. then: | - Текст строго равен «Показывать POI». + Ассерт строки 54 проходит на новой подписи. note: | - ДО правки кейс обязан падать (сейчас «POI»). Обновить ожидание - синхронно с правкой index.html. - covers: [REQ-F-01, AC-01] + ДО правки кейс обязан падать (сейчас в HTML «POI»). + Обновлять index.html:88 и test_poi_toggle.py:54 СТРОГО в одном + коммите, иначе make test красный. + covers: [REQ-F-01, AC-01, AC-09] # ─── Unit (python, регресс серверного контракта) ──────────────────── - id: TC-U-06 diff --git a/docs/work-items/ET-001/09-analyst-decision-required.md b/docs/work-items/ET-001/09-analyst-decision-required.md index 6d5d680..fe40a7f 100644 --- a/docs/work-items/ET-001/09-analyst-decision-required.md +++ b/docs/work-items/ET-001/09-analyst-decision-required.md @@ -246,3 +246,24 @@ POI закоммиченные артефакты не тронуты, дубл **Итог.** Этап «Анализ» завершён (пакет POI в наличии и корректен). Задача остаётся `blocked-needs-owner-decision`; рекомендация прежняя — **закрыть ET-001 как дубликат ET-002** (закрытие за Owner/CI, CLAUDE.md №4). + +## 11. Прогон #23 (2026-06-14) — точная привязка теста подписи + +Соглашаюсь с §10 (пакет POI выпущен, этап «Анализ» завершён). Единственное +добавление — **конкретизация, какой тест ломает дельту подписи REQ-F-01**, +т.к. прежние ТЗ/AC/тест-план указывали тест неточно: + +- Подпись жёстко проверяет **python-тест** `tests/unit/test_poi_toggle.py:54` + — `assert "POI" in html`. Его нужно обновить на + `Показывать POI` **в одном коммите** с правкой `index.html:88`. +- **JS-тест** `tests/unit/poi_toggle.test.js` подпись **не** проверяет + (извлекает поведенческий блок по маркерам) — правки не требует. + +Финализированы (точная привязка теста, без смены сути) только артефакты +аналитика: `01-brd` v4 (риск R1), `02-trz` v3 (REQ-F-01 + §4), +`03-acceptance-criteria` v3 (AC-09), `04-test-plan` v3 (TC-U-05). Файлы +чужих этапов (`00`, `06`, `07`, `12`, `13`) и архив не тронуты. + +**Рекомендация без изменений** — закрыть ET-001 как дубликат ET-002. Если +доводить: единственная работа — `index.html:88` + синхронно +`test_poi_toggle.py:54` (этап разработки, не аналитики).