doc: modified more test files

Author: moe-ad

tests/test_code_docstrings.py

@@ -28,44 +28,43 @@
 
 import doctest
 import os
-import pathlib
+from pathlib import Path
 
 import pytest
 
 
 @pytest.mark.skipif(True, reason="examples are created for windows")
 def test_doctest_allfiles():
     directory = r"../ansys/dpf/core"
-    actual_path = pathlib.Path(__file__).parent.absolute()
-    # actual_path = os.getcwd()
+    actual_path = Path(__file__).parent.absolute()
     print(actual_path)
-    for filename in os.listdir(os.path.join(actual_path, directory)):
+    for filename in os.listdir(actual_path / directory):
         if filename.endswith(".py"):
-            path = os.path.join(directory, filename)
+            path = Path(directory) / filename
             print(path)
-            doctest.testfile(path, verbose=True, raise_on_error=True)
+            doctest.testfile(str(path), verbose=True, raise_on_error=True)
         else:
             continue
 
 
 @pytest.mark.skipif(True, reason="examples are created for windows")
 def test_doctest_allexamples():
     directory = r"../examples"
-    actual_path = pathlib.Path(__file__).parent.absolute()
+    actual_path = Path(__file__).parent.absolute()
     handled_files = []
-    for root, subdirectories, files in os.walk(os.path.join(actual_path, directory)):
+    for root, subdirectories, files in os.walk(actual_path / directory):
         for subdirectory in subdirectories:
-            subdir = os.path.join(root, subdirectory)
+            subdir = Path(root) / subdirectory
             print(subdir)
             for filename in os.listdir(subdir):
                 if filename.endswith(".py"):
-                    path = os.path.join(subdir, filename)
-                    if ".ipynb_checkpoints" in path:
+                    path = subdir / filename
+                    if ".ipynb_checkpoints" in str(path):
                         continue
                     print(path)
-                    handled_files.append(path)
+                    handled_files.append(str(path))
                     exec(
-                        open(path, mode="r", encoding="utf8").read(),
+                        path.read_text(encoding="utf-8"),
                         globals(),
                         globals(),
                     )

tests/test_codegeneration.py

@@ -24,6 +24,7 @@
 import os
 import copy
 import tempfile
+from pathlib import Path
 
 import ansys.grpc.dpf
 import numpy as np
@@ -153,7 +154,7 @@ def test_operator_any_input(allkindofcomplexity):
     serialization.inputs.any_input3.connect(u.outputs)
 
     # create a temporary file at the default temp directory
-    path = os.path.join(tempfile.gettempdir(), "dpf_temp_ser.txt")
+    path = str(Path(tempfile.gettempdir()) / "dpf_temp_ser.txt")
     if not core.SERVER.local_server:
         core.upload_file_in_tmp_folder(examples.find_static_rst(return_local_path=True))
         path = core.path_utilities.join(core.make_tmp_dir_server(), "dpf_temp_ser.txt")
@@ -171,8 +172,9 @@ def test_operator_any_input(allkindofcomplexity):
 
     assert hasattr(fc, "outputs") == False
 
-    if os.path.exists(path):
-        os.remove(path)
+    path = Path(path)
+    if path.exists():
+        path.unlink()
 
 
 def test_create_op_with_inputs(plate_msup):

tests/test_data_tree.py

@@ -24,6 +24,7 @@
 import os
 import pytest
 import conftest
+from pathlib import Path
 
 
 @conftest.raises_for_servers_version_under("4.0")
@@ -174,16 +175,16 @@ def test_write_to_file_data_tree(tmpdir, server_type):
         to_fill.list_int = [1, 2]
         to_fill.list_double = [1.5, 2.5]
         to_fill.list_string = ["hello", "bye"]
-    data_tree.write_to_txt(os.path.join(tmpdir, "file.txt"))
-    data_tree = dpf.DataTree.read_from_txt(os.path.join(tmpdir, "file.txt"), server=server_type)
+    data_tree.write_to_txt(str(Path(tmpdir) / "file.txt"))
+    data_tree = dpf.DataTree.read_from_txt(str(Path(tmpdir) / "file.txt"), server=server_type)
     assert data_tree.has("int")
     assert data_tree.has("double")
     assert data_tree.has("string")
     assert data_tree.has("list_int")
     assert data_tree.has("list_double")
     assert data_tree.has("list_string")
-    data_tree.write_to_json(os.path.join(tmpdir, "file.json"))
-    data_tree = dpf.DataTree.read_from_json(os.path.join(tmpdir, "file.json"), server=server_type)
+    data_tree.write_to_json(str(Path(tmpdir) / "file.json"))
+    data_tree = dpf.DataTree.read_from_json(str(Path(tmpdir) / "file.json"), server=server_type)
     assert data_tree.has("int")
     assert data_tree.has("double")
     assert data_tree.has("string")
@@ -207,19 +208,19 @@ def test_write_to_file_remote_data_tree(tmpdir, server_clayer_remote_process):
         to_fill.list_int = [1, 2]
         to_fill.list_double = [1.5, 2.5]
         to_fill.list_string = ["hello", "bye"]
-    data_tree.write_to_txt(os.path.join(tmpdir, "file.txt"))
+    data_tree.write_to_txt(str(Path(tmpdir) / "file.txt"))
     data_tree = dpf.DataTree.read_from_txt(
-        os.path.join(tmpdir, "file.txt"), server=server_connected
+        str(Path(tmpdir) / "file.txt"), server=server_connected
     )
     assert data_tree.has("int")
     assert data_tree.has("double")
     assert data_tree.has("string")
     assert data_tree.has("list_int")
     assert data_tree.has("list_double")
     assert data_tree.has("list_string")
-    data_tree.write_to_json(os.path.join(tmpdir, "file.json"))
+    data_tree.write_to_json(str(Path(tmpdir) / "file.json"))
     data_tree = dpf.DataTree.read_from_json(
-        os.path.join(tmpdir, "file.json"), server=server_connected
+        str(Path(tmpdir) / "file.json"), server=server_connected
     )
     assert data_tree.has("int")
     assert data_tree.has("double")