Add names to all Tessie entities (#106267)

* Add name back to device tracker and lock

* Add name to media player

* Add name to update

* Update snapshot

* Keep none name on Update

* Re-add update

homeassistant/components/tessie/device_tracker.py

@@ -48,7 +48,6 @@ class TessieDeviceTrackerEntity(TessieEntity, TrackerEntity):
 class TessieDeviceTrackerLocationEntity(TessieDeviceTrackerEntity):
     """Vehicle Location Device Tracker Class."""
 
-    _attr_name = None
     key = "location"
 
     @property

homeassistant/components/tessie/lock.py

@@ -27,8 +27,6 @@ async def async_setup_entry(
 class TessieLockEntity(TessieEntity, LockEntity):
     """Lock entity for current charge."""
 
-    _attr_name = None
-
     def __init__(
         self,
         coordinator: TessieStateUpdateCoordinator,

homeassistant/components/tessie/media_player.py

@@ -33,7 +33,6 @@ async def async_setup_entry(
 class TessieMediaEntity(TessieEntity, MediaPlayerEntity):
     """Vehicle Location Media Class."""
 
-    _attr_name = None
     _attr_device_class = MediaPlayerDeviceClass.SPEAKER
 
     def __init__(

homeassistant/components/tessie/strings.json

@@ -24,6 +24,7 @@
   "entity": {
     "device_tracker": {
       "location": {
+        "name": "Location",
         "state_attributes": {
           "heading": {
             "name": "Heading"
@@ -52,6 +53,16 @@
         }
       }
     },
+    "lock": {
+      "vehicle_state_locked": {
+        "name": "[%key:component::lock::title%]"
+      }
+    },
+    "media_player": {
+      "media": {
+        "name": "[%key:component::media_player::title%]"
+      }
+    },
     "sensor": {
       "charge_state_usable_battery_level": {
         "name": "Battery level"
@@ -298,6 +309,11 @@
       "vehicle_state_speed_limit_mode_current_limit_mph": {
         "name": "Speed limit"
       }
+    },
+    "update": {
+      "update": {
+        "name": "[%key:component::update::title%]"
+      }
     }
   }
 }

homeassistant/components/tessie/update.py

@@ -30,7 +30,6 @@ class TessieUpdateEntity(TessieEntity, UpdateEntity):
     """Tessie Updates entity."""
 
     _attr_supported_features = UpdateEntityFeature.PROGRESS
-    _attr_name = None
 
     def __init__(
         self,

tests/components/tessie/snapshots/test_media_player.ambr

@@ -3,12 +3,12 @@
   StateSnapshot({
     'attributes': ReadOnlyDict({
       'device_class': 'speaker',
-      'friendly_name': 'Test',
+      'friendly_name': 'Test Media player',
       'supported_features': <MediaPlayerEntityFeature: 0>,
       'volume_level': 0.22580323309042688,
     }),
     'context': <ANY>,
-    'entity_id': 'media_player.test',
+    'entity_id': 'media_player.test_media_player',
     'last_changed': <ANY>,
     'last_updated': <ANY>,
     'state': 'idle',
@@ -18,12 +18,12 @@
   StateSnapshot({
     'attributes': ReadOnlyDict({
       'device_class': 'speaker',
-      'friendly_name': 'Test',
+      'friendly_name': 'Test Media player',
       'supported_features': <MediaPlayerEntityFeature: 0>,
       'volume_level': 0.22580323309042688,
     }),
     'context': <ANY>,
-    'entity_id': 'media_player.test',
+    'entity_id': 'media_player.test_media_player',
     'last_changed': <ANY>,
     'last_updated': <ANY>,
     'state': 'idle',

tests/components/tessie/test_device_tracker.py

@@ -19,7 +19,7 @@ async def test_device_tracker(hass: HomeAssistant) -> None:
 
     assert len(hass.states.async_all(DEVICE_TRACKER_DOMAIN)) == 2
 
-    entity_id = "device_tracker.test"
+    entity_id = "device_tracker.test_location"
     state = hass.states.get(entity_id)
     assert state.attributes.get(ATTR_LATITUDE) == STATES["drive_state"]["latitude"]
     assert state.attributes.get(ATTR_LONGITUDE) == STATES["drive_state"]["longitude"]

tests/components/tessie/test_lock.py

@@ -22,7 +22,7 @@ async def test_locks(hass: HomeAssistant) -> None:
 
     assert len(hass.states.async_all("lock")) == 1
 
-    entity_id = "lock.test"
+    entity_id = "lock.test_lock"
 
     assert (
         hass.states.get(entity_id).state == STATE_LOCKED

tests/components/tessie/test_media_player.py

@@ -35,12 +35,12 @@ async def test_media_player_idle(
 
     assert len(hass.states.async_all("media_player")) == 1
 
-    state = hass.states.get("media_player.test")
+    state = hass.states.get("media_player.test_media_player")
     assert state == snapshot
 
     # Trigger coordinator refresh since it has a different fixture.
     freezer.tick(WAIT)
     async_fire_time_changed(hass)
 
-    state = hass.states.get("media_player.test")
+    state = hass.states.get("media_player.test_media_player")
     assert state == snapshot

tests/components/tessie/test_update.py

@@ -21,7 +21,7 @@ async def test_updates(hass: HomeAssistant) -> None:
 
     assert len(hass.states.async_all("update")) == 1
 
-    entity_id = "update.test"
+    entity_id = "update.test_update"
     state = hass.states.get(entity_id)
     assert state.state == STATE_ON
     assert state.attributes.get(ATTR_IN_PROGRESS) is False