From 654ed9a28090a6f6533e8703c33d40122b8ba2e1 Mon Sep 17 00:00:00 2001 From: Aaron Gao Date: Tue, 20 Sep 2016 12:28:59 -0700 Subject: [PATCH] loose the assertion condition of rate_limiter_test 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 --- util/rate_limiter_test.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/util/rate_limiter_test.cc b/util/rate_limiter_test.cc index 211aac03a..0104be3e5 100644 --- a/util/rate_limiter_test.cc +++ b/util/rate_limiter_test.cc @@ -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); } } }