Skip to content

Commit 65767e9

Browse files
committed
[oidc] fix a flaky test
1 parent ed3a3a4 commit 65767e9

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

components/public-api-server/pkg/oidc/service_test.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -102,16 +102,16 @@ func TestGetClientConfigFromStartRequest(t *testing.T) {
102102
}
103103

104104
for _, tc := range testCases {
105-
t.Run(tc.Location, func(t *testing.T) {
105+
t.Run(tc.Location, func(te *testing.T) {
106106
request := httptest.NewRequest(http.MethodGet, tc.Location, nil)
107107
config, err := service.GetClientConfigFromStartRequest(request)
108108
if tc.ExpectedError == true {
109-
require.Error(t, err)
109+
require.Error(te, err)
110110
}
111111
if tc.ExpectedError != true {
112-
require.NoError(t, err)
113-
require.NotNil(t, config)
114-
require.Equal(t, tc.ExpectedId, config.ID)
112+
require.NoError(te, err)
113+
require.NotNil(te, config)
114+
require.Equal(te, tc.ExpectedId, config.ID)
115115
}
116116
})
117117
}
@@ -239,7 +239,8 @@ func createConfig(t *testing.T, dbConn *gorm.DB, config *ClientConfig) (db.OIDCC
239239
team, err := db.CreateTeam(context.Background(), dbConn, db.Team{
240240
ID: orgID,
241241
Name: "Org 1",
242-
Slug: "org-1",
242+
// creating random slug using UUID generator, because it's handy here
243+
Slug: uuid.New().String(),
243244
})
244245
require.NoError(t, err)
245246

0 commit comments

Comments
 (0)