From e76a8dcaefce5234b04086183104c9ef5dabe7a3 Mon Sep 17 00:00:00 2001
From: bot-backend <bot-backend@gendsgn.local>
Date: Tue, 16 Jun 2026 10:24:20 +0300
Subject: [PATCH] =?UTF-8?q?fix(tradein):=20review=20minors=20=E2=80=94=20c?=
 =?UTF-8?q?eiling=5Fheight=5Fm=20numeric(5,2)=20+=20strengthen=20finishing?=
 =?UTF-8?q?=20test?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Migration 111: ceiling_height_m numeric → numeric(5,2) (schema consistency with living_area_m2/kitchen_area_m2)
- test_finishing_stored_lowercase_stripped: fixture now uses padded uppercase '  Чистовая  ' to actually exercise .strip().lower(), asserts finishing == 'чистовая'
---
 .../backend/data/sql/111_listings_avito_detail_fields.sql     | 2 +-
 tradein-mvp/backend/tests/test_avito_detail_parse.py          | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tradein-mvp/backend/data/sql/111_listings_avito_detail_fields.sql b/tradein-mvp/backend/data/sql/111_listings_avito_detail_fields.sql
index 03a0427d..536ec406 100644
--- a/tradein-mvp/backend/data/sql/111_listings_avito_detail_fields.sql
+++ b/tradein-mvp/backend/data/sql/111_listings_avito_detail_fields.sql
@@ -11,7 +11,7 @@
 
 BEGIN;
 
-ALTER TABLE listings ADD COLUMN IF NOT EXISTS ceiling_height_m numeric;
+ALTER TABLE listings ADD COLUMN IF NOT EXISTS ceiling_height_m numeric(5,2);
 ALTER TABLE listings ADD COLUMN IF NOT EXISTS finishing text;
 
 COMMIT;
diff --git a/tradein-mvp/backend/tests/test_avito_detail_parse.py b/tradein-mvp/backend/tests/test_avito_detail_parse.py
index f8cc9988..e9e419fb 100644
--- a/tradein-mvp/backend/tests/test_avito_detail_parse.py
+++ b/tradein-mvp/backend/tests/test_avito_detail_parse.py
@@ -303,12 +303,12 @@ def test_finishing_stored_lowercase_stripped() -> None:
     <div data-marker="item-view/item-id">№ 8888888888</div>
     <span itemprop="price" content="5000000">5 000 000 ₽</span>
     <div data-marker="item-view/item-params">
-      <ul><li>Отделка: под ключ</li></ul>
+      <ul><li>Отделка:   Чистовая  </li></ul>
     </div>
     </body></html>
     """
     result = parse_detail_html(html, "https://www.avito.ru/test_8888888888")
-    assert result.finishing == "под ключ"
+    assert result.finishing == "чистовая"
 
 
 # ── _parse_height_m unit tests ────────────────────────────────────────────────