Add support for multipath parameter to app_dirs property. Ref jaraco#88.

jaraco · jaraco · commit 8ae026bb4e89 · 2015-07-27T22:44:40.000-04:00
diff --git a/path.py b/path.py
@@ -1493,7 +1493,8 @@ def get_dir(self, scope, class_):
         appdirs = importlib.import_module('appdirs')
         func_name = '{scope}_{class_}_dir'.format(**locals())
         func = getattr(appdirs, func_name)
-        return compose(self.path_class, func)
+        MultiPath = Multi.for_class(self.path_class)
+        return compose(MultiPath.detect, func)
 
     @staticmethod
     def ensure(target, *subs):
diff --git a/test_path.py b/test_path.py
@@ -962,6 +962,7 @@ def appdirs_installed(cls):
     def feign_linux(self, monkeypatch):
         monkeypatch.setattr("platform.system", lambda: "Linux")
         monkeypatch.setattr("sys.platform", "linux")
+        monkeypatch.setattr("os.pathsep", ":")
         # remove any existing import of appdirs, as it sets up some
         # state during import.
         sys.modules.pop('appdirs')
@@ -1015,6 +1016,20 @@ def test_other_parameters(self):
         res = Path.app_dirs.site.config(multipath=True)
         assert isinstance(res, str)
 
+    def test_multipath(self, feign_linux, monkeypatch, tmpdir):
+        """
+        If multipath is provided, on Linux return the XDG_CONFIG_DIRS
+        """
+        fake_config_1 = str(tmpdir / '_config1')
+        fake_config_2 = str(tmpdir / '_config2')
+        config_dirs = os.pathsep.join([fake_config_1, fake_config_2])
+        monkeypatch.setitem(os.environ, 'XDG_CONFIG_DIRS', config_dirs)
+        res = Path.app_dirs.site.config(multipath=True)
+        assert isinstance(res, Multi)
+        assert fake_config_1 in res
+        assert fake_config_2 in res
+        assert '_config1' in str(res)
+
 
 class TestMultiPath:
     def test_for_class(self):
