lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

fix(competitors): _ACTIVE_STATUSES = русские значения как у домрф (#1213) #1257

Merged
bot-backend merged 1 commit from fix/competitors-active-statuses-ru-values into main 2026-06-13 05:53:17 +00:00
Conversation 0 Commits 1 Files changed 2 +22 -11
2 changed files with 22 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
backend/app/services/site_finder/competitors.py
View file

@ -57,8 +57,14 @@ _TIME_WINDOW_MONTHS: dict[str, float] = {
"last_year": 12.0, "last_year": 12.0,
} }
# site_status значения, считающиеся «активными» # site_status значения, считающиеся «активными» (#1213). domrf_kn_objects.site_status
_ACTIVE_STATUSES = frozenset({"sales", "construction"}) # хранит русские значения ровно как scraper берёт из siteStatus дом.рф (см. domrf_kn.py:316
# — без перевода). Прод-аудит: 105_add_sales_started_flag.sql фильтрует по 'Строящиеся'
# (~1322 строки), partial index в 66_indexes_recommend.sql использует те же, фронтовые
# проверки MarketTab/CompetitorTable — тоже русские. Английские 'sales'/'construction'
# не совпадали никогда → у всех Competitor is_active=false и active_count=0 при любых
# данных. is_sold_out_filter сохранён как раньше — на стороне фронта.
_ACTIVE_STATUSES = frozenset({"Строящиеся"})
# #968 (949-A): радиус спатиального матча domrf↔complexes для velocity gap-fill. # #968 (949-A): радиус спатиального матча domrf↔complexes для velocity gap-fill.
# 200м + tolerant-name + nearest-per-domrf → подтверждаем «тот же ЖК» И гео, И # 200м + tolerant-name + nearest-per-domrf → подтверждаем «тот же ЖК» И гео, И

23
backend/tests/api/v1/test_parcel_competitors.py
View file

@ -26,7 +26,7 @@ def _coord_row(lat: float = 56.838, lon: float = 60.605) -> MagicMock:
def _obj_row( def _obj_row(
obj_id: int = 1, obj_id: int = 1,
distance_m: float = 400.0, distance_m: float = 400.0,
site_status: str = "sales", site_status: str = "Строящиеся",
obj_class: str | None = "comfort", obj_class: str | None = "comfort",
velocity: float = 5.0, velocity: float = 5.0,
flat_count: int | None = 200, flat_count: int | None = 200,
@ -148,11 +148,15 @@ def test_competitors_basic() -> None:
def test_competitors_summary_calc() -> None: def test_competitors_summary_calc() -> None:
"""summary: total_competitors, active_count, weighted_avg_velocity корректны.""" """summary: total_competitors, active_count, weighted_avg_velocity корректны.
site_status — реальные русские значения как у домрф (#1213 fix).
«Строящиеся» считаются активными, «Сданные» — нет.
"""
rows = [ rows = [
_obj_row(obj_id=1, site_status="sales", velocity=10.0), _obj_row(obj_id=1, site_status="Строящиеся", velocity=10.0),
_obj_row(obj_id=2, site_status="construction", velocity=6.0), _obj_row(obj_id=2, site_status="Строящиеся", velocity=6.0),
_obj_row(obj_id=3, site_status="completed", velocity=2.0), _obj_row(obj_id=3, site_status="Сданные", velocity=2.0),
] ]
db = _make_db(coord=_coord_row(), obj_rows=rows) db = _make_db(coord=_coord_row(), obj_rows=rows)
@ -391,11 +395,12 @@ def test_competitors_avg_price_populated() -> None:
def test_competitors_is_active_flag() -> None: def test_competitors_is_active_flag() -> None:
"""is_active=True для sales/construction, False для completed/null.""" """is_active=True для русского 'Строящиеся' (реальная prod-форма #1213),
False для 'Сданные'/null/прочих."""
rows = [ rows = [
_obj_row(obj_id=1, site_status="sales"), _obj_row(obj_id=1, site_status="Строящиеся"),
_obj_row(obj_id=2, site_status="construction"), _obj_row(obj_id=2, site_status="Строящиеся"),
_obj_row(obj_id=3, site_status="completed"), _obj_row(obj_id=3, site_status="Сданные"),
] ]
db = _make_db(coord=_coord_row(), obj_rows=rows) db = _make_db(coord=_coord_row(), obj_rows=rows)