HJ-140 - Fix tests on fides - External-Datastores

andres-torres-marroquin commented 3 weeks ago

Closes #HJ-140

Description Of Changes

Fix tests on fides - External-Datastores

Code Changes

[ ] Updated some passwords to pull them from 1Password intead of GH Secrets. .github/workflows/backend_checks.yml
[ ] Updated noxfiles/setup_tests_nox.py to copy some of those passwords into the running container.
[ ] Updated tests/fixtures/dynamodb_fixtures.py to be more granular on error showing.
[ ] Fixed some tests.

Steps to Confirm

[ ] Look at the checks on the CI.

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 ↗︎

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 5, 2024 0:52am

cypress[bot] commented 3 weeks ago

Codecov Report

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

Project coverage is 85.46%. Comparing base (dc3bfed) to head (c29e041). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #5460 +/- ## ========================================== - Coverage 85.47% 85.46% -0.01% ========================================== Files 384 384 Lines 24119 24119 Branches 2625 2625 ========================================== - Hits 20615 20614 -1 Misses 2950 2950 - Partials 554 555 +1 ```

: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 #10790

Run Properties: Passed #10790 • ba4c825f21: HJ-140 - Fix tests on fides - External-Datastores (#5460)

Project	`fides`
Branch Review	`main`
Run status	`Passed #10790`
Run duration	`00m 38s`
Commit	`ba4c825f21: HJ-140 - Fix tests on fides - External-Datastores (#5460)`
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 ↗︎

ethyca / fides