fix(ptica): фильтр класса в velocity ссылался на алиас, которого нет в CTE (#2464-G) #2865

Merged
bot-backend merged 4 commits from fix/2464-velocity-class-filter into main 2026-08-13 17:46:29 +00:00

4 commits

Author SHA1 Message Date
c9c7aab228 test(ptica): объявить пропуск нового EXPLAIN-теста в skip_allowlist
All checks were successful
CI Trade-In / changes (pull_request) Successful in 9s
CI / changes (pull_request) Successful in 10s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m30s
CI / backend-tests (pull_request) Successful in 16m41s
Причина четырёх красных прогонов backend-tests найдена, и это не инфраструктура:
сторож в tests/conftest.py (pytest_sessionfinish) роняет прогон, если тест
пропустился, но не объявлен в skip_allowlist.txt. Мои два новых
параметризованных EXPLAIN-теста скипаются без TEST_DATABASE_URL и в списке
не значились → session.exitstatus = 1 при зелёной сводке «4647 passed».

Сторож сработал ПРАВИЛЬНО и ровно за тем, зачем написан: пропуск без записи
неотличим от пройденной проверки. Его сообщение всё это время лежало в логе
на 1014-й строке, за две секунды до сводки, — я его не увидел, потому что
искал по английским «FAILED/ERROR» и смотрел хвост лога.

Почему проверку нельзя выполнить здесь (как требует шапка файла): EXPLAIN идёт
против КОПИИ ПРОДОВОЙ схемы через SSH-туннель, CI намеренно не задаёт
TEST_DATABASE_URL (комментарий в ci.yml у backend-tests). Где выполняется
вместо этого: вручную с туннелем — прогнан в этой ветке, 4 passed, и со
строкой из main падает 1 из 3.

Плюс backend/.gitignore: coverage.xml. Локальный прогон с --cov-report=xml
кладёт 1.2 МБ рядом с кодом; в этом заходе файл чуть не уехал в коммит и был
пойман только хуком check-added-large-files.

Refs #2464, #2871
2026-08-13 22:28:15 +05:00
8aacd1385e Merge branch 'main' into fix/2464-velocity-class-filter
Some checks failed
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 8s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m6s
CI / backend-tests (pull_request) Failing after 16m14s
2026-08-13 16:53:51 +00:00
ef6813d670 Merge branch 'main' into fix/2464-velocity-class-filter
Some checks failed
CI / changes (pull_request) Successful in 9s
CI Trade-In / changes (pull_request) Successful in 10s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m12s
CI / backend-tests (pull_request) Failing after 16m25s
2026-08-13 12:39:56 +00:00
e7d112b028 fix(ptica): фильтр класса в velocity ссылался на алиас, которого нет в CTE
Some checks failed
CI Trade-In / changes (pull_request) Successful in 9s
CI / changes (pull_request) Successful in 9s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m18s
CI / backend-tests (pull_request) Failing after 16m26s
class_filter подставляется ВНУТРЬ latest_obj, где FROM — голый
domrf_kn_objects, а алиас `o` появляется только во внешнем SELECT.
Прод-EXPLAIN 13.08: `missing FROM-clause entry for table "o"`.

Ветка мёртвая — единственный вызывающий (analyze_parcel) obj_class не
передаёт, поэтому в проде это не стреляло. Стрельнуло бы тихо: исключение
глотает except в compute_velocity, функция возвращает None, и блок
velocity просто исчезает из отчёта с одной строкой в логе.

SQL вынесен в модульную константу _COMPETITORS_SQL_TMPL — чтобы
integration-тест мог прогнать EXPLAIN по ОБЕИМ подстановкам, а не только
по той, что сегодня исполняется.

Один хунк в тесте — не мой: pre-commit ruff v0.7.4 против 0.15.12 в venv (#2864).

Refs #2464
2026-08-13 16:44:07 +05:00