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

test: implement framework for integration tests (DNA-19347: DNA-19404) #72

Merged
merged 4 commits into from
Oct 11, 2023

Conversation

cverhoef
Copy link
Contributor

@cverhoef cverhoef commented Oct 6, 2023

Description

Added a dbt project "Integration tests" that consists of the following:

  • A standard dbt_project.yml file without any specific settings.
  • The packages.yml that points to the local version of pm-utils.
  • A models folder where we can add .sql files with tests. Currently only one file exists for testing the concat() function. It creates a table with "actual values" using the function and "expected values"
  • A schema.yml file where we test the result of the models. In this example, are the actual and expected values the same.
  • To support testing whether values are the same, a test macro is implemented in the macros.

Note: when we merge this, we should add the additional files to the .zipignore in the AppTemplates repo: https://github.com/UiPath/ProcessMining-App-Templates/blob/main/.pipelines/.zipignore.

Release

  • Direct release (main)
  • Merge to dev (or other) branch
    • Why:

Did you consider?

  • Does it Work on Automation Suite / SQL Server
  • Does it Work on Automation Cloud / Snowflake
  • [] What is the performance impact?
  • [] The version number in dbt_project.yml

@cverhoef cverhoef marked this pull request as draft October 6, 2023 14:10
@cverhoef cverhoef changed the title test: implement framework for integration tests test: implement framework for integration tests (DNA-19347: DNA-19404) Oct 10, 2023
@cverhoef cverhoef marked this pull request as ready for review October 10, 2023 12:21
@cverhoef cverhoef merged commit 32ff042 into main Oct 11, 2023
3 checks passed
@cverhoef cverhoef deleted the test/implement_framework branch October 11, 2023 11:56
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.

2 participants