diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-12-16 09:31:47 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-12-16 09:31:47 +0000 |
commit | 3d088ac0ee4e2019eb8d3626fa4d3c19e67f105a (patch) | |
tree | 99d127eb0428f710ca430ba50705983faef6f318 /searchcore/src/tests/proton/server/disk_mem_usage_filter | |
parent | 5a37d3ead73e2837e505ebc764711e8e98a20fd7 (diff) |
No need to restart sampling or reconfigure anything unless the config has changed.
Diffstat (limited to 'searchcore/src/tests/proton/server/disk_mem_usage_filter')
-rw-r--r-- | searchcore/src/tests/proton/server/disk_mem_usage_filter/disk_mem_usage_filter_test.cpp | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/searchcore/src/tests/proton/server/disk_mem_usage_filter/disk_mem_usage_filter_test.cpp b/searchcore/src/tests/proton/server/disk_mem_usage_filter/disk_mem_usage_filter_test.cpp index dd38765c4f0..ccee7caa917 100644 --- a/searchcore/src/tests/proton/server/disk_mem_usage_filter/disk_mem_usage_filter_test.cpp +++ b/searchcore/src/tests/proton/server/disk_mem_usage_filter/disk_mem_usage_filter_test.cpp @@ -65,9 +65,17 @@ assertResourceUsage(double usage, double limit, double utilization, const Resour EXPECT_DOUBLE_EQ(utilization, state.utilization()); } +TEST_F(DiskMemUsageFilterTest, reconfig_with_identical_config_is_noop) +{ + EXPECT_TRUE(_filter.setConfig(Config(1.0, 0.8))); + assertResourceUsage(0.2, 0.8, 0.25, _filter.usageState().diskState()); + EXPECT_FALSE(_filter.setConfig(Config(1.0, 0.8))); + assertResourceUsage(0.2, 0.8, 0.25, _filter.usageState().diskState()); +} + TEST_F(DiskMemUsageFilterTest, disk_limit_can_be_reached) { - _filter.setConfig(Config(1.0, 0.8)); + EXPECT_TRUE(_filter.setConfig(Config(1.0, 0.8))); assertResourceUsage(0.2, 0.8, 0.25, _filter.usageState().diskState()); triggerDiskLimit(); testWrite("diskLimitReached: { " @@ -80,7 +88,7 @@ TEST_F(DiskMemUsageFilterTest, disk_limit_can_be_reached) TEST_F(DiskMemUsageFilterTest, memory_limit_can_be_reached) { - _filter.setConfig(Config(0.8, 1.0)); + EXPECT_TRUE(_filter.setConfig(Config(0.8, 1.0))); assertResourceUsage(0.3, 0.8, 0.375, _filter.usageState().memoryState()); triggerMemoryLimit(); testWrite("memoryLimitReached: { " @@ -95,7 +103,7 @@ TEST_F(DiskMemUsageFilterTest, memory_limit_can_be_reached) TEST_F(DiskMemUsageFilterTest, both_disk_limit_and_memory_limit_can_be_reached) { - _filter.setConfig(Config(0.8, 0.8)); + EXPECT_TRUE(_filter.setConfig(Config(0.8, 0.8))); triggerMemoryLimit(); triggerDiskLimit(); testWrite("memoryLimitReached: { " |