RSHB/arm-tabular-parser
9
0
Fork 0
Code Issues 8 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Follow-ups from #32 #36

New Issue
Closed
opened 2026-10-13 11:24:16 +03:00 by aep · 1 comment
aep commented 2026-10-13 11:24:16 +03:00
Owner
Copy Link

Отложенные находки задачи #32 (PR #33).

Маркер Находка Откуда
O-1 Интеграционный тест test_request_gets_ack_then_parser_error_and_duplicate_gets_only_ack флейкует («лишнее сообщение»), пока поднят локальный стенд: контейнер arm-ka-tabular-parser-1 подписан на тот же NATS localhost:4222 в той же queue group и забирает часть сообщений (2 из 3 прогонов зелёные). Нужна изоляция: свой subject/queue group в тесте или отдельный NATS make check-full при поднятом стенде, PR #33
O-2 rows[].extra дублирует значения объединённых ячеек: у ОСВ 60 Новопетровской строки несут extra C/G/K — имя из B:C и суммы из H:I/J:K (text_at отдаёт значение origin merged-диапазона). Поведение было до #32 разбор файла банка, PR #33
H-2 Все 8 раскладок «ИНН над/под × в дереве/нет × с суммами/без» по содержимому, включая дефект main «ИНН под, не в дереве, без сумм» → ИНН следующему контрагенту вынесено в #35 (сюда — для полноты)
Отложенные находки задачи #32 (PR #33). | Маркер | Находка | Откуда | |---|---|---| | O-1 | Интеграционный тест `test_request_gets_ack_then_parser_error_and_duplicate_gets_only_ack` флейкует («лишнее сообщение»), пока поднят локальный стенд: контейнер `arm-ka-tabular-parser-1` подписан на тот же NATS `localhost:4222` в той же queue group и забирает часть сообщений (2 из 3 прогонов зелёные). Нужна изоляция: свой subject/queue group в тесте или отдельный NATS | `make check-full` при поднятом стенде, PR #33 | | O-2 | `rows[].extra` дублирует значения объединённых ячеек: у ОСВ 60 Новопетровской строки несут `extra` C/G/K — имя из B:C и суммы из H:I/J:K (`text_at` отдаёт значение origin merged-диапазона). Поведение было до #32 | разбор файла банка, PR #33 | | H-2 | Все 8 раскладок «ИНН над/под × в дереве/нет × с суммами/без» по содержимому, включая дефект main «ИНН под, не в дереве, без сумм» → ИНН следующему контрагенту | вынесено в #35 (сюда — для полноты) |
aep commented 2026-10-25 19:17:14 +03:00
Author
Owner
Copy Link

Флейк снят (уникальные subject'ы в roundtrip), раскладки ИНН закрыты в #35. Дубль extra у объединённых ячеек — косметика. Закрываю.

Флейк снят (уникальные subject'ы в roundtrip), раскладки ИНН закрыты в #35. Дубль `extra` у объединённых ячеек — косметика. Закрываю.
aep closed this issue 2026-10-25 19:17:15 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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-tabular-parser#36
No description provided.
Delete Branch "%!s()"

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?