diff options
author | ADAM David Alan Martin <adam.martin@10gen.com> | 2018-10-02 18:01:59 -0400 |
---|---|---|
committer | ADAM David Alan Martin <adam.martin@10gen.com> | 2018-10-02 18:03:42 -0400 |
commit | 0380ac1465fcd05e2577351d5df226eb6bbccaa5 (patch) | |
tree | 68112921bf2524cb3da13f6255577edb7917ab7b /src/mongo/db/logical_clock_test.cpp | |
parent | 2d379ce39872fdfc04e6775ed8adea7ccdd1d1c1 (diff) | |
download | mongo-0380ac1465fcd05e2577351d5df226eb6bbccaa5.tar.gz |
SERVER-37365 Don't use `ignore()` in some tests.
It shouldn't be necessary to explicitly ignore status returns in
functions when testing their throw behavior - the test is sufficient
to indicate that `Status` is ignored.
Diffstat (limited to 'src/mongo/db/logical_clock_test.cpp')
-rw-r--r-- | src/mongo/db/logical_clock_test.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/mongo/db/logical_clock_test.cpp b/src/mongo/db/logical_clock_test.cpp index 0db83d6dbee..5a895dca22e 100644 --- a/src/mongo/db/logical_clock_test.cpp +++ b/src/mongo/db/logical_clock_test.cpp @@ -326,8 +326,7 @@ TEST_F(LogicalClockTest, RejectsLogicalTimesGreaterThanMaxTime) { auto almostMaxSecs = Seconds(maxVal) - LogicalClock::kMaxAcceptableLogicalClockDriftSecs + Seconds(10); setMockClockSourceTime(Date_t::fromDurationSinceEpoch(almostMaxSecs)); - ASSERT_THROWS(getClock()->advanceClusterTime(beyondMaxTime).transitional_ignore(), - std::exception); + ASSERT_THROWS(getClock()->advanceClusterTime(beyondMaxTime), std::exception); ASSERT_TRUE(getClock()->getClusterTime() == LogicalTime()); } |