add tests

.github/workflows/build_centos7.yml

@@ -176,10 +176,10 @@ jobs:
           cmake --build _build --config Release -j$(nproc)
 
 
-      - name: Run cucumber on outer_loop tests
+      - name: Tests with Cucumber
         uses: ./.github/workflows/cucumber-tests
         with:
-          feature: "features/outer_loop_tests.feature"
+          #          feature: "features/outer_loop_tests.feature"
           mpi_path: ${GITHUB_WORKSPACE}/_build/vcpkg_installed/x64-linux-release/tools/openmpi/bin
 
       - name: Cache vcpkg binary dir

.github/workflows/build_oracle8.yml

@@ -146,10 +146,10 @@ jobs:
           cmake --build _build --config Release -j$(nproc)
 
 
-      - name: Run cucumber on outer_loop tests
+      - name: Tests with Cucumber
         uses: ./.github/workflows/cucumber-tests
         with:
-          feature: "features/outer_loop_tests.feature"
+          #          feature: "features/outer_loop_tests.feature"
           mpi_path: ${GITHUB_WORKSPACE}/_build/vcpkg_installed/x64-linux-release/tools/openmpi/bin
 
       - name: Running unit tests

.github/workflows/build_ubuntu.yml

@@ -150,13 +150,12 @@ jobs:
         run: |
           cmake --build _build --config Release -j$(nproc)
 
-      - name: Run cucumber on outer_loop tests
+      - name: Tests with Cucumber
         uses: ./.github/workflows/cucumber-tests
         with:
-          feature: "features/outer_loop_tests.feature"
+          #          feature: "features/outer_loop_tests.feature"
           mpi_path: ${{ github.workspace }}/_build/vcpkg_installed/x64-linux-release/tools/openmpi/bin
 
-
       - name: Test
         run: |
           export PATH=${GITHUB_WORKSPACE}/_build/vcpkg_installed/x64-linux-release/tools/openmpi/bin:$PATH

.github/workflows/build_windows.yml

@@ -128,10 +128,10 @@ jobs:
           cmake --build _build --config Release -j4
 
 
-      - name: Run cucumber on outer_loop tests
+      - name: Tests with Cucumber
         uses: ./.github/workflows/cucumber-tests
         with:
-          feature: "features/outer_loop_tests.feature"
+          #          feature: "features/outer_loop_tests.feature"
           mpi_path: /c/Program Files/Microsoft MPI/Bin
 
 

src/cpp/benders/benders_core/BendersMathLogger.cpp

@@ -82,11 +82,16 @@ LogDestination::LogDestination(const std::filesystem::path& file_path,
     std::cerr << err_msg.str();
   }
 }
+void LogDestination::setDelimiter(const std::string& delimiter) {
+  delimiter_ = delimiter;
+}
 
 void MathLoggerBehaviour::write_header() {
   setHeadersList();
+  LogsDestination().InsertDelimiter();
   for (const auto& header : Headers()) {
     LogsDestination() << header;
+    LogsDestination().InsertDelimiter();
   }
   LogsDestination() << std::endl;
 }
