Fix VIN so it matches the check

home-assistant · May 20, 2024 · 52b36a1 · 52b36a1
1 parent b9abbde
commit 52b36a1
Show file tree

Hide file tree

Showing 9 changed files with 42 additions and 40 deletions.
diff --git a/homeassistant/components/teslemetry/coordinator.py b/homeassistant/components/teslemetry/coordinator.py
@@ -77,7 +77,9 @@ async def _async_update_data(self) -> dict[str, Any]:
         except VehicleOffline:
             self.data["state"] = TeslemetryState.OFFLINE
             return self.data
-        except (InvalidToken, Forbidden, SubscriptionRequired) as e:
+        except InvalidToken as e:
+            raise ConfigEntryAuthFailed from e
+        except SubscriptionRequired as e:
             raise ConfigEntryAuthFailed from e
         except TeslaFleetError as e:
             raise UpdateFailed(e.message) from e

diff --git a/homeassistant/components/teslemetry/manifest.json b/homeassistant/components/teslemetry/manifest.json
@@ -6,5 +6,5 @@
   "documentation": "https://www.home-assistant.io/integrations/teslemetry",
   "iot_class": "cloud_polling",
   "loggers": ["tesla-fleet-api"],
-  "requirements": ["tesla-fleet-api==0.5.11"]
+  "requirements": ["tesla-fleet-api==0.5.12"]
 }
diff --git a/requirements_all.txt b/requirements_all.txt
@@ -2695,7 +2695,7 @@ temperusb==1.6.1
 # tensorflow==2.5.0
 
 # homeassistant.components.teslemetry
-tesla-fleet-api==0.5.11
+tesla-fleet-api==0.5.12
 
 # homeassistant.components.powerwall
 tesla-powerwall==0.5.1

diff --git a/requirements_test_all.txt b/requirements_test_all.txt
@@ -2087,7 +2087,7 @@ temescal==0.5
 temperusb==1.6.1
 
 # homeassistant.components.teslemetry
-tesla-fleet-api==0.5.11
+tesla-fleet-api==0.5.12
 
 # homeassistant.components.powerwall
 tesla-powerwall==0.5.1

diff --git a/tests/components/teslemetry/fixtures/products.json b/tests/components/teslemetry/fixtures/products.json
@@ -4,7 +4,7 @@
       "id": 1234,
       "user_id": 1234,
       "vehicle_id": 1234,
-      "vin": "LRW3F7EK4KC700000",
+      "vin": "LRWXF7EK4KC700000",
       "color": null,
       "access_type": "OWNER",
       "display_name": "Test",

diff --git a/tests/components/teslemetry/fixtures/vehicle_data.json b/tests/components/teslemetry/fixtures/vehicle_data.json
@@ -3,7 +3,7 @@
     "id": 1234,
     "user_id": 1234,
     "vehicle_id": 1234,
-    "vin": "LRW3F7EK4KC700000",
+    "vin": "LRWXF7EK4KC700000",
     "color": null,
     "access_type": "OWNER",
     "granular_access": {

diff --git a/tests/components/teslemetry/fixtures/vehicle_data_alt.json b/tests/components/teslemetry/fixtures/vehicle_data_alt.json
@@ -3,7 +3,7 @@
     "id": 1234,
     "user_id": 1234,
     "vehicle_id": 1234,
-    "vin": "LRW3F7EK4KC700000",
+    "vin": "LRWXF7EK4KC700000",
     "color": null,
     "access_type": "OWNER",
     "granular_access": {

diff --git a/tests/components/teslemetry/snapshots/test_climate.ambr b/tests/components/teslemetry/snapshots/test_climate.ambr
@@ -41,7 +41,7 @@
     'previous_unique_id': None,
     'supported_features': <ClimateEntityFeature: 401>,
     'translation_key': <TeslemetryClimateSide.DRIVER: 'driver_temp'>,
-    'unique_id': 'LRW3F7EK4KC700000-driver_temp',
+    'unique_id': 'LRWXF7EK4KC700000-driver_temp',
     'unit_of_measurement': None,
   })
 # ---
@@ -116,7 +116,7 @@
     'previous_unique_id': None,
     'supported_features': <ClimateEntityFeature: 401>,
     'translation_key': <TeslemetryClimateSide.DRIVER: 'driver_temp'>,
