Updated mobject.py, test_copy.py, and conftest.py, from os to pathlib Path (#535)

Co-authored-by: kolibril13 <[email protected]>

Author: digitalvirtuoso

manim/mobject/mobject.py

@@ -8,10 +8,10 @@
 import copy
 import itertools as it
 import operator as op
-import os
 import random
 import sys
 
+from pathlib import Path
 from colour import Color
 import numpy as np
 
@@ -199,7 +199,7 @@ def show(self, camera=None):
 
     def save_image(self, name=None):
         self.get_image().save(
-            os.path.join(file_writer_config["video_dir"], (name or str(self)) + ".png")
+            Path(file_writer_config["video_dir"]).joinpath((name or str(self)) + ".png")
         )
 
     def copy(self):

tests/test_copy.py

@@ -1,4 +1,4 @@
-import os
+from pathlib import Path
 from manim import Mobject, BraceLabel, file_writer_config
 
 
@@ -20,11 +20,11 @@ def test_bracelabel_copy(tmp_path):
     # For this test to work, we need to tweak some folders temporarily
     original_text_dir = file_writer_config["text_dir"]
     original_tex_dir = file_writer_config["tex_dir"]
-    mediadir = os.path.join(tmp_path, "deepcopy")
-    file_writer_config["text_dir"] = os.path.join(mediadir, "Text")
-    file_writer_config["tex_dir"] = os.path.join(mediadir, "Tex")
+    mediadir = Path(tmp_path) / "deepcopy"
+    file_writer_config["text_dir"] = str(mediadir.joinpath("Text"))
+    file_writer_config["tex_dir"] = str(mediadir.joinpath("Tex"))
     for el in ["text_dir", "tex_dir"]:
-        os.makedirs(file_writer_config[el])
+        Path(file_writer_config[el]).mkdir(parents=True)
 
     # Before the refactoring of Mobject.copy(), the class BraceLabel was the
     # only one to have a non-trivial definition of copy.  Here we test that it

tests/test_scene_rendering/conftest.py

@@ -1,15 +1,15 @@
 import pytest
-import os
 
+from pathlib import Path
 
 from manim import file_writer_config
 
 
 @pytest.fixture
 def manim_cfg_file():
-    return os.path.join(os.path.dirname(__file__), "manim.cfg")
+    return str(Path(__file__).parent / "manim.cfg")
 
 
 @pytest.fixture
 def simple_scenes_path():
-    return os.path.join(os.path.dirname(__file__), "simple_scenes.py")
+    return str(Path(__file__).parent / "simple_scenes.py")