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

Development #87

Merged
merged 19 commits into from
Jul 3, 2023
Merged

Development #87

merged 19 commits into from
Jul 3, 2023

Conversation

mjirv
Copy link
Owner

@mjirv mjirv commented Jul 3, 2023

This is a:

  • bug fix PR with no breaking changes
  • new functionality
  • a breaking change

Description & motivation

Checklist

  • I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)
    • BigQuery
    • Postgres
    • Redshift
    • Snowflake
  • I have updated the README.md (if applicable)
  • I have added tests & descriptions to my macros (and models if applicable)

Add support for testing incremental models
* Added all existing unit tests in macro form.

* Refactored the test blocks into macros blocks.
Replaced the original code inside the test blocks with the new macros.

* Adjusted the render logic for the the input mapping.
This has to be done for the values and the keys. This is necessary so that the unit tests can be executed as macros.

* Added explanations to the README on how to setup the tests as macros.

---------

Co-authored-by: Leopold Gabelmann <[email protected]>
@mjirv mjirv merged commit 5a4c311 into main Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants