Files
HY2XS_flamy/apps/model/dto/normalize_test.go
T
founder c0a43ae915 fix(admin): закрыть обещания панели, которые продукт не выполнял
Девятый проход, по итогам приёмки v1.0.0-rc1 на живом Debian 13. Общая тема:
интерфейс обещал оператору то, что продукт умел, но до чего не доходило
управление.

Секрет пира. Подпись под полем предлагала оставить его пустым, сервер умел его
сгенерировать, и генерация была недостижима: в go-playground/validator тег
omitempty НЕ пропускает правило, если поле объявлено указателем и указатель не
nil — hasValue считает указатель на пустую строку «значением». Правило min=6
применялось к пустой строке и отказывало. Ловушка закрыта общим шагом
нормализации DTO, а не тегом на одном поле: та же ловушка ломала фильтр списка
пиров, где очищенный крестиком el-input отправляет `?name=`. Граница проходит по
каждому полю отдельно — у remark пустая строка означает «убрать пометку», у
disabled ноль означает «включён».

Отказы. Любая ошибка любого поля превращалась в слово `invalid`, а слой vo
определял код ответа СРАВНЕНИЕМ текста сообщения — тот же антипаттерн, который
запрещён панели, только на сервере. Ответ несёт errors[{code, field, message,
params}]; панель выбирает фразу по коду и подставляет причины под поля.

Сессия. Ветка «войдите заново» была недостижима дважды: сервер отвечает HTTP 200
на любой отказ, поэтому обработчик ошибок axios не вызывался, а условие в нём
проверяло code === "A0230" и поле msg, которых в этом API никогда не было.
Истёкший токен вдобавок уезжал с кодом системной ошибки.

Иконки. Контракт currentColor был объявлен в двух местах и не действовал: восемь
ассетов несли литеральный fill="#000000" на <path>, а атрибут представления
перебивает унаследованное CSS-свойство. Под это попадали все семь иконок
бокового меню на фоне #181818.

Имя пира. Два правила на одном поле противоречили друг другу (min=1 против
6-32), а копия набора символов в слое контроллеров несла неэкранированный дефис
и впускала `, - . / : ; <` — через панель проходило имя peer/name, которое
импорт того же пира отклонял. Набор символов ЛОГИНА сознательно не сужен и
закреплён тестом: он приходит из HY2XS_ADMIN_USER и оркестратором не
ограничивается.

Добавлены подпись «Разработано во Flamy» с адресом, принадлежащим приложению, и
контрактные тесты панели как обязательный шаг сборки. Их исполняет Bun, а не
vitest: jsdom не вычисляет currentColor и визуальной корректности не доказал бы,
зато vitest привёл бы в граф pnpm audit сотню транзитивных зависимостей.

