diff --git a/backend/tests/services/test_nspd_denorm.py b/backend/tests/services/test_nspd_denorm.py index 3420a761..ca194b10 100644 --- a/backend/tests/services/test_nspd_denorm.py +++ b/backend/tests/services/test_nspd_denorm.py @@ -304,12 +304,15 @@ def test_denorm_dump_empty_features() -> None: db = _make_mock_session() counts = denorm_dump(db, quarter_cad="66:41:0101001", features=[]) - assert counts == {"parcels": 0, "buildings": 0, "errors": 0} + assert counts == {"parcels": 0, "buildings": 0, "errors": 0, "skipped": 0} db.commit.assert_called_once() -def test_denorm_dump_no_cad_num_counted_as_error() -> None: - """Parcel без cad_num → denorm_parcel_feature returns False → errors += 1.""" +def test_denorm_dump_no_cad_num_counted_as_skipped() -> None: + """Parcel без cad_num → pre-check в denorm_dump → skipped += 1, не errors. + + denorm_parcel_feature не вызывается вовсе — пропуск штатный, не сбой UPSERT. + """ db = _make_mock_session() feature: dict[str, Any] = { "layer": "parcels", @@ -319,4 +322,5 @@ def test_denorm_dump_no_cad_num_counted_as_error() -> None: } counts = denorm_dump(db, quarter_cad="66:41:0101001", features=[feature]) assert counts["parcels"] == 0 - assert counts["errors"] == 1 + assert counts["errors"] == 0 + assert counts["skipped"] == 1