fixes the issue that bluesky calls stop on all devices at the end of …

src/secop_ophyd/SECoPDevices.py

@@ -758,10 +758,10 @@ async def stop(self, success=True):
         """
         self._success = success
 
-        self.logger.info(f"Stopping {self.name} success={success}")
-
-        await self._secclient.exec_command(self._module, "stop")
-        self._stopped = True
+        if not success:
+            self.logger.info(f"Stopping {self.name} success={success}")
+            await self._secclient.exec_command(self._module, "stop")
+            self._stopped = True
 
     async def locate(self) -> Location:
         # return current location of the device (setpoint and readback).

tests/test_commands.py

@@ -23,7 +23,7 @@ async def test_stop_cmd(cryo_sim, cryo_node_internal_loop: SECoPNodeDevice):
 
     await stat
 
-    assert cryo._stopped is True
+    assert cryo._stopped is False
 
     await cryo_node_internal_loop.disconnect_async()