From 1d096887924eec321b259cc272374c018630dbea Mon Sep 17 00:00:00 2001 From: kfanning Date: Mon, 25 Nov 2024 19:16:30 -0300 Subject: [PATCH] DM-47795: adjusting test case for the new wait, adding news fragment. --- doc/news/DM-47795.bugfix.rst | 1 + tests/maintel/test_mtcs.py | 13 ------------- 2 files changed, 1 insertion(+), 13 deletions(-) create mode 100644 doc/news/DM-47795.bugfix.rst diff --git a/doc/news/DM-47795.bugfix.rst b/doc/news/DM-47795.bugfix.rst new file mode 100644 index 00000000..630c476e --- /dev/null +++ b/doc/news/DM-47795.bugfix.rst @@ -0,0 +1 @@ +IN ``mtcs.py``, wait for both primary and secondary bump tests to finish to avoid a race condition where the last actuator has its test cancelled due to exiting engineering mode on M1M3. \ No newline at end of file diff --git a/tests/maintel/test_mtcs.py b/tests/maintel/test_mtcs.py index 1f448bd7..c2a1b3f0 100644 --- a/tests/maintel/test_mtcs.py +++ b/tests/maintel/test_mtcs.py @@ -1707,19 +1707,6 @@ async def test_run_m1m3_actuator_bump_test_both_fail(self) -> None: secondary_status, ) = await self.mtcs.get_m1m3_bump_test_status(actuator_id=actuator_id) - assert primary_status == idl.enums.MTM1M3.BumpTest.FAILED - assert secondary_status != idl.enums.MTM1M3.BumpTest.FAILED - - with pytest.raises(RuntimeError): - await self.mtcs._wait_bump_test_ok( - actuator_id=actuator_id, primary=False, secondary=True - ) - - ( - primary_status, - secondary_status, - ) = await self.mtcs.get_m1m3_bump_test_status(actuator_id=actuator_id) - assert primary_status == idl.enums.MTM1M3.BumpTest.FAILED assert secondary_status == idl.enums.MTM1M3.BumpTest.FAILED