test: runtime fixtures - bad files

2026-11-13 17:08:20 +00:00 · 2026-09-26 04:00:44 +05:00
parent 712fd3cb2e
commit 468beec998
10 changed files with 328 additions and 29 deletions
@@ -51,17 +51,17 @@
 только если темы нет в существующем документе; иначе обновляется источник
 истины.

-## Текущий baseline Q6c
+## Текущий baseline Q6d

-На момент фиксации Q6c:
+На момент фиксации Q6d:

 - production registry содержит 122 инструмента;
 - `pnpm --dir web check` проходит без ошибок и предупреждений;
- `pnpm --dir web test` — 42 test files / 455 tests;
+- `pnpm --dir web test` — 42 test files / 457 tests;
 - `pnpm --dir web test:rules` — 4 test files / 74 tests;
 - `pnpm --dir web lint:all` — 0 errors и 0 i18n warnings; остаются 4 unused
  design tokens от token audit;
- `pnpm verify` проходит; production E2E — 112 passed, 0 skipped.
+- `pnpm verify` проходит; production E2E — 121 passed, 0 skipped.

 Baseline и следующий шаг поддерживаются в `docs/plan-codebase-quality.md`.

@@ -107,7 +107,8 @@

 ## Качество production UI

-Q5 закрывает известные preview-проблемы; активных `test.fixme` в e2e больше нет.
+Q5–Q6d закрывают известные preview-проблемы и baseline special fixtures;
+активных `test.fixme` в e2e больше нет.

 - [x] Генераторы auto-run без кнопки Generate — `web/e2e/generators.spec.ts`.
 - [x] `resize-png` и `crop-png` используют размеры текущего source через общий
@@ -116,6 +117,11 @@ Q5 закрывает известные preview-проблемы; активн
      auto-run — `web/e2e/pipeline.spec.ts`.
 - [x] Ошибки хранят i18n key/vars и переводятся при выводе; изменение языка
      обновляет alert — `web/e2e/i18n.spec.ts`.
+- [x] Malformed/special PNG fixtures: CRC, truncated, palette и 16-bit проходят
+      через upload без краша — `web/e2e/png-fixtures.spec.ts`.
+- [ ] Ошибка декодирования изображения из `createImageBitmap` пока показывается
+      сырым сообщением браузера; нужен `ToolError`-маппер и i18n-ключ. Это
+      продуктовая задача, не Q6d.

 ## Идеи

@@ -19,12 +19,13 @@
 - [ ] PNG 1×1 — инструменты не падают (проверено авто: flip-png, см. suite).
 - [ ] PNG с полупрозрачностью/чётким альфа-краёв (чёрно-белая шахматка) —
      remove-background, feather-edges, clean-edges дают корректные края.
- [ ] Палитровый PNG (индексированные цвета) — конвертируется/анализируется.
+- [ ] Палитровый PNG (индексированные цвета) — конвертируется/анализируется
+      (авто-покрытие: `web/e2e/png-fixtures.spec.ts`).
 - [ ] Анимированный/APNG, если попадётся — не ломает конвейер.
 - [ ] Повреждённый/не-PNG файл — понятная ошибка, нет краша (авто-чека в
-      pipeline.spec.ts).
+      `pipeline.spec.ts` и `png-fixtures.spec.ts`).
 - [ ] PNG с битой CRC / 16-bit — как ведут себя анализаторы (ориентация, размер,
-      прозрачность).
+      прозрачность); CRC/16-bit авто-покрыты в `png-fixtures.spec.ts`.

 ### Файлы (`tests/fixtures/manual-edge-cases/`)

@@ -95,9 +96,9 @@
      корректный (не «кракозябры»).
 - [ ] remove-background на сложной полупрозрачности — края не «звенят».

-## G. Закрытые Q5/Q6b/Q6c-проблемы
+## G. Закрытые Q5/Q6b/Q6c/Q6d-проблемы

-> Q5/Q6b/Q6c-проблемы закрыты автоматическими тестами; для ручной проверки
+> Q5/Q6b/Q6c/Q6d-проблемы закрыты автоматическими тестами; для ручной проверки
 > остаются субъективные сценарии разделов A–F.

 - [x] **Генераторы:** auto-run по дефолтам, результат и изменение параметра без
@@ -114,6 +115,8 @@
      `web/e2e/pipeline.spec.ts`.
 - [x] **Output MIME/quality:** JPG/WebP/BMP имеют ожидаемые расширения и
      сигнатуры, quality 10/90 меняет размер файла; `web/e2e/pipeline.spec.ts`.
+- [x] **Malformed/special PNG:** CRC, truncated, palette и 16-bit fixtures
+      проходят через upload без краша; `web/e2e/png-fixtures.spec.ts`.

 ## Как долго

@@ -1,9 +1,9 @@
 # План: линтеры, тесты и качество кодовой базы

-> Статус: **draft — Q0–Q5 и Q6a–Q6c готовы к ревью; Q6d — следующий этап**.
+> Статус: **draft — Q0–Q5 и Q6a–Q6d готовы к ревью; Q6e — следующий этап**.
 >
 > Источник: `docs/backlog.md`, аудит конфигурации `web/`, e2e-тестов и
