Attempts to fix failing test

Joins all threads instead of only joining joinable ones

Author: JustCallMeRay

.github/workflows/cmake.yml

@@ -56,5 +56,5 @@ jobs:
       working-directory: ${{github.workspace}}/build/test
       # Execute tests defined by the CMake configuration.
       # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
-      run: ctest
+      run: ctest --output-on-failure
 

include/CXXGraph/Partitioning/Partitioner.hpp

@@ -192,20 +192,9 @@ CoordinatedPartitionState<T> Partitioner<T>::startCoordinated() {
     }
   }
   for (int t = 0; t < processors; ++t) {
-    if (myThreads[t].joinable()) {
-      myThreads[t].join();
-    }
-    // if(myRunnable[t] != nullptr){
-    //    delete myRunnable[t];
-    //}
-  }
-  /*
-  for (int t = 0; t < processors; ++t){
-      if (runnableList[t] != nullptr){
-          delete runnableList[t];
-      }
+    myThreads[t].join();
   }
-  */
+
   // new shared state is move constructed then returned
   //  (so has no effect on the shared ponter)
   return std::move(*shared_state);