pmdtester 1.1.2 → 1.4.0

data/lib/pmdtester/project.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'pathname'
+
 module PmdTester
   # This class represents all the information about the project
   class Project
@@ -12,7 +14,8 @@ module PmdTester
     attr_reader :connection
     attr_reader :webview_url
     attr_reader :tag
-    attr_reader :exclude_pattern
+    attr_reader :exclude_patterns
+    attr_reader :src_subpath
     attr_accessor :report_diff
     # key: pmd branch name as String => value: local path of pmd report
     attr_reader :build_command
@@ -20,7 +23,7 @@ module PmdTester
     # stores the auxclasspath calculated after cloning/preparing the project
     attr_accessor :auxclasspath
 
-    def initialize(project)
+    def initialize(project) # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
       @name = project.at_xpath('name').text
       @type = project.at_xpath('type').text
       @connection = project.at_xpath('connection').text
@@ -31,9 +34,10 @@ module PmdTester
       @webview_url = default_webview_url
       @webview_url = webview_url_element.text unless webview_url_element.nil?
 
-      @exclude_pattern = []
+      @src_subpath = project.at_xpath('src-subpath')&.text || '.'
+      @exclude_patterns = []
       project.xpath('exclude-pattern').each do |ep|
-        @exclude_pattern.push(ep.text)
+        @exclude_patterns.push(ep.text)
       end
 
       @build_command = project.at_xpath('build-command')&.text
@@ -56,17 +60,17 @@ module PmdTester
     # 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(%r{/#{local_source_path}}, @webview_url)
