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

CSS-9575 Add device flow integration tests #1268

Merged
merged 5 commits into from
Jul 17, 2024

Conversation

kian99
Copy link
Contributor

@kian99 kian99 commented Jul 12, 2024

Description

This PR adds integration tests for login via the device flow testing that the session token login provider works as expected and extends our model proxy test suite. The JIMM test suite now has a channel that simulates the polling JIMM does when it queries for an access token from the OIDC server.

In order to do this I've extended the mock OAuthAuthenticator to implement the device flow methods.

Also added:

  • A test for the model proxy that makes a juju status call.
  • A test for the model proxy that tests what happens when a user tries to access a model they don't have permission to access.

Partially addresses CSS-9575

Engineering checklist

Check only items that apply

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

@kian99 kian99 requested a review from a team as a code owner July 12, 2024 09:09
@kian99 kian99 force-pushed the CSS-9575-login-loop branch from 5c14ff0 to 2f71720 Compare July 15, 2024 16:36
internal/auth/oauth2_test.go Outdated Show resolved Hide resolved
internal/auth/oauth2_test.go Show resolved Hide resolved
@kian99 kian99 merged commit 10b1aa2 into canonical:v3 Jul 17, 2024
3 checks passed
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.

3 participants