mirror of
https://github.com/home-assistant/core.git
synced 2025-11-09 10:59:40 +00:00
* 2nd attempt to add energysensors to Adax component * Ruff format changes * I did not reuse the first call for information.. Now i do.. * Fixed some tests after the last change * Remove extra attributes * Dont use info logger * aggregate if not rooms * Raise error if no rooms are discovered * Move code out of try catch * Catch more specific errors * removed platforms from manifest.json * remove attribute translation key * Getting rid of the summation of energy used.. * Fixed errorness in test * set roomproperty in Init * concatenated the two functions * use raw Wh values and suggest a konversion for HomeAssistant * Use snapshot testing * Update homeassistant/components/adax/coordinator.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/strings.json Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Removing un needed logg * Removing initial value * Changing tests to snapshot_platform * Removing available property from sensor.py and doing a ruff formating.. * Fix a broken indent * Add fix for coordinator updates in Adax energisensor and namesetting * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/coordinator.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/coordinator.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * generated snapshots * Ruff changes * Even more ruff changes, that did not appear on ruff command locally * Trying to fix CI updates * Update homeassistant/components/adax/sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> * Improve AdaxEnergySensor by simplifying code and ensuring correct handling of energy values. Adjust how room and device information is retrieved to avoid duplication and improve readability. * Removed a test för device_id as per request.. * Make supersure that value is int and not "Any" * removing executable status * Update tests/components/adax/test_sensor.py Co-authored-by: Josef Zweck <josef@zweck.dev> --------- Co-authored-by: Josef Zweck <josef@zweck.dev>
122 lines
4.0 KiB
Python
122 lines
4.0 KiB
Python
"""Test Adax sensor entity."""
|
|
|
|
from unittest.mock import AsyncMock, patch
|
|
|
|
from syrupy.assertion import SnapshotAssertion
|
|
|
|
from homeassistant.const import Platform
|
|
from homeassistant.core import HomeAssistant
|
|
from homeassistant.helpers import entity_registry as er
|
|
|
|
from . import setup_integration
|
|
|
|
from tests.common import MockConfigEntry, snapshot_platform
|
|
|
|
|
|
async def test_sensor_cloud(
|
|
hass: HomeAssistant,
|
|
mock_adax_cloud: AsyncMock,
|
|
mock_cloud_config_entry: MockConfigEntry,
|
|
snapshot: SnapshotAssertion,
|
|
entity_registry: er.EntityRegistry,
|
|
) -> None:
|
|
"""Test sensor setup for cloud connection."""
|
|
with patch("homeassistant.components.adax.PLATFORMS", [Platform.SENSOR]):
|
|
await setup_integration(hass, mock_cloud_config_entry)
|
|
# Now we use fetch_rooms_info as primary method
|
|
mock_adax_cloud.fetch_rooms_info.assert_called_once()
|
|
|
|
await snapshot_platform(
|
|
hass, entity_registry, snapshot, mock_cloud_config_entry.entry_id
|
|
)
|
|
|
|
|
|
async def test_sensor_local_not_created(
|
|
hass: HomeAssistant,
|
|
mock_adax_local: AsyncMock,
|
|
mock_local_config_entry: MockConfigEntry,
|
|
) -> None:
|
|
"""Test that sensors are not created for local connection."""
|
|
with patch("homeassistant.components.adax.PLATFORMS", [Platform.SENSOR]):
|
|
await setup_integration(hass, mock_local_config_entry)
|
|
|
|
# No sensor entities should be created for local connection
|
|
sensor_entities = hass.states.async_entity_ids("sensor")
|
|
adax_sensors = [e for e in sensor_entities if "adax" in e or "room" in e]
|
|
assert len(adax_sensors) == 0
|
|
|
|
|
|
async def test_multiple_devices_create_individual_sensors(
|
|
hass: HomeAssistant,
|
|
mock_adax_cloud: AsyncMock,
|
|
mock_cloud_config_entry: MockConfigEntry,
|
|
snapshot: SnapshotAssertion,
|
|
entity_registry: er.EntityRegistry,
|
|
) -> None:
|
|
"""Test that multiple devices create individual sensors."""
|
|
# Mock multiple devices for both fetch_rooms_info and get_rooms (fallback)
|
|
multiple_devices_data = [
|
|
{
|
|
"id": "1",
|
|
"homeId": "1",
|
|
"name": "Room 1",
|
|
"temperature": 15,
|
|
"targetTemperature": 20,
|
|
"heatingEnabled": True,
|
|
"energyWh": 1500,
|
|
},
|
|
{
|
|
"id": "2",
|
|
"homeId": "1",
|
|
"name": "Room 2",
|
|
"temperature": 18,
|
|
"targetTemperature": 22,
|
|
"heatingEnabled": True,
|
|
"energyWh": 2500,
|
|
},
|
|
]
|
|
|
|
mock_adax_cloud.fetch_rooms_info.return_value = multiple_devices_data
|
|
mock_adax_cloud.get_rooms.return_value = multiple_devices_data
|
|
|
|
with patch("homeassistant.components.adax.PLATFORMS", [Platform.SENSOR]):
|
|
await setup_integration(hass, mock_cloud_config_entry)
|
|
|
|
await snapshot_platform(
|
|
hass, entity_registry, snapshot, mock_cloud_config_entry.entry_id
|
|
)
|
|
|
|
|
|
async def test_fallback_to_get_rooms(
|
|
hass: HomeAssistant,
|
|
mock_adax_cloud: AsyncMock,
|
|
mock_cloud_config_entry: MockConfigEntry,
|
|
snapshot: SnapshotAssertion,
|
|
entity_registry: er.EntityRegistry,
|
|
) -> None:
|
|
"""Test fallback to get_rooms when fetch_rooms_info returns empty list."""
|
|
# Mock fetch_rooms_info to return empty list, get_rooms to return data
|
|
mock_adax_cloud.fetch_rooms_info.return_value = []
|
|
mock_adax_cloud.get_rooms.return_value = [
|
|
{
|
|
"id": "1",
|
|
"homeId": "1",
|
|
"name": "Room 1",
|
|
"temperature": 15,
|
|
"targetTemperature": 20,
|
|
"heatingEnabled": True,
|
|
"energyWh": 0, # No energy data from get_rooms
|
|
}
|
|
]
|
|
|
|
with patch("homeassistant.components.adax.PLATFORMS", [Platform.SENSOR]):
|
|
await setup_integration(hass, mock_cloud_config_entry)
|
|
|
|
# Should call both methods
|
|
mock_adax_cloud.fetch_rooms_info.assert_called_once()
|
|
mock_adax_cloud.get_rooms.assert_called_once()
|
|
|
|
await snapshot_platform(
|
|
hass, entity_registry, snapshot, mock_cloud_config_entry.entry_id
|
|
)
|