HJ-124 - Moving TestBigQueryQueryConfig to its own file

ethyca / fides

The Privacy Engineering & Compliance Framework

https://ethyca.com/docs

Apache License 2.0

361 stars 72 forks source link

HJ-124 - Moving TestBigQueryQueryConfig to its own file #5444

Closed andres-torres-marroquin closed 3 weeks ago

andres-torres-marroquin commented 3 weeks ago

Closes #HJ-124

Description Of Changes

Moving TestBigQueryQueryConfig to its own file

Code Changes

[ ] Added tests/ops/service/connectors/test_bigquery_queryconfig.py with all the tests

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:
- [ ] documentation complete, PR opened in fidesdocs
- [ ] documentation issue created in fidesdocs
- [ ] if there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated
If there are any database migrations:
- [ ] Ensure that your downrev is up to date with the latest revision on main
- [ ] Ensure that your downgrade() migration is correct and works
- [ ] If a downgrade migration is not possible for this change, please call this out in the PR description!

vercel[bot] commented 3 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

cypress[bot] commented 3 weeks ago

fides Run #10730

Run Properties: Passed #10730 • e2648ed47f ℹ️: Merge 20dc63b0f8ff584ff1c81fb0a966f279313df8f7 into d60ec78344f18bd0118e088612f4...

Project	`fides`
Branch Review	`refs/pull/5444/merge`
Run status	`Passed #10730`
Run duration	`00m 38s`
Commit	`e2648ed47f ℹ️: Merge 20dc63b0f8ff584ff1c81fb0a966f279313df8f7 into d60ec78344f18bd0118e088612f4...`
Committer	`Andres Torres`
View all properties for this run ↗︎

Test results
Failures	`0`
Flaky	`0`
Pending	`0`
Skipped	`0`
Passing	`4`
View all changes introduced in this branch ↗︎

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.47%. Comparing base (d60ec78) to head (20dc63b). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #5444 +/- ## ======================================= Coverage 85.47% 85.47% ======================================= Files 384 384 Lines 24118 24118 Branches 2624 2624 ======================================= Hits 20614 20614 Misses 2950 2950 Partials 554 554 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cypress[bot] commented 3 weeks ago

fides Run #10731

Run Properties: Passed #10731 • 652aa76dbc: HJ-124 - Moving TestBigQueryQueryConfig to its own file (#5444)

Project	`fides`
Branch Review	`main`
Run status	`Passed #10731`
Run duration	`00m 40s`
Commit	`652aa76dbc: HJ-124 - Moving TestBigQueryQueryConfig to its own file (#5444)`
Committer	`Andres Torres`
View all properties for this run ↗︎

Test results
Failures	`0`
Flaky	`0`
Pending	`0`
Skipped	`0`
Passing	`4`
View all changes introduced in this branch ↗︎