bot-backend
afa648b21f
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
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 2m26s
CI / backend-tests (pull_request) Successful in 17m14s
create_profile/update_profile делают «снять is_default у всех → поставить новому»
двумя отдельными операторами. Между ними инвариант нарушен, и при одновременных
запросах у пользователя может оказаться ДВА профиля с is_default=TRUE. А читающий
_SELECT_DEFAULT брал LIMIT 1 БЕЗ ORDER BY — выбор молча перескакивал между ними от
запроса к запросу.
Два рубежа, а не один:
миграция 190 — частичный уникальный индекс (user_id) WHERE is_default: два
дефолта становятся невозможными на уровне БД;
ORDER BY id — детерминированный выбор, если индекс когда-нибудь снимут.
Соседние запросы этого файла тай-брейк по id уже имеют.
Индекс не мешает штатной переустановке дефолта: порядок операторов в коде уже
правильный (сначала снять у всех, потом поставить), поэтому в момент проверки
дефолтов ноль. Это отдельно проверено тестом.
Безопасность миграции: на проде нарушений нет — у admin один дефолт, у __system__
ноль, ни одного пользователя с двумя. Таблица в 4 строки, индексируется мгновенно.
lock_timeout проставлен по #2752.
Тест проверяет ПОВЕДЕНИЕ на живом Postgres: вторая установка дефолта отвергается
базой. Плюс фальсификация — без индекса два дефолта вставляются молча; без неё
зелёный тест неотличим от «оно и так не вставлялось». Плюс два контроля:
переустановка дефолта работает, разные пользователи сохраняют свои.
Тест про ORDER BY вынесен в tests/services/site_finder, а НЕ внесён в
skip_allowlist: живой БД он не требует, и пропускаться вместе с DB-тестами ему
незачем. Против origin/main он краснеет, показывая запрос без тай-брейка.
Прогоны: без БД — 648 passed rc=0; с БД — 5 passed rc=0.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
||
|---|---|---|
| .. | ||
| test_2464_capacity_savepoints.py | ||
| test_2464_default_profile_order.py | ||
| test_2464_heat_loader_tx.py | ||
| test_2464_ird_lookups_savepoint.py | ||
| test_2464a_savepoint_regressions.py | ||
| test_best_layouts.py | ||
| test_competitor_relevance.py | ||
| test_competitors_avg_price_sql.py | ||
| test_competitors_parking.py | ||
| test_competitors_sold_count_sql.py | ||
| test_connection_capacity_savepoint.py | ||
| test_developer_attribution.py | ||
| test_district_resolver.py | ||
| test_future_supply.py | ||
| test_locations.py | ||
| test_macro.py | ||
| test_market_metrics.py | ||
| test_okn_lookup.py | ||
| test_own_portfolio.py | ||
| test_parcel_financial.py | ||
| test_pat_subzones.py | ||
| test_permits_nearby.py | ||
| test_premises_lookup.py | ||
| test_program_optimizer.py | ||
| test_quarter_dump_savepoint.py | ||
| test_riasurt_lookup.py | ||
| test_supply_layers.py | ||