Skip to content

Commit

Permalink
test: Modified refresh_token flow test to catch regressions
Browse files Browse the repository at this point in the history
  • Loading branch information
vivshankar authored and aeneasr committed Feb 12, 2024
1 parent aa20d1f commit 23b3ac5
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions handler/oauth2/flow_refresh_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,11 @@ func TestRefreshFlow_HandleTokenEndpointRequest(t *testing.T) {
require.NoError(t, err)

areq.Form.Add("refresh_token", token)

orReqID := areq.GetID() + "_OR"
areq.Form.Add("or_request_id", orReqID)
err = store.CreateRefreshTokenSession(context.Background(), sig, &fosite.Request{
ID: orReqID,
Client: areq.Client,
GrantedScope: fosite.Arguments{"foo", "offline"},
RequestedScope: fosite.Arguments{"foo", "bar", "offline"},
Expand All @@ -177,6 +181,7 @@ func TestRefreshFlow_HandleTokenEndpointRequest(t *testing.T) {
assert.NotEqual(t, url.Values{"foo": []string{"bar"}}, areq.Form)
assert.Equal(t, time.Now().Add(time.Hour).UTC().Round(time.Second), areq.GetSession().GetExpiresAt(fosite.AccessToken))
assert.Equal(t, time.Now().Add(time.Hour).UTC().Round(time.Second), areq.GetSession().GetExpiresAt(fosite.RefreshToken))
assert.EqualValues(t, areq.Form.Get("or_request_id"), areq.GetID(), "Requester ID should be replaced based on the refresh token session")
},
},
{
Expand Down

0 comments on commit 23b3ac5

Please sign in to comment.