diff options
| author | Matthew Saltz <matthew.saltz@mongodb.com> | 2018-10-08 18:29:05 -0400 |
|---|---|---|
| committer | Matthew Saltz <matthew.saltz@mongodb.com> | 2018-10-10 11:10:16 -0400 |
| commit | 0d4e8a8b21b7ba1e765d42feabafa7e835686b08 (patch) | |
| tree | 958c08439e91da97083022b0592a2f64020444ac | |
| parent | 1b51df8420ef3c57afb2d753d7188814e90eee51 (diff) | |
| download | mongo-0d4e8a8b21b7ba1e765d42feabafa7e835686b08.tar.gz | |
SERVER-37386 Make periodic_runner_impl_test more diagnosable
| -rw-r--r-- | src/mongo/util/periodic_runner_impl_test.cpp | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/src/mongo/util/periodic_runner_impl_test.cpp b/src/mongo/util/periodic_runner_impl_test.cpp index 7deeb18b72d..280d1378eb1 100644 --- a/src/mongo/util/periodic_runner_impl_test.cpp +++ b/src/mongo/util/periodic_runner_impl_test.cpp @@ -198,7 +198,9 @@ TEST_F(PeriodicRunnerImplTest, OnePausableJobPausesCorrectly) { for (int i = 0; i < 10; i++) { clockSource().advance(interval); } - ASSERT_TRUE(count == numExecutionsBeforePause || count == numExecutionsBeforePause + 1); + ASSERT_TRUE(count == numExecutionsBeforePause || count == numExecutionsBeforePause + 1) + << "Actual values: count: " << count + << ", numExecutionsBeforePause: " << numExecutionsBeforePause; tearDown(); } @@ -234,7 +236,10 @@ TEST_F(PeriodicRunnerImplTest, OnePausableJobResumesCorrectly) { } auto countBeforePause = count; ASSERT_TRUE(countBeforePause == numFastForwardsForIterationWhileActive || - countBeforePause == numFastForwardsForIterationWhileActive + 1); + countBeforePause == numFastForwardsForIterationWhileActive + 1) + << "Actual values: countBeforePause: " << countBeforePause + << ", numFastForwardsForIterationWhileActive: " << numFastForwardsForIterationWhileActive; + handle->pause(); // Fast forward ten times, we shouldn't run anymore for (int i = 0; i < 10; i++) { @@ -252,7 +257,9 @@ TEST_F(PeriodicRunnerImplTest, OnePausableJobResumesCorrectly) { // This is slightly racy so once in a while count will be one extra ASSERT_TRUE(count == numFastForwardsForIterationWhileActive * 2 || - count == numFastForwardsForIterationWhileActive * 2 + 1); + count == numFastForwardsForIterationWhileActive * 2 + 1) + << "Actual values: count: " << count + << ", numFastForwardsForIterationWhileActive: " << numFastForwardsForIterationWhileActive; tearDown(); } |