Follow-ups from #38 #40

New Issue

aep · 2026-11-13T18:54:50+03:00

aep commented

2026-11-13 18:54:50 +03:00

Отложенное из #38 (PR #39, merge cc1bc59) — генератор synthgen/. Маркер — откуда взялось: Tn.k — ревью задачи n плана, M-n — финальное ревью ветки, O-n — наблюдение контроллера.

Сверка с реальностью

O-6 — подпись колонки «За 3 месяца» в ББ/ОФР за 1 квартал не сверена с реальной выгрузкой 1С; строки 2340/2350 всегда нулевые (наблюдение контроллера).
O-3 — в ОСВ 58 есть внешние контрагенты (займы 58.03), 76.05 только по кредиту (наблюдение, Task 3).
M-2 — 90.01–90.08 закрываются ежемесячно, ОСВ 90 выглядит иначе, чем у компаний, закрывающих 90 в конце года (финальное ревью).
T2.8 — у ~15 ИП словаря фамилия совпадает с реальными людьми, инициалы другие (ревью Task 2).
T6.3 — налог 2410 в масштабированных сравнительных колонках не зависит от прибыли (ревью Task 6).

Код и устойчивость

M-3 — префиксы, различающиеся только регистром, дают разные ИНН, но похожие имена (финальное ревью).
M-4 — абсолютные пути автора в скриптах сборки шаблонов и тестах (финальное ревью).
T5.4 — TP_WT по умолчанию указывает в недостижимое место, приёмка парсером без переменной пропускается (ревью Task 5).
T6.5 — Ledger.post не проверяет kind; ruff по пакету не прогонялся (ревью Task 6).
T7.4 — крупный прогон 198 с при лимите 300 — мало запаса (ревью Task 7).
T7.5 — README добавляется в manifest.files вручную (ревью Task 7).
T4.10 — счёт 80 у головной без аналитики по участникам (описано в README набора, ревью Task 4).
T1.1 — dump_config пишет Decimal как float (ревью Task 1).

Тесты

T1.2 — абсолютный путь к seed плана счетов в test_ledger (ревью Task 1).
T1.3 — мало негативных тестов конфигурации (ревью Task 1).
T2.7 — test_variant_name проверяет тривиальное условие (ревью Task 2).
T3.3–T3.6 — частично тавтологичный test_books_balanced, общий seen в test_externals_used, доли 3/20/5 % без тестов, веса во float до Decimal (ревью Task 3).
T4.5 — test_dividends_no_balance привязан к 2025 Q2 (ревью Task 4).
T4.8, T4.9 — слабый тест расхождения займа; условие «< 2 × транша» смотрит на момент проводки (ревью Task 4).
T5.5, T5.6–T5.8 — ИНН внешних контрагентов не сверяются с выходом парсера; проверка 10-значных чисел, _DIRECTORY, _natural_key (ревью Task 5).
T6.2 — слабый допуск в test_net_profit_equals_99_balance (ре��ью Task 6).
T7.2 — поиск расхождений в README по подстроке (ревью Task 7).
T7.3 — граничные конфигурации проверяются только на отсутствие ошибок (ревью Task 7).

Отложенное из #38 (PR #39, merge cc1bc59) — генератор `synthgen/`. Маркер — откуда взялось: Tn.k — ревью задачи n плана, M-n — финальное ревью ветки, O-n — наблюдение контроллера. **Сверка с реальностью** - O-6 — подпись колонки «За 3 месяца» в ББ/ОФР за 1 квартал не сверена с реальной выгрузкой 1С; строки 2340/2350 всегда нулевые (наблюдение контроллера). - O-3 — в ОСВ 58 есть внешние контрагенты (займы 58.03), 76.05 только по кредиту (наблюдение, Task 3). - M-2 — 90.01–90.08 закрываются ежемесячно, ОСВ 90 выглядит иначе, чем у компаний, закрывающих 90 в конце года (финальное ревью). - T2.8 — у ~15 ИП словаря фамилия совпадает с реальными людьми, инициалы другие (ревью Task 2). - T6.3 — налог 2410 в масштабированных сравнительных колонках не зависит от прибыли (ревью Task 6). **Код и устойчивость** - M-3 — префиксы, различающиеся только регистром, дают разные ИНН, но похожие имена (финальное ревью). - M-4 — абсолютные пути автора в скриптах сборки шаблонов и тестах (финальное ревью). - T5.4 — `TP_WT` по умолчанию указывает в недостижимое место, приёмка парсером без переменной пропускается (ревью Task 5). - T6.5 — `Ledger.post` не проверяет `kind`; ruff по пакету не прогонялся (ревью Task 6). - T7.4 — крупный прогон 198 с при лимите 300 — мало запаса (ревью Task 7). - T7.5 — README добавляется в `manifest.files` вручную (ревью Task 7). - T4.10 — счёт 80 у головной без аналитики по участникам (описано в README набора, ревью Task 4). - T1.1 — `dump_config` пишет Decimal как float (ревью Task 1). **Тесты** - T1.2 — абсолютный путь к seed плана счетов в test_ledger (ревью Task 1). - T1.3 — мало негативных тестов конфигурации (ревью Task 1). - T2.7 — test_variant_name проверяет тривиальное условие (ревью Task 2). - T3.3–T3.6 — частично тавтологичный test_books_balanced, общий `seen` в test_externals_used, доли 3/20/5 % без тестов, веса во float до Decimal (ревью Task 3). - T4.5 — test_dividends_no_balance привязан к 2025 Q2 (ревью Task 4). - T4.8, T4.9 — слабый тест расхождения займа; условие «< 2 × транша» смотрит на момент проводки (ревью Task 4). - T5.5, T5.6–T5.8 — ИНН внешних контрагентов не сверяются с выходом парсера; проверка 10-значных чисел, `_DIRECTORY`, `_natural_key` (ревью Task 5). - T6.2 — слабый допуск в test_net_profit_equals_99_balance (ре��ью Task 6). - T7.2 — поиск расхождений в README по подстроке (ревью Task 7). - T7.3 — граничные конфигурации проверяются только на отсутствие ошибок (ревью Task 7).

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RSHB/arm-ka-fixtures#40