-    'unique_id': 'LRW3F7EK4KC700000-driver_temp',
+    'unique_id': 'LRWXF7EK4KC700000-driver_temp',
     'unit_of_measurement': None,
   })
 # ---
@@ -191,7 +191,7 @@
     'previous_unique_id': None,
     'supported_features': <ClimateEntityFeature: 401>,
     'translation_key': <TeslemetryClimateSide.DRIVER: 'driver_temp'>,
-    'unique_id': 'LRW3F7EK4KC700000-driver_temp',
+    'unique_id': 'LRWXF7EK4KC700000-driver_temp',
     'unit_of_measurement': None,
   })
 # ---

diff --git a/tests/components/teslemetry/snapshots/test_sensor.ambr b/tests/components/teslemetry/snapshots/test_sensor.ambr
@@ -867,7 +867,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_battery_level',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_battery_level',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_battery_level',
     'unit_of_measurement': '%',
   })
 # ---
@@ -940,7 +940,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_battery_range',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_battery_range',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_battery_range',
     'unit_of_measurement': <UnitOfLength.KILOMETERS: 'km'>,
   })
 # ---
@@ -1005,7 +1005,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_conn_charge_cable',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_conn_charge_cable',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_conn_charge_cable',
     'unit_of_measurement': None,
   })
 # ---
@@ -1069,7 +1069,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charge_energy_added',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charge_energy_added',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charge_energy_added',
     'unit_of_measurement': <UnitOfEnergy.KILO_WATT_HOUR: 'kWh'>,
   })
 # ---
@@ -1139,7 +1139,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charge_rate',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charge_rate',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charge_rate',
     'unit_of_measurement': <UnitOfSpeed.KILOMETERS_PER_HOUR: 'km/h'>,
   })
 # ---
@@ -1206,7 +1206,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charger_actual_current',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charger_actual_current',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charger_actual_current',
     'unit_of_measurement': <UnitOfElectricCurrent.AMPERE: 'A'>,
   })
 # ---
@@ -1273,7 +1273,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charger_power',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charger_power',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charger_power',
     'unit_of_measurement': <UnitOfPower.KILO_WATT: 'kW'>,
   })
 # ---
@@ -1340,7 +1340,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charger_voltage',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charger_voltage',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charger_voltage',
     'unit_of_measurement': <UnitOfElectricPotential.VOLT: 'V'>,
   })
 # ---
@@ -1414,7 +1414,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_charging_state',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_charging_state',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_charging_state',
     'unit_of_measurement': None,
   })
 # ---
@@ -1496,7 +1496,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_active_route_miles_to_arrival',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_active_route_miles_to_arrival',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_active_route_miles_to_arrival',
     'unit_of_measurement': <UnitOfLength.KILOMETERS: 'km'>,
   })
 # ---
@@ -1566,7 +1566,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'climate_state_driver_temp_setting',
-    'unique_id': 'LRW3F7EK4KC700000-climate_state_driver_temp_setting',
+    'unique_id': 'LRWXF7EK4KC700000-climate_state_driver_temp_setting',
     'unit_of_measurement': <UnitOfTemperature.CELSIUS: '°C'>,
   })
 # ---
@@ -1639,7 +1639,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_est_battery_range',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_est_battery_range',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_est_battery_range',
     'unit_of_measurement': <UnitOfLength.KILOMETERS: 'km'>,
   })
 # ---
@@ -1704,7 +1704,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_fast_charger_type',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_fast_charger_type',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_fast_charger_type',
     'unit_of_measurement': None,
   })
 # ---
@@ -1771,7 +1771,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_ideal_battery_range',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_ideal_battery_range',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_ideal_battery_range',
     'unit_of_measurement': <UnitOfLength.KILOMETERS: 'km'>,
   })
 # ---
@@ -1841,7 +1841,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'climate_state_inside_temp',
-    'unique_id': 'LRW3F7EK4KC700000-climate_state_inside_temp',
+    'unique_id': 'LRWXF7EK4KC700000-climate_state_inside_temp',
     'unit_of_measurement': <UnitOfTemperature.CELSIUS: '°C'>,
   })
 # ---
@@ -1914,7 +1914,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'vehicle_state_odometer',
-    'unique_id': 'LRW3F7EK4KC700000-vehicle_state_odometer',
+    'unique_id': 'LRWXF7EK4KC700000-vehicle_state_odometer',
     'unit_of_measurement': <UnitOfLength.KILOMETERS: 'km'>,
   })
 # ---
