test (сквозное): анти-паттерн «спека тестирует копию продакшен-логики» + неработающие сьюты security-фич #96

New Issue

Ghost · 2026-06-21T02:33:45+03:00

Ghost commented

2026-06-21 02:33:45 +03:00

Сквозная тема (tracking), найдена в multi-aspect code review изменений с коммита 053a9c0d.

Грань: test-coverage / conventions · Severity: warning

Повторяющийся анти-паттерн: спека определяет локальную копию продакшен-логики вместо импорта реального кода → тест остаётся зелёным при реальной регрессии. Плюс часть серверного тест-слоя вообще не исполняется.

Входящие проблемы

#75 — sidebar-pages-tree.spec проверяет копию shapeTree вместо getSidebarPagesTree (и неверное обоснование про jest-config).
#79 — спеки transclusion не компилируются (TS2554) → 0 тестов, отключены все проверки безопасности page-templates.
public-share-chat.spec: локальный decidePageInShare переписывает deriveShareAccess вместо его вызова (suggestion-уровень, отдельной issue не заводилось).
#56 — 16 падающих серверных jest-сьютов (NestJS DI без провайдеров + lib0 ESM вне transformIgnorePatterns).

Что сделать
Тестировать реальные функции/классы (импорт или Test/TestingModule с моками repo); починить компиляцию/инфраструктуру (#79, #56). Эта issue — зонтик для отслеживания темы; конкретные правки — в дочерних.

Сквозная тема (tracking), найдена в multi-aspect code review изменений с коммита `053a9c0d`. **Грань:** test-coverage / conventions · **Severity:** warning Повторяющийся анти-паттерн: спека определяет локальную копию продакшен-логики вместо импорта реального кода → тест остаётся зелёным при реальной регрессии. Плюс часть серверного тест-слоя вообще не исполняется. **Входящие проблемы** - #75 — sidebar-pages-tree.spec проверяет копию shapeTree вместо getSidebarPagesTree (и неверное обоснование про jest-config). - #79 — спеки transclusion не компилируются (TS2554) → 0 тестов, отключены все проверки безопасности page-templates. - public-share-chat.spec: локальный decidePageInShare переписывает deriveShareAccess вместо его вызова (suggestion-уровень, отдельной issue не заводилось). - #56 — 16 падающих серверных jest-сьютов (NestJS DI без провайдеров + lib0 ESM вне transformIgnorePatterns). **Что сделать** Тестировать реальные функции/классы (импорт или Test/TestingModule с моками repo); починить компиляцию/инфраструктуру (#79, #56). Эта issue — зонтик для отслеживания темы; конкретные правки — в дочерних.

Ghost closed this issue

2026-06-21 04:15:11 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vvzvlad/gitmost#96