Skip to content

Commit

Permalink
test: enable changing subscribe level testing
Browse files Browse the repository at this point in the history
This test doesn't pass previously because of a bug. After fixing it, we
can now enable this test.

Signed-off-by: Zhiqiang Wang <wonzhq@hotmail.com>
  • Loading branch information
wonzhq committed Dec 24, 2015
1 parent 7485668 commit 9af4d56
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/test/librados/cmd.cc
Original file line number Diff line number Diff line change
Expand Up @@ -182,15 +182,11 @@ TEST(LibRadosCmd, WatchLog) {
}
ASSERT_TRUE(l.contains("onexx"));

/*
changing the subscribe level is currently broken.
cmd[0] = (char *)"{\"prefix\":\"log\", \"logtext\":[\"twoxx\"]}";
ASSERT_EQ(0, rados_monitor_log(cluster, "err", log_cb, &l));
ASSERT_EQ(0, rados_mon_command(cluster, (const char **)cmd, 1, "", 0, &buf, &buflen, &st, &stlen));
sleep(2);
ASSERT_FALSE(l.contains("twoxx"));
*/

ASSERT_EQ(0, rados_monitor_log(cluster, "info", log_cb, &l));
cmd[0] = (char *)"{\"prefix\":\"log\", \"logtext\":[\"threexx\"]}";
Expand Down

0 comments on commit 9af4d56

Please sign in to comment.