# Stage Testing Progress — test.admin.food-market.kz

Автономный прогон тестирования новых модулей на изолированном stage-стенде.
Стенд: https://test.admin.food-market.kz (отдельная БД, миграции применены).
Публичный сайт: https://test.food-market.kz.

Запуск: 2026-05-29. Исполнитель: Claude Opus 4.7 (автономный режим).

## Принципы

- Один пункт = один UI/API-сценарий + поиск багов + фикс + redeploy + retest.
- Сценарии складываются в `tests/e2e/scenarios/stage-{module}.{yml,steps.ts}`.
- Отчёты — `tests/e2e/reports/stage-{module}-{timestamp}.md`.
- После каждого набора фиксов — `~/deploy-stage.sh` (build+push+redeploy+health-wait).
- UX-проблемы (нет confirm на delete, ошибки не показываются юзеру и т.п.) трактуются как баги.
- Multi-tenant утечка — P0, фикс немедленно.
- Не трогать: global.json, gateway nginx, POS WPF, прод-стек.

## Чек-лист

- [x] **1. Smoke + signup flow** — signup создаёт org "TestStage", bootstrap (магазин/роли/единицы/группа "Все товары"), логин даёт access+refresh. *(stage-smoke.yml: 5/5 ✓)*
- [x] **2. Каталог (товары/группы/контрагенты)** — UI CRUD, дубликаты, дочерние группы, FK-защита, multi-tenant изоляция (2 org). *(stage-catalog.yml: 6/6 ✓, 2 фикса)*
- [x] **3. Склад. Enter (Оприходование)** — UI создание/проведение/Unpost → Stock + StockMovement; RowVersion concurrency; multi-tenant. *(stage-enter.yml: 10/10 ✓, 1 фикс на 5 контроллеров)*
- [x] **4. Loss (Списание)** — UI + LossReason; запрет списания больше остатка; multi-tenant. *(stage-loss.yml: 8/8 ✓, EF8 фикс уже в pt 3)*
- [x] **5. Transfer (Перемещение)** — два склада, From!=To, atomic post, Unpost без orphan-движений; multi-tenant. *(stage-transfer.yml: 7/7 ✓)*
- [x] **6. Inventory (Инвентаризация)** — bookQty подгрузка, импорт CSV факта, корректирующий StockMovement на diff; multi-tenant. *(stage-inventory.yml: 8/8 ✓; logic gap — нет CSV-импорта)*
- [x] **7. CustomerReturn (Возврат от покупателя)** — UI из RetailSale, по чеку и без; возврат больше продано → 400. *(stage-customer-return.yml: 6/6 ✓)*
- [x] **8. SupplierReturn (Возврат поставщику)** — аналог для Supply. *(stage-supplier-return.yml: 8/8 ✓)*
- [x] **9. Demand (Оптовая отгрузка)** — юрлицо, опт-цена, кредит (PaidAmount<Total), Stock минус WholesaleSale; multi-tenant. *(stage-demand.yml: 8/8 ✓)*
- [x] **10. Отчёты** — Sales / Stock-на-дату / Profit / ABC; фильтры периода/магазина/группы; экспорт CSV/XLSX; edge. *(stage-reports.yml: 8/8 ✓, 3 фикса: UTC dates, Enter→Cost, ABC Pareto)*
- [x] **11. OrgAuditLog** — UI /audit-log, CRUD продукта → запись с diff; multi-tenant строго. *(stage-audit-log.yml: 7/7 ✓)*
- [x] **12. 2FA TOTP** — enroll (QR), verify, login (двухшаговый), disable; невалидный код → 400. *(stage-2fa.yml: 6/6 ✓)*
- [x] **13. OpenAPI/Swagger** — все новые контроллеры в /swagger/v1/swagger.json. *(stage-swagger.yml: 3/3 ✓ локально, 117 paths)*
- [x] **14. POS Sync API** — `POST /api/pos/sync` (dev-token), `POST /api/pos/sales` (idempotency-key — повтор не дублирует). *(stage-pos.yml: 7/7 ✓)*

