From: Kefu Chai <kchai@redhat.com>
Date: Fri, 21 Aug 2020 05:30:24 +0000 (+0800)
Subject: test/pybind: remove object for testing after done with it
X-Git-Tag: v16.1.0~1358^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F36722%2Fhead;p=ceph.git

test/pybind: remove object for testing after done with it

* add vertical spacing for better readability
* remove object for testing after done with it

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/test/pybind/test_rados.py b/src/test/pybind/test_rados.py
index 580062a4499f..1f2f8ba0af1f 100644
--- a/src/test/pybind/test_rados.py
+++ b/src/test/pybind/test_rados.py
@@ -518,22 +518,27 @@ class TestIoctx(object):
             write_op.new(0)
             self.ioctx.operate_write_op(write_op, "write_ops")
             eq(self.ioctx.read('write_ops'), b'')
+
             write_op.write_full(b'1')
             write_op.append(b'2')
             self.ioctx.operate_write_op(write_op, "write_ops")
             eq(self.ioctx.read('write_ops'), b'12')
+
             write_op.write_full(b'12345')
             write_op.write(b'x', 2)
             self.ioctx.operate_write_op(write_op, "write_ops")
             eq(self.ioctx.read('write_ops'), b'12x45')
+
             write_op.write_full(b'12345')
             write_op.zero(2, 2)
             self.ioctx.operate_write_op(write_op, "write_ops")
             eq(self.ioctx.read('write_ops'), b'12\x00\x005')
+
             write_op.write_full(b'12345')
             write_op.truncate(2)
             self.ioctx.operate_write_op(write_op, "write_ops")
             eq(self.ioctx.read('write_ops'), b'12')
+
             write_op.remove()
             self.ioctx.operate_write_op(write_op, "write_ops")
             with assert_raises(ObjectNotFound):
@@ -606,20 +611,24 @@ class TestIoctx(object):
             write_op.new(LIBRADOS_CREATE_EXCLUSIVE)
             for key, value in xattrs.items():
                 write_op.set_xattr(key, value)
-                self.ioctx.operate_write_op(write_op, "abc")
+                self.ioctx.operate_write_op(write_op, 'abc')
                 eq(self.ioctx.get_xattr('abc', key), value)
+
             stored_xattrs_1 = {}
             for key, value in self.ioctx.get_xattrs('abc'):
                 stored_xattrs_1[key] = value
             eq(stored_xattrs_1, xattrs)
+
             for key in xattrs.keys():
                 write_op.rm_xattr(key)
-                self.ioctx.operate_write_op(write_op, "abc")
+                self.ioctx.operate_write_op(write_op, 'abc')
             stored_xattrs_2 = {}
             for key, value in self.ioctx.get_xattrs('abc'):
                 stored_xattrs_2[key] = value
             eq(stored_xattrs_2, {})
+
             write_op.remove()
+            self.ioctx.operate_write_op(write_op, 'abc')
 
     def test_locator(self):
         self.ioctx.set_locator_key("bar")