Follow-ups from #190 #231

New Issue

aep · 2026-11-07T07:58:11+03:00

aep commented

2026-11-07 07:58:11 +03:00

Отложенные внутренние находки ревью метрик качества (PR #229, AURORA-1759). Дефекты продукта заведены в Jira (эпик AURORA-1353, связаны с AURORA-1759).

Тесты:

T1.1 — тест test_orm_matches_dbml_for_all_43_tables называется «43», покрывает 48 таблиц (задача 1).
T2.1 — нет теста, что поля шапки из Document сохраняются рядом с header_source (задача 2).
T2.2 — нет теста пути «header_source опущен» через build_outcome_payload (задача 2).
T3.1 — тест кода использует 9999/9998, которые parse_code не распознаёт (задача 3).
T4.2 — из семи server-default колонок фактов явная ассерция только у одной (задача 4).
T8.1 — хелперы _outcome_fact / _outcome_facts почти дубли (задача 8).
T11.1 — CommittedJobs.cleanup не учитывает raw_s3_object_id (задача 11).
T12.1 — _digital_pdf_bytes() дублируется в двух тестовых файлах (задача 12).
M-9 — тест лимита страниц has_text_layer проверяет только 0, игнор страницы 11 не доказывает.
T16.8 — тест меню проверяет общую функцию, а не реальный config Sidebar.

Код:

T3.2 — лишний abs() в атрибуции (безвредно).
T3.4 — docstring «единственный вызов arm_statements» в statements_engine.py / stored_form.py устарел.
T4.1 — **columns без аннотации : Any.
T5.3 — молчаливый пропуск факта при owner is None (сейчас недостижимо) — хотя бы лог.
T5.4 — N+1 запросов на документ в транзакции маппинга.
T9.2 — хук факта вшит в supersede_previous, не вынесен в функцию.
T10.2 — _FailureDimensions дублируется в recognition_exchange.py и result_mapping.py.
T16.12 — лишний эффект сброса страницы в DocumentDrillList; MonthlyTrend перезапрашивает на скрытой вкладке.

Отложенные внутренние находки ревью метрик качества (PR #229, AURORA-1759). Дефекты продукта заведены в Jira (эпик AURORA-1353, связаны с AURORA-1759). Тесты: - T1.1 — тест `test_orm_matches_dbml_for_all_43_tables` называется «43», покрывает 48 таблиц (задача 1). - T2.1 — нет теста, что поля шапки из Document сохраняются рядом с header_source (задача 2). - T2.2 — нет теста пути «header_source опущен» через build_outcome_payload (задача 2). - T3.1 — тест кода использует 9999/9998, которые parse_code не распознаёт (задача 3). - T4.2 — из семи server-default колонок фактов явная ассерция только у одной (задача 4). - T8.1 — хелперы `_outcome_fact` / `_outcome_facts` почти дубли (задача 8). - T11.1 — `CommittedJobs.cleanup` не учитывает `raw_s3_object_id` (задача 11). - T12.1 — `_digital_pdf_bytes()` дублируется в двух тестовых файлах (задача 12). - M-9 — тест лимита страниц `has_text_layer` проверяет только 0, игнор страницы 11 не доказывает. - T16.8 — тест меню проверяет общую функцию, а не реальный config Sidebar. Код: - T3.2 — лишний `abs()` в атрибуции (безвредно). - T3.4 — docstring «единственный вызов arm_statements» в `statements_engine.py` / `stored_form.py` устарел. - T4.1 — `**columns` без аннотации `: Any`. - T5.3 — молчаливый пропуск факта при `owner is None` (сейчас недостижимо) — хотя бы лог. - T5.4 — N+1 запросов на документ в транзакции маппинга. - T9.2 — хук факта вшит в `supersede_previous`, не вынесен в функцию. - T10.2 — `_FailureDimensions` дублируется в `recognition_exchange.py` и `result_mapping.py`. - T16.12 — лишний эффект сброса страницы в DocumentDrillList; MonthlyTrend перезапрашивает на скрытой вкладке.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RSHB/arm-core#231