test: sidebar-pages-tree.spec тестирует копию продакшен-логики; обоснование про jest-config неверно #75

New Issue

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

Ghost commented

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

Найдено в multi-aspect code review всех изменений с коммита 053a9c0d (ветка develop).

Грань: conventions · Severity: warning
Где: apps/server/src/core/page/services/sidebar-pages-tree.spec.ts:1-45

Проблема
Спека определяет локальный shapeTree — копию PageService.getSidebarPagesTree — и проверяет копию, никогда не задевая продакшен-путь (ложная уверенность: зелёный тест при регрессии реального getSidebarPagesTree). Заголовок оправдывает это «отсутствием ^src/(.*)$ mapping в jest-конфиге сервера», но apps/server/package.json (jest.moduleNameMapper) такой mapping содержит — заявленного блокера нет. Все остальные серверные спеки импортируют реальный код (в т.ч. соседние ws-tree.service.spec.ts, page-ws.listener.spec.ts).

Предлагаемый фикс
Импортировать и тестировать реальную логику (вынести чистую часть getSidebarPagesTree в импортируемый хелпер либо поднять PageService через Test/TestingModule с моками repo), либо, если реальный блокер всё же есть, исправить неверный заголовок.

Связанные: #56

Найдено в multi-aspect code review всех изменений с коммита `053a9c0d` (ветка develop). **Грань:** conventions · **Severity:** warning **Где:** `apps/server/src/core/page/services/sidebar-pages-tree.spec.ts:1-45` **Проблема** Спека определяет локальный shapeTree — копию PageService.getSidebarPagesTree — и проверяет копию, никогда не задевая продакшен-путь (ложная уверенность: зелёный тест при регрессии реального getSidebarPagesTree). Заголовок оправдывает это «отсутствием ^src/(.*)$ mapping в jest-конфиге сервера», но apps/server/package.json (jest.moduleNameMapper) такой mapping содержит — заявленного блокера нет. Все остальные серверные спеки импортируют реальный код (в т.ч. соседние ws-tree.service.spec.ts, page-ws.listener.spec.ts). **Предлагаемый фикс** Импортировать и тестировать реальную логику (вынести чистую часть getSidebarPagesTree в импортируемый хелпер либо поднять PageService через Test/TestingModule с моками repo), либо, если реальный блокер всё же есть, исправить неверный заголовок. **Связанные:** #56

Ghost referenced this issue

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

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

Ghost referenced this issue from a commit

2026-06-21 03:30:31 +03:00

test(page): exercise the real getSidebarPagesTree via an extracted pure helper (#75)

~~Ghost referenced this issue 2026-06-21 04:15:11 +03:00~~

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

Ghost referenced this issue

2026-06-21 04:19:31 +03:00

fix: review/red-team batch 2 — 30 issues (security, ws, page-templates, html-embed, mcp, tests, docs) #101

Ghost closed this issue