From 9f892c32487695c77408bdd78757e376b95ba025 Mon Sep 17 00:00:00 2001
From: anagainaru <ana.gainaru@gmail.com>
Date: Wed, 30 Nov 2022 14:57:45 -0500
Subject: [PATCH] Forcing the SCR test to always use MPI

---
 testing/adios2/engine/bp/CMakeLists.txt       |  4 +-
 .../adios2/engine/bp/TestBPWriteReadSCR.cpp   | 42 ++-----------------
 2 files changed, 6 insertions(+), 40 deletions(-)

diff --git a/testing/adios2/engine/bp/CMakeLists.txt b/testing/adios2/engine/bp/CMakeLists.txt
index ac397ff026..fa5055212a 100644
--- a/testing/adios2/engine/bp/CMakeLists.txt
+++ b/testing/adios2/engine/bp/CMakeLists.txt
@@ -115,8 +115,8 @@ bp_gtest_add_tests_helper(WriteReadVector MPI_ALLOW)
 bp_gtest_add_tests_helper(WriteReadAttributesMultirank MPI_ALLOW)
 bp_gtest_add_tests_helper(LargeMetadata MPI_ALLOW)
 
-if(ADIOS2_HAVE_SCR)
-  bp_gtest_add_tests_helper(WriteReadSCR MPI_ALLOW)
+if(ADIOS2_HAVE_SCR AND ADIOS2_HAVE_MPI)
+  bp_gtest_add_tests_helper(WriteReadSCR MPI_ONLY)
   foreach(tgt ${Test.Engine.BP.WriteReadSCR-TARGETS})
     target_link_libraries(${tgt} SCR::SCR)
   endforeach()
diff --git a/testing/adios2/engine/bp/TestBPWriteReadSCR.cpp b/testing/adios2/engine/bp/TestBPWriteReadSCR.cpp
index 45e6411703..f489852460 100644
--- a/testing/adios2/engine/bp/TestBPWriteReadSCR.cpp
+++ b/testing/adios2/engine/bp/TestBPWriteReadSCR.cpp
@@ -35,25 +35,17 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCR1D)
     // form a mpiSize * Nx 1D array
     const std::string fname("SCRBPWriteRead1D8.bp");
 
-    int mpiRank = 0, mpiSize = 1;
+    int mpiRank, mpiSize;
     // Number of rows
     const size_t Nx = 8;
 
     // Number of steps
     const size_t NSteps = 3;
-
-#if ADIOS2_USE_MPI
     MPI_Comm_rank(MPI_COMM_WORLD, &mpiRank);
     MPI_Comm_size(MPI_COMM_WORLD, &mpiSize);
-#endif
 
     // Write test data using BP
-
-#if ADIOS2_USE_MPI
     adios2::ADIOS adios(MPI_COMM_WORLD);
-#else
-    adios2::ADIOS adios;
-#endif
     {
         adios2::IO io = adios.DeclareIO("TestIO");
 
@@ -411,7 +403,7 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCR2D)
     // form a 2D 2 * (numberOfProcess*Nx) matrix where Nx is 4 here
     const std::string fname("SCRBPWriteRead2D2x4Test.bp");
 
-    int mpiRank = 0, mpiSize = 1;
+    int mpiRank, mpiSize;
     // Number of rows
     const std::size_t Nx = 4;
 
@@ -420,19 +412,11 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCR2D)
 
     // Number of steps
     const std::size_t NSteps = 3;
-
-#if ADIOS2_USE_MPI
     MPI_Comm_rank(MPI_COMM_WORLD, &mpiRank);
     MPI_Comm_size(MPI_COMM_WORLD, &mpiSize);
-#endif
 
     // Write test data using ADIOS2
-
-#if ADIOS2_USE_MPI
     adios2::ADIOS adios(MPI_COMM_WORLD);
-#else
-    adios2::ADIOS adios;
-#endif
     {
         adios2::IO io = adios.DeclareIO("TestIO");
 
@@ -743,14 +727,11 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCR10)
     // form a 10D NumberOfProcess x 2 x ... x 2) array
     const std::string fname("SCRBPWriteRead10D2x2Test.bp");
 
-    int mpiRank = 0, mpiSize = 1;
+    int mpiRank, mpiSize;
     // Number of steps
     const std::size_t NSteps = 3;
-
-#if ADIOS2_USE_MPI
     MPI_Comm_rank(MPI_COMM_WORLD, &mpiRank);
     MPI_Comm_size(MPI_COMM_WORLD, &mpiSize);
-#endif
 
     size_t NX = static_cast<unsigned int>(mpiSize);
     size_t OX = static_cast<unsigned int>(mpiRank);
@@ -762,12 +743,7 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCR10)
     std::array<std::complex<double>, 512> CR64w, CR64r;
 
     // Write test data using ADIOS2
-
-#if ADIOS2_USE_MPI
     adios2::ADIOS adios(MPI_COMM_WORLD);
-#else
-    adios2::ADIOS adios;
-#endif
     {
         adios2::IO io = adios.DeclareIO("TestIO");
 
@@ -935,19 +911,15 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCRDeferred)
 {
     const std::string fname("SCRBPWriteReadDeferred.bp");
 
-    int mpiRank = 0, mpiSize = 1;
+    int mpiRank, mpiSize;
     const size_t Nx = 8;
     const std::size_t Ny = 2;
     const size_t NSteps = 3;
 
-#if ADIOS2_USE_MPI
     MPI_Comm_rank(MPI_COMM_WORLD, &mpiRank);
     MPI_Comm_size(MPI_COMM_WORLD, &mpiSize);
 
     adios2::ADIOS adios(MPI_COMM_WORLD);
-#else
-    adios2::ADIOS adios;
-#endif
     {
         adios2::IO io = adios.DeclareIO("TestIO");
         const adios2::Dims shape_1d{static_cast<size_t>(Nx * mpiSize)};
@@ -1068,13 +1040,9 @@ TEST_F(BPWriteReadTestSCR, ADIOS2BPWriteReadSCRDeferred)
 
 int main(int argc, char **argv)
 {
-#if ADIOS2_USE_MPI
     int provided;
-
-    // MPI_THREAD_MULTIPLE is only required if you enable the SST MPI_DP
     MPI_Init_thread(nullptr, nullptr, MPI_THREAD_MULTIPLE, &provided);
     SCR_Init();
-#endif
 
     int result;
     ::testing::InitGoogleTest(&argc, argv);
@@ -1089,10 +1057,8 @@ int main(int argc, char **argv)
     }
     result = RUN_ALL_TESTS();
 
-#if ADIOS2_USE_MPI
     SCR_Finalize();
     MPI_Finalize();
-#endif
 
     return result;
 }