junte/Model
1
0
Fork 0
mirror of https://github.com/Hestia-Homes/Model.git synced 2026-06-30 13:10:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor(fuel-rates): name the adapter aggregate-first per house convention

Browse source 
PR feedback: adapters here are <aggregate>_<backend>_repository (e.g.
property_baseline_postgres_repository). Rename the fuel-rates adapter to
match — file static_file_fuel_rates_repository.py ->
fuel_rates_static_file_repository.py and class StaticFileFuelRatesRepository
-> FuelRatesStaticFileRepository, plus its test. git mv preserves history.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Khalim Conn-Kowlessar 2026-06-02 14:05:57 +00:00
parent b24e4d46e4
commit c431453d75
2 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
repositories/fuel_rates/static_file_fuel_rates_repository.py → repositories/fuel_rates/fuel_rates_static_file_repository.py
View file

@ -11,7 +11,7 @@ from repositories.fuel_rates.fuel_rates_repository import FuelRatesRepository
_DEFAULT_SNAPSHOT = Path(__file__).parent / "data" / "fuel_rates_2026_q2.json" _DEFAULT_SNAPSHOT = Path(__file__).parent / "data" / "fuel_rates_2026_q2.json"
class StaticFileFuelRatesRepository(FuelRatesRepository): class FuelRatesStaticFileRepository(FuelRatesRepository):
"""Reads Fuel Rates from a committed JSON snapshot (ADR-0014). """Reads Fuel Rates from a committed JSON snapshot (ADR-0014).
Only **single-rate** fuels (those lodging a ``unit_rate_p_per_kwh``) are Only **single-rate** fuels (those lodging a ``unit_rate_p_per_kwh``) are

10
tests/repositories/fuel_rates/test_static_file_fuel_rates_repository.py → tests/repositories/fuel_rates/test_fuel_rates_static_file_repository.py
View file

@ -3,14 +3,14 @@ from __future__ import annotations
import pytest import pytest
from domain.fuel_rates.fuel import Fuel, UnpricedFuel from domain.fuel_rates.fuel import Fuel, UnpricedFuel
from repositories.fuel_rates.static_file_fuel_rates_repository import ( from repositories.fuel_rates.fuel_rates_static_file_repository import (
StaticFileFuelRatesRepository, FuelRatesStaticFileRepository,
) )
def test_get_current_loads_the_committed_snapshot_mains_gas_rate() -> None: def test_get_current_loads_the_committed_snapshot_mains_gas_rate() -> None:
# Arrange # Arrange
repository = StaticFileFuelRatesRepository() repository = FuelRatesStaticFileRepository()
# Act # Act
rates = repository.get_current() rates = repository.get_current()
@ -21,7 +21,7 @@ def test_get_current_loads_the_committed_snapshot_mains_gas_rate() -> None:
def test_snapshot_prices_metered_and_delivered_fuels_plus_seg() -> None: def test_snapshot_prices_metered_and_delivered_fuels_plus_seg() -> None:
# Arrange # Arrange
rates = StaticFileFuelRatesRepository().get_current() rates = FuelRatesStaticFileRepository().get_current()
# Act / Assert — electricity carries a daily standing charge; oil is # Act / Assert — electricity carries a daily standing charge; oil is
# delivered (no meter) so its standing charge is 0; SEG is a flat credit. # delivered (no meter) so its standing charge is 0; SEG is a flat credit.
@ -38,7 +38,7 @@ def test_snapshot_prices_metered_and_delivered_fuels_plus_seg() -> None:
def test_unpriced_fuels_raise_rather_than_defaulting(fuel: Fuel) -> None: def test_unpriced_fuels_raise_rather_than_defaulting(fuel: Fuel) -> None:
# Arrange — house coal + heat network have no national rate, and off-peak # Arrange — house coal + heat network have no national rate, and off-peak
# needs the day/night split a later slice adds (ADR-0014). # needs the day/night split a later slice adds (ADR-0014).
rates = StaticFileFuelRatesRepository().get_current() rates = FuelRatesStaticFileRepository().get_current()
# Act / Assert # Act / Assert
with pytest.raises(UnpricedFuel): with pytest.raises(UnpricedFuel):