diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2021-11-23 21:46:41 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-11-23 21:46:41 +0100 |
commit | 3fc1bdd5329c6b3796bbf2619f78225a675e705b (patch) | |
tree | 267354537445d8a5169b483d3570b04819c488cc /searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp | |
parent | 6e7385e7858ee5491f028c7012d9928ea340d678 (diff) |
Revert "Continue serving search queries when in Maintenance node state [run-systemtest]"
Diffstat (limited to 'searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp')
-rw-r--r-- | searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp | 43 |
1 files changed, 1 insertions, 42 deletions
diff --git a/searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp b/searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp index cadfa8cd72f..29748a2010c 100644 --- a/searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp +++ b/searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp @@ -140,11 +140,6 @@ struct Fixture setNodeUp(bool value) { _calc->setNodeUp(value); - _calc->setNodeMaintenance(false); - _handler.notifyClusterStateChanged(_calc); - } - void setNodeMaintenance(bool value) { - _calc->setNodeMaintenance(value); _handler.notifyClusterStateChanged(_calc); } }; @@ -228,7 +223,7 @@ TEST_F("require that unready bucket can be reported as active", Fixture) } -TEST_F("node going down (but not into maintenance state) deactivates all buckets", Fixture) +TEST_F("require that node being down deactivates buckets", Fixture) { f._handler.handleSetCurrentState(f._ready.bucket(2), BucketInfo::ACTIVE, f._genResult); @@ -257,42 +252,6 @@ TEST_F("node going down (but not into maintenance state) deactivates all buckets EXPECT_EQUAL(true, f._bucketInfo.getInfo().isActive()); } -TEST_F("node going into maintenance state does _not_ deactivate any buckets", Fixture) -{ - f._handler.handleSetCurrentState(f._ready.bucket(2), - BucketInfo::ACTIVE, f._genResult); - f.sync(); - f.setNodeMaintenance(true); - f.sync(); - f.handleGetBucketInfo(f._ready.bucket(2)); - EXPECT_TRUE(f._bucketInfo.getInfo().isActive()); -} - -TEST_F("node going from maintenance to up state deactivates all buckets", Fixture) -{ - f._handler.handleSetCurrentState(f._ready.bucket(2), - BucketInfo::ACTIVE, f._genResult); - f.sync(); - f.setNodeMaintenance(true); - f.sync(); - f.setNodeUp(true); - f.sync(); - f.handleGetBucketInfo(f._ready.bucket(2)); - EXPECT_FALSE(f._bucketInfo.getInfo().isActive()); -} - -TEST_F("node going from maintenance to down state deactivates all buckets", Fixture) -{ - f._handler.handleSetCurrentState(f._ready.bucket(2), - BucketInfo::ACTIVE, f._genResult); - f.sync(); - f.setNodeMaintenance(true); - f.sync(); - f.setNodeUp(false); - f.sync(); - f.handleGetBucketInfo(f._ready.bucket(2)); - EXPECT_FALSE(f._bucketInfo.getInfo().isActive()); -} TEST_MAIN() { |