search

zendesk / prop

Puts a cork in their requests
Apache License 2.0
120 stars 13 forks source link

the concurrency test in test_limiter.rb seems flaky #36

Open vanchi-zendesk opened 5 years ago

vanchi-zendesk commented 5 years ago

This test here: https://github.com/zendesk/prop/blob/master/test/test_limiter.rb#L29 seems to be flaky and occasionally returns 1 instead of 2.

sergey-alekseev commented 5 years ago

Introduced by me in https://github.com/zendesk/prop/pull/33. Needs to be improved.