From 30381313c9e465e92cdf4956402c5fed08ac40a8 Mon Sep 17 00:00:00 2001 From: "igor udot (horw)" Date: Tue, 19 Sep 2023 09:26:21 +0800 Subject: [PATCH] fix: make force naming of reset parameter, change pop(reset) to get(reset) --- pytest-embedded-idf/pytest_embedded_idf/unity_tester.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pytest-embedded-idf/pytest_embedded_idf/unity_tester.py b/pytest-embedded-idf/pytest_embedded_idf/unity_tester.py index 85d013db..ed363260 100644 --- a/pytest-embedded-idf/pytest_embedded_idf/unity_tester.py +++ b/pytest-embedded-idf/pytest_embedded_idf/unity_tester.py @@ -254,7 +254,7 @@ def wrapper(self, *args, **kwargs): try: # do it here since the first hard reset before test case shouldn't be counted in duration time if 'reset' in kwargs: - if kwargs.pop('reset') and self._hard_reset_func: + if kwargs.get('reset') and self._hard_reset_func: try: self._hard_reset_func() except NotImplementedError: @@ -327,6 +327,7 @@ def _add_single_unity_test_case( def _run_normal_case( self, case: UnittestMenuCase, + *, reset: bool = False, # noqa timeout: float = 30, ) -> None: @@ -352,6 +353,7 @@ def _run_normal_case( def _run_multi_stage_case( self, case: UnittestMenuCase, + *, reset: bool = False, # noqa timeout: float = 30, ) -> None: @@ -792,7 +794,7 @@ def run_normal_case(self, case: UnittestMenuCase, reset: bool = False, timeout: reset: whether do a hardware reset before running the case timeout: timeout in second """ - self.first_dut._run_normal_case(case, reset, timeout=timeout) + self.first_dut._run_normal_case(case, reset=reset, timeout=timeout) def run_multi_stage_case(self, case: UnittestMenuCase, reset: bool = False, timeout: int = 90) -> None: """