remove racy test assertions

Do not assert before the loop waiting for the thread to complete the
expected side effect. The whole point of the loop is to make sure
there is no window of opportunity for a race condition and asserting
before it means taking a useless risk. If run enough times, it will
happen.

Signed-off-by: Loic Dachary <loic@dachary.org>

1 files changed, 0 insertions, 10 deletions

diff --git a/src/test/test_osd_types.cc b/src/test/test_osd_types.cc
index fa4ae6163ac..730a8ffdc5d 100644
--- a/src/test/test_osd_types.cc
+++ b/src/test/test_osd_types.cc
@@ -1081,11 +1081,6 @@ TEST_F(ObjectContextTest, read_write_lock)
 
     obc.ondisk_write_unlock();
 
-    EXPECT_EQ(1, obc.readers_waiting);
-    EXPECT_EQ(0, obc.readers);
-    EXPECT_EQ(0, obc.writers_waiting);
-    EXPECT_EQ(0, obc.unstable_writes);
-
     do {
       cout << "Trying (2) with delay " << delay << "us\n";
       usleep(delay);
@@ -1144,11 +1139,6 @@ TEST_F(ObjectContextTest, read_write_lock)
 
     obc.ondisk_read_unlock();
 
-    EXPECT_EQ(0, obc.readers_waiting);
-    EXPECT_EQ(0, obc.readers);
-    EXPECT_EQ(1, obc.writers_waiting);
-    EXPECT_EQ(0, obc.unstable_writes);
-
     do {
       cout << "Trying (4) with delay " << delay << "us\n";
       usleep(delay);