]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
test/pybind: remove object for testing after done with it
authorKefu Chai <kchai@redhat.com>
Fri, 21 Aug 2020 05:30:24 +0000 (13:30 +0800)
committerKefu Chai <kchai@redhat.com>
Fri, 21 Aug 2020 08:41:31 +0000 (16:41 +0800)
* add vertical spacing for better readability
* remove object for testing after done with it

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/test/pybind/test_rados.py

index 580062a4499fe319af0c4ac584b589391d79e503..1f2f8ba0af1faa231cd6e015c32c0b4d5e1f4322 100644 (file)
@@ -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")