From 3cd0bce21ac3a67ccedbf9baa2f2dd4f2af6560e Mon Sep 17 00:00:00 2001 From: darkrain-nl <24763370+darkrain-nl@users.noreply.github.com> Date: Wed, 26 Aug 2026 17:58:18 +0200 Subject: [PATCH] Remove the Sofar system time sensor (#180302) --- homeassistant/components/sofar/icons.json | 3 -- homeassistant/components/sofar/sensor.py | 7 --- homeassistant/components/sofar/strings.json | 3 -- .../sofar/snapshots/test_sensor.ambr | 50 ------------------- tests/components/sofar/test_sensor.py | 4 +- 5 files changed, 2 insertions(+), 65 deletions(-) diff --git a/homeassistant/components/sofar/icons.json b/homeassistant/components/sofar/icons.json index 77facf4fea5b..1c22d93641ac 100644 --- a/homeassistant/components/sofar/icons.json +++ b/homeassistant/components/sofar/icons.json @@ -42,9 +42,6 @@ }, "battery_state_of_health_total": { "default": "mdi:battery-heart" - }, - "rtc": { - "default": "mdi:clock" } } } diff --git a/homeassistant/components/sofar/sensor.py b/homeassistant/components/sofar/sensor.py index 8e057075c361..b3effe0f2859 100644 --- a/homeassistant/components/sofar/sensor.py +++ b/homeassistant/components/sofar/sensor.py @@ -227,13 +227,6 @@ SENSOR_DESCRIPTIONS: tuple[SofarSensorDescription, ...] = ( state_class=SensorStateClass.MEASUREMENT, entity_category=EntityCategory.DIAGNOSTIC, ), - SofarSensorDescription( - key="rtc", - component="identity", - translation_key="rtc", - entity_category=EntityCategory.DIAGNOSTIC, - entity_registry_enabled_default=False, - ), SofarSensorDescription( key="serial_number", component="identity", diff --git a/homeassistant/components/sofar/strings.json b/homeassistant/components/sofar/strings.json index 31ef4c6a41d9..a685637117b6 100644 --- a/homeassistant/components/sofar/strings.json +++ b/homeassistant/components/sofar/strings.json @@ -640,9 +640,6 @@ "reactive_power_pcc_total": { "name": "Reactive power PCC total" }, - "rtc": { - "name": "System time" - }, "serial_number": { "name": "Serial number" }, diff --git a/tests/components/sofar/snapshots/test_sensor.ambr b/tests/components/sofar/snapshots/test_sensor.ambr index 510313d0abf0..e2b57dd19590 100644 --- a/tests/components/sofar/snapshots/test_sensor.ambr +++ b/tests/components/sofar/snapshots/test_sensor.ambr @@ -9593,56 +9593,6 @@ 'state': 'grid_connected', }) # --- -# name: test_all_entities[sensor.hydxxktl_3p_system_time-entry] - EntityRegistryEntrySnapshot({ - 'aliases': list([ - None, - ]), - 'area_id': None, - 'capabilities': None, - 'config_entry_id': , - 'config_subentry_id': , - 'device_class': None, - 'device_id': , - 'disabled_by': None, - 'domain': 'sensor', - 'entity_category': , - 'entity_id': 'sensor.hydxxktl_3p_system_time', - 'has_entity_name': True, - 'hidden_by': None, - 'icon': None, - 'id': , - 'labels': set({ - }), - 'name': None, - 'object_id_base': 'System time', - 'options': dict({ - }), - 'original_device_class': None, - 'original_icon': None, - 'original_name': 'System time', - 'platform': 'sofar', - 'previous_unique_id': None, - 'suggested_object_id': None, - 'supported_features': 0, - 'translation_key': 'rtc', - 'unique_id': 'SP1XXES100XX_rtc', - 'unit_of_measurement': None, - }) -# --- -# name: test_all_entities[sensor.hydxxktl_3p_system_time-state] - StateSnapshot({ - 'attributes': ReadOnlyDict({ - : 'HYDxxKTL-3P System time', - }), - 'context': , - 'entity_id': 'sensor.hydxxktl_3p_system_time', - 'last_changed': , - 'last_reported': , - 'last_updated': , - 'state': 'unknown', - }) -# --- # name: test_all_entities[sensor.hydxxktl_3p_update_system_time_operation_result-entry] EntityRegistryEntrySnapshot({ 'aliases': list([ diff --git a/tests/components/sofar/test_sensor.py b/tests/components/sofar/test_sensor.py index 0f7d9a0f4193..4235d7be3fa9 100644 --- a/tests/components/sofar/test_sensor.py +++ b/tests/components/sofar/test_sensor.py @@ -96,12 +96,12 @@ async def test_sensor_entities_created_and_state( @pytest.mark.parametrize( ("serial", "model", "seed", "created", "enabled"), [ - pytest.param(MOCK_SERIAL, MOCK_MODEL, seed_pv_inverter, 74, 23, id="pv"), + pytest.param(MOCK_SERIAL, MOCK_MODEL, seed_pv_inverter, 73, 23, id="pv"), pytest.param( MOCK_HYBRID_SERIAL, MOCK_HYBRID_MODEL, seed_hybrid_inverter, - 182, + 181, 39, id="hybrid", ),