test (сквозное): security-нагруженные гейты исполнения и fail-closed ветки системно не покрыты #97

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 · Severity: warning

Самые security-нагруженные решения — клиентский гейт исполнения JS и серверные fail-closed ветки — именно те, что без покрытия. Систематически «гейт работает» проверяется присутствием кода, а не поведением.

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

#77 — клиентский гейт исполнения JS htmlEmbed (shouldExecute): read-only всегда исполняет, editable гейтит на тоггл — ни одна ветка не покрыта.
#81 — catch-ветка lookupTemplate (fail-closed в not_found при битом контенте) не покрыта.
#85 — положительная ветка getSharePage (обязательный updatePublicAttachments перед отдачей анонимной модели) не покрыта.

Что сделать
Для каждого — вынести решение в чистый хелпер где уместно и покрыть позитив+негатив ветки. Эта issue — зонтик; конкретные правки — в дочерних #77/#81/#85.

Сквозная тема (tracking), найдена в multi-aspect code review изменений с коммита `053a9c0d`. **Грань:** test-coverage · **Severity:** warning Самые security-нагруженные решения — клиентский гейт исполнения JS и серверные fail-closed ветки — именно те, что без покрытия. Систематически «гейт работает» проверяется присутствием кода, а не поведением. **Входящие проблемы** - #77 — клиентский гейт исполнения JS htmlEmbed (shouldExecute): read-only всегда исполняет, editable гейтит на тоггл — ни одна ветка не покрыта. - #81 — catch-ветка lookupTemplate (fail-closed в not_found при битом контенте) не покрыта. - #85 — положительная ветка getSharePage (обязательный updatePublicAttachments перед отдачей анонимной модели) не покрыта. **Что сделать** Для каждого — вынести решение в чистый хелпер где уместно и покрыть позитив+негатив ветки. Эта issue — зонтик; конкретные правки — в дочерних #77/#81/#85.

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#97