RSHB/DFiP_Budget_planing
9
0
Fork 0
Code Issues 13 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

fix(tests): test_form2_ahr_limit падает KeyError после null-strip (AURORA-1897, часть #296) #299

Merged
aep merged 1 commits from feat/296-ahr-limit-tests-null-strip into test 2026-11-08 17:10:07 +03:00
Conversation 0 Commits 1 Files Changed 1 +12 -12
aep commented 2026-11-08 15:04:43 +03:00
Owner
Copy Link

Часть follow-up #296 (раздел «Баг в тестах AURORA-1897/1906»), только test_form2_ahr_limit.py.

Проблема. 6 из 7 тестов падали KeyError: 'q1' ещё на чтении состояния «до»: после AURORA-1848 GET листа вырезает null-поля, у заголовка статьи без плана ключа plan.qN нет.

Правка. ["plan"]["qN"] → ["plan"].get("qN") (выражения уже были ... or 0). Продуктовый код не трогается.

Проверка. pytest tests/integration/test_form2_ahr_limit.py --init_db на чистой БД (Postgres 16, стенд perf): было 6 failed / 1 passed, стало 7 passed. Тем самым подтверждено, что AURORA-1897 на test после PR #295 работает: лист «АХР Лимиты» отдаётся (200), годовые нормативы /4, суммы без НДС, план АХР в тыс. руб., amount_year, аренда без деления.

#296 остаётся открытой: остальные пункты (1850 vs 1897/1887, depth=3 в get_form2_ahr_satellite_delta, перф-долг дельты, тесты 1906) этим PR не закрываются.

🤖 Generated with Claude Code

Часть follow-up #296 (раздел «Баг в тестах AURORA-1897/1906»), только `test_form2_ahr_limit.py`. **Проблема.** 6 из 7 тестов падали `KeyError: 'q1'` ещё на чтении состояния «до»: после AURORA-1848 GET листа вырезает null-поля, у заголовка статьи без плана ключа `plan.qN` нет. **Правка.** `["plan"]["qN"]` → `["plan"].get("qN")` (выражения уже были `... or 0`). Продуктовый код не трогается. **Проверка.** `pytest tests/integration/test_form2_ahr_limit.py --init_db` на чистой БД (Postgres 16, стенд perf): было 6 failed / 1 passed, стало **7 passed**. Тем самым подтверждено, что AURORA-1897 на test после PR #295 работает: лист «АХР Лимиты» отдаётся (200), годовые нормативы /4, суммы без НДС, план АХР в тыс. руб., `amount_year`, аренда без деления. #296 остаётся открытой: остальные пункты (1850 vs 1897/1887, depth=3 в `get_form2_ahr_satellite_delta`, перф-долг дельты, тесты 1906) этим PR не закрываются. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
aep added 1 commit 2026-11-08 15:04:45 +03:00 
После AURORA-1848 GET листа вырезает null-поля, поэтому у заголовка статьи
без плана в квартале ключа plan.qN нет вовсе, и чтение «до» падало
KeyError до проверки самой логики. Заменено на plan.get("qN") or 0.
Продуктовый код не меняется: с правкой все 7 тестов зелёные на test
(после PR #295).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aep merged commit 1965f2a939 into test 2026-11-08 17:10:07 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
auto-assigned
Бот автоматически назначил ревьюеров
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/DFiP_Budget_planing#299
No description provided.
Delete Branch "feat/296-ahr-limit-tests-null-strip"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?