docs/ разложена по слоям, 11-testing-and-acceptance.md (117 КБ) разбит на пять
частей, добавлен docs/acceptance/ с отчётом о прогоне rc1 и перечнем дефектов.
Обход документации в приёмке стал рекурсивным: плоский docs/*.md после
разнесения по каталогам совпадал бы ровно с одним файлом.
2026-09-01 07:27:15 +05:00

123 lines
4.6 KiB
Go

package dto
import "testing"
func strPtr(v string) *string { return &v }
func i64Ptr(v int64) *int64 { return &v }
func intPtr(v int) *int { return &v }
// Граница проходит по КАЖДОМУ полю отдельно, и это главное свойство
// нормализации.
//
// Общее правило «пусто → не задано» выглядит соблазнительно и молча ломает
// смысл: у комментария пустая строка означает «убрать пометку», у флага
// disabled ноль — «включён», у квоты ноль — «нулевая квота». Тест закрепляет,
// что эти три случая не попали под общий гребень.
func TestPeerSaveNormalizeTreatsBlankSecretAsAbsent(t *testing.T) {
for _, blank := range []string{"", " ", "\t", "\n", " \t\n "} {
d := PeerSaveDto{Name: strPtr("client-01"), Secret: strPtr(blank)}
d.Normalize()
if d.Secret != nil {
t.Errorf("секрет %q не приведён к «не задано»: %q", blank, *d.Secret)
}
}
}
func TestPeerSaveNormalizeKeepsManualSecretTrimmed(t *testing.T) {
d := PeerSaveDto{Name: strPtr("client-01"), Secret: strPtr(" s3cret-value ")}
d.Normalize()
if d.Secret == nil {
t.Fatal("заданный секрет потерян")
}
if *d.Secret != "s3cret-value" {
t.Fatalf("секрет не обрезан по краям: %q", *d.Secret)
}
}
func TestPeerSaveNormalizeKeepsBlankRemarkAsValue(t *testing.T) {
d := PeerSaveDto{Name: strPtr("client-01"), Remark: strPtr(" ")}
d.Normalize()
if d.Remark == nil {
t.Fatal("пустая пометка превращена в «не задано»: очистить комментарий станет нечем")
}
if *d.Remark != "" {
t.Fatalf("пометка не обрезана: %q", *d.Remark)
}
}
func TestPeerUpdateNormalizeTreatsBlankIdentityFieldsAsAbsent(t *testing.T) {
d := PeerUpdateDto{Name: strPtr(" "), Secret: strPtr("")}
d.Normalize()
if d.Name != nil {
t.Error("пустое имя при изменении обязано означать «не менять»")
}
if d.Secret != nil {
t.Error("пустой секрет при изменении обязан означать «не менять»")
}
}
func TestPeerUpdateNormalizeKeepsZeroValuedFlags(t *testing.T) {
d := PeerUpdateDto{
Disabled: i64Ptr(0),
QuotaBytes: i64Ptr(0),
MaxDevices: i64Ptr(1),
}
d.Normalize()
if d.Disabled == nil || *d.Disabled != 0 {
t.Error("disabled=0 означает «включён», а не «не задано»")
}
if d.QuotaBytes == nil || *d.QuotaBytes != 0 {
t.Error("quotaBytes=0 означает нулевую квоту, а не «не задано»")
}
}
// Регрессия: очищенный крестиком фильтр отправлялся как `?name=` и отказывал
// правилом длины, то есть список пиров ломался в один клик.
func TestPeerPageNormalizeDropsClearedFilters(t *testing.T) {
d := PeerPageDto{Name: strPtr(""), Remark: strPtr(" ")}
d.Normalize()
if d.Name != nil || d.Remark != nil {
t.Fatalf("очищенный фильтр не снят: name=%v remark=%v", d.Name, d.Remark)
}
}
func TestBaseNormalizeDropsZeroTimestamps(t *testing.T) {
d := BaseDto{StartTime: i64Ptr(0), EndTime: i64Ptr(0)}
d.Normalize()
if d.StartTime != nil || d.EndTime != nil {
t.Fatal("нулевая отметка времени означает отсутствие фильтра")
}
kept := BaseDto{StartTime: i64Ptr(1), EndTime: i64Ptr(2)}
kept.Normalize()
if kept.StartTime == nil || kept.EndTime == nil {
t.Fatal("заданные отметки времени потеряны")
}
}
func TestLogNormalizeDropsZeroLineCount(t *testing.T) {
d := LogDto{NumLine: intPtr(0)}
d.Normalize()
if d.NumLine != nil {
t.Fatal("«показать 0 строк» — это пропущенный параметр, а не запрос")
}
}
// Все нормализуемые DTO обязаны реализовывать интерфейс: слой контроллеров
// вызывает Normalize через него, и забытая реализация означала бы молча
// пропущенный шаг.
func TestNormalizableIsImplemented(t *testing.T) {
var _ Normalizable = (*PeerSaveDto)(nil)
var _ Normalizable = (*PeerUpdateDto)(nil)
var _ Normalizable = (*PeerPageDto)(nil)
var _ Normalizable = (*BaseDto)(nil)
var _ Normalizable = (*LogDto)(nil)
}