aboutsummaryrefslogtreecommitdiffstats
path: root/searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2021-11-23 21:46:41 +0100
committerGitHub <noreply@github.com>2021-11-23 21:46:41 +0100
commit3fc1bdd5329c6b3796bbf2619f78225a675e705b (patch)
tree267354537445d8a5169b483d3570b04819c488cc /searchcore/src/tests/proton/documentdb/buckethandler/buckethandler_test.cpp
parent6e7385e7858ee5491f028c7012d9928ea340d678 (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.cpp43
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()
{