From 3a4b99df5510f9c0ce9901fde755c20471afbdda Mon Sep 17 00:00:00 2001 From: Julian Stirling Date: Fri, 8 Aug 2025 09:32:31 +0000 Subject: [PATCH] Apply suggestions from code review of branch faster-stage-tests Co-authored-by: Beth Probert --- tests/mock_things/mock_cancel.py | 2 +- tests/test_stage.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/mock_things/mock_cancel.py b/tests/mock_things/mock_cancel.py index fd380894..1e9960a7 100644 --- a/tests/mock_things/mock_cancel.py +++ b/tests/mock_things/mock_cancel.py @@ -14,6 +14,6 @@ import time class MockCancel: """A class to use when directly calling an action with CancelHook dependency.""" - def sleep(self, time_in_seconds: float): + def sleep(self, time_in_seconds: float) -> None: """Sleep for the input number of seconds.""" time.sleep(time_in_seconds) diff --git a/tests/test_stage.py b/tests/test_stage.py index 54b5c107..e4d395b6 100644 --- a/tests/test_stage.py +++ b/tests/test_stage.py @@ -234,7 +234,7 @@ def test_move_relative(stage_client, dummy_stage, path): inverted when appropriate. NOTE: it is essential that `stage_client` is imported even if any time it is used - dummy client could be used. This is because the fixture that creates stage client + dummy stage could be used. This is because the fixture that creates stage client handles adding the dummy_stage to a server. It needs to have been added to a server for it not to throw errors about not being connected to a server. """ @@ -295,7 +295,7 @@ def test_move_absolute(stage_client, dummy_stage, path): inverted when appropriate. NOTE: it is essential that `stage_client` is imported even if any time it is used - dummy client could be used. This is because the fixture that creates stage client + dummy stage could be used. This is because the fixture that creates stage client handles adding the dummy_stage to a server. It needs to have been added to a server for it not to throw errors about not being connected to a server. """