Skip to content

Commit a343da9

Browse files
committed
Fix unit test
1 parent a6fdac8 commit a343da9

File tree

2 files changed

+19
-5
lines changed

2 files changed

+19
-5
lines changed

coderd/userauth_test.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88
"io"
99
"net/http"
1010
"net/url"
11+
"strings"
1112
"testing"
1213
"time"
1314

@@ -244,7 +245,7 @@ func TestUserOAuth2Github(t *testing.T) {
244245
resp := oauth2Callback(t, client)
245246
require.Equal(t, http.StatusTemporaryRedirect, resp.StatusCode)
246247

247-
client.SessionToken = resp.Cookies()[0].Value
248+
client.SessionToken = authCookieValue(resp.Cookies())
248249
user, err := client.User(context.Background(), "me")
249250
require.NoError(t, err)
250251
require.Equal(t, "kyle@coder.com", user.Email)
@@ -374,7 +375,7 @@ func TestUserOIDC(t *testing.T) {
374375
defer cancel()
375376

376377
if tc.Username != "" {
377-
client.SessionToken = resp.Cookies()[0].Value
378+
client.SessionToken = authCookieValue(resp.Cookies())
378379
user, err := client.User(ctx, "me")
379380
require.NoError(t, err)
380381
require.Equal(t, tc.Username, user.Username)
@@ -503,3 +504,12 @@ func oidcCallback(t *testing.T, client *codersdk.Client) *http.Response {
503504
func i64ptr(i int64) *int64 {
504505
return &i
505506
}
507+
508+
func authCookieValue(cookies []*http.Cookie) string {
509+
for _, cookie := range cookies {
510+
if strings.Contains(cookie.Name, "token") {
511+
return cookie.Value
512+
}
513+
}
514+
return ""
515+
}

coderd/users_test.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -281,10 +281,14 @@ func TestPostLogout(t *testing.T) {
281281
require.Equal(t, http.StatusOK, res.StatusCode)
282282

283283
cookies := res.Cookies()
284-
require.Len(t, cookies, 1, "Exactly one cookie should be returned")
284+
require.Len(t, cookies, 2, "Exactly one cookie should be returned")
285285

286-
require.Equal(t, codersdk.SessionTokenKey, cookies[0].Name, "Cookie should be the auth cookie")
287-
require.Equal(t, -1, cookies[0].MaxAge, "Cookie should be set to delete")
286+
for _, cookie := range cookies {
287+
if cookie.Name == codersdk.SessionTokenKey {
288+
require.Equal(t, codersdk.SessionTokenKey, cookies[0].Name, "Cookie should be the auth cookie")
289+
require.Equal(t, -1, cookies[0].MaxAge, "Cookie should be set to delete")
290+
}
291+
}
288292

289293
_, err = client.GetAPIKey(ctx, admin.UserID.String(), keyID)
290294
sdkErr := &codersdk.Error{}

0 commit comments

Comments
 (0)