diff options
| author | Geir Storli <geirstorli@yahoo.no> | 2018-07-19 09:56:01 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-07-19 09:56:01 +0200 |
| commit | cb90c0622c301d16cefa83a19385576dbbca1365 (patch) | |
| tree | 0adb8685c5f25ac356201e47af53b204e84fc25c | |
| parent | aaccad8be603b7727f3a5f7004624e18011bc21d (diff) | |
| parent | 9df684ab0857b573443c00ac7cf1007822d17dfc (diff) | |
Merge pull request #6426 from vespa-engine/balder/fix-incorrect-test
Actually test what you claim
| -rw-r--r-- | searchcore/src/tests/proton/matching/matching_stats_test.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/searchcore/src/tests/proton/matching/matching_stats_test.cpp b/searchcore/src/tests/proton/matching/matching_stats_test.cpp index e467c4b58b7..85960de2021 100644 --- a/searchcore/src/tests/proton/matching/matching_stats_test.cpp +++ b/searchcore/src/tests/proton/matching/matching_stats_test.cpp @@ -290,9 +290,9 @@ TEST("requireThatSoftDoomFacorIsComputedCorrectlyForUpAdjustment") { stats.updatesoftDoomFactor(0.0009, 0.9, 0.1); // hard limits less than 1ms should be ignored EXPECT_EQUAL(1ul, stats.softDoomed()); EXPECT_EQUAL(0.516, stats.softDoomFactor()); - stats.updatesoftDoomFactor(1.0, 0.9, 0.1); // soft limits less than 1ms should be ignored + stats.updatesoftDoomFactor(1.0, 0.0009, 0.1); // soft limits less than 1ms should be ignored EXPECT_EQUAL(1ul, stats.softDoomed()); - EXPECT_EQUAL(0.524, stats.softDoomFactor()); + EXPECT_EQUAL(0.516, stats.softDoomFactor()); stats.softDoomFactor(0.1); stats.updatesoftDoomFactor(1.0, 0.9, 0.001); // Prevent changes above 5% EXPECT_EQUAL(1ul, stats.softDoomed()); |