From 0585d43c3d0261110144e50bd035e5869e8d9eb2 Mon Sep 17 00:00:00 2001 From: hiltonlima <116589806+hiltonlima@users.noreply.github.com> Date: Wed, 23 Oct 2024 10:25:46 -0300 Subject: [PATCH] Do not error while parsing SDK python tests (#141) * replace erro with warning * Update test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com> * black error fix * pr review --------- Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com> --- .../python_testing/models/python_test_parser.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py b/test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py index b9f1fbf6..8e390740 100644 --- a/test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py +++ b/test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py @@ -158,8 +158,8 @@ def __parse_test_case( try: tc_desc = __retrieve_description(desc_method) except Exception as e: - logger.error( - f"Error while parsing description for {tc_name}, Error:{str(e)}" + logger.warning( + f"Failed parsing description method for {tc_name}, Error:{str(e)}" ) # If the python test does not implement the steps template method, @@ -170,14 +170,14 @@ def __parse_test_case( try: tc_steps = __retrieve_steps(steps_method) except Exception as e: - logger.error(f"Error while parsing steps for {tc_name}, Error:{str(e)}") + logger.warning(f"Failed parsing steps method for {tc_name}, Error:{str(e)}") pics_method = __get_method_by_name(pics_method_name, methods) if pics_method: try: tc_pics = __retrieve_pics(pics_method) except Exception as e: - logger.error(f"Error while parsing PICS for {tc_name}, Error:{str(e)}") + logger.warning(f"Failed parsing PICS method for {tc_name}, Error:{str(e)}") # - PythonTestType.COMMISSIONING: test cases that have a commissioning first step # - PythonTestType.NO_COMMISSIONING: test cases that follow the expected template @@ -237,7 +237,9 @@ def __retrieve_steps(method: FunctionDefType) -> List[MatterTestStep]: step_name = step.args[ARG_STEP_DESCRIPTION_INDEX].value parsed_step_name = step_name except Exception as e: - logger.error(f"Error while parsing step from {method.name}, Error:{str(e)}") + logger.warning( + f"Failed parsing step name from {method.name}, Error:{str(e)}" + ) parsed_step_name = "UNABLE TO PARSE TEST STEP NAME" python_steps.append(