-
-
Notifications
You must be signed in to change notification settings - Fork 813
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tidy up #519
Tidy up #519
Conversation
@@ -82,7 +82,7 @@ func TestAttemptWithDelay(t *testing.T) { | |||
}) | |||
|
|||
is.Equal(iter1, 1) | |||
is.Greater(dur1, 0*time.Millisecond) | |||
is.GreaterOrEqual(dur1, 0*time.Millisecond) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How could be invalid?
I mean this test will always pass. A duration is always positive, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be my hardware or the fact that I'm running on Windows, but for me the first call to AttemptWithDelay
(which succeeds immediately) returns within the minimum monotonic time period, so these tests always fail.
@@ -187,7 +187,7 @@ func TestAttemptWhileWithDelay(t *testing.T) { | |||
}) | |||
|
|||
is.Equal(iter1, 1) | |||
is.Greater(dur1, 0*time.Millisecond) | |||
is.GreaterOrEqual(dur1, 0*time.Millisecond) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same?
(tests are broken) |
Were you aware that I'm Australian before posting that gif? 😛 Don't worry, we don't take ourselves too seriously! |
Ahah, sorry. I missed that little detail. 😳 🤣 |
Just some minor tidy up - mostly spelling, grammar, newline inconsistencies, trailing whitespace, redundant types, etc.