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

Add contract testing for OIDC/OAuth2 protected resources #41

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

lbroudoux
Copy link
Member

@lbroudoux lbroudoux commented Jan 28, 2025

Pull Request

Proposed Changes

Complete the OAuth2 related model classes to allow contract-testing of OIDC/OAuth2 protected API

See issue #40

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run dotnet test and ensure you have test coverage for the lines you are introducing
  • run dotnet husky run and fix any issues that you have introduced

Reviewer

  • Label as either feature, fix, documentation, enhancement, maintenance or breaking

@lbroudoux lbroudoux added this to the 0.2.0 milestone Jan 28, 2025
Copy link

👋 @lbroudoux

Welcome to the Microcks community! 💖

Thanks and congrats 🎉 for opening your first pull request here! Be sure to follow the pull request template or please update it accordingly.

Hope you have a great time there!

Copy link
Member

@SebastienDegodez SebastienDegodez left a comment

Choose a reason for hiding this comment

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

I added some of my preferences.
Well done for a 1st PR in .NET

@lbroudoux lbroudoux added component/documentation Improvements or additions to documentation kind/enhancement Enhancement of existing feature component/runtime Runtime behavior of test container labels Jan 30, 2025
@lbroudoux lbroudoux merged commit af189ad into microcks:main Jan 30, 2025
9 checks passed
@lbroudoux lbroudoux deleted the feat/add_oidc_support branch January 30, 2025 12:11
Copy link

🎉 @lbroudoux

You are now a Microcks community contributor! 💖

Thanks and congrats 🚀 on merging your first pull request! We are delighted and very proud of you! 👏

📢 If you're using Microcks in your organization, please add your company name to this list. 🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact.

If you need to know why and how to add yourself to the list, please read the blog post "Join the Microcks Adopters list and Empower the vibrant open source Community 🙌"

Kudos and please keep going, we need you 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/documentation Improvements or additions to documentation component/runtime Runtime behavior of test container kind/enhancement Enhancement of existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants