Skip to content

Commit

Permalink
test: temporarily disable tests which relied on MDS' incorrect GID-ad…
Browse files Browse the repository at this point in the history
…ding behavior

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
  • Loading branch information
gregsfortytwo committed Sep 26, 2016
1 parent 7d2a8b7 commit 6da13e9
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/test/libcephfs/access.cc
Expand Up @@ -308,9 +308,10 @@ TEST(AccessTest, User) {
// chown and chgrp
ASSERT_EQ(0, ceph_chmod(admin, dir.c_str(), 0700));
ASSERT_EQ(0, ceph_chown(admin, dir.c_str(), 123, 456));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), 123, 789));
// FIXME: Re-enable these 789 tests once we can set multiple GIDs via libcephfs/config
// ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), 123, 789));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), 123, 456));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), -1, 789));
// ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), -1, 789));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), -1, 456));
ASSERT_EQ(-EACCES, ceph_chown(cmount, dir.c_str(), 123, 1));
ASSERT_EQ(-EACCES, ceph_chown(cmount, dir.c_str(), 1, 456));
Expand All @@ -327,7 +328,7 @@ TEST(AccessTest, User) {

ASSERT_EQ(0, ceph_chown(admin, dir.c_str(), 123, 1));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), -1, 456));
ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), 123, 789));
// ASSERT_EQ(0, ceph_chown(cmount, dir.c_str(), 123, 789));

ceph_shutdown(cmount);

Expand Down

0 comments on commit 6da13e9

Please sign in to comment.