@@ -1984,7 +1984,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'climate_state_outside_temp',
-    'unique_id': 'LRW3F7EK4KC700000-climate_state_outside_temp',
+    'unique_id': 'LRWXF7EK4KC700000-climate_state_outside_temp',
     'unit_of_measurement': <UnitOfTemperature.CELSIUS: '°C'>,
   })
 # ---
@@ -2054,7 +2054,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'climate_state_passenger_temp_setting',
-    'unique_id': 'LRW3F7EK4KC700000-climate_state_passenger_temp_setting',
+    'unique_id': 'LRWXF7EK4KC700000-climate_state_passenger_temp_setting',
     'unit_of_measurement': <UnitOfTemperature.CELSIUS: '°C'>,
   })
 # ---
@@ -2121,7 +2121,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_power',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_power',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_power',
     'unit_of_measurement': <UnitOfPower.KILO_WATT: 'kW'>,
   })
 # ---
@@ -2193,7 +2193,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_shift_state',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_shift_state',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_shift_state',
     'unit_of_measurement': None,
   })
 # ---
@@ -2271,7 +2271,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_speed',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_speed',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_speed',
     'unit_of_measurement': <UnitOfSpeed.KILOMETERS_PER_HOUR: 'km/h'>,
   })
 # ---
@@ -2338,7 +2338,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_active_route_energy_at_arrival',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_active_route_energy_at_arrival',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_active_route_energy_at_arrival',
     'unit_of_measurement': '%',
   })
 # ---
@@ -2403,7 +2403,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_active_route_minutes_to_arrival',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_active_route_minutes_to_arrival',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_active_route_minutes_to_arrival',
     'unit_of_measurement': None,
   })
 # ---
@@ -2464,7 +2464,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_minutes_to_full_charge',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_minutes_to_full_charge',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_minutes_to_full_charge',
     'unit_of_measurement': None,
   })
 # ---
@@ -2533,7 +2533,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'vehicle_state_tpms_pressure_fl',
-    'unique_id': 'LRW3F7EK4KC700000-vehicle_state_tpms_pressure_fl',
+    'unique_id': 'LRWXF7EK4KC700000-vehicle_state_tpms_pressure_fl',
     'unit_of_measurement': <UnitOfPressure.PSI: 'psi'>,
   })
 # ---
@@ -2606,7 +2606,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'vehicle_state_tpms_pressure_fr',
-    'unique_id': 'LRW3F7EK4KC700000-vehicle_state_tpms_pressure_fr',
+    'unique_id': 'LRWXF7EK4KC700000-vehicle_state_tpms_pressure_fr',
     'unit_of_measurement': <UnitOfPressure.PSI: 'psi'>,
   })
 # ---
@@ -2679,7 +2679,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'vehicle_state_tpms_pressure_rl',
-    'unique_id': 'LRW3F7EK4KC700000-vehicle_state_tpms_pressure_rl',
+    'unique_id': 'LRWXF7EK4KC700000-vehicle_state_tpms_pressure_rl',
     'unit_of_measurement': <UnitOfPressure.PSI: 'psi'>,
   })
 # ---
@@ -2752,7 +2752,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'vehicle_state_tpms_pressure_rr',
-    'unique_id': 'LRW3F7EK4KC700000-vehicle_state_tpms_pressure_rr',
+    'unique_id': 'LRWXF7EK4KC700000-vehicle_state_tpms_pressure_rr',
     'unit_of_measurement': <UnitOfPressure.PSI: 'psi'>,
   })
 # ---
@@ -2819,7 +2819,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'drive_state_active_route_traffic_minutes_delay',
-    'unique_id': 'LRW3F7EK4KC700000-drive_state_active_route_traffic_minutes_delay',
+    'unique_id': 'LRWXF7EK4KC700000-drive_state_active_route_traffic_minutes_delay',
     'unit_of_measurement': <UnitOfTime.MINUTES: 'min'>,
   })
 # ---
@@ -2886,7 +2886,7 @@
     'previous_unique_id': None,
     'supported_features': 0,
     'translation_key': 'charge_state_usable_battery_level',
-    'unique_id': 'LRW3F7EK4KC700000-charge_state_usable_battery_level',
+    'unique_id': 'LRWXF7EK4KC700000-charge_state_usable_battery_level',
     'unit_of_measurement': '%',
   })
 # ---