Refactors parallel barrier test

[libcds.git] / test / stress / misc / barrier_driver.cpp

diff --git a/test/stress/misc/barrier_driver.cpp b/test/stress/misc/barrier_driver.cpp
index e7878fdc3d734d859aa784c27ee20bcc8e60f490..272e3c43d3eef3253d5fa3fb469432923479b577 100644 (file)
--- a/test/stress/misc/barrier_driver.cpp
+++ b/test/stress/misc/barrier_driver.cpp
@@ -6,6 +6,7 @@
 #include <cds_test/stress_test.h>
 #include <iostream>
 #include <memory>
+#include <vector>
 #include <thread>
 
 using namespace std;
@@ -14,7 +15,7 @@ namespace {
 
 typedef cds_others::SpinBarrier Barrier;
 
-static size_t s_nBarrierThreadCount = 4;
+static size_t s_nBarrierThreadCount = 2;
 static size_t s_nBarrierPassCount = 100000000;
 
 class BarrierTest : public cds_test::stress_fixture {
@@ -28,14 +29,8 @@ protected:
     GetConfig(BarrierThreadCount);
   }
 
-  static void TearDownTestCase() {
-    if (count != s_nBarrierPassCount) {
-      cout << "Incorrect" << endl;
-    }
-  }
-
   static void Thread() {
-    for (ullong i = 0; i < s_nBarrierPassCount; i++) {
+    for (size_t i = 0; i < s_nBarrierPassCount; i++) {
       if (barrier->wait()) {
         count++;
       }
@@ -48,14 +43,16 @@ size_t BarrierTest::count;
 
 TEST_F(BarrierTest, Wait) {
   barrier = new Barrier(s_nBarrierThreadCount);
-  std::unique_ptr<std::thread[]> threads(new std::thread[s_nBarrierThreadCount]);
+  std::vector<std::thread> threads;
   for (size_t i = 0; i < s_nBarrierThreadCount; i++) {
-    threads[i] = std::thread(Thread);
+    threads.push_back(std::thread(Thread));
   }
 
-  for (size_t i = 0; i < s_nBarrierThreadCount; i++) {
-    threads[i].join();
+  for (auto& thrd : threads) {
+    thrd.join();
   }
+  EXPECT_EQ(count, s_nBarrierPassCount);
+  delete barrier;
 }
 
 } // namespace