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

E2E oauth2 integration #2958

Merged
merged 4 commits into from
Sep 11, 2024
Merged

E2E oauth2 integration #2958

merged 4 commits into from
Sep 11, 2024

Conversation

arbulu89
Copy link
Contributor

Description

Add E2E tests to OAUTH2 integration. As the execution and usage as user doesn't change with OIDC, the tests are exactly the same, except some basic configuration.
I have renamed some files to make them more generic as well

How was this tested?

E2E tests

@arbulu89 arbulu89 added enhancement New feature or request test integration labels Sep 10, 2024
@arbulu89 arbulu89 force-pushed the e2e-oauth2-integration branch from a4dac52 to 895503b Compare September 10, 2024 12:55
@arbulu89 arbulu89 marked this pull request as ready for review September 10, 2024 13:31
Copy link
Member

@CDimonaco CDimonaco left a comment

Choose a reason for hiding this comment

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

Perfect, just commented a nitpick

@arbulu89 arbulu89 merged commit 9fbc611 into main Sep 11, 2024
31 checks passed
@arbulu89 arbulu89 deleted the e2e-oauth2-integration branch September 11, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants