diff options
Diffstat (limited to 'src/mongo/db/pipeline/memory_usage_tracker_test.cpp')
| -rw-r--r-- | src/mongo/db/pipeline/memory_usage_tracker_test.cpp | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/src/mongo/db/pipeline/memory_usage_tracker_test.cpp b/src/mongo/db/pipeline/memory_usage_tracker_test.cpp index cc354e7fd69..f60933a4655 100644 --- a/src/mongo/db/pipeline/memory_usage_tracker_test.cpp +++ b/src/mongo/db/pipeline/memory_usage_tracker_test.cpp @@ -99,28 +99,28 @@ TEST_F(MemoryUsageTrackerTest, UpdateUsageUpdatesGlobal) { ASSERT_EQ(_tracker.maxMemoryBytes(), 150LL); } -DEATH_TEST_F(MemoryUsageTrackerTest, - UpdateGlobalToNegativeIsDisallowed, - "Underflow on memory tracking") { - _tracker.set(50LL); +// TODO SERVER-61281: Switch to 'DEATH_TEST_F' checking the underflow case. +TEST_F(MemoryUsageTrackerTest, UpdateFunctionUsageToNegativeIsDisallowed) { + _funcTracker.set(50LL); + ASSERT_EQ(_funcTracker.currentMemoryBytes(), 50LL); + ASSERT_EQ(_funcTracker.maxMemoryBytes(), 50LL); ASSERT_EQ(_tracker.currentMemoryBytes(), 50LL); ASSERT_EQ(_tracker.maxMemoryBytes(), 50LL); - _tracker.update(-100); + _funcTracker.update(-100); + ASSERT_EQ(_tracker.currentMemoryBytes(), 0LL); + ASSERT_EQ(_tracker.maxMemoryBytes(), 50LL); } -TEST_F(MemoryUsageTrackerTest, UpdateFunctionUsageToNegativeIsDisallowed) { - _funcTracker.set(50LL); +// TODO SERVER-61281: Switch to 'DEATH_TEST_F' checking the underflow case. +TEST_F(MemoryUsageTrackerTest, UpdateMemUsageToNegativeIsDisallowed) { + _tracker.set(50LL); ASSERT_EQ(_tracker.currentMemoryBytes(), 50LL); ASSERT_EQ(_tracker.maxMemoryBytes(), 50LL); - // TODO SERVER-61281: Temporarily disable the assert (and associated test) in - // PerFunctionMemoryTracker.update() to prevent inaccurate tracking to cause underflow errors - // Once accurate tracking is implemented and no underflow should happen, this negative test - // could be restored to verify that "Underflow on memory tracking" is reported. - - _funcTracker.update(-100); + _tracker.update(-100); ASSERT_EQ(_tracker.currentMemoryBytes(), 0LL); + ASSERT_EQ(_tracker.maxMemoryBytes(), 50LL); } } // namespace |
