Merge pull request #6701 from pradyunsg/tests/remove-makedirs

Change Path.makedirs() -> Path.mkdir(parents=True)

Author: pradyunsg

tests/functional/test_install.py

@@ -896,7 +896,7 @@ def test_install_editable_with_prefix(script):
 
     # make sure target path is in PYTHONPATH
     pythonpath = script.scratch_path / site_packages
-    pythonpath.makedirs()
+    pythonpath.mkdir(parents=True)
     script.environ["PYTHONPATH"] = pythonpath
 
     # install pkga package into the absolute prefix directory

tests/functional/test_uninstall_user.py

@@ -51,7 +51,7 @@ def test_uninstall_editable_from_usersite(self, script, data):
         """
         Test uninstall editable local user install
         """
-        script.user_site_path.makedirs()
+        script.user_site_path.mkdir(parents=True)
 
         # install
         to_install = data.packages.joinpath("FSPkg")

tests/functional/test_wheel.py

@@ -16,7 +16,7 @@ def auto_with_wheel(with_wheel):
 
 
 def add_files_to_dist_directory(folder):
-    (folder / 'dist').makedirs()
+    (folder / 'dist').mkdir(parents=True)
     (folder / 'dist' / 'a_name-0.0.1.tar.gz').write_text("hello")
     # Not adding a wheel file since that confuses setuptools' backend.
     # (folder / 'dist' / 'a_name-0.0.1-py2.py3-none-any.whl').write_text(

tests/lib/__init__.py

@@ -471,7 +471,7 @@ def __init__(self, base_path, *args, **kwargs):
 
         # create easy-install.pth in user_site, so we always have it updated
         #   instead of created
-        self.user_site_path.makedirs()
+        self.user_site_path.mkdir(parents=True)
         self.user_site_path.joinpath("easy-install.pth").touch()
 
     def _ignore_file(self, fn):

tests/lib/path.py

@@ -161,20 +161,17 @@ def exists(self):
         """
         return os.path.exists(self)
 
-    def mkdir(self, mode=0x1FF):  # 0o777
+    def mkdir(self, mode=0x1FF, parents=False):  # 0o777
         """
         Creates a directory, if it doesn't exist already.
-        """
-        if not self.exists():
-            os.mkdir(self, mode)
-        return self
 
-    def makedirs(self, mode=0x1FF):  # 0o777
+        :param parents: Whether to create parent directories.
         """
-        Like mkdir(), but also creates parent directories.
-        """
-        if not self.exists():
-            os.makedirs(self, mode)
+        if self.exists():
+            return self
+
+        maker_func = os.makedirs if parents else os.mkdir
+        maker_func(self, mode)
         return self
 
     def unlink(self):

tests/lib/venv.py

@@ -72,7 +72,7 @@ def _create(self, clear=False):
                 context = builder.ensure_directories(self.location)
                 builder.create_configuration(context)
                 builder.setup_python(context)
-                self.site.makedirs()
+                self.site.mkdir(parents=True)
             self.sitecustomize = self._sitecustomize
             self.user_site_packages = self._user_site_packages
 

tests/unit/test_download.py

@@ -25,7 +25,7 @@
 @pytest.fixture(scope="function")
 def cache_tmpdir(tmpdir):
     cache_dir = tmpdir.joinpath("cache")
-    cache_dir.makedirs()
+    cache_dir.mkdir(parents=True)
     yield cache_dir