+      file_path.gsub(%r{/#{clone_root_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(%r{/#{local_source_path}}, @name)
+      file_path.gsub(%r{/#{clone_root_path}}, @name)
     end
 
     def get_local_path(file_path)
-      file_path.sub(%r{/#{local_source_path}/}, '')
+      file_path.sub(%r{/#{clone_root_path}/}, '')
     end
 
     def get_pmd_report_path(branch_name)
@@ -93,6 +97,19 @@ module PmdTester
       end
     end
 
+    ##
+    # Path to the sources to analyze (below or equal to clone_root_path)
+    def local_source_path
+      # normalize path
+      Pathname.new("#{clone_root_path}/#{src_subpath}").cleanpath
+    end
+
+    ##
+    # Path to the clone directory
+    def clone_root_path
+      "#{REPOSITORIES_PATH}/#{@name}"
+    end
+
     def get_project_target_dir(branch_name)
       branch_filename = PmdBranchDetail.branch_filename(branch_name)
       dir = "target/reports/#{branch_filename}/#{@name}"
@@ -100,10 +117,6 @@ module PmdTester
       dir
     end
 
-    def local_source_path
-      "#{REPOSITORIES_PATH}/#{@name}"
-    end
-
     def compute_report_diff(base_branch, patch_branch, filter_set)
       self.report_diff = build_report_diff(get_pmd_report_path(base_branch),
                                            get_pmd_report_path(patch_branch),

data/lib/pmdtester/report_diff.rb

@@ -57,28 +57,45 @@ module PmdTester
                 :configerrors_by_rule,
                 :exec_time,
                 :timestamp,
-                :infos_by_rule
+                :file
 
     def initialize(report_document: nil,
+                   file: '',
                    exec_time: 0,
                    timestamp: '0')
       initialize_empty
       initialize_with_report_document report_document unless report_document.nil?
       @exec_time = exec_time
       @timestamp = timestamp
+      @file = file
     end
 
     def self.empty
       new
     end
 
+    def rule_summaries
+      summary = {}
+      @violations_by_file.each_value do |violation|
+        unless summary.key?(violation.rule_name)
+          summary[violation.rule_name] = {
+            'name' => violation.rule_name,
+            'info_url' => violation.info_url,
+            'count' => 0
+          }
+        end
+        summary[violation.rule_name]['count'] += 1
+      end
+
+      summary.values
+    end
+
     private
 
     def initialize_with_report_document(report_document)
       @violations_by_file = report_document.violations
       @errors_by_file = report_document.errors
       @configerrors_by_rule = report_document.configerrors
-      @infos_by_rule = report_document.infos_by_rules
 
       PmdTester.logger.debug("Loaded #{@violations_by_file.total_size} violations " \
                              "in #{@violations_by_file.num_files} files")
@@ -91,7 +108,6 @@ module PmdTester
       @violations_by_file = CollectionByFile.new
       @errors_by_file = CollectionByFile.new
       @configerrors_by_rule = {}
-      @infos_by_rule = {}
     end
   end
 
@@ -122,7 +138,7 @@ module PmdTester
       @error_diffs_by_file = {}
       @configerror_diffs_by_rule = {}
 
-      @rule_infos_union = base_report.infos_by_rule.dup
+      @rule_infos_union = {}
       @base_report = base_report
       @patch_report = patch_report
 

data/lib/pmdtester/runner.rb

@@ -33,8 +33,11 @@ module PmdTester
     def run_local_mode
       logger.info "Mode: #{@options.mode}"
       get_projects(@options.project_list) unless @options.nil?
-      rule_sets = RuleSetBuilder.new(@options).build if @options.auto_config_flag
-      return if rule_sets&.empty?
+      if @options.auto_config_flag
+        run_required = RuleSetBuilder.new(@options).build?
+        logger.debug "Run required: #{run_required}"
+        return unless run_required
+      end
 
       base_branch_details = create_pmd_report(config: @options.base_config, branch: @options.base_branch)
       patch_branch_details = create_pmd_report(config: @options.patch_config, branch: @options.patch_branch)
@@ -51,7 +54,8 @@ module PmdTester
       get_projects(project_list)
 
       if @options.auto_config_flag
-        return if RuleSetBuilder.new(@options).build.empty?
+        logger.info 'Autogenerating a dynamic ruleset based on source changes'
+        return unless RuleSetBuilder.new(@options).build?
       elsif @options.patch_config == Options::DEFAULT_CONFIG_PATH
         # patch branch build pmd reports with same configuration as base branch
         # if not specified otherwise. This allows to use a different config (e.g. less rules)
@@ -59,6 +63,7 @@ module PmdTester
         @options.patch_config = "#{baseline_path}/config.xml"
       else
         logger.info "Using config #{@options.patch_config} which might differ from baseline"
+        RuleSetBuilder.new(@options).calculate_filter_set if @options.filter_with_patch_config
       end
 
       patch_branch_details = create_pmd_report(config: @options.patch_config, branch: @options.patch_branch)

data/lib/pmdtester/semver.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module PmdTester
+  # Utility to deal with semantic versions
+  class Semver
+    def self.compare(version_a, version_b)
+      PmdTester.logger.debug "Comparing #{version_a} <=> #{version_b}"
+      m = /(\d+)\.(\d+)\.(\d+)(.*)/.match(version_a)
+      a_major = m[1].to_i
+      a_minor = m[2].to_i
+      a_patch = m[3].to_i
+      a_snapshot = m[4]
+      PmdTester.logger.debug "  a_major: #{a_major} a_minor: #{a_minor} a_patch: #{a_patch} a_snapshot: #{a_snapshot}"
+
+      m = /(\d+)\.(\d+)\.(\d+)(.*)/.match(version_b)
+      b_major = m[1].to_i
+      b_minor = m[2].to_i
+      b_patch = m[3].to_i
+      b_snapshot = m[4]
+      PmdTester.logger.debug "  b_major: #{b_major} b_minor: #{b_minor} b_patch: #{b_patch} b_snapshot: #{b_snapshot}"
+
+      return a_major <=> b_major if a_major != b_major
+      return a_minor <=> b_minor if a_minor != b_minor
+      return a_patch <=> b_patch if a_patch != b_patch
+
+      compare_snapshots(a_snapshot, b_snapshot)
+    end
+
+    private_class_method def self.compare_snapshots(a_snapshot, b_snapshot)
+      return -1 if a_snapshot == '-SNAPSHOT' && b_snapshot == ''
+      return 1 if a_snapshot == '' && b_snapshot == '-SNAPSHOT'
+
+      a_snapshot <=> b_snapshot
+    end
+  end
+end

data/lib/pmdtester.rb

@@ -15,6 +15,7 @@ require_relative 'pmdtester/project'
 require_relative 'pmdtester/report_diff'
 require_relative 'pmdtester/resource_locator'
 require_relative 'pmdtester/runner'
+require_relative 'pmdtester/semver'
 
 require_relative 'pmdtester/builders/simple_progress_logger'
 require_relative 'pmdtester/builders/project_builder'
@@ -32,7 +33,7 @@ require_relative 'pmdtester/parsers/projects_parser'
 # and unexpected behaviors will not be introduced to PMD project
 # after fixing an issue and new rules can work as expected.
 module PmdTester
-  VERSION = '1.1.2'
+  VERSION = '1.4.0'
   BASE = 'base'
   PATCH = 'patch'
   PR_NUM_ENV_VAR = 'PMD_CI_PULL_REQUEST_NUMBER' # see PmdBranchDetail

data/pmdtester.gemspec

@@ -1,22 +1,22 @@
 # DO NOT EDIT THIS FILE. Instead, edit Rakefile, and run `rake hoe:spec`.
 
 # -*- encoding: utf-8 -*-
-# stub: pmdtester 1.1.2 ruby lib
+# stub: pmdtester 1.4.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "pmdtester".freeze
-  s.version = "1.1.2"
+  s.version = "1.4.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.metadata = { "bug_tracker_uri" => "https://github.com/pmd/pmd-regression-tester/issues", "homepage_uri" => "https://pmd.github.io", "source_code_uri" => "https://github.com/pmd/pmd-regression-tester" } if s.respond_to? :metadata=
   s.require_paths = ["lib".freeze]
   s.authors = ["Andreas Dangel".freeze, "Binguo Bao".freeze, "Cl\u00E9ment Fournier".freeze]
-  s.date = "2021-04-20"
+  s.date = "2022-03-24"
   s.description = "A regression testing tool ensure that new problems and unexpected behaviors will not be introduced to PMD project after fixing an issue , and new rules can work as expected.".freeze
   s.email = ["andreas.dangel@pmd-code.org".freeze, "djydewang@gmail.com".freeze, "clement.fournier76@gmail.com".freeze]
   s.executables = ["pmdtester".freeze]
   s.extra_rdoc_files = ["History.md".freeze, "Manifest.txt".freeze, "README.rdoc".freeze]
-  s.files = [".ci/build.sh".freeze, ".ci/files/env.gpg".freeze, ".ci/inc/install-openjdk.inc".freeze, ".ci/manual-integration-tests.sh".freeze, ".github/workflows/build.yml".freeze, ".github/workflows/manual-integration-tests.yml".freeze, ".gitignore".freeze, ".hoerc".freeze, ".rubocop.yml".freeze, ".rubocop_todo.yml".freeze, ".ruby-version".freeze, "Gemfile".freeze, "History.md".freeze, "LICENSE".freeze, "Manifest.txt".freeze, "README.rdoc".freeze, "Rakefile".freeze, "bin/pmdtester".freeze, "config/all-java.xml".freeze, "config/design.xml".freeze, "config/project-list.xml".freeze, "config/projectlist_1_0_0.xsd".freeze, "config/projectlist_1_1_0.xsd".freeze, "lib/pmdtester.rb".freeze, "lib/pmdtester/builders/liquid_renderer.rb".freeze, "lib/pmdtester/builders/pmd_report_builder.rb".freeze, "lib/pmdtester/builders/project_builder.rb".freeze, "lib/pmdtester/builders/project_hasher.rb".freeze, "lib/pmdtester/builders/rule_set_builder.rb".freeze, "lib/pmdtester/builders/simple_progress_logger.rb".freeze, "lib/pmdtester/builders/summary_report_builder.rb".freeze, "lib/pmdtester/cmd.rb".freeze, "lib/pmdtester/collection_by_file.rb".freeze, "lib/pmdtester/parsers/options.rb".freeze, "lib/pmdtester/parsers/pmd_report_document.rb".freeze, "lib/pmdtester/parsers/projects_parser.rb".freeze, "lib/pmdtester/pmd_branch_detail.rb".freeze, "lib/pmdtester/pmd_configerror.rb".freeze, "lib/pmdtester/pmd_error.rb".freeze, "lib/pmdtester/pmd_report_detail.rb".freeze, "lib/pmdtester/pmd_tester_utils.rb".freeze, "lib/pmdtester/pmd_violation.rb".freeze, "lib/pmdtester/project.rb".freeze, "lib/pmdtester/report_diff.rb".freeze, "lib/pmdtester/resource_locator.rb".freeze, "lib/pmdtester/runner.rb".freeze, "pmdtester.gemspec".freeze, "resources/_includes/diff_pill_row.html".freeze, "resources/css/bootstrap.min.css".freeze, "resources/css/datatables.min.css".freeze, "resources/css/pmd-tester.css".freeze, "resources/js/bootstrap.min.js".freeze, "resources/js/code-snippets.js".freeze, "resources/js/datatables.min.js".freeze, "resources/js/jquery-3.2.1.slim.min.js".freeze, "resources/js/jquery.min.js".freeze, "resources/js/popper.min.js".freeze, "resources/js/project-report.js".freeze, "resources/project_diff_report.html".freeze, "resources/project_index.html".freeze]
+  s.files = [".ci/build.sh".freeze, ".ci/inc/fetch_ci_scripts.bash".freeze, ".ci/manual-integration-tests.sh".freeze, ".github/workflows/build.yml".freeze, ".github/workflows/manual-integration-tests.yml".freeze, ".gitignore".freeze, ".hoerc".freeze, ".rubocop.yml".freeze, ".rubocop_todo.yml".freeze, ".ruby-version".freeze, "Gemfile".freeze, "History.md".freeze, "LICENSE".freeze, "Manifest.txt".freeze, "README.rdoc".freeze, "Rakefile".freeze, "bin/pmdtester".freeze, "config/all-java.xml".freeze, "config/design.xml".freeze, "config/project-list.xml".freeze, "config/projectlist_1_0_0.xsd".freeze, "config/projectlist_1_1_0.xsd".freeze, "config/projectlist_1_2_0.xsd".freeze, "lib/pmdtester.rb".freeze, "lib/pmdtester/builders/liquid_renderer.rb".freeze, "lib/pmdtester/builders/pmd_report_builder.rb".freeze, "lib/pmdtester/builders/project_builder.rb".freeze, "lib/pmdtester/builders/project_hasher.rb".freeze, "lib/pmdtester/builders/rule_set_builder.rb".freeze, "lib/pmdtester/builders/simple_progress_logger.rb".freeze, "lib/pmdtester/builders/summary_report_builder.rb".freeze, "lib/pmdtester/cmd.rb".freeze, "lib/pmdtester/collection_by_file.rb".freeze, "lib/pmdtester/parsers/options.rb".freeze, "lib/pmdtester/parsers/pmd_report_document.rb".freeze, "lib/pmdtester/parsers/projects_parser.rb".freeze, "lib/pmdtester/pmd_branch_detail.rb".freeze, "lib/pmdtester/pmd_configerror.rb".freeze, "lib/pmdtester/pmd_error.rb".freeze, "lib/pmdtester/pmd_report_detail.rb".freeze, "lib/pmdtester/pmd_tester_utils.rb".freeze, "lib/pmdtester/pmd_violation.rb".freeze, "lib/pmdtester/project.rb".freeze, "lib/pmdtester/report_diff.rb".freeze, "lib/pmdtester/resource_locator.rb".freeze, "lib/pmdtester/runner.rb".freeze, "lib/pmdtester/semver.rb".freeze, "pmdtester.gemspec".freeze, "resources/_includes/diff_pill_row.html".freeze, "resources/css/bootstrap.min.css".freeze, "resources/css/datatables.min.css".freeze, "resources/css/pmd-tester.css".freeze, "resources/js/bootstrap.min.js".freeze, "resources/js/code-snippets.js".freeze, "resources/js/datatables.min.js".freeze, "resources/js/jquery-3.2.1.slim.min.js".freeze, "resources/js/jquery.min.js".freeze, "resources/js/popper.min.js".freeze, "resources/js/project-report.js".freeze, "resources/project_diff_report.html".freeze, "resources/project_index.html".freeze, "resources/project_pmd_report.html".freeze]
   s.homepage = "https://pmd.github.io".freeze
   s.licenses = ["BSD-2-Clause".freeze]
   s.rdoc_options = ["--main".freeze, "README.rdoc".freeze]
@@ -29,35 +29,35 @@ Gem::Specification.new do |s|
   end
 
   if s.respond_to? :add_runtime_dependency then
-    s.add_runtime_dependency(%q<nokogiri>.freeze, [">= 1.11.0.rc4"])
+    s.add_runtime_dependency(%q<nokogiri>.freeze, ["~> 1.13"])
     s.add_runtime_dependency(%q<slop>.freeze, ["~> 4.6"])
     s.add_runtime_dependency(%q<differ>.freeze, ["~> 0.1"])
-    s.add_runtime_dependency(%q<rufus-scheduler>.freeze, ["~> 3.5"])
+    s.add_runtime_dependency(%q<rufus-scheduler>.freeze, ["~> 3.8"])
     s.add_runtime_dependency(%q<logger-colors>.freeze, ["~> 1.0"])
-    s.add_runtime_dependency(%q<liquid>.freeze, [">= 4.0"])
+    s.add_runtime_dependency(%q<liquid>.freeze, ["~> 5.2"])
     s.add_development_dependency(%q<hoe-bundler>.freeze, ["~> 1.5"])
     s.add_development_dependency(%q<hoe-git>.freeze, ["~> 1.6"])
     s.add_development_dependency(%q<minitest>.freeze, ["~> 5.10"])
     s.add_development_dependency(%q<mocha>.freeze, ["~> 1.5"])
-    s.add_development_dependency(%q<rubocop>.freeze, ["~> 0.81"])
-    s.add_development_dependency(%q<test-unit>.freeze, ["~> 3.2"])
-    s.add_development_dependency(%q<rdoc>.freeze, [">= 4.0", "< 7"])
-    s.add_development_dependency(%q<hoe>.freeze, ["~> 3.22"])
+    s.add_development_dependency(%q<rubocop>.freeze, ["~> 0.93"])
+    s.add_development_dependency(%q<test-unit>.freeze, ["~> 3.5"])
+    s.add_development_dependency(%q<rdoc>.freeze, ["~> 6.4"])
+    s.add_development_dependency(%q<hoe>.freeze, ["~> 3.23"])
   else
-    s.add_dependency(%q<nokogiri>.freeze, [">= 1.11.0.rc4"])
+    s.add_dependency(%q<nokogiri>.freeze, ["~> 1.13"])
     s.add_dependency(%q<slop>.freeze, ["~> 4.6"])
     s.add_dependency(%q<differ>.freeze, ["~> 0.1"])
-    s.add_dependency(%q<rufus-scheduler>.freeze, ["~> 3.5"])
+    s.add_dependency(%q<rufus-scheduler>.freeze, ["~> 3.8"])
     s.add_dependency(%q<logger-colors>.freeze, ["~> 1.0"])
-    s.add_dependency(%q<liquid>.freeze, [">= 4.0"])
+    s.add_dependency(%q<liquid>.freeze, ["~> 5.2"])
     s.add_dependency(%q<hoe-bundler>.freeze, ["~> 1.5"])
     s.add_dependency(%q<hoe-git>.freeze, ["~> 1.6"])
     s.add_dependency(%q<minitest>.freeze, ["~> 5.10"])
     s.add_dependency(%q<mocha>.freeze, ["~> 1.5"])
-    s.add_dependency(%q<rubocop>.freeze, ["~> 0.81"])
-    s.add_dependency(%q<test-unit>.freeze, ["~> 3.2"])
-    s.add_dependency(%q<rdoc>.freeze, [">= 4.0", "< 7"])
-    s.add_dependency(%q<hoe>.freeze, ["~> 3.22"])
+    s.add_dependency(%q<rubocop>.freeze, ["~> 0.93"])
+    s.add_dependency(%q<test-unit>.freeze, ["~> 3.5"])
+    s.add_dependency(%q<rdoc>.freeze, ["~> 6.4"])
+    s.add_dependency(%q<hoe>.freeze, ["~> 3.23"])
   end
 end
 

data/resources/css/pmd-tester.css

@@ -10,9 +10,25 @@ code {
   font-size: 13pt;
   white-space: pre;
 }
-code.highlight {
+code.highlight, tr.highlight {
   background-color: yellow;
 }
+table.code-snippet > tbody > tr > td {
+  padding: 0px;
+  border: none;
+}
+table.code-snippet > tbody > tr > td.line-number > code:before {
+  content: attr(data-line-number);
+}
+.btn-clipboard {
+  margin-top: 1rem;
+  display: block;
+  padding: .25rem .5rem;
+  color: #0d6efd;
+  background-color: #fff;
+  border: 1px solid;
+  border-radius: .25rem;
+}
 a {
   text-decoration: none;
 }

data/resources/js/code-snippets.js

@@ -15,18 +15,18 @@
 
     // returns text, not html
     function formatLineNumber(number) {
-        let prefix;
+        let prefix = '';
         if (number < 10) {
-            prefix =  nbsp.repeat(3);
+            prefix = nbsp.repeat(3);
         } else if (number < 100) {
-            prefix =  nbsp.repeat(2);
+            prefix = nbsp.repeat(2);
         } else if (number < 1000) {
             prefix = nbsp;
         }
         return prefix + number;
     }
 
-    function fetchSnippet(document, container, url, line, weburl) {
+    function fetchSnippet(document, container, url, violationLineNumber, weburl) {
         var weburl, requestUrl, oReq;
 
         requestUrl = url.replace(/github.com/, "raw.githubusercontent.com");
@@ -34,37 +34,70 @@
 
         oReq = new XMLHttpRequest();
         oReq.addEventListener("load", function() {
-            let lines, start, deleteCount;
+            let lines, start, deleteCount, lineSeparator;
 
             // we'll append stuff in the loop below
             container.innerHTML = '<p><a href="' + weburl + '" target="_blank" rel="noopener noreferrer">' + weburl + '</a></p>';
 
-            lines = this.responseText.split(/\r\n|\n/);
-            start = line - contextLines;
+            if (this.responseText.indexOf('\r\n') >= 0) {
+                lineSeparator = '\r\n';
+            } else {
+                lineSeparator = '\n';
+            }
+            lines = this.responseText.split(lineSeparator);
+            start = violationLineNumber - contextLines;
             if (start > 0) {
                 lines.splice(0, start); // remove lines before
             }
             deleteCount = lines.length - (2 * contextLines) + 1;
             lines.splice(2 * contextLines - 1, deleteCount); // delete lines after
 
+            let table = document.createElement('table');
+            table.classList.add('code-snippet');
+            let tableBody = document.createElement('tbody');
+            table.appendChild(tableBody);
             // now we have just the lines which will be displayed
             lines.forEach(line => {
                 start++;
-                let lineElt = document.createElement("code");
-                if (start === line) {
-                    lineElt.classList.add("highlight");
+                let tableRow = document.createElement('tr');
+                if (start === violationLineNumber) {
+                    tableRow.classList.add("highlight");
                 }
+
+                let lineNumberColumn = document.createElement('td');
+                lineNumberColumn.classList.add('line-number');
+                tableRow.appendChild(lineNumberColumn);
+                let lineNumberElement = document.createElement('code');
+                lineNumberColumn.appendChild(lineNumberElement);
+                lineNumberElement.setAttribute('data-line-number', formatLineNumber(start));
+
+                let codeColumn = document.createElement('td');
+                tableRow.appendChild(codeColumn);
+                let codeElement = document.createElement("code");
+                codeColumn.appendChild(codeElement);
                 // createTextNode escapes special chars
-                lineElt.appendChild(document.createTextNode(formatLineNumber(start) + nbsp + line));
-                lineElt.appendChild(document.createElement("br"));
+                codeElement.appendChild(document.createTextNode(line));
 
-                container.appendChild(lineElt); // append to the container
+                tableBody.appendChild(tableRow); // append row to the table
             });
+            container.appendChild(table);
+
+            if (navigator.clipboard) {
+                let copyButton = document.createElement('button');
+                copyButton.classList.add('btn-clipboard');
+                copyButton.setAttribute('title', 'Copy to clipboard');
+                copyButton.appendChild(document.createTextNode('copy'));
+                copyButton.onclick = function() {
+                    navigator.clipboard.writeText(lines.join(lineSeparator));
+                }
+                container.appendChild(copyButton);
+            }
         });
-        oReq.open("GET", requestUrl);
-        oReq.send();
 
         container.innerHTML = "<samp>fetching...</samp>";
+
+        oReq.open("GET", requestUrl);
+        oReq.send();
     }
 
     window.pmd_code_snippets = {

data/resources/js/project-report.js

@@ -52,12 +52,13 @@ $(document).ready(function () {
         ],
         deferRender: true,
         // scrollY: "6000px",
-        dom: 'Pfrtip', // Search Panes, filtering input, processing display element, table, table information summary, pagination control
+        dom: 'Pfrtipl', // Search Panes, filtering input, processing display element, table, table information summary, pagination control, length changing input control
         searchPanes: {
             viewTotal: true,
             cascadePanes: true,
             columns: [0, 1, 3],
-            order: ['Rule', 'Location (click row to expand)', 'Type']
+            order: ['Rule', 'Location (click row to expand)', 'Type'],
+            threshold: 1 // always show filters in search pane (default: 0.6)
         },
         // scrollCollapse: true,
         // paging: false,
@@ -112,12 +113,13 @@ $(document).ready(function () {
             },
         ],
         displayLength: 25,
+        lengthMenu: [ [10, 20, 25, 50, 100, -1], [10, 20, 25, 50, 100, "All"] ],
         rowCallback(row, data, index) {
             $(row).addClass(cssClass[data.t]);
         },
     });
 
-    $('#violationsTable tbody').on('click', 'tr', function() {
+    $('#violationsTable tbody').on('click', 'tr[role=row]', function() {
         var tr = $(this).closest('tr');
         var row = table.row( tr );
 

data/resources/project_diff_report.html

@@ -18,6 +18,9 @@
 
 </head>
 <body>
+<div class="section">
+    <h1>PMD Regression Tester Diff Report for {{project_name}}</h1>
+</div>
 <div class="section">
     <h2>Summary</h2>
     <div class="section-content">
@@ -68,6 +71,12 @@
                 <td class="patch">{{diff.patch_timestamp}}</td>
                 <td class="diff"></td>
             </tr>
+            <tr>
+                <td class="item">Full Report</td>
+                <td class="base"><a href="base_pmd_report.html">Base PMD Report</a></td>
+                <td class="patch"><a href="patch_pmd_report.html">Patch PMD Report</a></td>
+                <td class="diff"></td>
+            </tr>
             </tbody>
         </table>
     </div>

data/resources/project_index.html

@@ -13,6 +13,12 @@
     <link rel="stylesheet" href="css/pmd-tester.css">
 </head>
 <body>
+<div class="section">
+    <h1>PMD Regression Tester Report</h1>
+    <p>
+        <a href="https://github.com/pmd/pmd-regression-tester">github.com/pmd/pmd-regression-tester</a>
+    </p>
+</div>
 <div class="section">
     <h2>Branch details</h2>
     <div class="section-content">
@@ -37,6 +43,11 @@
                         <span class="external-link-secondary"><a href="{{ comparison_url }}">[Compare]</a></span>
                     </td>
                 </tr>
+                <tr>
+                    <td class="item">Timestamp</td>
+                    <td class="base">{{base.timestamp}}</td>
+                    <td class="patch">{{patch.timestamp}}</td>
+                </tr>
                 <tr>
                     <td class="item">Total PMD runtime</td>
                     <td class="base">{{base.execution_time}}</td>