Merge pull request #78 from Bachmann1234/issue-77

Strip the source property of trailing and leading whitespace
Bachmann1234 · Jul 11, 2018 · 39edb0c · 39edb0c
2 parents 1a91bcf + 5b1ced3
commit 39edb0c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/diff_cover/tests/fixtures/dotnet_coverage.xml b/diff_cover/tests/fixtures/dotnet_coverage.xml
@@ -3,7 +3,9 @@
 <coverage line-rate="0.75" branch-rate="1" lines-covered="9" lines-valid="12" branches-covered="0" branches-valid="0" complexity="0" version="0" timestamp="1410969572">
   <sources>
     <source>--source</source>
-    <source>/code/samplediff/SampleApp</source>
+    <source>
+      /code/samplediff/SampleApp 
+    </source>
   </sources>
   <packages>
     <package name="SampleApp" line-rate="0.75" branch-rate="1" complexity="0">

diff --git a/diff_cover/violationsreporters/violations_reporter.py b/diff_cover/violationsreporters/violations_reporter.py
@@ -84,7 +84,7 @@ def _get_classes(self, xml_document, src_path):
              src_abs_path in [
                  self._to_unix_path(
                      os.path.join(
-                         source,
+                         source.strip(),
                          clazz.get('filename')
                      )
                  ) for source in sources]]