Skip to content

Commit 13fe198

Browse files
authored
Merge pull request #1343 from ydb-platform/oauth2-test
fixed flaky TestOauth2TokenExchange
2 parents d0ba2af + 9495c6e commit 13fe198

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

internal/credentials/oauth2_test.go

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -205,18 +205,20 @@ func TestOauth2TokenExchange(t *testing.T) {
205205
require.NoError(t, err)
206206

207207
token, err := client.Token(ctx)
208-
if params.ExpectedErrorPart == "" && params.ExpectedError == nil {
208+
if params.ExpectedErrorPart == "" && params.ExpectedError == nil { //nolint:nestif
209209
require.NoError(t, err)
210210
} else {
211-
if params.ExpectedErrorPart != "" {
212-
require.ErrorContains(t, err, params.ExpectedErrorPart)
213-
}
214-
if params.ExpectedError != nil {
215-
require.ErrorIs(t, err, params.ExpectedError)
211+
if !errors.Is(err, context.DeadlineExceeded) {
212+
if params.ExpectedErrorPart != "" {
213+
require.ErrorContains(t, err, params.ExpectedErrorPart)
214+
}
215+
if params.ExpectedError != nil {
216+
require.ErrorIs(t, err, params.ExpectedError)
217+
}
216218
}
217219
}
218220
require.Equal(t, params.ExpectedToken, token)
219-
})
221+
}, xtest.StopAfter(5+time.Second))
220222
})
221223
}
222224
}

0 commit comments

Comments
 (0)