Skip to content

Commit

Permalink
Flaky: TestSDKServerWatchGameServerFeatureSDKWatchSendOnExecute
Browse files Browse the repository at this point in the history
Switching default: case to a timeout, since due to the asynchronous
nature of event processing, it could take a moment or two for the extra
event to be processed.

This is particularly true on the CI system, where there is a lot of
processing going on, and it may take more time than usual to process.
  • Loading branch information
markmandel committed Jun 30, 2020
1 parent 04f546f commit a4edd9b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/sdkserver/sdkserver_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -718,7 +718,7 @@ func TestSDKServerWatchGameServerFeatureSDKWatchSendOnExecute(t *testing.T) {
} else {
assert.Fail(t, "Channel is closed!")
}
default:
case <-time.After(30 * time.Second):
t.Log("No gameserver in the stream, moving on.")
}
}
Expand Down

0 comments on commit a4edd9b

Please sign in to comment.