diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-09-15 09:41:55 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-09-15 09:41:55 +0000 |
commit | 221587249f2bcba5b416011b34c7eecdd3d59ac3 (patch) | |
tree | 4f539e1b10eb176a4acce3e627389086e58dcefe /searchcore | |
parent | 4035eae8042a8920e3c05b784c25385e1b44b516 (diff) |
On retired nodes no buckets should be ready.
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/tests/proton/documentdb/documentbucketmover/documentbucketmover_test.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/searchcore/src/tests/proton/documentdb/documentbucketmover/documentbucketmover_test.cpp b/searchcore/src/tests/proton/documentdb/documentbucketmover/documentbucketmover_test.cpp index b53338f4d0b..44075de4179 100644 --- a/searchcore/src/tests/proton/documentdb/documentbucketmover/documentbucketmover_test.cpp +++ b/searchcore/src/tests/proton/documentdb/documentbucketmover/documentbucketmover_test.cpp @@ -433,9 +433,9 @@ TEST_F(ControllerFixture, require_that_de_activated_bucket_is_not_moved_if_new_c TEST_F(ControllerFixture, ready_bucket_not_moved_to_not_ready_if_node_is_marked_as_retired) { - _calc->setNodeRetired(true); // Bucket 2 would be moved from ready to not ready in a non-retired case, but not when retired. - addReady(_ready.bucket(1)); + remReady(_ready.bucket(1)); + _calc->setNodeRetired(true); masterExecute([this]() { _bmj->recompute(); _bmj->scanAndMove(4, 3); @@ -449,10 +449,10 @@ TEST_F(ControllerFixture, ready_bucket_not_moved_to_not_ready_if_node_is_marked_ // but test this case for the sake of completion. TEST_F(ControllerFixture, inactive_not_ready_bucket_not_moved_to_ready_if_node_is_marked_as_retired) { + remReady(_ready.bucket(1)); + remReady(_ready.bucket(2)); + remReady(_notReady.bucket(3)); _calc->setNodeRetired(true); - addReady(_ready.bucket(1)); - addReady(_ready.bucket(2)); - addReady(_notReady.bucket(3)); masterExecute([this]() { _bmj->recompute(); _bmj->scanAndMove(4, 3); @@ -464,10 +464,10 @@ TEST_F(ControllerFixture, inactive_not_ready_bucket_not_moved_to_ready_if_node_i TEST_F(ControllerFixture, explicitly_active_not_ready_bucket_can_not_be_moved_to_ready_if_node_is_marked_as_retired) { + remReady(_ready.bucket(1)); + remReady(_ready.bucket(2)); + remReady(_notReady.bucket(3)); _calc->setNodeRetired(true); - addReady(_ready.bucket(1)); - addReady(_ready.bucket(2)); - addReady(_notReady.bucket(3)); _bmj->recompute(); masterExecute([this]() { activateBucket(_notReady.bucket(3)); |