]> 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>
Mon, 24 Aug 2020 09:51:32 +0000 (17:51 +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 4fdc0efefc43bc83c8f0c54a9f4bb34d16f6b8b3..1e4d6ba4b895231cd4ad14de4c8193aa86bf12fa 100644 (file)
@@ -507,22 +507,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):
@@ -595,20 +600,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")