add pmd xml driver

diff_cover/diff_quality_tool.py

@@ -25,7 +25,7 @@
     jshint_driver, eslint_driver, pydocstyle_driver,
     pycodestyle_driver)
 from diff_cover.violationsreporters.java_violations_reporter import (
-    CheckstyleXmlDriver, checkstyle_driver, FindbugsXmlDriver)
+    CheckstyleXmlDriver, checkstyle_driver, FindbugsXmlDriver, PmdXmlDriver)
 
 QUALITY_DRIVERS = {
     'pycodestyle': pycodestyle_driver,
@@ -37,7 +37,8 @@
     'pydocstyle': pydocstyle_driver,
     'checkstyle': checkstyle_driver,
     'checkstylexml': CheckstyleXmlDriver(),
-    'findbugs': FindbugsXmlDriver()
+    'findbugs': FindbugsXmlDriver(),
+    'pmd': PmdXmlDriver()
 }
 
 VIOLATION_CMD_HELP = "Which code quality tool to use (%s)" % "/".join(sorted(QUALITY_DRIVERS))

diff_cover/violationsreporters/java_violations_reporter.py

@@ -3,6 +3,7 @@
 """
 from __future__ import unicode_literals
 
+import os
 from collections import defaultdict
 
 from xml.etree import cElementTree
@@ -115,3 +116,46 @@ def installed(self):
         Returns: boolean False: As findbugs analyses bytecode, it would be hard to run it from outside the build framework.
         """
         return False
+
+
+class PmdXmlDriver(QualityDriver):
+    def __init__(self):
+        """
+        See super for args
+        """
+        super(PmdXmlDriver, self).__init__(
+            'pmd',
+            ['java'],
+            []
+        )
+
+    def parse_reports(self, reports):
+        """
+        Args:
+            reports: list[str] - output from the report
+        Return:
+            A dict[Str:Violation]
+            Violation is a simple named tuple Defined above
+        """
+        violations_dict = defaultdict(list)
+        for report in reports:
+            xml_document = cElementTree.fromstring("".join(report))
+            files = xml_document.findall(".//file")
+            for file in files:
+                for error in file.findall('violation'):
+                    line_number = error.get('beginline')
+                    error_str = "{}: {}".format(error.get('rule'),
+                                                error.text.strip())
+                    violation = Violation(int(line_number), error_str)
+                    filename = GitPathTool.relative_path(file.get('name'))
+                    filename = filename.replace(os.sep, "/")
+                    violations_dict[filename].append(violation)
+
+        return violations_dict
+
+    def installed(self):
+        """
+        Method checks if the provided tool is installed.
+        Returns: boolean False: As findbugs analyses bytecode, it would be hard to run it from outside the build framework.
+        """
+        return True