Reorganize unit tests into subdirectories mirroring include/ structure

Move 42 test files from flat test/ into subdirectories:
- convex_bodies/        (hpolytope, vpolytope, orderpolytope, convex_body)
- convex_bodies/correlation_matrices/  (matrix sampling)
- convex_bodies/spectrahedra/          (SDPA format)
- diagnostics/          (MCMC diagnostics)
- generators/           (polytope + zonotope generators)
- integration/          (Monte Carlo integration)
- lp_oracles/           (boundary oracles)
- misc/                 (poset, print_table, small_utils)
- ode_solvers/          (ODE solvers, leapfrog)
- preprocess/           (rounding, full_dimensional, inner points, max ellipsoid)
- preprocess/crhmc/     (CRHMC preprocessing)
- random_walks/         (sampling, shake-and-bake, logconcave)
- random_walks/crhmc/   (CRHMC sampling)
- root_finders/         (root finders)
- sampling/             (MMCS)
- volume/               (cooling balls, cooling gaussians, sequence of balls)
- benchmarks/           (performance benchmarks)

Each subdirectory has its own CMakeLists.txt with test registration.
Root CMakeLists.txt retains shared setup (dependencies, coverage_config)
and uses add_subdirectory() for each module.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: vissarion

examples/crhmc_prepare/crhmc_prepare.cpp

@@ -41,7 +41,7 @@ int main(int argc, char *argv[])
   {
     std::cout << "Usage: ./crhmc_prepare [file_name_string]\n";
     std::cout << "Example file name= "
-                 "../../test/metabolic_full_dim/polytope_e_coli.ine\n";
+                 "../data/metabolic_full_dim/polytope_e_coli.ine\n";
     exit(1);
   }
   std::string fileName(argv[1]);

examples/volume_spectrahedron/volume_cb_spectrahedra.cpp

