Skip to content

Commit

Permalink
loose the assertion condition of rate_limiter_test
Browse files Browse the repository at this point in the history
Summary: 0.9 can make the test flaky since just found one test fail with 0.88

Test Plan: make all check

Reviewers: sdong, andrewkr

Reviewed By: andrewkr

Subscribers: andrewkr, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D63939
  • Loading branch information
lightmark committed Sep 20, 2016
1 parent e4d3f5d commit 654ed9a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions util/rate_limiter_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ TEST_F(RateLimiterTest, Rate) {
arg.request_size - 1, target / 1024, rate / 1024,
elapsed / 1000000.0);

ASSERT_GE(rate / target, 0.9);
ASSERT_LE(rate / target, 1.1);
ASSERT_GE(rate / target, 0.85);
ASSERT_LE(rate / target, 1.15);
}
}
}
Expand Down

0 comments on commit 654ed9a

Please sign in to comment.