Skip to content

test: add full OIDC fake IDP #9317

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.

Sign up for GitHub

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

Merged
merged 20 commits into from
Aug 25, 2023
Merged

test: add full OIDC fake IDP #9317

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
46c2c62
test: implement fake OIDC provider with full functionality
Emyrk Aug 23, 2023
7d16bc1
full idp fake for oidc
Emyrk Aug 24, 2023
8bfd428
Refactor existing tests
Emyrk Aug 24, 2023
da69b16
create refresh helper
Emyrk Aug 24, 2023
f906fae
Merge remote-tracking branch 'origin/main' into stevenmasley/full_oid…
Emyrk Aug 24, 2023
a8a9633
refactor all old tests, delete old fake
Emyrk Aug 24, 2023
a1b716b
comments
Emyrk Aug 24, 2023
dffeb16
Fix import
Emyrk Aug 24, 2023
3be7d31
Fix comments
Emyrk Aug 24, 2023
4b31e28
Fix issue with expirary json
Emyrk Aug 24, 2023
722e36f
fixup! Fix issue with expirary json
Emyrk Aug 24, 2023
00b4760
Linting
Emyrk Aug 24, 2023
1c7e8b4
Pr feedback, add authorized redirect urls
Emyrk Aug 25, 2023
930e7dd
PR Feedback
Emyrk Aug 25, 2023
c61ceb9
Linting
Emyrk Aug 25, 2023
6e6084b
Linting
Emyrk Aug 25, 2023
9a600c0
Move sync map to utils
Emyrk Aug 25, 2023
6030a5f
Fix panic
Emyrk Aug 25, 2023
2b0ff9c
Linting
Emyrk Aug 25, 2023
64866da
Merge remote-tracking branch 'origin/main' into stevenmasley/full_oid…
Emyrk Aug 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix import
  • Loading branch information
@Emyrk
Emyrk committed Aug 24, 2023
commit dffeb16bb4ef240181d9ad3ddcfd14b37df30b71
4 changes: 1 addition & 3 deletions coderd/coderdtest/oidctest/idp.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,6 @@ func (f *FakeIDP) authenticateOIDClientRequest(t testing.TB, req *http.Request)
values, err := url.ParseQuery(string(data))
if !assert.NoError(t, err, "parse token request values") {
return nil, xerrors.New("invalid token request")

}

if !assert.Equal(t, f.clientID, values.Get("client_id"), "client_id mismatch") {
Expand Down Expand Up @@ -419,7 +418,7 @@ func (f *FakeIDP) httpHandler(t testing.TB) http.Handler {
state := r.URL.Query().Get("state")

scope := r.URL.Query().Get("scope")
var _ = scope
_ = scope

responseType := r.URL.Query().Get("response_type")
switch responseType {
Expand Down Expand Up @@ -557,7 +556,6 @@ func (f *FakeIDP) httpHandler(t testing.TB) http.Handler {
http.Error(rw, fmt.Sprintf("invalid user info request: %s", err.Error()), http.StatusBadRequest)
return
}
var _ = token

email, ok := f.accessTokens.Load(token)
if !ok {
Expand Down
14 changes: 2 additions & 12 deletions coderd/userauth_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,6 @@ import (
"github.com/coder/coder/v2/coderd/database/dbgen"
"github.com/coder/coder/v2/coderd/database/dbtestutil"
"github.com/coder/coder/v2/codersdk"
"github.com/coder/coder/v2/enterprise/coderd/coderdenttest"
"github.com/coder/coder/v2/enterprise/coderd/license"
"github.com/coder/coder/v2/testutil"
)

Expand All @@ -49,16 +47,8 @@ func TestOIDCOauthLoginWithExisting(t *testing.T) {
cfg.IgnoreUserInfo = true
})

client, _, api, _ := coderdenttest.NewWithAPI(t, &coderdenttest.Options{
Options: &coderdtest.Options{
OIDCConfig: cfg,
},
LicenseOptions: &coderdenttest.LicenseOptions{
Features: license.Features{
codersdk.FeatureUserRoleManagement: 1,
codersdk.FeatureTemplateRBAC: 1,
},
},
client, _, api := coderdtest.NewWithAPI(t, &coderdtest.Options{
OIDCConfig: cfg,
})

const username = "alice"
Expand Down