Fix wrong way to catch exception in test

Author: jlatusek

scripts/py_matter_yamltests/matter_yamltests/runner.py

@@ -172,61 +172,56 @@ async def _run_with_timeout(self, parser: TestParser, config: TestRunnerConfig):
 
     async def _run(self, parser: TestParser, config: TestRunnerConfig):
         status = True
-        try:
-            hooks = config.hooks
-            hooks.test_start(parser.filename, parser.name, parser.tests.count)
-
-            test_duration = 0
-            for idx, request in enumerate(parser.tests):
-                if not request.is_pics_enabled:
-                    hooks.step_skipped(request.label, request.pics)
-                    continue
-                elif not config.adapter:
-                    hooks.step_start(request)
-                    hooks.step_unknown()
-                    continue
-                elif config.pseudo_clusters.is_manual_step(request):
-                    hooks.step_start(request)
-                    await hooks.step_manual()
-                    continue
-                else:
-                    hooks.step_start(request)
-
-                start = time.time()
-                if config.pseudo_clusters.supports(request):
-                    responses, logs = await config.pseudo_clusters.execute(request, parser.definitions)
-                else:
-                    encoded_request = config.adapter.encode(request)
-                    encoded_response = await self.execute(encoded_request)
-                    responses, logs = config.adapter.decode(encoded_response)
-                duration = round((time.time() - start) * 1000, 2)
-                test_duration += duration
-
-                logger = request.post_process_response(responses)
-
-                if logger.is_failure():
-                    hooks.step_failure(logger, logs, duration,
-                                       request, responses)
-                else:
-                    hooks.step_success(logger, logs, duration, request)
-
-                if logger.is_failure() and config.options.stop_on_error:
-                    status = False
-                    break
-
-                if logger.warnings and config.options.stop_on_warning:
-                    status = False
-                    break
-
-                if (idx + 1) == config.options.stop_at_number:
-                    break
-
-                if config.options.delay_in_ms:
-                    await asyncio.sleep(config.options.delay_in_ms / 1000)
-
-            hooks.test_stop(round(test_duration))
-
-        except Exception as exception:
-            status = exception
-        finally:
-            return status
+        hooks = config.hooks
+        hooks.test_start(parser.filename, parser.name, parser.tests.count)
+
+        test_duration = 0
+        for idx, request in enumerate(parser.tests):
+            if not request.is_pics_enabled:
+                hooks.step_skipped(request.label, request.pics)
+                continue
+            elif not config.adapter:
+                hooks.step_start(request)
+                hooks.step_unknown()
+                continue
+            elif config.pseudo_clusters.is_manual_step(request):
+                hooks.step_start(request)
+                await hooks.step_manual()
+                continue
+            else:
+                hooks.step_start(request)
+
+            start = time.time()
+            if config.pseudo_clusters.supports(request):
+                responses, logs = await config.pseudo_clusters.execute(request, parser.definitions)
+            else:
+                encoded_request = config.adapter.encode(request)
+                encoded_response = await self.execute(encoded_request)
+                responses, logs = config.adapter.decode(encoded_response)
+            duration = round((time.time() - start) * 1000, 2)
+            test_duration += duration
+
+            logger = request.post_process_response(responses)
+
+            if logger.is_failure():
+                hooks.step_failure(logger, logs, duration,
+                                    request, responses)
+            else:
+                hooks.step_success(logger, logs, duration, request)
+
+            if logger.is_failure() and config.options.stop_on_error:
+                status = False
+                break
+
+            if logger.warnings and config.options.stop_on_warning:
+                status = False
+                break
+
+            if (idx + 1) == config.options.stop_at_number:
+                break
+
+            if config.options.delay_in_ms:
+                await asyncio.sleep(config.options.delay_in_ms / 1000)
+
+        hooks.test_stop(round(test_duration))
+        return status