Give warning for git repository problems [fix #41]

psss · Sep 23, 2015 · 450a94c · 450a94c
1 parent ac9dcfb
commit 450a94c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/did/plugins/git.py b/did/plugins/git.py
@@ -72,9 +72,9 @@ def commits(self, user, options):
                         summary, 8 * " ", directory))
                 return commits
         else:
-            log.error(errors.strip())
-            raise ReportError(
-                "Unable to check commits in '{0}'".format(self.path))
+            log.debug(errors.strip())
+            log.warn("Unable to check commits in '{0}'".format(self.path))
+            return []
 
 
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

diff --git a/tests/plugins/test_git.py b/tests/plugins/test_git.py
@@ -56,9 +56,7 @@ def test_git_invalid():
     try:
         did.cli.main(INTERVAL)
     except SystemExit:
-        pass
-    else:
-        raise RuntimeError("Expected failure")
+        raise RuntimeError("Expected warning only")
 
 def test_git_non_existent():
     """ Non-existent git repo """