Make VCS error messages and Storage test failures more descriptive

4 files changed, 11 insertions, 8 deletions

diff --git a/libbe/storage/vcs/arch.py b/libbe/storage/vcs/arch.py
index 74ba371..bfccf59 100644
--- a/libbe/storage/vcs/arch.py
+++ b/libbe/storage/vcs/arch.py
@@ -72,7 +72,8 @@ class Arch(base.VCS):
 
     def _vcs_version(self):
         status,output,error = self._u_invoke_client('--version')
-        return output
+        version = '\n'.join(output.splitlines()[:2])
+        return version
 
     def _vcs_detect(self, path):
         """Detect whether a directory is revision-controlled using Arch"""
@@ -307,7 +308,8 @@ class Arch(base.VCS):
             return base.VCS._vcs_get_file_contents(self, relpath)
 
     def _vcs_path(self, id, revision):
-        raise NotImplementedError
+        raise NotImplementedError(
+            'Too little Arch understanding at the moment...')
 
     def _vcs_commit(self, commitfile, allow_empty=False):
         if allow_empty == False:
@@ -356,7 +358,8 @@ class Arch(base.VCS):
         return '%s--%s' % (self._archive_project_name(), log)
 
     def _vcs_changed(self, revision):
-        raise NotImplementedError
+        raise NotImplementedError(
+            'Too little Arch understanding at the moment...')
 
 
 if libbe.TESTING == True:
diff --git a/libbe/storage/vcs/base.py b/libbe/storage/vcs/base.py
index 8335cfa..337576e 100644
--- a/libbe/storage/vcs/base.py
+++ b/libbe/storage/vcs/base.py
@@ -515,7 +515,7 @@ os.listdir(self.get_path("bugs")):
         """
         Does the path exist in a given revision? (True/False)
         """
-        raise NotImplementedError
+        raise NotImplementedError('Lazy BE developers')
 
     def _vcs_remove(self, path):
         """
diff --git a/libbe/storage/vcs/darcs.py b/libbe/storage/vcs/darcs.py
index c6892b4..7c6f069 100644
--- a/libbe/storage/vcs/darcs.py
+++ b/libbe/storage/vcs/darcs.py
@@ -53,7 +53,7 @@ class Darcs(base.VCS):
 
     def _vcs_version(self):
         status,output,error = self._u_invoke_client('--version')
-        return output.rstrip('\n')
+        return output.strip()
 
     def version_cmp(self, *args):
         """
@@ -191,8 +191,8 @@ class Darcs(base.VCS):
             if path in children:
                 return True
             return False
-        # Darcs versions <= 2.3.1 lack the --patch option for 'show files'
-        raise NotImplementedError
+        raise NotImplementedError(
+            'Darcs versions <= 2.3.1 lack the --patch option for "show files"')
 
     def _vcs_listdir(self, path, revision):
         if self.version_cmp(2, 3, 1) == 1:
diff --git a/libbe/storage/vcs/git.py b/libbe/storage/vcs/git.py
index 6d3aa87..e2c1b83 100644
--- a/libbe/storage/vcs/git.py
+++ b/libbe/storage/vcs/git.py
@@ -49,7 +49,7 @@ class Git(base.VCS):
 
     def _vcs_version(self):
         status,output,error = self._u_invoke_client('--version')
-        return output
+        return output.strip()
 
     def _vcs_get_user_id(self):
         status,output,error = \