fix for coverage reports w/ absolute file path (#69)

fix for coverage reports w/ absolute file path

CHANGELOG

@@ -1,3 +1,7 @@
+12/19/2017 v1.0.2
+
+ * Fix bug where clover uses absolute path in its xml report
+
 12/08/2017 v1.0.1
 
  * Add Support for clover and checkstyle

diff_cover/__init__.py

@@ -1,3 +1,3 @@
-VERSION = '1.0.1'
+VERSION = '1.0.2'
 DESCRIPTION = 'Automatically find diff lines that need test coverage.'
 QUALITY_DESCRIPTION = 'Automatically find diff lines with quality violations.'

diff_cover/command_runner.py

@@ -1,3 +1,4 @@
+import re
 import six
 import subprocess
 
@@ -38,6 +39,9 @@ def execute(command):
         raise
 
     stderr = _ensure_unicode(stderr)
+    # after version 1.8.0 pylint writes a message to stderr:
+    # Using config file {}
+    stderr = re.sub(r'^Using config file .*$', '', stderr).strip()
     # If we get a non-empty output to stderr, raise an exception
     if bool(stderr) and process.returncode:
         raise CommandError(stderr)

diff_cover/tests/test_java_violations_reporter.py

@@ -45,7 +45,7 @@ class CloverXmlCoverageReporterTest(unittest.TestCase):
 
     def setUp(self):
         # Paths generated by git_path are always the given argument
-        _git_path_mock = patch('diff_cover.violationsreporters.violations_reporter.GitPathTool').start()
+        _git_path_mock = patch('diff_cover.violationsreporters.java_violations_reporter.GitPathTool').start()
         _git_path_mock.relative_path = lambda path: path
         _git_path_mock.absolute_path = lambda path: path
 

diff_cover/violationsreporters/java_violations_reporter.py

@@ -39,7 +39,7 @@ def _get_src_path_line_nodes(xml_document, src_path):
         """
         files = [file_tree
                  for file_tree in xml_document.findall(".//file")
-                 if file_tree.get('path') == src_path
+                 if GitPathTool.relative_path(file_tree.get('path')) == src_path
                  or []]
         if not files:
             return None