- По данным реальных сделок, квартиры продаются в среднем на 10–18% дешевле,
- чем заявлено в объявлениях (Екатеринбург, 2026)
+ {deals_discount_text}
@@ -1558,7 +1641,7 @@ def _build_offer_page(estimate: AggregatedEstimate, brand) -> str: # type: igno
brand_short = _html.escape(brand.name)
trade_in_label = f"{brand_short}.Обмен" if brand.slug != "generic" else "Trade-In"
- report_num = _report_number(estimate.estimate_id)
+ report_num = _report_number(estimate)
today = dt.date.today()
footer_html = _page_footer(
brand,
@@ -2015,7 +2098,7 @@ def _build_insufficient_data_page(estimate: AggregatedEstimate, input_snapshot:
Предотвращает публикацию «0,0 млн» и fabricated таблиц потерь.
"""
address = _html.escape(input_snapshot.get("address", "—"))
- report_num = _report_number(estimate.estimate_id)
+ report_num = _report_number(estimate)
today = dt.date.today()
return f"""
diff --git a/tradein-mvp/backend/tests/services/test_trade_in_pdf_dual_price.py b/tradein-mvp/backend/tests/services/test_trade_in_pdf_dual_price.py
index ac78775a..2f405445 100644
--- a/tradein-mvp/backend/tests/services/test_trade_in_pdf_dual_price.py
+++ b/tradein-mvp/backend/tests/services/test_trade_in_pdf_dual_price.py
@@ -30,7 +30,7 @@ os.environ.setdefault("DATABASE_URL", "postgresql+psycopg://test:test@localhost:
_wp_mock = MagicMock()
sys.modules.setdefault("weasyprint", _wp_mock)
-from app.schemas.trade_in import AggregatedEstimate # noqa: E402
+from app.schemas.trade_in import AggregatedEstimate, AnalogLot # noqa: E402
from app.services.brand import Brand # noqa: E402
from app.services.exporters import trade_in_pdf as mod # noqa: E402
@@ -79,7 +79,16 @@ class _WellFormed(HTMLParser):
"""Minimal balanced-tag check: every non-void tag that opens must close."""
_VOID: ClassVar[set[str]] = {
- "br", "img", "hr", "meta", "input", "rect", "path", "circle", "line", "polyline",
+ "br",
+ "img",
+ "hr",
+ "meta",
+ "input",
+ "rect",
+ "path",
+ "circle",
+ "line",
+ "polyline",
}
def __init__(self) -> None:
@@ -235,3 +244,108 @@ def test_build_cover_without_sold_renders_only_asking() -> None:
# existing layout intact
assert "Диапазон цен в объявлениях" in html
assert "Диапазон цен по фактическим сделкам" in html
+
+
+# ── _days_on_market_range — no fabricated (4, 118) corridor (#pdf-honesty) ─────
+
+
+def _lot(days_on_market: int | None, price_rub: int = 10_000_000) -> AnalogLot:
+ return AnalogLot(
+ address="Екатеринбург, ул. Ленина, 1",
+ area_m2=50.0,
+ rooms=2,
+ floor=3,
+ total_floors=9,
+ price_rub=price_rub,
+ price_per_m2=int(price_rub / 50),
+ listing_date=None,
+ days_on_market=days_on_market,
+ )
+
+
+def test_days_range_none_when_no_lots() -> None:
+ # Раньше возвращал хардкод (4, 118) — выдуманный срок экспозиции.
+ assert mod._days_on_market_range([]) is None
+
+
+def test_days_range_none_when_all_days_missing() -> None:
+ assert mod._days_on_market_range([_lot(None), _lot(None)]) is None
+
+
+def test_days_range_min_max_when_data_present() -> None:
+ assert mod._days_on_market_range([_lot(12), _lot(None), _lot(88)]) == (12, 88)
+
+
+def test_cover_without_days_data_still_renders_range_chart() -> None:
+ # Нет days_on_market → show_days=False, но ценовой бар всё равно строится.
+ est = _estimate(analogs=[_lot(None)])
+ html = mod._build_cover(est, _SNAPSHOT, _BRAND)
+ _assert_well_formed(html)
+ assert "Диапазон цен в объявлениях" in html
+
+
+# ── _report_number — city-aware prefix, never hardcoded «EKБ» (#pdf-honesty) ───
+
+
+def test_report_number_ekb_prefix_for_ekb_address() -> None:
+ est = _estimate(target_address="Екатеринбург, ул. Ленина, 1")
+ assert mod._report_number(est).startswith("ЕКБ-")
+
+
+def test_report_number_city_prefix_for_non_ekb_address() -> None:
+ assert mod._report_number(_estimate(target_address="Нижний Тагил, ул. Мира, 5")).startswith(
+ "НТ-"
+ )
+ assert mod._report_number(_estimate(target_address="Серов, ул. Ленина, 3")).startswith("СЕР-")
+
+
+def test_report_number_neutral_prefix_when_city_unknown() -> None:
+ # None адрес и адрес вне региона → нейтральный «МЕРА», НЕ ложный «EKБ».
+ assert mod._report_number(_estimate(target_address=None)).startswith("МЕРА-")
+ assert mod._report_number(_estimate(target_address="Москва, ул. Тверская, 1")).startswith(
+ "МЕРА-"
+ )
+
+
+def test_report_number_never_hardcodes_old_ekb_literal() -> None:
+ # Старый баг: «EKБ-» (латинские EK + кириллическая Б) для ЛЮБОГО адреса.
+ for addr in (None, "Серов, ул. Ленина, 3", "Москва, ул. Тверская, 1"):
+ assert not mod._report_number(_estimate(target_address=addr)).startswith("EK")
+
+
+def test_report_number_falls_back_to_canonical_address() -> None:
+ est = _estimate(target_address=None, canonical_address="Первоуральск, ул. Вайнера, 2")
+ assert mod._report_number(est).startswith("ПРВ-")
+
+
+# ── Static «10–18%» haircut replaced by computed «−N%» (#pdf-honesty) ──────────
+
+
+def test_cover_advice_uses_computed_discount_not_static_range() -> None:
+ est = _estimate(expected_sold_price_rub=8_000_000, asking_to_sold_ratio=0.80) # −20%
+ html = mod._build_cover(est, _SNAPSHOT, _BRAND)
+ assert "10–18%" not in html
+ assert "на 20%" in html
+
+
+def test_cover_advice_neutral_when_no_discount() -> None:
+ html = mod._build_cover(_estimate(expected_sold_price_rub=None), _SNAPSHOT, _BRAND)
+ assert "10–18%" not in html
+ assert "проходят ниже цен в объявлениях" in html
+
+
+def test_deals_banner_uses_computed_discount_not_static_range() -> None:
+ est = _estimate(expected_sold_price_rub=8_000_000, asking_to_sold_ratio=0.80) # −20%
+ html = mod._build_deals_page(est, _SNAPSHOT, _BRAND)
+ _assert_well_formed(html)
+ assert "10–18%" not in html
+ # Ложный локейшн-хвост «(Екатеринбург, 2026)» удалён.
+ assert "Екатеринбург, 2026" not in html
+ assert "на 20%" in html
+
+
+def test_deals_banner_neutral_when_no_discount() -> None:
+ html = mod._build_deals_page(_estimate(expected_sold_price_rub=None), _SNAPSHOT, _BRAND)
+ _assert_well_formed(html)
+ assert "10–18%" not in html
+ assert "продаются дешевле, чем заявлено" in html
diff --git a/tradein-mvp/backend/tests/test_pdf_security.py b/tradein-mvp/backend/tests/test_pdf_security.py
index 0f04bfe7..7e05c92b 100644
--- a/tradein-mvp/backend/tests/test_pdf_security.py
+++ b/tradein-mvp/backend/tests/test_pdf_security.py
@@ -122,7 +122,10 @@ def test_insufficient_data_page_contains_address_and_report_num() -> None:
est = _zero_estimate()
html = mod._build_insufficient_data_page(est, _SNAPSHOT, _GENERIC)
assert "Ленина" in html # address
- assert "EKБ-" in html # report number prefix
+ # #pdf-honesty: № отчёта city-aware — у оценки без target_address город не распознан
+ # → нейтральный префикс «МЕРА» (раньше был захардкожен «EKБ» для любого объекта).
+ assert "МЕРА-" in html
+ assert "EKБ-" not in html # старый mixed-script хардкод убран
def test_generate_pdf_insufficient_skips_offer_page() -> None: