Skip to content

Fixes 35274: Check test parameters against their definitions and fix open-window and boolean-set runs - #35629

Merged
Shreyansh100704 merged 14 commits into
mainfrom
fix/dq-generated-test-parameters
Oct 17, 2026
Merged

Fixes 35274: Check test parameters against their definitions and fix open-window and boolean-set runs#35629
Shreyansh100704 merged 14 commits into
mainfrom
fix/dq-generated-test-parameters

test(dq): create IT fixtures with a definition that supports their BI…

41aa53c
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ingestion] SonarCloud Code Analysis succeeded Oct 17, 2026 in 2m 21s

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
68.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 89 in ingestion/tests/unit/observability/data_quality/validations/test_utils.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AaFFMU-9CiUwZk3hZofi&open=AaFFMU-9CiUwZk3hZofi&pullRequest=35629

Check warning on line 55 in ingestion/src/metadata/data_quality/validations/mixins/pandas_validator_mixin.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Remove this redundant return. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AaFFMVSWCiUwZk3hZofk&open=AaFFMVSWCiUwZk3hZofk&pullRequest=35629

Check warning on line 79 in ingestion/tests/unit/observability/data_quality/validations/test_utils.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this exception test to have only one invocation possibly throwing an exception. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AaFFMU-9CiUwZk3hZofg&open=AaFFMU-9CiUwZk3hZofg&pullRequest=35629

Check warning on line 84 in ingestion/tests/unit/observability/data_quality/validations/test_utils.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this exception test to have only one invocation possibly throwing an exception. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AaFFMU-9CiUwZk3hZofh&open=AaFFMU-9CiUwZk3hZofh&pullRequest=35629

Check warning on line 90 in ingestion/tests/unit/observability/data_quality/validations/test_utils.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AaFFMU-9CiUwZk3hZofj&open=AaFFMU-9CiUwZk3hZofj&pullRequest=35629
View more details on SonarQubeCloud