twister: PyTest plugin's DUT release sequence change

Change Twister PyTest plugin's test finalizing sequence to release the DUT it is used as the very last operation, after the Test Instance status becomes fully updated from the execution results. This also fix a race condition possible when pytest plugin releases the DUT and it becomes acquired by another test while the current test is not yet finalized completely. Signed-off-by: Dmitrii Golovanov <[email protected]>
zephyrproject-rtos · Aug 14, 2024 · 1827ab4 · 1827ab4
1 parent 9909031
commit 1827ab4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/scripts/pylib/twister/twisterlib/harness.py b/scripts/pylib/twister/twisterlib/harness.py
@@ -366,10 +366,10 @@ def pytest_run(self, timeout):
             self.status = TwisterStatus.FAIL
             self.instance.reason = str(pytest_exception)
         finally:
+            self.instance.record(self.recording)
+            self._update_test_status()
             if self.reserved_dut:
                 self.instance.handler.make_dut_available(self.reserved_dut)
-        self.instance.record(self.recording)
-        self._update_test_status()
 
     def generate_command(self):
         config = self.instance.testsuite.harness_config