-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
convert adapter table materialization test #6595
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
{{ | ||
config( | ||
materialized = "table", | ||
sort = 'first_name', |
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.
This is the only specific part for this test, otherwise we could actually remove it
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.
Mostly LGTM, maybe add a comment to explain what a mat test is?
@@ -0,0 +1,97 @@ | |||
import pytest |
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.
Nit: Rename file to test_table_materialization.py
.
return {'materialized.sql': model_sql} | ||
|
||
def test_table_materialization_sort_dist_no_op(self, project): | ||
# basic table mat test, sort and dist is not supported by postgres so the result table would still be same as input |
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.
Nit: mat
-> materialization
.
resolves #6576
Description
move adapter ddl test to new test framework. Since it is adapter specific so moving to adapter folder
Checklist
changie new
to create a changelog entry