From 53b73bd0bd371b0868832293ad3f31341b1459fa Mon Sep 17 00:00:00 2001 From: Martin Hjelmare Date: Thu, 25 Jan 2024 13:25:17 +0100 Subject: [PATCH] Make device tracker latitude and longitude optional (#108838) * Make device tracker latitude and longitude optional * Update test --- homeassistant/components/device_tracker/config_entry.py | 4 ++-- tests/components/device_tracker/test_config_entry.py | 9 +++------ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/homeassistant/components/device_tracker/config_entry.py b/homeassistant/components/device_tracker/config_entry.py index c169c78cacc..20ac365b33b 100644 --- a/homeassistant/components/device_tracker/config_entry.py +++ b/homeassistant/components/device_tracker/config_entry.py @@ -241,12 +241,12 @@ class TrackerEntity(BaseTrackerEntity): @property def latitude(self) -> float | None: """Return latitude value of the device.""" - raise NotImplementedError + return None @property def longitude(self) -> float | None: """Return longitude value of the device.""" - raise NotImplementedError + return None @property def state(self) -> str | None: diff --git a/tests/components/device_tracker/test_config_entry.py b/tests/components/device_tracker/test_config_entry.py index fe52ec1219a..ba258af068e 100644 --- a/tests/components/device_tracker/test_config_entry.py +++ b/tests/components/device_tracker/test_config_entry.py @@ -506,13 +506,10 @@ def test_tracker_entity() -> None: entity = TrackerEntity() with pytest.raises(NotImplementedError): assert entity.source_type is None - with pytest.raises(NotImplementedError): - assert entity.latitude is None - with pytest.raises(NotImplementedError): - assert entity.longitude is None + assert entity.latitude is None + assert entity.longitude is None assert entity.location_name is None - with pytest.raises(NotImplementedError): - assert entity.state is None + assert entity.state is None assert entity.battery_level is None assert entity.should_poll is False assert entity.force_update is True