fix(pd): fix unitest (#5113)

This pull request refactors several test files to update import paths
for the `expand_sys_str` utility and related modules, reflecting a
migration away from the `deepmd.tf` namespace. Additionally, it
temporarily skips a test due to compatibility issues with numpy 2.4 and
improves file cleanup logic in one test. The main themes are import path
updates and test maintenance.

**Import path updates:**

* Updated imports of `expand_sys_str` in multiple test files
(`test_descriptor.py`, `test_embedding_net.py`, `test_model.py`,
`test_saveload_dpa1.py`, `test_saveload_se_e2_a.py`, `test_sampler.py`)
to use `deepmd.common` instead of `deepmd.tf.common`.
[[1]](diffhunk://#diff-026c6e70e7f5fd37843904e99c43910d638b0da8607e1bc0ab84702f256f3264R16-R18)
[[2]](diffhunk://#diff-026c6e70e7f5fd37843904e99c43910d638b0da8607e1bc0ab84702f256f3264L34-L36)
[[3]](diffhunk://#diff-9d16c9bf54ca3ba29bd1045d492b6a4eb11956305c05631ac8691fe681271535R21-R23)
[[4]](diffhunk://#diff-9d16c9bf54ca3ba29bd1045d492b6a4eb11956305c05631ac8691fe681271535L37-L39)
[[5]](diffhunk://#diff-a3bf7b66e3c105cf0d73116440343c6b0be07ff6246557f04f6707bec0e21af3R22-R24)
[[6]](diffhunk://#diff-a3bf7b66e3c105cf0d73116440343c6b0be07ff6246557f04f6707bec0e21af3L35-L37)
[[7]](diffhunk://#diff-0d7e97b8c00caf452b143cae472e2fa0a97473ba4b685f588bb60c21c504b934R16-R18)
[[8]](diffhunk://#diff-0d7e97b8c00caf452b143cae472e2fa0a97473ba4b685f588bb60c21c504b934L35-L37)
[[9]](diffhunk://#diff-74518d33e0df93e8af8027509351b11fbbe1f32b8c84f6d3a165e1df0953048fR16-R18)
[[10]](diffhunk://#diff-74518d33e0df93e8af8027509351b11fbbe1f32b8c84f6d3a165e1df0953048fL35-L37)
[[11]](diffhunk://#diff-fd952e5725b949f09ee868247a1a5b0b1b383a25b096766fa994a102a3aeca31R16-R24)
* Changed import of `DeepmdDataSystem` in `test_sampler.py` from
`deepmd.tf.utils.data_system` to `deepmd.utils.data_system`.

**Test maintenance and compatibility:**

* Temporarily skipped the `test_sampler_debug_info` test in
`test_sampler.py` due to issues with numpy 2.4.
* Enhanced file cleanup in `test_training.py` by checking for file
existence before attempting to remove files or directories.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Tests**
* Improved test robustness by adding existence checks before file
cleanup operations to prevent errors.
  * Temporarily skipped one test due to numpy compatibility issues.

* **Chores**
  * Reorganized internal module imports for improved code structure.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Related PR: <PaddlePaddle/Paddle#77013>

Author: HydrogenSulfate

source/tests/pd/model/test_descriptor.py

@@ -13,6 +13,9 @@
     Path,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.pd.model.descriptor import (
     prod_env_mat,
 )
@@ -31,9 +34,6 @@
 from deepmd.pd.utils.nlist import (
     extend_input_and_build_neighbor_list,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 from deepmd.tf.env import (
     op_module,
 )

source/tests/pd/model/test_embedding_net.py

@@ -18,6 +18,9 @@
     Path,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.pd.model.descriptor import (
     DescrptSeA,
 )
@@ -34,9 +37,6 @@
 from deepmd.pd.utils.nlist import (
     extend_input_and_build_neighbor_list,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 from deepmd.tf.descriptor import DescrptSeA as DescrptSeA_tf
 
 from ..test_finetune import (

source/tests/pd/model/test_model.py

@@ -19,6 +19,9 @@
     Path,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.dpmodel.utils.learning_rate import LearningRateExp as MyLRExp
 from deepmd.pd.loss import (
     EnergyStdLoss,
@@ -32,9 +35,6 @@
 from deepmd.pd.utils.env import (
     DEVICE,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 from deepmd.tf.descriptor import DescrptSeA as DescrptSeA_tf
 from deepmd.tf.fit import (
     EnerFitting,

source/tests/pd/model/test_saveload_dpa1.py

@@ -13,6 +13,9 @@
     DataLoader,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.pd.loss import (
     EnergyStdLoss,
 )
@@ -32,9 +35,6 @@
 from deepmd.pd.utils.stat import (
     make_stat_input,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 
 
 def get_dataset(config):

source/tests/pd/model/test_saveload_se_e2_a.py

@@ -13,6 +13,9 @@
     DataLoader,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.pd.loss import (
     EnergyStdLoss,
 )
@@ -32,9 +35,6 @@
 from deepmd.pd.utils.stat import (
     make_stat_input,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 
 
 def get_dataset(config):

source/tests/pd/test_sampler.py

@@ -13,15 +13,15 @@
     DataLoader,
 )
 
+from deepmd.common import (
+    expand_sys_str,
+)
 from deepmd.pd.utils.dataloader import (
     DpLoaderSet,
     get_weighted_sampler,
 )
-from deepmd.tf.common import (
-    expand_sys_str,
-)
 from deepmd.tf.utils import random as tf_random
-from deepmd.tf.utils.data_system import (
+from deepmd.utils.data_system import (
     DeepmdDataSystem,
 )
 
@@ -55,6 +55,7 @@ def setUp(self):
         tf_random.seed(10)
         self.dp_dataset = DeepmdDataSystem(self.systems, self.batch_size, 1, self.rcut)
 
+    @unittest.skip("Temporarily skipped due to changes in numpy 2.4")
     def test_sampler_debug_info(self):
         dataloader = DataLoader(
             self.my_dataset,

source/tests/pd/test_training.py

@@ -136,11 +136,14 @@ def test_trainable(self) -> None:
     def tearDown(self) -> None:
         for f in os.listdir("."):
             if f.startswith("model") and f.endswith(".pd"):
-                os.remove(f)
+                if os.path.exists(f):
+                    os.remove(f)
             if f in ["lcurve.out"]:
-                os.remove(f)
+                if os.path.exists(f):
+                    os.remove(f)
             if f in ["stat_files"]:
-                shutil.rmtree(f)
+                if os.path.exists(f):
+                    shutil.rmtree(f)
 
 
 class TestEnergyModelSeA(unittest.TestCase, DPTrainTest):