Add is_vehicle_tracking_enabled on Vehicle

bimmer_connected/vehicle.py

@@ -137,6 +137,11 @@ def has_weekly_planner_service(self) -> bool:
         """Return True if charging control (weekly planner) is available."""
         return self.attributes["capabilities"]["isChargingPlanSupported"]
 
+    @property
+    def is_vehicle_tracking_enabled(self) -> bool:
+        """Return True if vehicle finder is enabled in vehicle."""
+        return self.attributes["capabilities"]["vehicleFinder"]["isEnabled"]
+
     @property
     def drive_train_attributes(self) -> List[str]:
         """Get list of attributes available for the drive train of the vehicle.

bimmer_connected/vehicle_status.py

@@ -199,7 +199,7 @@ def gps_position(self) -> Tuple[float, float]:
         This only provides data, if the vehicle tracking is enabled!
         """
         if self.is_vehicle_active:
-            _LOGGER.warning('Vehicle was moving at last update, no position available')
+            _LOGGER.info('Vehicle was moving at last update, no position available')
             return None
         if not self._remote_service_position and "vehicleLocation" not in self.properties:
             _LOGGER.info("No vehicle location data available.")
@@ -224,7 +224,7 @@ def gps_heading(self) -> int:
         This only provides data, if the vehicle tracking is enabled!
         """
         if self.is_vehicle_active:
-            _LOGGER.warning('Vehicle was moving at last update, no position available')
+            _LOGGER.info('Vehicle was moving at last update, no position available')
             return None
 
         if not self._remote_service_position and "vehicleLocation" not in self.properties:

test/responses/F11/vehicles_v2_bmw_0.json

@@ -86,7 +86,7 @@
       },
       "vehicleFinder": {
         "executionMessage": "Find your vehicle now? Remote functions may take a few seconds.",
-        "isEnabled": true,
+        "isEnabled": false,
         "isPinAuthenticationRequired": false
       }
     },

test/test_utils.py

@@ -34,6 +34,7 @@ def test_drive_train(self):
                 "has_internal_combustion_engine",
                 "has_range_extender",
                 "has_weekly_planner_service",
+                "is_vehicle_tracking_enabled",
                 "lsc_type",
                 "name",
                 "to_json",

test/test_vehicle.py

@@ -117,6 +117,14 @@ def test_set_observer_position(self):
         with self.assertRaises(ValueError):
             vehicle.set_observer_position(12.1, None)
 
+    def test_get_is_tracking_enabled(self):
+        """Test setting observer position"""
+        vehicle = get_mocked_account().get_vehicle(VIN_F11)
+        self.assertEqual(False, vehicle.is_vehicle_tracking_enabled)
+
+        vehicle = get_mocked_account().get_vehicle(VIN_F31)
+        self.assertEqual(True, vehicle.is_vehicle_tracking_enabled)
+
     def test_available_attributes(self):
         """Check that available_attributes returns exactly the arguments we have in our test data."""
         account = get_mocked_account()

test/test_vehicle_status.py

@@ -166,11 +166,12 @@ def test_parse_f31_no_position(self):
 
     def test_parse_f11_no_position_vehicle_active(self):
         """Test parsing of F11 data with vehicle beeing active."""
-        status = get_mocked_account().get_vehicle(VIN_F48).status
+        vehicle = get_mocked_account().get_vehicle(VIN_F48)
+        status = vehicle.status
 
-        # self.assertTrue(status.is_vehicle_tracking_enabled)
-        with self.assertLogs(level=logging.WARNING):
-            self.assertTrue(status.is_vehicle_active)
+        self.assertTrue(vehicle.is_vehicle_tracking_enabled)
+        self.assertTrue(status.is_vehicle_active)
+        with self.assertLogs(level=logging.INFO):
             self.assertIsNone(status.gps_position)
             self.assertIsNone(status.gps_heading)