Remove return statement in finally clause

Author: andchiind

src/isar/robot/robot_service.py

@@ -176,8 +176,7 @@ def request_inspection_upload(task: InspectionTask) -> None:
                 )
         except asyncio.CancelledError:
             pass
-        finally:
-            return mission if remaining_mission is None else remaining_mission
+        return mission if remaining_mission is None else remaining_mission
 
     def _register_status_threads(self) -> None:
         self.status_thread = RobotStatusThread(

tests/isar/services/robot/test_robot_service.py

@@ -134,7 +134,7 @@ def test_successful_stop_with_no_remaining_tasks(
 
     mocker.patch(
         "isar.robot.robot_service.robot_monitor_mission",
-        return_value=(None, mission),
+        return_value=(None, mission, True),
     )
 
     async def test_stop_mission_handler() -> None: