readSolution and readBasis don't require column and row names to exist, but always use them if they do

Author: jajhall

check/TestCheckSolution.cpp

@@ -565,7 +565,7 @@ TEST_CASE("read-lp-file-rgn", "[highs_check_solution]") {
   REQUIRE(integral);
   REQUIRE(feasible);
 
-  //  std::remove(model_file_name.c_str());
+  std::remove(model_file_name.c_str());
   std::remove(solution_file_name.c_str());
 
   h.resetGlobalScheduler(true);
@@ -689,7 +689,7 @@ void runSetLpSolution(const std::string model) {
 
   highs.clear();
 
-  //  std::remove(solution_file.c_str());
+  std::remove(solution_file.c_str());
 
   highs.resetGlobalScheduler(true);
 }

check/TestMipSolver.cpp

@@ -946,6 +946,7 @@ TEST_CASE("issue-2409", "[highs_test_mip_solver]") {
   const HighsModelStatus require_model_status = HighsModelStatus::kOptimal;
   const double optimal_objective = 0.1;
   Highs highs;
+  highs.setOptionValue("output_flag", dev_run);
   REQUIRE(highs.passModel(lp) == HighsStatus::kOk);
   if (dev_run) printf("Testing that presolve reduces the problem to empty\n");
   REQUIRE(highs.presolve() == HighsStatus::kOk);
@@ -982,6 +983,7 @@ TEST_CASE("issue-2432", "[highs_test_mip_solver]") {
   const HighsModelStatus require_model_status = HighsModelStatus::kOptimal;
   const double optimal_objective = -3777.57124352;
   Highs highs;
+  highs.setOptionValue("output_flag", dev_run);
   REQUIRE(highs.passModel(lp) == HighsStatus::kOk);
   if (dev_run) printf("Testing that presolve reduces the problem\n");
   REQUIRE(highs.presolve() == HighsStatus::kOk);