diff options
| author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-01-29 11:40:10 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-29 11:40:10 +0100 |
| commit | cbe4909b5d23dcee552d6777733387966053d25a (patch) | |
| tree | 568d7725f7822989f388d775398936e106260dd5 /slobrok/src/tests/backoff/testbackoff.cpp | |
| parent | 1892897a2ab3e543d0f6511436e4764269d38d50 (diff) | |
| parent | ea22e1cc3c456936d4b2fc5a470ec7957d57c0f4 (diff) | |
Merge pull request #25777 from vespa-engine/arnej/avoid-silly-warning
Arnej/avoid warning
Diffstat (limited to 'slobrok/src/tests/backoff/testbackoff.cpp')
| -rw-r--r-- | slobrok/src/tests/backoff/testbackoff.cpp | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/slobrok/src/tests/backoff/testbackoff.cpp b/slobrok/src/tests/backoff/testbackoff.cpp index 8125d8625a8..50d68b72adf 100644 --- a/slobrok/src/tests/backoff/testbackoff.cpp +++ b/slobrok/src/tests/backoff/testbackoff.cpp @@ -17,9 +17,14 @@ Test::Main() TEST_INIT("backoff_test"); BackOff one; - EXPECT_TRUE(one.shouldWarn()); + EXPECT_FALSE(one.shouldWarn()); EXPECT_EQUAL(0.500, one.get()); - for (int i = 2; i < 41; i++) { + EXPECT_FALSE(one.shouldWarn()); + EXPECT_EQUAL(1.000, one.get()); + EXPECT_FALSE(one.shouldWarn()); + EXPECT_EQUAL(1.500, one.get()); + EXPECT_TRUE(one.shouldWarn()); + for (int i = 4; i < 41; i++) { EXPECT_EQUAL(0.5 * i, one.get()); } for (int i = 1; i < 1000; i++) { @@ -28,22 +33,24 @@ Test::Main() TEST_FLUSH(); BackOff two; + EXPECT_FALSE(two.shouldWarn()); for (int i = 1; i < 50; i++) { double expect = 0.5 * i; if (expect > 20.0) expect = 20.0; EXPECT_EQUAL(expect, two.get()); - if (i == 1 || i == 7 || i == 18) { + if (i == 3 || i == 8 || i == 18) { EXPECT_TRUE(two.shouldWarn()); } else { EXPECT_FALSE(two.shouldWarn()); } } two.reset(); + EXPECT_FALSE(two.shouldWarn()); for (int i = 1; i < 50; i++) { double expect = 0.5 * i; if (expect > 20.0) expect = 20.0; EXPECT_EQUAL(expect, two.get()); - if (i == 1 || i == 7 || i == 18) { + if (i == 3 || i == 8 || i == 18) { EXPECT_TRUE(two.shouldWarn()); } else { EXPECT_FALSE(two.shouldWarn()); |