Map secondary_heating on full-SAP certs for the calculator 🟥

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Khalim Conn-Kowlessar 2026-06-25 20:40:30 +00:00
parent bc394ce5f5
commit 7b06d9dbab

View file

@ -116,6 +116,17 @@ class TestFromSapSchema17_1RebaselineFields:
assert result.built_form == "1"
def test_maps_secondary_heating_element(self) -> None:
# secondary_heating is read by the calculator (cert_to_inputs) — dropping
# it under-counts a real secondary heater. The fixture lodges "None" (no
# secondary); assert it's carried through rather than silently dropped.
schema = from_dict(SapSchema17_1, load("sap_17_1.json"))
result = EpcPropertyDataMapper.from_sap_schema_17_1(schema)
assert result.secondary_heating is not None
assert result.secondary_heating.description == "None"
class TestFullSapHasHotWaterCylinderFallback:
"""Some full-SAP certs (e.g. SAP-Schema-17.0 cert 8265-7433-3220-9736-7902)