camzalewski/personal-site
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Fixed potential test failures due to timing.

Browse source
This commit is contained in:
mikestefanello 2022-05-17 08:57:43 -04:00
parent acebcf0ba0
commit 8fa6ebb8cf
2 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
context/context_test.go
View file

@ -16,7 +16,7 @@ func TestIsCanceled(t *testing.T) {
assert.True(t, IsCanceledError(ctx.Err())) assert.True(t, IsCanceledError(ctx.Err()))
ctx, cancel = context.WithTimeout(context.Background(), time.Microsecond) ctx, cancel = context.WithTimeout(context.Background(), time.Microsecond)
time.Sleep(time.Microsecond * 2) time.Sleep(time.Microsecond * 5)
cancel() cancel()
assert.False(t, IsCanceledError(ctx.Err())) assert.False(t, IsCanceledError(ctx.Err()))

5
services/auth_test.go
View file

@ -70,12 +70,13 @@ func TestAuthClient_GetValidPasswordToken(t *testing.T) {
assert.Equal(t, pt.ID, pt2.ID) assert.Equal(t, pt.ID, pt2.ID)
// Expire the token by pushing the date far enough back // Expire the token by pushing the date far enough back
_, err = c.ORM.PasswordToken. count, err := c.ORM.PasswordToken.
Update(). Update().
SetCreatedAt(time.Now().Add(-(c.Config.App.PasswordToken.Expiration + 3600))). SetCreatedAt(time.Now().Add(-(c.Config.App.PasswordToken.Expiration + time.Hour))).
Where(passwordtoken.ID(pt.ID)). Where(passwordtoken.ID(pt.ID)).
Save(context.Background()) Save(context.Background())
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 1, count)
// Expired tokens should not be valid // Expired tokens should not be valid
_, err = c.Auth.GetValidPasswordToken(ctx, usr.ID, pt.ID, token) _, err = c.Auth.GetValidPasswordToken(ctx, usr.ID, pt.ID, token)