Add makedirs_p to cross-compatible API (#60)

This is a no-op directory methods for swift paths

docs/release_notes.rst

@@ -1,6 +1,12 @@
 Release Notes
 =============
 
+v1.5.4
+------
+
+* Add ``stor.makedirs_p(path, mode=0o777)`` to cross-compatible API. This does
+  nothing on OBS-paths (just there for convenience).
+
 v1.5.3
 ------
 

stor/__init__.py

@@ -75,6 +75,7 @@ def wrapper(path, *args, **kwargs):
 islink = _delegate_to_path('islink')
 ismount = _delegate_to_path('ismount')
 getsize = _delegate_to_path('getsize')
+makedirs_p = _delegate_to_path('makedirs_p')
 remove = _delegate_to_path('remove')
 rmtree = _delegate_to_path('rmtree')
 walkfiles = _delegate_to_path('walkfiles')

stor/base.py

@@ -342,6 +342,11 @@ def rmtree(self):
         See shutil.rmtree"""
         raise NotImplementedError
 
+    def makedirs_p(self, mode=0o777):
+        """ Like :func:`os.makedirs`, but does not raise an exception if the
+        directory already exists. """
+        raise NotImplementedError
+
     def walkfiles(self, pattern=None):
         """Iterate over files recursively.
 

stor/obs.py

@@ -186,6 +186,12 @@ def islink(self):
     def ismount(self):
         return True
 
+    # NOTE: we only have makedirs_p() because the other mkdir/mkdir_p/makedirs methods are expected
+    # to raise errors if directories exist or intermediate directories don't exist.
+    def makedirs_p(self, mode=0o777):
+        """No-op (no directories on OBS)"""
+        return
+
     def getsize(self):
         """Returns size, in bytes of path."""
         raise NotImplementedError

stor/tests/test_swift.py

@@ -231,6 +231,10 @@ def setUp(self):
         super(TestSwiftFile, self).setUp()
         settings.update({'swift': {'num_retries': 5}})
 
+    def test_makedirs_p_does_nothing(self):
+        # dumb test... but why not?
+        SwiftPath('swift://tenant/container/obj').makedirs_p()
+
     def test_invalid_buffer_mode(self):
         swift_f = SwiftPath('swift://tenant/container/obj').open()
         swift_f.mode = 'invalid'