Skip to content

Commit 5aafaf3

Browse files
committed
fix: use target.type instead of target.name for Databricks float precision exclusion
The equality test for fct_test_coverage was failing on Databricks when using custom target names (e.g., 'dx') because the exclusion logic checked target.name instead of target.type. Also added test_to_model_ratio to the exclusion list as it has the same float32 precision issue.
1 parent 7333c0c commit 5aafaf3

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

integration_tests/seeds/tests/tests_seeds.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ seeds:
3232
- total_models
3333
- total_tests
3434
- tested_models
35-
- "{{ 'test_coverage_pct' if not target.name in ['duckdb', 'databricks', 'trino'] else 'tested_models' }}"
36-
- test_to_model_ratio
35+
- "{{ 'test_coverage_pct' if target.type not in ['duckdb', 'databricks', 'trino'] else 'tested_models' }}"
36+
- "{{ 'test_to_model_ratio' if target.type not in ['duckdb', 'databricks', 'trino'] else 'tested_models' }}"
3737
- staging_test_coverage_pct
3838
- intermediate_test_coverage_pct
3939
- marts_test_coverage_pct

0 commit comments

Comments
 (0)