tools/psoc6/run_test_plan.py: WIP. FIXUP.

Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>

Author: jaenrig-ifx

tools/psoc6/run_test_plan.py

@@ -660,70 +660,68 @@ def get_test_list(self, test_name_list=[]):
         return test_list
 
 
-class TestPlanParser:
+class TestPlanCLI:
     def __init__(self):
-        self.tests_parser = argparse.ArgumentParser(description="MicroPython test suites runner.")
-        self.tests_parser.add_argument(
-            "test_suite", nargs="*", type=str, help="Test suite to run."
-        )
-        self.tests_parser.add_argument(
+        self.parser = argparse.ArgumentParser(description="MicroPython test suites runner.")
+        self.parser.add_argument("test_suite", nargs="*", type=str, help="Test suite to run.")
+        self.parser.add_argument(
             "--test-plan", type=str, default=None, help="Path to the test plan file."
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "--hil-devs", type=str, default=None, help="Path to the HIL devices file."
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "-b",
             "--board",
             type=str,
             default=None,
             help="Test board name (only used with --hil-devs).",
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "-d",
             "--dut-port",
             type=str,
             default=None,
             help="Device under test port. Default is /dev/ttyACM0.",
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "-s",
             "--stub-port",
             type=str,
             default=None,
             help="Stub device port. Default is /dev/ttyACM1.",
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "--max-retries",
             type=int,
             default=0,
             help="Maximum number of retries for failed tests.",
         )
-        self.tests_parser.add_argument(
+        self.parser.add_argument(
             "--mpy-root-dir",
             type=str,
             default=None,
             help="Path to the root of the MicroPython repository. Default is two levels up from this script.",
         )
 
     def parse(self):
-        args = self.tests_parser.parse_args()
+        args = self.parser.parse_args()
         args = self.set_validate_args(args)
         return args
 
     def set_validate_args(self, args):
         if args.hil_devs:
             args.hil_devs = os.path.abspath(args.hil_devs)
             if args.board is None:
-                self.tests_parser.error("--board is required when --hil-devs is provided")
+                self.parser.error("--board is required when --hil-devs is provided")
 
             if args.dut_port or args.stub_port:
-                self.tests_parser.error(
+                self.parser.error(
                     "--dut-port and --stub-port are not supported when --hil-devs is provided"
                 )
         else:
             if args.board is not None:
-                self.tests_parser.error("--hil-devs is required when --board is provided")
+                self.parser.error("--hil-devs is required when --board is provided")
 
             # If the ports are not provide, the default values are set.
             if args.dut_port is None:
@@ -749,14 +747,13 @@ def set_validate_args(self, args):
 
 
 if __name__ == "__main__":
-    test_parser = TestPlanParser()
-    parser_args = test_parser.parse()
-    print(parser_args)
+    test_plan_cli = TestPlanCLI()
+    cli_args = test_plan_cli.parse()
     test_plan_runner = TestPlanRunner(
-        parser_args.test_plan,
-        parser_args.hil_devs,
-        parser_args.max_retries,
-        parser_args.dut_port,
-        parser_args.stub_port,
+        cli_args.test_plan,
+        cli_args.hil_devs,
+        cli_args.max_retries,
+        cli_args.dut_port,
+        cli_args.stub_port,
     )
-    test_plan_runner.run(parser_args.board, parser_args.test_suite)
+    test_plan_runner.run(cli_args.board, cli_args.test_suite)