Make testing methods private

Author: utf

src/atomate2/utils/testing/vasp.py

@@ -144,16 +144,16 @@ def fake_run_vasp(
     logger.info("Running fake VASP.")
 
     if "incar" in check_inputs:
-        check_incar(ref_path, incar_settings, incar_exclude)
+        _check_incar(ref_path, incar_settings, incar_exclude)
 
     if "kpoints" in check_inputs:
-        check_kpoints(ref_path)
+        _check_kpoints(ref_path)
 
     if "poscar" in check_inputs:
-        check_poscar(ref_path)
+        _check_poscar(ref_path)
 
     if "potcar" in check_inputs:
-        check_potcar(ref_path)
+        _check_potcar(ref_path)
 
     # This is useful to check if the WAVECAR has been copied
     if "wavecar" in check_inputs and not Path("WAVECAR").exists():
@@ -162,15 +162,15 @@ def fake_run_vasp(
     logger.info("Verified inputs successfully")
 
     if clear_inputs:
-        clear_vasp_inputs()
+        _clear_vasp_inputs()
 
-    copy_vasp_outputs(ref_path)
+    _copy_vasp_outputs(ref_path)
 
     # pretend to run VASP by copying pre-generated outputs from reference dir
     logger.info("Generated fake vasp outputs")
 
 
-def check_incar(
+def _check_incar(
     ref_path: Path,
     incar_settings: Sequence[str] | None,
     incar_exclude: Sequence[str] | None,
@@ -194,7 +194,7 @@ def check_incar(
             )
 
 
-def check_kpoints(ref_path: Path) -> None:
+def _check_kpoints(ref_path: Path) -> None:
     """Check that KPOINTS file is consistent with the reference calculation."""
     user_kpoints_exists = (user_kpt_path := zpath("KPOINTS")).exists()
     ref_kpoints_exists = (
@@ -234,7 +234,7 @@ def check_kpoints(ref_path: Path) -> None:
             )
 
 
-def check_poscar(ref_path: Path) -> None:
+def _check_poscar(ref_path: Path) -> None:
     """Check that POSCAR information is consistent with the reference calculation."""
     user_poscar_path = zpath("POSCAR")
     ref_poscar_path = zpath(ref_path / "inputs" / "POSCAR")
@@ -263,7 +263,7 @@ def check_poscar(ref_path: Path) -> None:
         )
 
 
-def check_potcar(ref_path: Path) -> None:
+def _check_potcar(ref_path: Path) -> None:
     """Check that POTCAR information is consistent with the reference calculation."""
     potcars = {"reference": None, "user": None}
     paths = {"reference": ref_path / "inputs", "user": Path(".")}
@@ -283,7 +283,7 @@ def check_potcar(ref_path: Path) -> None:
         )
 
 
-def clear_vasp_inputs() -> None:
+def _clear_vasp_inputs() -> None:
     """Clean up VASP input files."""
     for vasp_file in (
         "INCAR",
@@ -300,7 +300,7 @@ def clear_vasp_inputs() -> None:
     logger.info("Cleared vasp inputs")
 
 
-def copy_vasp_outputs(ref_path: Path) -> None:
+def _copy_vasp_outputs(ref_path: Path) -> None:
     """Copy VASP output files from the reference directory."""
     output_path = ref_path / "outputs"
     for output_file in output_path.iterdir():