Change device class of Auto Seat Heater sensors in Tessie (#109240)

homeassistant/components/tessie/binary_sensor.py

@@ -62,17 +62,14 @@ DESCRIPTIONS: tuple[TessieBinarySensorEntityDescription, ...] = (
     ),
     TessieBinarySensorEntityDescription(
         key="climate_state_auto_seat_climate_left",
-        device_class=BinarySensorDeviceClass.HEAT,
         entity_category=EntityCategory.DIAGNOSTIC,
     ),
     TessieBinarySensorEntityDescription(
         key="climate_state_auto_seat_climate_right",
-        device_class=BinarySensorDeviceClass.HEAT,
         entity_category=EntityCategory.DIAGNOSTIC,
     ),
     TessieBinarySensorEntityDescription(
         key="climate_state_auto_steering_wheel_heat",
-        device_class=BinarySensorDeviceClass.HEAT,
         entity_category=EntityCategory.DIAGNOSTIC,
     ),
     TessieBinarySensorEntityDescription(

homeassistant/components/tessie/icons.json

@@ -0,0 +1,23 @@
+{
+  "entity": {
+    "sensor": {
+      "drive_state_shift_state": {
+        "default": "mdi:car-shift-pattern"
+      },
+      "drive_state_active_route_destination": {
+        "default": "mdi:map-marker"
+      }
+    },
+    "binary_sensor": {
+      "climate_state_auto_seat_climate_left": {
+        "default": "mdi:car-seat-heater"
+      },
+      "climate_state_auto_seat_climate_right": {
+        "default": "mdi:car-seat-heater"
+      },
+      "climate_state_auto_steering_wheel_heat": {
+        "default": "mdi:steering"
+      }
+    }
+  }
+}

homeassistant/components/tessie/sensor.py

@@ -122,7 +122,6 @@ DESCRIPTIONS: tuple[TessieSensorEntityDescription, ...] = (
     ),
     TessieSensorEntityDescription(
         key="drive_state_shift_state",
-        icon="mdi:car-shift-pattern",
         options=["p", "d", "r", "n"],
         device_class=SensorDeviceClass.ENUM,
         value_fn=lambda x: x.lower() if isinstance(x, str) else x,
@@ -231,7 +230,6 @@ DESCRIPTIONS: tuple[TessieSensorEntityDescription, ...] = (
     ),
     TessieSensorEntityDescription(
         key="drive_state_active_route_destination",
-        icon="mdi:map-marker",
         entity_category=EntityCategory.DIAGNOSTIC,
     ),
 )

homeassistant/components/tessie/strings.json

@@ -258,7 +258,7 @@
       "climate_state_auto_seat_climate_right": {
         "name": "Auto seat climate right"
       },
-      "climate_state_auto_steering_wheel_heater": {
+      "climate_state_auto_steering_wheel_heat": {
         "name": "Auto steering wheel heater"
       },
       "climate_state_cabin_overheat_protection": {

tests/components/tessie/snapshots/test_binary_sensors.ambr

@@ -19,7 +19,7 @@
     'name': None,
     'options': dict({
     }),
-    'original_device_class': <BinarySensorDeviceClass.HEAT: 'heat'>,
+    'original_device_class': None,
     'original_icon': None,
     'original_name': 'Auto seat climate left',
     'platform': 'tessie',
@@ -33,7 +33,6 @@
 # name: test_binary_sensors[binary_sensor.test_auto_seat_climate_left-state]
   StateSnapshot({
     'attributes': ReadOnlyDict({
-      'device_class': 'heat',
       'friendly_name': 'Test Auto seat climate left',
     }),
     'context': <ANY>,
@@ -63,7 +62,7 @@
     'name': None,
     'options': dict({
     }),
-    'original_device_class': <BinarySensorDeviceClass.HEAT: 'heat'>,
+    'original_device_class': None,
     'original_icon': None,
     'original_name': 'Auto seat climate right',
     'platform': 'tessie',
@@ -77,7 +76,6 @@
 # name: test_binary_sensors[binary_sensor.test_auto_seat_climate_right-state]
   StateSnapshot({
     'attributes': ReadOnlyDict({
-      'device_class': 'heat',
       'friendly_name': 'Test Auto seat climate right',
     }),
     'context': <ANY>,
@@ -87,6 +85,49 @@
     'state': 'on',
   })
 # ---
+# name: test_binary_sensors[binary_sensor.test_auto_steering_wheel_heater-entry]
+  EntityRegistryEntrySnapshot({
+    'aliases': set({
+    }),
+    'area_id': None,
+    'capabilities': None,
+    'config_entry_id': <ANY>,
+    'device_class': None,
+    'device_id': <ANY>,
+    'disabled_by': None,
+    'domain': 'binary_sensor',
+    'entity_category': <EntityCategory.DIAGNOSTIC: 'diagnostic'>,
+    'entity_id': 'binary_sensor.test_auto_steering_wheel_heater',
+    'has_entity_name': True,
+    'hidden_by': None,
+    'icon': None,
+    'id': <ANY>,
+    'name': None,
+    'options': dict({
+    }),
+    'original_device_class': None,
+    'original_icon': None,
+    'original_name': 'Auto steering wheel heater',
+    'platform': 'tessie',
+    'previous_unique_id': None,
+    'supported_features': 0,
+    'translation_key': 'climate_state_auto_steering_wheel_heat',
+    'unique_id': 'VINVINVIN-climate_state_auto_steering_wheel_heat',
+    'unit_of_measurement': None,
+  })
+# ---
+# name: test_binary_sensors[binary_sensor.test_auto_steering_wheel_heater-state]
+  StateSnapshot({
+    'attributes': ReadOnlyDict({
+      'friendly_name': 'Test Auto steering wheel heater',
+    }),
+    'context': <ANY>,
+    'entity_id': 'binary_sensor.test_auto_steering_wheel_heater',
+    'last_changed': <ANY>,
+    'last_updated': <ANY>,
+    'state': 'on',
+  })
+# ---
 # name: test_binary_sensors[binary_sensor.test_battery_heater-entry]
   EntityRegistryEntrySnapshot({
     'aliases': set({
@@ -527,50 +568,6 @@
     'state': 'off',
   })
 # ---
-# name: test_binary_sensors[binary_sensor.test_heat-entry]
-  EntityRegistryEntrySnapshot({
-    'aliases': set({
-    }),
-    'area_id': None,
-    'capabilities': None,
-    'config_entry_id': <ANY>,
-    'device_class': None,
-    'device_id': <ANY>,
-    'disabled_by': None,
-    'domain': 'binary_sensor',
-    'entity_category': <EntityCategory.DIAGNOSTIC: 'diagnostic'>,
-    'entity_id': 'binary_sensor.test_heat',
-    'has_entity_name': True,
-    'hidden_by': None,
-    'icon': None,
-    'id': <ANY>,
-    'name': None,
-    'options': dict({
-    }),
-    'original_device_class': <BinarySensorDeviceClass.HEAT: 'heat'>,
-    'original_icon': None,
-    'original_name': 'Heat',
-    'platform': 'tessie',
-    'previous_unique_id': None,
-    'supported_features': 0,
-    'translation_key': 'climate_state_auto_steering_wheel_heat',
-    'unique_id': 'VINVINVIN-climate_state_auto_steering_wheel_heat',
-    'unit_of_measurement': None,
-  })
-# ---
-# name: test_binary_sensors[binary_sensor.test_heat-state]
-  StateSnapshot({
-    'attributes': ReadOnlyDict({
-      'device_class': 'heat',
-      'friendly_name': 'Test Heat',
-    }),
-    'context': <ANY>,
-    'entity_id': 'binary_sensor.test_heat',
-    'last_changed': <ANY>,
-    'last_updated': <ANY>,
-    'state': 'on',
-  })
-# ---
 # name: test_binary_sensors[binary_sensor.test_preconditioning_enabled-entry]
   EntityRegistryEntrySnapshot({
     'aliases': set({

tests/components/tessie/snapshots/test_sensor.ambr

@@ -368,7 +368,7 @@
     'options': dict({
     }),
     'original_device_class': None,
-    'original_icon': 'mdi:map-marker',
+    'original_icon': None,
     'original_name': 'Destination',
     'platform': 'tessie',
     'previous_unique_id': None,
@@ -382,7 +382,6 @@
   StateSnapshot({
     'attributes': ReadOnlyDict({
       'friendly_name': 'Test Destination',
-      'icon': 'mdi:map-marker',
     }),
     'context': <ANY>,
     'entity_id': 'sensor.test_destination',
@@ -776,7 +775,7 @@
     'options': dict({
     }),
     'original_device_class': <SensorDeviceClass.ENUM: 'enum'>,
-    'original_icon': 'mdi:car-shift-pattern',
+    'original_icon': None,
     'original_name': 'Shift state',
     'platform': 'tessie',
     'previous_unique_id': None,
@@ -791,7 +790,6 @@
     'attributes': ReadOnlyDict({
       'device_class': 'enum',
       'friendly_name': 'Test Shift state',
-      'icon': 'mdi:car-shift-pattern',
       'options': list([
         'p',
         'd',