-
Notifications
You must be signed in to change notification settings - Fork 49
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
Migrate to new adapter testing framework #65
Conversation
9a7d54c
to
0fb1a03
Compare
b581d52
to
a3d10f6
Compare
a3d10f6
to
668d54f
Compare
668d54f
to
68a7d27
Compare
68a7d27
to
48c5c93
Compare
@hovaesco are we planning to move to a structure similar to the dbt-snowflake adapter ? |
Snowflake uses old tests structure in the folder you sent. We are moving to the new one. |
48c5c93
to
6736940
Compare
211d0b0
to
fd4954f
Compare
This PR looks very good. A few minor comments are still open. |
fd4954f
to
d967d92
Compare
# seed seeds | ||
results = run_dbt(["seed"], expect_pass=True) | ||
assert len(results) == 1 | ||
# run models two times to check on_table_exists = 'drop' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could use the same mechanism as in test-incremental to change the seed before running the second time. This way, the test looks more realistic.
…ng-framework-v1.1 Migrate to new adapter testing framework
Overview
Checklist
README.md
updated and added information about my changeCHANGELOG.md
updated and added information about my change