]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
unittest: add an assert for chain_listxattr.
authorDongsheng Yang <dongsheng.yang@easystack.cn>
Thu, 24 Mar 2016 06:12:44 +0000 (02:12 -0400)
committerDongsheng Yang <dongsheng.yang@easystack.cn>
Mon, 25 Apr 2016 04:58:28 +0000 (00:58 -0400)
As there is no assert after chain_listxattr(), and memset actual to '\0'
immediately, we did not test chain_listxattr() at all.

This patch add an assert to do the unittest for chain_listxattr().

Signed-off-by: Dongsheng Yang <dongsheng.yang@easystack.cn>
src/test/objectstore/chain_xattr.cc

index 65ab2536db4879d2dc26912a5689522ef40b82b0..83493e016af5c86adc2caf0959d1ec2ee18a7a9f 100644 (file)
@@ -240,6 +240,7 @@ TEST(chain_xattr, listxattr) {
   char* actual = (char*)calloc(1, buffer_size);
   ASSERT_LT(buffer_size, chain_listxattr(file, NULL, 0)); // size evaluation is conservative
   chain_listxattr(file, actual, buffer_size);
+  ASSERT_EQ(0, ::memcmp(expected, actual, buffer_size));
   ::memset(actual, '\0', buffer_size);
   chain_flistxattr(fd, actual, buffer_size);
   ASSERT_EQ(0, ::memcmp(expected, actual, buffer_size));