Added support for passing file names as pathlib.Path

rarfile.py

@@ -813,6 +813,8 @@ def extract(self, member, path=None, pwd=None):
         """
         if isinstance(member, RarInfo):
             fname = member.filename
+        elif _have_pathlib and isinstance(member, Path):
+            fname = str(member)
         else:
             fname = member
         self._extract([fname], path, pwd)
@@ -898,6 +900,8 @@ def _extract(self, fnlist, path=None, psw=None):
 
             # destination path
             if path is not None:
+                if _have_pathlib and isinstance(path, Path):
+                    path = str(path)
                 cmd.append(path + os.sep)
 
             # call
@@ -967,6 +971,8 @@ def getinfo(self, member):
         """
         if isinstance(member, RarInfo):
             fname = member.filename
+        elif _have_pathlib and isinstance(member, Path):
+            fname = str(member)
         else:
             fname = member
 
@@ -2738,7 +2744,12 @@ def _parse_xtime(flag, data, pos, basetime=None):
 def is_filelike(obj):
     """Filename or file object?
     """
-    if isinstance(obj, (bytes, unicode)):
+    if _have_pathlib:
+        filename_types = (bytes, unicode, Path)
+    else:
+        filename_types = (bytes, unicode)
+
+    if isinstance(obj, filename_types):
         return False
     res = True
     for a in ('read', 'tell', 'seek'):

test/test_api.py

@@ -1,6 +1,5 @@
 """API tests.
 """
-
 import sys
 import io
 import os
@@ -9,6 +8,9 @@
 
 import rarfile
 
+if rarfile._have_pathlib:
+    from pathlib import Path
+
 #
 # test start
 #
@@ -49,6 +51,11 @@ def test_open_psw_late_rar5():
     rf.open('stest1.txt', 'r', 'password').read()
     rf.open('stest1.txt', 'r', u'password').read()
 
+if rarfile._have_pathlib:
+    def test_open_pathlib_path():
+        rf = rarfile.RarFile('test/files/rar5-psw.rar')
+        rf.open(Path('stest1.txt'), 'r', 'password').read()
+
 def test_read_psw_late_rar3():
     rf = rarfile.RarFile('test/files/rar3-comment-psw.rar')
     rf.read('file1.txt', 'password')
@@ -64,20 +71,21 @@ def test_open_psw_late():
     rf = rarfile.RarFile('test/files/rar5-psw.rar')
     rf.read('stest1.txt', 'password222')
 
-def test_open_pathlib_path():
-    try:
-        from pathlib import Path
+if rarfile._have_pathlib:
+    def test_create_from_pathlib_path():
         # Make sure we can open both relative and absolute Paths
         rarfile.RarFile(Path('test/files/rar5-psw.rar'))
         rarfile.RarFile(Path('test/files/rar5-psw.rar').resolve())
-    except ImportError:
-        pass
 
 def test_detection():
     eq_(rarfile.is_rarfile('test/files/ctime4.rar.exp'), False)
     eq_(rarfile.is_rarfile('test/files/ctime4.rar'), True)
     eq_(rarfile.is_rarfile('test/files/rar5-crc.rar'), True)
 
+    if rarfile._have_pathlib:
+        eq_(rarfile.is_rarfile(Path('test/files/rar5-crc.rar')), True)
+
+
 @raises(rarfile.BadRarFile)
 def test_signature_error():
     rarfile.RarFile('test/files/ctime4.rar.exp')
@@ -178,6 +186,12 @@ def test_extract():
     rf.extractall('tmp/extract3', [rf.getinfo('stest2.txt')])
     assert_true(os.path.isfile('tmp/extract3/stest2.txt'))
 
+    if rarfile._have_pathlib:
+        os.makedirs('tmp/extract1_pathlib')
+        rf.extractall(Path('tmp/extract1'))
+        assert_true(os.path.isfile('tmp/extract1/stest1.txt'))
+        assert_true(os.path.isfile('tmp/extract1/stest2.txt'))
+
 @with_setup(clean_extract_dirs, clean_extract_dirs)
 def test_extract_mem():
     os.makedirs('tmp/extract1')