Added specific test for roughly checking configuration paths.

Byron · Byron · commit dfb0a9c4bca5 · 2015-01-12T10:25:15.000+01:00
For some reason, I didn't trust the existing one as it tests that code more indirectly. Related to gitpython-developers#160
diff --git a/.gitignore b/.gitignore
@@ -10,6 +10,6 @@ cover/
 /doc/_build
 nbproject
 *.sublime-workspace
-
+.DS_Store
 /*egg-info
 /.tox
diff --git a/git/repo/base.py b/git/repo/base.py
@@ -354,7 +354,7 @@ def _get_config_path(self, config_level):
         if config_level == "system":
             return "/etc/gitconfig"
         elif config_level == "user":
-            config_home = os.environ.get("XDG_CONFIG_HOME") or join(os.environ.get("HOME", '~'), ".config")
+            config_home = os.environ.get("XDG_CONFIG_HOME") or os.path.join(os.environ.get("HOME", '~'), ".config")
             return os.path.expanduser(join(config_home, "git", "config"))
         elif config_level == "global":
             return os.path.normpath(os.path.expanduser("~/.gitconfig"))
diff --git a/git/test/test_repo.py b/git/test/test_repo.py
@@ -365,6 +365,11 @@ def test_config_writer(self):
                 pass
         # END for each config level
 
+    def test_config_level_paths(self):
+        for config_level in self.rorepo.config_level:
+            assert self.rorepo._get_config_path(config_level)
+        # end for each config level
+
     def test_creation_deletion(self):
         # just a very quick test to assure it generally works. There are
         # specialized cases in the test_refs module