@@ -36,7 +36,7 @@ void example_volume(){
     std::ifstream in1;
     spectrahedron spectra;
     Point objFunction;
-    in1.open("./../../test/spectra_data/sdp_prob_20_20.txt", std::ifstream::in);
+    in1.open("./../../test/data/spectra_data/sdp_prob_20_20.txt", std::ifstream::in);
     sdpaFormatManager.loadSDPAFormatFile(in1, spectra, objFunction);
 
     // Setup the parameters

test/.gitignore

@@ -0,0 +1,15 @@
+# Out-of-source build artifacts
+build/
+CMakeCache.txt
+CMakeFiles/
+cmake_install.cmake
+CTestTestfile.cmake
+Makefile
+*.o
+*.a
+*.so
+*.gcda
+*.gcno
+*.gcov
+Testing/
+compile_commands.json

test/CMakeLists.txt

@@ -131,7 +131,7 @@ static SDPTestResult run_sdp_test(const SDPTestCase& test_case) {
 // Individual test case wrappers for better test organization
 static void test_sdp_2x8() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp__2_8_d.txt",
+        .filename = "../data/spectra_data/sdp__2_8_d.txt",
         .expected_minimum = -1.3888e+00,
         .relative_error = 1e-2,
         .additive_error_factor = 5e-2, 
@@ -147,7 +147,7 @@ static void test_sdp_2x8() {
 
 static void test_sdp_20x20() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp_prob_20_20_d.txt",
+        .filename = "../data/spectra_data/sdp_prob_20_20_d.txt",
         .expected_minimum = -1.6535356e+00,
         .relative_error = 1e-3,
         .additive_error_factor = 5e-2,  
@@ -163,7 +163,7 @@ static void test_sdp_20x20() {
 
 static void test_sdp_20x30() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp__20_30_d.txt",
+        .filename = "../data/spectra_data/sdp__20_30_d.txt",
         .expected_minimum = -1.89076e+01,
         .relative_error = 1e-3,
         .additive_error_factor = 5e-2, 
@@ -179,7 +179,7 @@ static void test_sdp_20x30() {
 
 static void test_sdp_50x40() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp__50_40_d.txt",
+        .filename = "../data/spectra_data/sdp__50_40_d.txt",
         .expected_minimum = -3.1513966779338073e+01,
         .relative_error = 1e-3,
         .additive_error_factor = 5e-2, 
@@ -195,7 +195,7 @@ static void test_sdp_50x40() {
 
 static void test_sdp_20x100() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp__20_100_d.txt",
+        .filename = "../data/spectra_data/sdp__20_100_d.txt",
         .expected_minimum = -6.74974e+00,
         .relative_error = 1e-3,
         .additive_error_factor = 5e-2,
@@ -211,7 +211,7 @@ static void test_sdp_20x100() {
 
 static void test_sdp_50x200() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp__50_200_d.txt",
+        .filename = "../data/spectra_data/sdp__50_200_d.txt",
         .expected_minimum = -7.71954e+00,
         .relative_error = 1e-3,
         .additive_error_factor = 5e-2,
@@ -227,7 +227,7 @@ static void test_sdp_50x200() {
 
 static void test_sdp_200x15() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp_prob_200_15_d.txt",
+        .filename = "../data/spectra_data/sdp_prob_200_15_d.txt",
         .expected_minimum = -3.1610166002684946e+12,
         .relative_error = 1e-4,
         .additive_error_factor = 5e-2, 
@@ -243,7 +243,7 @@ static void test_sdp_200x15() {
 
 static void test_sdp_400x20() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp_prob_400_20_d.txt",
+        .filename = "../data/spectra_data/sdp_prob_400_20_d.txt",
         .expected_minimum = -8.7728166762094482e+11,
         .relative_error = 1e-4,
         .additive_error_factor = 5e-2, 
@@ -259,7 +259,7 @@ static void test_sdp_400x20() {
 
 static void test_sdp_600x25() {
     SDPTestCase test_case = {
-        .filename = "../spectra_data/sdp_prob_600_25_d.txt",
+        .filename = "../data/spectra_data/sdp_prob_600_25_d.txt",
         .expected_minimum = -1.3547280395306393e+11,
         .relative_error = 1e-2,
         .additive_error_factor = 5e-2,  

test/SDP/test_solve_sdp.cpp

@@ -0,0 +1,15 @@
+# Benchmark executables (not run via ctest by default)
+
+add_executable(benchmarks_sob benchmarks_sob.cpp)
+add_executable(benchmarks_cg benchmarks_cg.cpp)
+add_executable(benchmarks_cb benchmarks_cb.cpp)
+
+TARGET_LINK_LIBRARIES(benchmarks_sob lp_solve ${MKL_LINK} coverage_config)
+TARGET_LINK_LIBRARIES(benchmarks_cg lp_solve ${MKL_LINK} coverage_config)
+TARGET_LINK_LIBRARIES(benchmarks_cb lp_solve ${MKL_LINK} coverage_config)
+
+# CRHMC benchmarks (disabled by default)
+# add_executable(benchmarks_crhmc benchmarks_crhmc.cpp)
+# add_executable(benchmarks_crhmc_sampling benchmarks_crhmc_sampling.cpp)
+# TARGET_LINK_LIBRARIES(benchmarks_crhmc_sampling lp_solve ${MKL_LINK} QD_LIB coverage_config)
+# TARGET_LINK_LIBRARIES(benchmarks_crhmc lp_solve ${MKL_LINK} QD_LIB coverage_config)

test/benchmarks/CMakeLists.txt

@@ -93,22 +93,22 @@ int main() {
   assert(preparation_time < 0.5);
   std::cout << "Assertion (preparation_time< 0.5 secs) passed!" << std::endl
             << std::endl;
-  if (exists_check("../test/metabolic_full_dim/polytope_e_coli.ine")) {
+  if (exists_check("../data/metabolic_full_dim/polytope_e_coli.ine")) {
     preparation_time =
-        benchmark("../test/metabolic_full_dim/polytope_e_coli.ine");
+        benchmark("../data/metabolic_full_dim/polytope_e_coli.ine");
     assert(preparation_time < 2.0);
     std::cout << "Assertion (preparation_time< 2 secs) passed!" << std::endl
               << std::endl;
   }
-  if (exists_check("../test/netlib/afiro.ine")) {
-    preparation_time = benchmark("../test/netlib/afiro.ine");
+  if (exists_check("../data/netlib/afiro.ine")) {
+    preparation_time = benchmark("../data/netlib/afiro.ine");
     assert(preparation_time < 0.3);
     std::cout << "Assertion (preparation_time< 0.3 secs) passed!" << std::endl
               << std::endl;
   }
-  if (exists_check("../test/metabolic_full_dim/polytope_iAB_RBC_283.ine")) {
+  if (exists_check("../data/metabolic_full_dim/polytope_iAB_RBC_283.ine")) {
     preparation_time =
-        benchmark("../test/metabolic_full_dim/polytope_iAB_RBC_283.ine");
+        benchmark("../data/metabolic_full_dim/polytope_iAB_RBC_283.ine");
     assert(preparation_time < 400);
     std::cout << "Assertion (preparation_time< 400 secs) passed!" << std::endl
               << std::endl;

test/benchmarks_cb.cpp test/benchmarks/benchmarks_cb.cpptest/benchmarks_cb.cpp renamed to test/benchmarks/benchmarks_cb.cpp

@@ -369,17 +369,17 @@ void call_test_benchmark_polytope() {
     test_benchmark_polytope<NT, Point, Hpolytope, simdLen>(stream, P, name, target_time, centered);
   }
 
-  if (exists_check("../test/netlib/afiro.ine")) {
-    Hpolytope P = read_polytope<Hpolytope, NT>("../test/netlib/afiro.ine");
+  if (exists_check("../data/netlib/afiro.ine")) {
+    Hpolytope P = read_polytope<Hpolytope, NT>("../data/netlib/afiro.ine");
     std::string name = "afiro";
     bool centered = true;
     double target_time = 100; // secs
     test_benchmark_polytope<NT, Point, Hpolytope, simdLen>(stream, P, name, target_time, centered);
   }
 
-  if (exists_check("../test/metabolic_full_dim/polytope_e_coli.ine")) {
+  if (exists_check("../data/metabolic_full_dim/polytope_e_coli.ine")) {
     Hpolytope P =
-        read_polytope<Hpolytope, NT>("../test/metabolic_full_dim/polytope_e_coli.ine");
+        read_polytope<Hpolytope, NT>("../data/metabolic_full_dim/polytope_e_coli.ine");
     std::string name = "e_coli";
     bool centered = true;
     double target_time = 600; // secs