Keep parameter.h inside the macro region in para_json_test.cpp

CI caught what the local build could not: source_io/module_json is only added
with ENABLE_RAPIDJSON, so MODULE_IO_JSON_OUTPUT_TEST was never compiled here.

para_json_test.cpp drives a local `Parameter param;` and writes `param.sys` /
`param.input`. The scoping pass hoisted parameter.h out of the macro region
because its 'does this file write PARAM' test only matched the uppercase
`PARAM.input.` singleton, not a lowercase local object -- the same miss already
fixed for lj_pot_test.cpp. Restored inside the region.

Reproduced and verified on a minimal translation unit: the hoisted arrangement
gives exactly the two 'is private within this context' errors CI reported, and
the restored one compiles clean.

Also drops the relax_criteria.h include from the three headers that do not
name the type (ions_move_bfgs2.h, ions_move_lbfgs.h, lattice_change_basic.h).

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

Author: Critsium-xy

source/source_io/module_json/test/para_json_test.cpp

@@ -1,7 +1,7 @@
 #include "gtest/gtest.h"
 
-#include "source_io/module_parameter/parameter.h"
 #define private public
+#include "source_io/module_parameter/parameter.h"
 #include "source_io/module_json/abacusjson.h"
 #include "source_io/module_json/general_info.h"
 #include "source_io/module_json/init_info.h"

source/source_relax/ions_move_bfgs2.h

@@ -1,7 +1,6 @@
 #ifndef IONS_MOVE_BFGS2_H
 #define IONS_MOVE_BFGS2_H
 
-#include "relax_criteria.h"
 #include <vector>
 #include <tuple> 
 #include <algorithm>

source/source_relax/ions_move_lbfgs.h

@@ -1,7 +1,6 @@
 #ifndef IONS_MOVE_LBFGS_H
 #define IONS_MOVE_LBFGS_H
 
-#include "relax_criteria.h"
 #include <vector>
 #include <tuple> 
 #include <algorithm>

source/source_relax/lattice_change_basic.h

@@ -1,7 +1,6 @@
 #ifndef LATTICE_CHANGE_BASIC_H
 #define LATTICE_CHANGE_BASIC_H
 
-#include "relax_criteria.h"
 #include <fstream>
 #include <vector>
 #include "relax_data.h"