]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
tests: librbd: adapt test_mock_RefreshRequest for jewel 16124/head
authorNathan Cutler <ncutler@suse.com>
Thu, 6 Jul 2017 16:01:33 +0000 (18:01 +0200)
committerNathan Cutler <ncutler@suse.com>
Thu, 6 Jul 2017 16:03:02 +0000 (18:03 +0200)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Signed-off-by: Nathan Cutler <ncutler@suse.com>
src/test/librbd/image/test_mock_RefreshRequest.cc

index 93dc2a5fa0d227b635830050edfe24b3ed240163..2938a9fef923e0420c6d66a7d1e81829d3f2f821 100644 (file)
@@ -690,6 +690,8 @@ TEST_F(TestMockImageRefreshRequest, JournalDisabledByPolicy) {
 TEST_F(TestMockImageRefreshRequest, ExclusiveLockWithCoR) {
   REQUIRE_FEATURE(RBD_FEATURE_EXCLUSIVE_LOCK);
 
+  REQUIRE(!is_feature_enabled(RBD_FEATURE_OBJECT_MAP | RBD_FEATURE_FAST_DIFF | RBD_FEATURE_JOURNALING))
+
   std::string val;
   ASSERT_EQ(0, _rados.conf_get("rbd_clone_copy_on_read", val));
   if (val == "false") {
@@ -706,28 +708,12 @@ TEST_F(TestMockImageRefreshRequest, ExclusiveLockWithCoR) {
   MockExclusiveLock mock_exclusive_lock;
   mock_image_ctx.exclusive_lock = &mock_exclusive_lock;
 
-  if (ictx->test_features(RBD_FEATURE_JOURNALING)) {
-    ASSERT_EQ(0, ictx->operations->update_features(RBD_FEATURE_JOURNALING,
-                                                   false));
-  }
-
-  if (ictx->test_features(RBD_FEATURE_FAST_DIFF)) {
-    ASSERT_EQ(0, ictx->operations->update_features(RBD_FEATURE_FAST_DIFF,
-                                                   false));
-  }
-
-  if (ictx->test_features(RBD_FEATURE_OBJECT_MAP)) {
-    ASSERT_EQ(0, ictx->operations->update_features(RBD_FEATURE_OBJECT_MAP,
-                                                   false));
-  }
-
   expect_op_work_queue(mock_image_ctx);
   expect_test_features(mock_image_ctx);
 
   InSequence seq;
   expect_get_mutable_metadata(mock_image_ctx, 0);
   expect_get_flags(mock_image_ctx, 0);
-  expect_get_group(mock_image_ctx, 0);
   expect_refresh_parent_is_required(mock_refresh_parent_request, false);
   expect_is_lock_required(mock_image_ctx, true);
   expect_set_require_lock_on_read(mock_image_ctx);