google / openhtf / 30292890051

64%

master: 64%

push

github

Fix thread deadlock during TestExecutor teardown when test_state is None.

When TestState fails during initialization (e.g. InvalidPlugError for unsubstituted placeholders), self.test_state remains None. During thread teardown in the finally block, accessing self.running_test_state raised a secondary TestStopError, which prevented self._execution_finished.set() from being called and caused test.execute() to hang indefinitely until test timeout.

This change guards against None test_state in teardown and close(), and wraps teardown in a try/finally block so _execution_finished.set() is always signaled.

PiperOrigin-RevId: 954728594

8 of 11 new or added lines in 1 file covered. (72.73%)

4853 of 7629 relevant lines covered (63.61%)

3.18 hits per line