RSHB/arm-core
9
0
Fork 0
Code Issues 26 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Issue #211: golden-тест учитывает D-7 для синтетики s48/49/50/53 #212

Merged
aep merged 1 commits from fix/211-golden-dup-check into main 2026-11-02 19:33:43 +03:00
Conversation 0 Commits 1 Files Changed 1 +13 -5
aep commented 2026-11-02 19:33:20 +03:00
Owner
Copy Link

Closes #211.

test_all_golden_snapshots_mapped сравнивал созданные документы 1:1 с листами снимка. Синтетика s48_other_system_titles, s49_balance_and_turnover_by_side, s50_sap_in_out_glued_side, s53_balance_by_side (tp#81, AURORA-1717/1732) намеренно кладёт несколько листов под один счёт/период в одном файле — проверяет распознавание разных названий ведомости разными системами. По D-7 (services/result_mapping._split_duplicates) core оставляет документом только первый лист, остальные — предупреждением у него же. Тест этого не учитывал и падал и на main, и в ветке #208 (не связано с ревью корректировок).

Фикс — ожидаемый набор локаторов теперь считается той же _split_duplicates, что и продакшен-код, а не 1:1 по листам снимка.

make lint зелёный, test_mapping_golden.py и смежные (267 тестов: result_mapping/mapping_statements/mapping_golden/intercompany_matrix_golden) зелёные.

🤖 Generated with Claude Code

Closes #211. `test_all_golden_snapshots_mapped` сравнивал созданные документы 1:1 с листами снимка. Синтетика `s48_other_system_titles`, `s49_balance_and_turnover_by_side`, `s50_sap_in_out_glued_side`, `s53_balance_by_side` (tp#81, AURORA-1717/1732) намеренно кладёт несколько листов под один счёт/период в одном файле — проверяет распознавание разных названий ведомости разными системами. По D-7 (`services/result_mapping._split_duplicates`) core оставляет документом только первый лист, остальные — предупреждением у него же. Тест этого не учитывал и падал и на main, и в ветке #208 (не связано с ревью корректировок). Фикс — ожидаемый набор локаторов теперь считается той же `_split_duplicates`, что и продакшен-код, а не 1:1 по листам снимка. `make lint` зелёный, `test_mapping_golden.py` и смежные (267 тестов: result_mapping/mapping_statements/mapping_golden/intercompany_matrix_golden) зелёные. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
aep added 1 commit 2026-11-02 19:33:22 +03:00 
test_all_golden_snapshots_mapped сравнивал документы 1:1 с листами
снимка. Синтетика s48_other_system_titles, s49_balance_and_turnover_by_side,
s50_sap_in_out_glued_side, s53_balance_by_side (tp#81, AURORA-1717/1732)
намеренно кладёт несколько листов под один счёт/период в одном файле,
проверяя распознавание разных названий ведомости — по D-7
(services/result_mapping._split_duplicates) core оставляет только
первый лист документом, остальные — предупреждением. Ожидаемый набор
локаторов теперь считается той же _split_duplicates, что и продакшен-код.
aep merged commit fcd57311bd into main 2026-11-02 19:33:43 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
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/arm-core#212
No description provided.
Delete Branch "fix/211-golden-dup-check"

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?