Merged PR 3075: Remove edeposit email flag test
# Description Deactivates the accepted values test that we had in the edeposit verifications staging table. The table is more complex that we initially thought (multiple values can appear comma separated) and we don't really care enough to increase to make it worth it to increase the complexity of the test, so we are removing it. # Checklist - [NA] The edited models and dependants run properly with production data. - [NA] The edited models are sufficiently documented. - [X] The edited models contain PK tests, and I've ran and passed them. - [NA] I have checked for DRY opportunities with other models and docs. - [NA] I've picked the right materialization for the affected models. # Other - [ ] Check if a full-refresh is required after this PR is merged.
This commit is contained in:
commit
c503d797b6
1 changed files with 0 additions and 6 deletions
|
|
@ -77,12 +77,6 @@ models:
|
|||
description: |
|
||||
Null if the email shows no issues, otherwise it details the problems
|
||||
attached to the given email.
|
||||
tests:
|
||||
- accepted_values:
|
||||
values:
|
||||
- "Email address is not deliverable"
|
||||
- "Email domain is not registered"
|
||||
- "Email domain listed as disposable"
|
||||
|
||||
- name: phone_flag
|
||||
data_type: text
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue