Fix RemoteReporter test race condition (#135)

* Fix remote reporter test

Signed-off-by: Isaac Hier <[email protected]>

* Add TODO for future fix

Signed-off-by: Isaac Hier <[email protected]>

Author: isaachier

src/jaegertracing/reporters/ReporterTest.cpp

@@ -74,13 +74,11 @@ TEST(Reporter, testRemoteReporter)
     constexpr auto kNumReports = 100;
     for (auto i = 0; i < kNumReports; ++i) {
         reporter.report(span);
-        if ((i + 1) % kFixedQueueSize > 0) {
-            std::this_thread::sleep_for(std::chrono::milliseconds(5));
-            std::lock_guard<std::mutex> lock(mutex);
-            ASSERT_EQ(i + 1, spans.size());
-        }
+        // TODO(isaachier): Find a way to make this test more rigorous.
+        std::this_thread::sleep_for(std::chrono::milliseconds(1));
     }
     reporter.close();
+    ASSERT_EQ(spans.size(), kNumReports);
 }
 
 TEST(Reporter, testNullReporter)