Merge pull request #250 from pubs/fix/note_expanduser

Fix #249: ~ expansion in filebroker.DocBroker

pubs/filebroker.py

@@ -154,7 +154,7 @@ class DocBroker(object):
 
     def __init__(self, directory, scheme='docsdir', subdir='doc'):
         self.scheme = scheme
-        self.docdir = os.path.join(directory, subdir)
+        self.docdir = os.path.expanduser(os.path.join(directory, subdir))
         if not check_directory(self.docdir, fail=False):
             os.mkdir(system_path(self.docdir))
 

tests/test_filebroker.py

@@ -71,6 +71,16 @@ class TestFileBroker(fake_env.TestFakeFs):
 
 class TestDocBroker(fake_env.TestFakeFs):
 
+    def test_expanduser(self):
+        """Test that real_docpath expand the user ~"""
+
+        self.fs.add_real_directory(os.path.join(self.rootpath, 'data'), read_only=False)
+        fb = filebroker.FileBroker('~/testrepo', create = True)
+        docb = filebroker.DocBroker('~/testrepo')
+        self.assertTrue(os.path.isabs(docb.docdir))
+        self.assertTrue(os.path.isabs(docb.real_docpath('docsdir://abc')))
+
+
     def test_doccopy(self):
 
         self.fs.add_real_directory(os.path.join(self.rootpath, 'data'), read_only=False)