From 09b61f09fc86506751ee6626aee5f021b4c1ca0b Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Tue, 6 Mar 2018 11:10:14 +0800 Subject: [PATCH] test: add a simple test for conf_get() and conf_set() Signed-off-by: Kefu Chai --- src/test/librados/misc.cc | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/src/test/librados/misc.cc b/src/test/librados/misc.cc index 93ac62ddca9..a07c344bf49 100644 --- a/src/test/librados/misc.cc +++ b/src/test/librados/misc.cc @@ -9,6 +9,7 @@ #include "include/rados.h" #include "include/rados/librados.h" #include "include/rados/librados.hpp" +#include "include/scope_guard.h" #include "include/stringify.h" #include "common/Checksummer.h" #include "global/global_context.h" @@ -1405,3 +1406,18 @@ TEST_F(LibRadosMiscPP, MinCompatClient) { ASSERT_LE(-1, require_min_compat_client); ASSERT_GT(CEPH_RELEASE_MAX, require_min_compat_client); } + +TEST_F(LibRadosMiscPP, Conf) { + const char* const option = "bluestore_throttle_bytes"; + size_t new_size = 1 << 20; + std::string original; + ASSERT_EQ(0, cluster.conf_get(option, original)); + auto restore_setting = make_scope_guard([&] { + cluster.conf_set(option, original.c_str()); + }); + std::string expected = std::to_string(new_size); + ASSERT_EQ(0, cluster.conf_set(option, expected.c_str())); + std::string actual; + ASSERT_EQ(0, cluster.conf_get(option, actual)); + ASSERT_EQ(expected, actual); +} -- 2.39.5