From 49bdf1a112a63d039067ed1ed755a525c4786f6b Mon Sep 17 00:00:00 2001 From: Pere Miquel Brull Date: Mon, 22 Sep 2025 16:34:36 +0200 Subject: [PATCH] MINOR - Report status for tests that blow up (#23326) * MINOR - Report status for tests that blow up * format --- .../data_quality/interface/test_suite_interface.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/ingestion/src/metadata/data_quality/interface/test_suite_interface.py b/ingestion/src/metadata/data_quality/interface/test_suite_interface.py index 6a7cc8cf848..3f82f09dd63 100644 --- a/ingestion/src/metadata/data_quality/interface/test_suite_interface.py +++ b/ingestion/src/metadata/data_quality/interface/test_suite_interface.py @@ -27,7 +27,7 @@ from metadata.data_quality.validations.runtime_param_setter.param_setter_factory ) from metadata.generated.schema.entity.data.table import Table from metadata.generated.schema.entity.services.databaseService import DatabaseConnection -from metadata.generated.schema.tests.basic import TestCaseResult +from metadata.generated.schema.tests.basic import TestCaseResult, TestCaseStatus from metadata.generated.schema.tests.testCase import TestCase from metadata.generated.schema.tests.testDefinition import TestDefinition from metadata.ingestion.ometa.ometa_api import OpenMetadata @@ -134,10 +134,16 @@ class TestSuiteInterface(ABC): try: return validator.run_validation() except Exception as err: - logger.error( + message = ( f"Error executing {test_case.testDefinition.fullyQualifiedName} - {err}" ) - raise RuntimeError(err) + logger.exception(message) + return validator.get_test_case_result_object( + validator.execution_date, + TestCaseStatus.Aborted, + message, + [], + ) def _get_table_config(self): """Get the sampling configuration for the data quality tests"""