Skip to content

Commit

Permalink
fix TestReadBadZookieFlake
Browse files Browse the repository at this point in the history
occasionally, the timetamps in the changelog can be behind now-gcwindow
by 5-7ms. bumping the gc window up a bit stabilizes the test

Signed-off-by: Evan Cordell <cordell.evan@gmail.com>
  • Loading branch information
ecordell committed Oct 4, 2021
1 parent 9df7471 commit 2089465
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/services/v0/acl_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ func TestRead(t *testing.T) {
func TestReadBadZookie(t *testing.T) {
require := require.New(t)

client, stop, revision, _ := newACLServicer(require, 0, 10*time.Millisecond, 0)
client, stop, revision, _ := newACLServicer(require, 0, 20*time.Millisecond, 0)
defer stop()

_, err := client.Read(context.Background(), &v0.ReadRequest{
Expand All @@ -266,7 +266,7 @@ func TestReadBadZookie(t *testing.T) {
require.NoError(err)

// Wait until the gc window expires
time.Sleep(20 * time.Millisecond)
time.Sleep(40 * time.Millisecond)

_, err = client.Read(context.Background(), &v0.ReadRequest{
Tuplesets: []*v0.RelationTupleFilter{
Expand Down

0 comments on commit 2089465

Please sign in to comment.