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

La 202 update testing for flushing oauth tokens #5587

Merged
merged 8 commits into from
Dec 12, 2024

Conversation

Vagoasdf
Copy link
Contributor

@Vagoasdf Vagoasdf commented Dec 10, 2024

Closes LA#202

Description Of Changes

Updating the refresh tests and ouath2 authorization strategies schemas

Code Changes

  • Adding client credentials to the schema of authorization strategies
  • improving saas_example_fixtures.py to include said authorization

Steps to Confirm

  1. This changes testing, so checking that all CI tests pass sucessfully

Pre-Merge Checklist

  • Issue requirements met

  • All CI pipelines succeeded

  • CHANGELOG.md updated

  • Followup issues:

    • No followup issues
  • Database migrations:

    • No migrations
  • Documentation:

    • No documentation updates required

Copy link

vercel bot commented Dec 10, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
fides-plus-nightly ⬜️ Ignored (Inspect) Visit Preview Dec 11, 2024 0:45am

@Vagoasdf Vagoasdf force-pushed the LA-202_update_testing_for_flushing_oauth_tokens branch from 5e5c998 to bd8acb8 Compare December 10, 2024 15:53
Copy link

cypress bot commented Dec 10, 2024

fides    Run #11420

Run Properties:  status check passed Passed #11420  •  git commit 2ccc004210 ℹ️: Merge 83037d7a466d2420a78cf3ff9411b700f4dac091 into e6609fa45da1b211680902f3811c...
Project fides
Branch Review refs/pull/5587/merge
Run status status check passed Passed #11420
Run duration 00m 39s
Commit git commit 2ccc004210 ℹ️: Merge 83037d7a466d2420a78cf3ff9411b700f4dac091 into e6609fa45da1b211680902f3811c...
Committer Bruno Gutierrez Rios
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 4
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

@Vagoasdf Vagoasdf marked this pull request as ready for review December 11, 2024 12:46
@Vagoasdf Vagoasdf requested a review from galvana December 11, 2024 14:38
Copy link
Contributor

@galvana galvana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for cleaning up the models and making things more consistent!

@Vagoasdf Vagoasdf merged commit 7e1832c into main Dec 12, 2024
35 of 37 checks passed
@Vagoasdf Vagoasdf deleted the LA-202_update_testing_for_flushing_oauth_tokens branch December 12, 2024 16:19
Copy link

cypress bot commented Dec 12, 2024

fides    Run #11463

Run Properties:  status check passed Passed #11463  •  git commit 7e1832c802: La 202 update testing for flushing oauth tokens (#5587)
Project fides
Branch Review main
Run status status check passed Passed #11463
Run duration 00m 35s
Commit git commit 7e1832c802: La 202 update testing for flushing oauth tokens (#5587)
Committer Bruno Gutierrez Rios
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 4
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

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