@@ -154,36 +159,54 @@ double getDurationNotSolving(double iteration, double master,
 void PrintBendersData(LogDestination& log_destination,
                       const CurrentIterationData& data, const HEADERSTYPE& type,
                       const BENDERSMETHOD& method) {
+  log_destination.InsertDelimiter();
   log_destination << data.it;
+  log_destination.InsertDelimiter();
   log_destination << std::scientific << std::setprecision(10) << data.lb;
+  log_destination.InsertDelimiter();
   if (method == BENDERSMETHOD::BENDERS) {
     log_destination << std::scientific << std::setprecision(10) << data.ub;
+    log_destination.InsertDelimiter();
     log_destination << std::scientific << std::setprecision(10) << data.best_ub;
+    log_destination.InsertDelimiter();
     log_destination << std::scientific << std::setprecision(2)
                     << data.best_ub - data.lb;
+    log_destination.InsertDelimiter();
     log_destination << std::scientific << std::setprecision(2)
                     << (data.best_ub - data.lb) / data.best_ub;
+    log_destination.InsertDelimiter();
   }
   log_destination << data.min_simplexiter;
+  log_destination.InsertDelimiter();
   log_destination << data.max_simplexiter;
+  log_destination.InsertDelimiter();
   if (type == HEADERSTYPE::LONG || method == BENDERSMETHOD::BENDERS_BY_BATCH) {
     log_destination << data.number_of_subproblem_solved;
+    log_destination.InsertDelimiter();
   }
   if (type == HEADERSTYPE::LONG) {
     log_destination << data.cumulative_number_of_subproblem_solved;
+    log_destination.InsertDelimiter();
   }
 
   log_destination << std::setprecision(2) << data.iteration_time;
+  log_destination.InsertDelimiter();
+
   log_destination << std::setprecision(2) << data.timer_master;
+  log_destination.InsertDelimiter();
+
   log_destination << std::setprecision(2) << data.subproblems_walltime;
+  log_destination.InsertDelimiter();
 
   if (type == HEADERSTYPE::LONG) {
     log_destination << std::setprecision(2)
                     << data.subproblems_cumulative_cputime;
+    log_destination.InsertDelimiter();
     log_destination << std::setprecision(2)
                     << getDurationNotSolving(data.iteration_time,
                                              data.timer_master,
                                              data.subproblems_walltime);
+    log_destination.InsertDelimiter();
   }
   log_destination << std::endl;
 }
@@ -192,20 +215,27 @@ void PrintExternalLoopData(LogDestination& log_destination,
                            const CurrentIterationData& data,
                            const HEADERSTYPE& type,
                            const BENDERSMETHOD& method) {
+  log_destination.InsertDelimiter();
   log_destination << data.outer_loop_current_iteration_data.benders_num_run;
+  log_destination.InsertDelimiter();
   log_destination << std::scientific << std::setprecision(10)
                   << data.outer_loop_current_iteration_data.max_criterion;
+  log_destination.InsertDelimiter();
   log_destination << data.outer_loop_current_iteration_data.max_criterion_area;
+  log_destination.InsertDelimiter();
 
   log_destination
       << std::scientific << std::setprecision(10)
       << data.outer_loop_current_iteration_data.outer_loop_bilevel_best_ub;
+  log_destination.InsertDelimiter();
   log_destination
       << std::scientific << std::setprecision(10)
       << data.outer_loop_current_iteration_data.external_loop_lambda;
+  log_destination.InsertDelimiter();
   log_destination
       << std::scientific << std::setprecision(10)
       << data.outer_loop_current_iteration_data.external_loop_lambda_min;
+  log_destination.InsertDelimiter();
   log_destination
       << std::scientific << std::setprecision(10)
       << data.outer_loop_current_iteration_data.external_loop_lambda_max;

src/cpp/benders/benders_core/include/antares-xpansion/benders/benders_core/BendersMathLogger.h

@@ -37,11 +37,16 @@ class LogDestination {
 
   template <class T>
   std::ostream& operator<<(const T& obj);
+  std::ostream& InsertDelimiter() { return *stream_ << delimiter_; }
 
  private:
   std::ofstream file_stream_;
   std::ostream* stream_;
   std::streamsize width_ = 40;
+  std::string delimiter_ = "\t";
+
+ public:
+  void setDelimiter(const std::string& delimiter);
 };
 template <class T>
 std::ostream& LogDestination::operator<<(const T& obj) {
@@ -214,10 +219,14 @@ void MathLoggerExternalLoopSpecific<T>::setHeadersList() {
 template <class T>
 void MathLoggerExternalLoopSpecific<T>::Print(
     const CurrentIterationData& data) {
+  LogsDestination().InsertDelimiter();
   LogsDestination() << data.outer_loop_current_iteration_data.benders_num_run;
+  LogsDestination().InsertDelimiter();
   LogsDestination() << data.it;
+  LogsDestination().InsertDelimiter();
   for (const auto& t : data.outer_loop_current_iteration_data.*ptr_) {
     LogsDestination() << std::scientific << std::setprecision(10) << t;
+    LogsDestination().InsertDelimiter();
   }
   LogsDestination() << std::endl;
 }

src/cpp/benders/benders_core/include/antares-xpansion/benders/benders_core/BendersStructsDatas.h

@@ -15,8 +15,8 @@ struct OuterLoopCurrentIterationData{
   double external_loop_lambda = 0.;
   double external_loop_lambda_min = 0.;
   double external_loop_lambda_max = 0.;
-  std::string max_criterion_area;
-  std::string max_criterion_area_best_it;
+  std::string max_criterion_area = "N/A";
+  std::string max_criterion_area_best_it = "N/A";
 };
 /*! \struct
  * struct that hold current Benders iteration

tests/cpp/logger/logger_test.cpp

@@ -743,6 +743,7 @@ TEST_F(MasterLoggerTest, LogSwitchToInteger) {
   ASSERT_TRUE(_logger->_switchToIntegerCall);
   ASSERT_TRUE(_logger2->_switchToIntegerCall);
 }
+static constexpr const char* const DELIMITER = "\t";
 
 TEST(LogDestinationTest, WithInvalidEmptyFilePath) {
   const std::filesystem::path invalid_file_path("");
@@ -867,8 +868,10 @@ TEST(MathLoggerBendersByBatchTest, HeadersListStdOutShort) {
   std::streamsize width = 25;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   for (const auto& header : headers_manager.HeadersList()) {
     expected_msg << std::setw(width) << std::left << header;
+    expected_msg << DELIMITER;
   }
   expected_msg << std::endl;
   std::stringstream redirectedStdout;
@@ -886,8 +889,10 @@ TEST(MathLoggerBendersByBatchTest, HeadersListFileLong) {
   std::streamsize width = 25;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   for (const auto& header : headers_manager.HeadersList()) {
     expected_msg << std::setw(width) << std::left << header;
+    expected_msg << DELIMITER;
   }
   expected_msg << std::endl;
   auto log_file =
@@ -919,27 +924,39 @@ TEST(MathLoggerBendersByBatchTest, DataInFileLong) {
       data.iteration_time - data.timer_master - data.subproblems_walltime;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.it;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.min_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.max_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width)
                << data.number_of_subproblem_solved;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width)
                << data.cumulative_number_of_subproblem_solved;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.iteration_time;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.timer_master;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_walltime;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_cumulative_cputime;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << time_not_solving;
+  expected_msg << DELIMITER;
   expected_msg << std::endl;
   auto log_file =
       CreateRandomSubDir(std::filesystem::temp_directory_path()) / "log.txt";
@@ -970,20 +987,29 @@ TEST(MathLoggerBendersByBatchTest, DataInStdOutShort) {
       data.iteration_time - data.timer_master - data.subproblems_walltime;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.it;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.min_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.max_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width)
                << data.number_of_subproblem_solved;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.iteration_time;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.timer_master;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_walltime;
+  expected_msg << DELIMITER;
 
   expected_msg << std::endl;
   std::stringstream redirectedStdout;
@@ -1016,36 +1042,52 @@ TEST(MathLoggerBendersBaseTest, DataInFileLong) {
       data.iteration_time - data.timer_master - data.subproblems_walltime;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.it;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.ub;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.best_ub;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(2) << data.best_ub - data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(2)
                << (data.best_ub - data.lb) / data.best_ub;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.min_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.max_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width)
                << data.number_of_subproblem_solved;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width)
                << data.cumulative_number_of_subproblem_solved;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.iteration_time;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.timer_master;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_walltime;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_cumulative_cputime;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << time_not_solving;
+  expected_msg << DELIMITER;
   expected_msg << std::endl;
   auto log_file =
       CreateRandomSubDir(std::filesystem::temp_directory_path()) / "log.txt";
@@ -1076,28 +1118,40 @@ TEST(MathLoggerBendersBaseTest, DataInStdOutShort) {
       data.iteration_time - data.timer_master - data.subproblems_walltime;
 
   std::ostringstream expected_msg;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.it;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.ub;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(10) << data.best_ub;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(2) << data.best_ub - data.lb;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::scientific
                << std::setprecision(2)
                << (data.best_ub - data.lb) / data.best_ub;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width) << data.min_simplexiter;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << data.max_simplexiter;
+  expected_msg << DELIMITER;
 
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.iteration_time;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.timer_master;
+  expected_msg << DELIMITER;
   expected_msg << std::left << std::setw(width) << std::setprecision(2)
                << data.subproblems_walltime;
+  expected_msg << DELIMITER;
 
   expected_msg << std::endl;
   std::stringstream redirectedStdout;