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

feat: Add checks for invalid manifest jsonnet configuration #2188

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Matovidlo
Copy link
Contributor

@Matovidlo Matovidlo commented Dec 18, 2024

Jira: PSGO-955

Changes:

  • Add new test for testing bigquery writer template using if conditions on HasProjectBackend function
  • Add new test for testing snowflake writer template using if conditions on HasProjectBackend function
  • Adjust manifest checks to not return panic

First version of template returns errors as the len configurations == 0
Second version of template returns error as the configurations == 1 and
the first configuration is `nil`.
Third version of template returns bigquery writer templated side by side
with snowflake writer.
This test should be duplicated for `snowflake` backend and swapped
checks within if statement.
Copy bigquery test case for snowflake initial state.
First we check that configurations == 0 because of wrong if statement
Secondly we check that configurations == 1 because of wrong if statement
Thirdly the template is created appropriatelly for backend selected
…r when records are missing

Used on empty manifest file which will be filled using `Save()` Method.
Used on non empty manifest to store new records which will be filled
using `Save()` Method.
@Matovidlo Matovidlo force-pushed the feat-add-checks-for-invalid-manifest-jsonnet-configuration branch from 96b9c5a to 49f8406 Compare December 19, 2024 09:43
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.

1 participant