Fix: close the access-control regions that ran to end of file

Sixteen test files opened `#define private public` (usually `#define protected
public` too) and never closed it, so from the include block onwards the whole
translation unit compiled with `private` and `protected` meaning `public` -- up to
1016 lines in `charge_mixing_test.cpp`. `elecstate_pw_test.cpp` was a partial case:
it closed `protected` 318 lines later at the end of the file, and never closed
`private` at all.

Each `#undef` now sits immediately after the include block it is meant to cover.
No include is moved: in every one of these files the standard headers already came
after the project headers, so they simply fall outside the region once it is
closed. That removes the incidental treatment of `<omp.h>` (charge_mixing_test),
`<string>` (elecstate_energy_test), `<fstream>` (verlet_test), `<mpi.h>` and
`<source_base/module_external/scalapack_connector.h>` (propagator_test1/2/3),
`<source_base/macros.h>` (test_hsolver) and `"mpi.h"` (read_wf2rho_pw_test).

Closing a region early is safe: the class definitions the tests reach into have
already been compiled by the time the `#undef` is seen.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: zxy.monado

source/source_esolver/test/esolver_dp_test.cpp

@@ -6,6 +6,7 @@
 #define protected public
 #include "../esolver_dp.h"
 #include "source_io/module_parameter/input_parameter.h"
+#undef protected
 #undef private
 /************************************************
  *  unit tests of class ESolver_DP

source/source_estate/test/charge_mixing_test.cpp

@@ -6,6 +6,7 @@
 #include "source_basis/module_pw/pw_basis.h"
 #include "source_hamilt/module_xc/xc_functional.h"
 #include "source_io/module_parameter/parameter.h"
+#undef private
 
 #ifdef _OPENMP
 #include <omp.h>

source/source_estate/test/charge_test.cpp

@@ -8,6 +8,8 @@
 #include "source_hamilt/module_xc/xc_functional.h"
 #include "source_io/module_parameter/parameter.h"
 #include "prepare_unitcell.h"
+#undef protected
+#undef private
 // mock functions for UnitCell
 
 Magnetism::Magnetism()

source/source_estate/test/elecstate_energy_test.cpp

@@ -4,6 +4,7 @@
 #include "source_estate/elecstate.h"
 #include "source_hamilt/module_xc/xc_functional.h"
 #include "source_io/module_parameter/parameter.h"
+#undef private
 
 #include <string>
 Parameter PARMA;

source/source_estate/test/elecstate_pw_test.cpp

@@ -10,6 +10,8 @@
 #include "source_pw/module_pwdft/vnl_pw.h"
 #include "source_pw/module_pwdft/soc.h"
 #include "source_io/module_parameter/parameter.h"
+#undef protected
+#undef private
 // mock functions for testing
 int XC_Functional::func_type = 1;
 namespace elecstate
@@ -323,4 +325,3 @@ TEST_F(ElecStatePWTest, ParallelKSingle)
     EXPECT_NO_THROW(elecstate_pw_s->parallelK());
 }
 
-#undef protected

source/source_hsolver/test/test_hsolver.cpp

@@ -6,6 +6,7 @@
 #include "hsolver_supplementary_mock.h"
 #include "source_hamilt/hamilt.h"
 #include "source_hsolver/hsolver.h"
+#undef protected
 
 #include <source_base/macros.h>
 

source/source_io/test/read_wf2rho_pw_test.cpp

@@ -15,6 +15,7 @@
 #include "source_io/module_wf/read_wf2rho_pw.h"
 #include "source_io/module_wf/write_wfc_pw.h"
 #include "source_psi/psi.h"
+#undef private
 
 #ifdef __MPI
 #include "mpi.h"

source/source_lcao/module_rt/test/propagator_test1.cpp

@@ -5,6 +5,8 @@
 #include "source_lcao/module_rt/propagator.h"
 #include "source_io/module_parameter/parameter.h"
 #include "tddft_test.h"
+#undef protected
+#undef private
 
 #include <source_base/module_external/scalapack_connector.h>
 #include <mpi.h>

source/source_lcao/module_rt/test/propagator_test2.cpp

@@ -5,6 +5,8 @@
 #include "source_lcao/module_rt/propagator.h"
 #include "source_io/module_parameter/parameter.h"
 #include "tddft_test.h"
+#undef protected
+#undef private
 
 #include <source_base/module_external/scalapack_connector.h>
 #include <mpi.h>

source/source_lcao/module_rt/test/propagator_test3.cpp

@@ -5,6 +5,8 @@
 #include "source_lcao/module_rt/propagator.h"
 #include "source_io/module_parameter/parameter.h"
 #include "tddft_test.h"
+#undef protected
+#undef private
 
 #include <source_base/module_external/scalapack_connector.h>
 #include <mpi.h>