Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix default.vw_card_res_char tests to better match the reality of the view #110

Merged
merged 2 commits into from
Aug 31, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 2 additions & 18 deletions dbt/models/default/schema.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,27 +69,11 @@ models:
tests:
# Unique by card and year
- dbt_utils.unique_combination_of_columns:
name: vw_card_res_char_unique_by_card_and_year
name: vw_card_res_char_unique_by_pin_card_and_year
combination_of_columns:
- pin
- year
- card
config:
error_if: ">913"
wrridgeway marked this conversation as resolved.
Show resolved Hide resolved
# char_recent_renovation correlates with char_renovation
- expression_is_true:
name: vw_card_res_char_renovation_fields_match
# char_renovation can be null, which we count here as falsey
expression: >-
char_recent_renovation = (
case when char_renovation = '1' then true else false end
)
select_columns:
- card
- year
- char_renovation
- char_recent_renovation
config:
error_if: ">73942"
# TODO: Characteristics columns should adhere to pre-determined criteria
- name: default.vw_pin_address
description: '{{ doc("vw_pin_address") }}'
Expand Down
Loading