-> инфраструктурных скриптов. Q0–Q5 и Q6a–Q6c выполнены; Q6d–Q8 выполняются
+> инфраструктурных скриптов. Q0–Q5 и Q6a–Q6d выполнены; Q6e–Q8 выполняются
 > отдельными атомарными шагами.
 >
 > Цель: сделать качество кодовой базы воспроизводимым, убрать ложную «зелёность»
@@ -243,9 +243,24 @@ JPG/WebP/BMP, не превращаясь в UX-райз кнопки Download.
 - E2E проверяет JPEG `FF D8 FF`, WebP `RIFF/WEBP`, BMP `BM` и PNG-фолбэк.
 - Ошибки кодирования и unsupported MIME используют существующие i18n-ключи.

-#### Q6d–Q6f. Оставшиеся направления
+#### Q6d. Malformed и special PNG fixtures
+
+**Статус:** выполнен. Runtime-fixtures покрывают CRC, truncated, palette и
+16-bit без бинарных файлов в git. E2E фиксирует браузерный контракт: critical
+CRC/truncated отклоняются без падения, ancillary CRC и хвост после `IEND`
+допускаются, palette/16-bit декодируются. Unit фиксирует контракт анализаторов
+для RGBA-данных, которые браузер отдаёт после нормализации.
+
+- `web/e2e/helpers/fixtures.ts` собирает indexed, 16-bit, CRC-mutation,
+  truncation и tail-варианты из уже существующих `crc32`/`chunk`.
+- Проверки выполняются в Chromium; browser matrix остаётся Q6f.
+- Поведение `createImageBitmap` является частью browser-контракта, а не
+  самостоятельного PNG-парсера product-кода.
+- Сырое сообщение браузера при decode-ошибке не локализуется в Q6d; это
+  отдельная задача в backlog.
+
+#### Q6e–Q6f. Оставшиеся направления

- malformed и special PNG fixtures: CRC, truncated, palette, 16-bit;
 - smoke всех типов инструментов из registry;
 - component-тесты `SchemaToolView`, `SchemaPreview` и schema fields;
 - browser matrix, mobile viewport, visual snapshots и coverage thresholds.
@@ -408,7 +423,6 @@ refactor, новые i18n-правила и исправление продук
 ## 7. Решения, которые нужно принять на ревью

 - Какие executor/worker/fallback сценарии войдут в первый Q6 coverage PR?
- Какие special PNG fixtures обязательны для baseline?
 - Нужны ли browser/mobile matrix и visual snapshots до coverage thresholds?
 - Какие unused design tokens можно удалить, а какие должны остаться?
 - Какие browser/e2e проверки должны быть обязательными для PR, а какие —
@@ -35,7 +35,9 @@ pnpm --dir web test:e2e
 ```

 Запускать его обязательно для изменений маршрутов, загрузки файла, результата,
-download, темы или локализации. В текущем CI это отдельный non-blocking
+download, темы или локализации. Изменения runtime fixtures/загрузки файла
+проверяются отдельным `web/e2e/png-fixtures.spec.ts`; ручные бинарники из
+`tests/fixtures/` не входят в `verify`. В текущем CI это отдельный non-blocking
 baseline; после закрытия известных ошибок его можно сделать blocking.

 ## Быстрые подмножества
@@ -46,6 +48,7 @@ baseline; после закрытия известных ошибок его м
 | TypeScript/Svelte            | `pnpm --dir web check` и `pnpm --dir web test`          |
 | Lint-правило или lint-конфиг | `pnpm --dir web test:rules` и `pnpm --dir web lint:all` |
 | Пользовательский поток       | `pnpm --dir web test:e2e`                               |
+| Fixtures/загрузка файла      | `pnpm --dir web test:e2e`                               |
 | Полный локальный gate        | `pnpm verify`                                           |

 ## Errors и warnings
@@ -53,9 +53,11 @@ CSS-классы и конкретный формат временного URL

 Изображения создаются воспроизводимыми helpers в `web/e2e/helpers/fixtures.ts`.
 Тесты не добавляют большие бинарные файлы и не зависят от локальных
-пользовательских файлов. `toPass()` допускается только для ограниченного
-ожидания hydration или асинхронного результата; он не должен превращать проверку
-продукта в бесконечный retry.
+пользовательских файлов. Runtime-fixtures для malformed/special PNG проверяют
+browser-контракт: critical CRC и truncated отклоняются, ancillary CRC и хвост
+после `IEND` допускаются, palette/16-bit декодируются. `toPass()` допускается
+только для ограниченного ожидания hydration или асинхронного результата; он не
+должен превращать проверку продукта в бесконечный retry.

 ## 5. Helpers и `test.fixme`

@@ -68,7 +70,7 @@ ZIP-сценарием: общий image-smoke не применяется к н

 `test.fixme` допускается только для зарегистрированной проблемы в backlog или
 checklist. В комментарии указывается ссылка на неё. После исправления тест и
-ссылка обновляются вместе; в текущем Q6a baseline активных fixme нет.
+ссылка обновляются вместе; в текущем Q6d baseline активных fixme нет.

 ## 6. Проверки