when reading pmd report, remove working dir and leave only relative path

Author: djydewang

lib/pmdtester/builders/diff_builder.rb

@@ -14,17 +14,19 @@ class DiffBuilder
     # http://pmd.sourceforge.net/report_2_0_0.xsd
     def build(base_report_filename, patch_report_filename, base_info, patch_info, filter_set = nil)
       report_diffs = ReportDiff.new
-      base_violations, base_errors = parse_pmd_report(base_report_filename, 'base', filter_set)
-      patch_violations, patch_errors = parse_pmd_report(patch_report_filename, 'patch')
+      base_details, patch_details = report_diffs.calculate_details(base_info, patch_info)
+      base_violations, base_errors = parse_pmd_report(base_report_filename, 'base',
+                                                      base_details.working_dir, filter_set)
+      patch_violations, patch_errors = parse_pmd_report(patch_report_filename, 'patch',
+                                                        patch_details.working_dir)
       report_diffs.calculate_violations(base_violations, patch_violations)
       report_diffs.calculate_errors(base_errors, patch_errors)
-      report_diffs.calculate_details(base_info, patch_info)
 
       report_diffs
     end
 
-    def parse_pmd_report(report_filename, branch, filter_set = nil)
-      doc = PmdReportDocument.new(branch, filter_set)
+    def parse_pmd_report(report_filename, branch, working_dir, filter_set = nil)
+      doc = PmdReportDocument.new(branch, working_dir, filter_set)
       parser = Nokogiri::XML::SAX::Parser.new(doc)
       parser.parse_file(report_filename) unless report_filename.nil?
       [doc.violations, doc.errors]

lib/pmdtester/parsers/pmd_report_document.rb

@@ -9,7 +9,7 @@ module PmdTester
   class PmdReportDocument < Nokogiri::XML::SAX::Document
     attr_reader :violations
     attr_reader :errors
-    def initialize(branch_name, filter_set = nil)
+    def initialize(branch_name, working_dir, filter_set = nil)
       @violations = PmdViolations.new
       @errors = PmdErrors.new
       @current_violations = []
@@ -18,6 +18,7 @@ def initialize(branch_name, filter_set = nil)
       @current_element = ''
       @filename = ''
       @filter_set = filter_set
+      @working_dir = working_dir
       @branch_name = branch_name
     end
 
@@ -28,11 +29,11 @@ def start_element(name, attrs = [])
       case name
       when 'file'
         @current_violations = []
-        @current_filename = attrs['name']
+        @current_filename = attrs['name'].sub(/^#{@working_dir}/, '')
       when 'violation'
         @current_violation = PmdViolation.new(attrs, @branch_name)
       when 'error'
-        @current_filename = attrs['filename']
+        @current_filename = attrs['filename'].sub(/^#{@working_dir}/, '')
         @current_error = PmdError.new(attrs, @branch_name)
       end
     end

lib/pmdtester/pmd_report_detail.rb

@@ -7,14 +7,16 @@ module PmdTester
   class PmdReportDetail
     attr_accessor :execution_time
     attr_accessor :timestamp
+    attr_reader :working_dir
 
     def initialize
       @execution_time = 0
       @timestamp = ''
+      @working_dir = Dir.getwd
     end
 
     def save(report_info_path)
-      hash = { execution_time: @execution_time, timestamp: @timestamp }
+      hash = { execution_time: @execution_time, timestamp: @timestamp, working_dir: @working_dir }
       file = File.new(report_info_path, 'w')
       file.puts JSON.generate(hash)
       file.close
@@ -25,6 +27,7 @@ def load(report_info_path)
         hash = JSON.parse(File.read(report_info_path))
         @execution_time = hash['execution_time']
         @timestamp = hash['timestamp']
+        @working_dir = hash['working_dir']
         hash
       else
         puts "#{report_info_path} doesn't exist"

lib/pmdtester/project.rb

@@ -51,13 +51,13 @@ def default_webview_url
     # Change the file path from 'LOCAL_DIR/SOURCE_CODE_PATH' to
     # 'WEB_VIEW_URL/SOURCE_CODE_PATH'
     def get_webview_url(file_path)
-      file_path.gsub(/#{local_source_path}/, @webview_url)
+      file_path.gsub(%r{/#{local_source_path}}, @webview_url)
     end
 
     # Change the file path from 'LOCAL_DIR/SOURCE_CODE_PATH' to
     # 'PROJECT_NAME/SOURCE_CODE_PATH'
     def get_path_inside_project(file_path)
-      file_path.gsub(/#{local_source_path}/, @name)
+      file_path.gsub(%r{/#{local_source_path}}, @name)
     end
 
     def get_pmd_report_path(branch_name)
@@ -84,7 +84,7 @@ def get_project_target_dir(branch_name)
     end
 
     def local_source_path
-      "#{Dir.getwd}/#{REPOSITORIES_PATH}/#{@name}"
+      "#{REPOSITORIES_PATH}/#{@name}"
     end
 
     def target_diff_report_path

lib/pmdtester/report_diff.rb

@@ -77,6 +77,7 @@ def calculate_details(base_info, patch_info)
 
       @base_timestamp = base_details.timestamp
       @patch_timestamp = patch_details.timestamp
+      [base_details, patch_details]
     end
 
     def build_diffs(base_hash, patch_hash)

test/integration_test_runner.rb

@@ -39,14 +39,15 @@ def test_single_mode
   end
 
   def test_online_mode
-    argv = '-r target/repositories/pmd -m online -b test_branch -p pmd_releases/6.3.0'
+    base_branch = 'test_branch_2'
+    argv = "-r target/repositories/pmd -m online -b #{base_branch} -p pmd_releases/6.3.0"
     # This test depends on the file test_branch-baseline.zip being available on sourceforge.
 
     system("bundle exec bin/pmdtester #{argv}")
 
-    assert_path_exist('target/reports/test_branch-baseline.zip')
-    assert_path_exist('target/reports/test_branch/checkstyle/pmd_report.xml')
-    assert_path_exist('target/reports/test_branch/spring-framework/pmd_report.xml')
+    assert_path_exist("target/reports/#{base_branch}-baseline.zip")
+    assert_path_exist("target/reports/#{base_branch}/checkstyle/pmd_report.xml")
+    assert_path_exist("target/reports/#{base_branch}/spring-framework/pmd_report.xml")
     assert_path_exist('target/reports/pmd_releases_6.3.0/checkstyle/pmd_report.xml')
     assert_path_exist('target/reports/pmd_releases_6.3.0/spring-framework/pmd_report.xml')
     assert_path_exist('target/reports/diff/checkstyle/index.html')

test/resources/diff_builder/base_report_info.json

@@ -0,0 +1 @@
+{"execution_time":121,"timestamp":"base time stamp","working_dir":""}

test/resources/diff_builder/patch_report_info.json

@@ -0,0 +1 @@
+{"execution_time":65,"timestamp":"patch time stamp","working_dir":""}

test/resources/html_report_builder/base_report_info.json

@@ -1 +1 @@
-{"execution_time":121,"timestamp":"base time stamp"}
+{"execution_time":121,"timestamp":"base time stamp","working_dir":"SHOULD_BE_REPLACED"}

test/resources/html_report_builder/patch_report_info.json

@@ -1 +1 @@
-{"execution_time":65,"timestamp":"patch time stamp"}
+{"execution_time":65,"timestamp":"patch time stamp","working_dir":"SHOULD_BE_REPLACED"}