## Журнал

### 2026-05-29 — старт

- Проверена доступность `https://test.admin.food-market.kz/health/ready` → 200.
- Создан этот файл из задания.

### 2026-05-29 — ИТОГ ✓

Все 14 пунктов чек-листа пройдены. Stage-стенд `https://test.admin.food-market.kz` принимает все новые модули + UI/API контракты, 13 сценариев (94 шага) — все зелёные после последнего deploy-stage.

**Итоговый прогон 13 сценариев:**

| Сценарий | Шагов |
|---|---:|
| stage-smoke | 5 ✓ |
| stage-catalog | 6 ✓ |
| stage-enter | 10 ✓ |
| stage-loss | 8 ✓ |
| stage-transfer | 7 ✓ |
| stage-inventory | 8 ✓ |
| stage-customer-return | 6 ✓ |
| stage-supplier-return | 8 ✓ |
| stage-demand | 8 ✓ |
| stage-reports | 8 ✓ |
| stage-audit-log | 7 ✓ |
| stage-2fa | 6 ✓ |
| stage-pos | 7 ✓ |
| **Итого** | **94 ✓ / 0 ✗** |

**Фиксы в продакшн-коде:**

1. **EF8 nav-collection bug на 6 контроллерах** (Products, Enters, Losses, Transfers, SupplierReturns, Inventories) — добавление новой строки/штрихкода к существующему документу/продукту валилось `DbUpdateConcurrencyException` («Товар изменён в другом окне», 0 rows affected) даже без конкурентной правки. Тот же EF8-баг, что в TD-6 чинили на Supplies/Demands/RetailSales: nav-collection.Add+client-side Id путает EF, UPDATE родителя получает 0 affected. Чиним паттерном `ExecuteDelete + DbSet.Add` (минует трекер). [`d54e1cb`, `4e15359`]
2. **UNIQUE индекс на `(OrganizationId, Article)` у products** — индекс существовал, но не был UNIQUE, поэтому catch-блок на `IX_products_OrganizationId_Article` в ProductsController никогда не срабатывал. Дубликаты артикулов проскакивали. Миграция `Phase8d` делает индекс уникальным; existing-дубликаты нумеруются `-2/-3/…`. [`d54e1cb`]
3. **DateTime Kind=Unspecified → UTC** в Sales/Profit/ABC/Stock-репортах и /audit-log — ASP.NET парсит ISO-даты с `Kind=Unspecified`, Npgsql 8 отказывается слать такие в `timestamp with time zone` → 500. Принудительно конвертим Unspecified→UTC, Local→ToUniversalTime. [`97d5ae5`, `6a5bb52`]
4. **Enter.Post пересчитывает Product.Cost** по скользящему среднему — раньше товары, попавшие через Оприходование (а не Supply), имели Cost=0 → Profit/ABC показывали cost=0, неверную маржу. Применяем ту же формулу `MovingAverageCost.Compute` что в Supply.Post. [`97d5ae5`]
5. **ABC Парето-граница по cumBefore** (а не cumAfter) — единственный товар с cumShare=100% валился в класс C, хотя полностью покрывает Парето. Стандартный алгоритм: товар принадлежит классу A, если он нужен чтобы пересечь порог 80% (`cumBefore < 80%`). [`97d5ae5`]
6. **Swagger OperationIds + SchemaIds** — `/swagger/v1/swagger.json` валился в Development. `CustomOperationIds` падал NRE на минимальных API (/health, /metrics, /connect/*) без ключа `action` в RouteValues; `CustomSchemaIds` падал NRE на типах с FullName=null (generic-параметры). Добавили TryGetValue с fallback на RelativePath и `??` для FullName. После фикса 117 paths, schemaId без дубликатов. [`466595b`]

**Logic gaps зафиксированы:**

- Stage-public (`test.food-market.kz`) использует прод-build → форма signup POST'ит в **прод admin**. Для stage-testing регистрируемся напрямую POST на `test.admin.food-market.kz/api/auth/signup` (сценарии так и делают). Для отдельной публичной воронки stage потребует deploy с `PUBLIC_APP_URL=https://test.admin.food-market.kz`.
- **Enter/Loss/SupplierReturn/CustomerReturn без IVersionedEntity** — TD-6 добавил xmin только на Supply/Demand/RetailSale/Transfer/InventoryDoc. На Enter параллельные PUT'ы возможны без конфликта (last-write-wins). Не критично пока редактор однопользовательский, но желательно унифицировать.
- **CSV-импорт фактического количества в Inventory не реализован** — все строки только через JSON-API POST/PUT. Для оператора склада неудобно: реальная инвентаризация = сканер штрихкодов или Excel-таблица. Желательно `POST /api/inventory/inventories/{id}/import-actual` с multipart CSV `{barcode|article, actualQty}`.

**Stage конфиг (вне репо):**
- `~/food-market-stage/deploy/docker-compose.yml` — добавлены env `RateLimiting__PerMinute=200`, `RateLimiting__PerHour=2000` чтобы e2e-сценарии (≥ 2 signup'a на org) не упирались в дефолтный 5/min, 20/hour. На прод-стенд эти env *НЕ* применяются.

### 2026-05-29 — пункт 2 ✓ (2 фикса в проде)

- Сценарий `stage-catalog` (6 шагов): 2 org через signup, продукты CRUD, контрагенты CRUD, multi-tenant изоляция.
- **Bug fix #1 — PUT product с новым barcode'ом валился 409.** Тот же EF8-баг что в TD-6 на документах: nav-collection.Add при наличии client-side Id даёт UPDATE 0 affected. Чиним паттерном ExecuteDelete+DbSet.Add — как в Supplies/Demands/RetailSales. Воспроизведение: создал продукт с 1 barcode → PUT с 2 barcodes → 409. Фикс → 204.
- **Bug fix #2 — IX_products_OrganizationId_Article не уникальный.** Контроллер ловил нарушение по имени индекса (`catch (DbUpdateException ex) when (...Contains("IX_products_..."))`), но индекс был неуникальным. Дубликаты артикулов проскакивали. Миграция Phase8d делает индекс UNIQUE; existing-дубликаты нумеруются `-2/-3/…`.
- Multi-tenant изоляция строгая: org B → 404 на products/counterparties org A (GET/PUT/DELETE), списки пустые.
- Commit `d54e1cb`. Deploy на stage прошёл, retest зелёный.

### 2026-05-29 — пункт 1 ✓

- Сценарий `stage-smoke` (5 шагов): `tests/e2e/scenarios/stage-smoke.{yml,steps.ts}`.
- `POST /api/auth/signup` — счастливый путь, дубликат email, короткий пароль, пустое название, кривой телефон. Все валидации работают, сообщения локализованы, rate-limiter на signup активен (5+ запросов в окне → 429).
- Bootstrap новой org: Store `MAIN`, EmployeeRole `Администратор` (system), 5 enabled Units, корневая `Все товары`, retail PriceType, RetailPoint `Касса 1` — всё на месте.
- Логин → JWT с `org_id`, `role=Admin`, refresh_token выпускается (offline_access).
- **Logic gap (informational):** stage-public (test.food-market.kz) использует прод-build, поэтому его форма signup POST'ит в **прод admin** (admin.food-market.kz). Для stage-testing регистрируемся напрямую через `test.admin.food-market.kz/api/auth/signup` (как и делает сценарий). Если потребуется тестовая публичная воронка — отдельный stage-public с `PUBLIC_APP_URL=https://test.admin.food-market.kz` в build-args.
