pmdtester 1.6.1 → 1.7.0

data/config/project-list-with-cpd.xml

@@ -0,0 +1,268 @@
+<?xml version="1.0"?>
+
+<projectlist xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+             xsi:noNamespaceSchemaLocation="projectlist_1_3_0.xsd">
+  <description>Standard Projects</description>
+
+  <project>
+    <name>checkstyle</name>
+    <type>git</type>
+    <connection>https://github.com/checkstyle/checkstyle</connection>
+    <tag>checkstyle-9.1</tag>
+
+    <exclude-pattern>.*/target/test-classes/com/puppycrawl/tools/checkstyle/.*</exclude-pattern>
+    <exclude-pattern>.*/target/generated-sources/.*</exclude-pattern>
+    <exclude-pattern>.*/src/test/resources-noncompilable/com/puppycrawl/tools/checkstyle/javaparser/InputJavaParserNoFreezeOnDeeplyNestedLambdas.java</exclude-pattern>
+
+    <build-command><![CDATA[#!/usr/bin/env bash
+if test -e classpath.txt; then
+  exit
+fi
+
+set -e
+
+# Make sure to use java11. This is already installed by build.yml/publish-snapshot.yml/publish-release.yml
+export JAVA_HOME=${HOME}/openjdk11
+export PATH=$JAVA_HOME/bin:$PATH
+
+mvn test-compile -B
+mvn dependency:build-classpath -DincludeScope=test -Dmdep.outputFile=classpath.txt -B
+]]></build-command>
+    <auxclasspath-command>echo -n "${HOME}/openjdk11/lib/jrt-fs.jar:$(pwd)/target/classes:$(pwd)/target/test-classes:"; cat classpath.txt</auxclasspath-command>
+    <cpd-options>
+      <language>java</language>
+      <minimum-tokens>50</minimum-tokens>
+      <max-memory>4g</max-memory>
+      <directories>
+        <!-- this avoids analyzing src/test/resources-noncompilable ... -->
+        <directory>src/main/java</directory>
+        <directory>src/test/java</directory>
+      </directories>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>spring-framework</name>
+    <type>git</type>
+    <connection>https://github.com/spring-projects/spring-framework</connection>
+    <tag>v5.3.13</tag>
+
+    <exclude-pattern>.*/build/generated-sources/.*</exclude-pattern>
+    <exclude-pattern>.*/build/resources/.*</exclude-pattern>
+
+    <build-command><![CDATA[#!/usr/bin/env bash
+## Skip gradle execution
+if test -e classpath.txt; then
+  exit
+fi
+
+set -e
+
+# Make sure to use java11. This is already installed by build.yml/publish-snapshot.yml/publish-release.yml
+export JAVA_HOME=${HOME}/openjdk11
+export PATH=$JAVA_HOME/bin:$PATH
+
+## Patches
+# keep the tabs!!
+# Patch 1: See https://github.com/spring-projects/spring-framework/commit/381b7d035a16d430b8783b7390c1677c9e7d1f68
+#          and https://github.com/spring-projects/spring-framework/commit/9e1ed6c7718d38c4b9fe5f75921abad33264307c
+(cat <<EOF
+diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java
+index 37f5884e67..53022443ee 100644
+--- a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java
++++ b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java
+@@ -539,7 +539,9 @@ public class AutowiredAnnotationBeanPostProcessor implements SmartInstantiationA
+ 	 */
+ 	@SuppressWarnings({"deprecation", "cast"})
+ 	protected boolean determineRequiredStatus(MergedAnnotation<?> ann) {
+-		return determineRequiredStatus(
++		// Cast to (AnnotationAttributes) is required. Otherwise, the :spring-beans:compileGroovy
++		// task fails in the Gradle build.
++		return determineRequiredStatus((AnnotationAttributes)
+ 				ann.asMap(mergedAnnotation -> new AnnotationAttributes(mergedAnnotation.getType())));
+ 	}
+ 
+EOF
+) | patch --strip=1
+
+# Patch 2: Ignore compiler warnings
+(cat <<EOF
+diff --git a/buildSrc/src/main/java/org/springframework/build/compile/CompilerConventionsPlugin.java b/buildSrc/src/main/java/org/springframework/build/compile/CompilerConventionsPlugin.java
+index f2424c549e..b6ec8b04da 100644
+--- a/buildSrc/src/main/java/org/springframework/build/compile/CompilerConventionsPlugin.java
++++ b/buildSrc/src/main/java/org/springframework/build/compile/CompilerConventionsPlugin.java
+@@ -51,7 +51,7 @@ public class CompilerConventionsPlugin implements Plugin<Project> {
+ 		COMPILER_ARGS.addAll(commonCompilerArgs);
+ 		COMPILER_ARGS.addAll(Arrays.asList(
+ 				"-Xlint:varargs", "-Xlint:fallthrough", "-Xlint:rawtypes", "-Xlint:deprecation",
+-				"-Xlint:unchecked", "-Werror"
++				"-Xlint:unchecked"//, "-Werror"
+ 		));
+ 		TEST_COMPILER_ARGS = new ArrayList<>();
+ 		TEST_COMPILER_ARGS.addAll(commonCompilerArgs);
+diff --git a/spring-beans/spring-beans.gradle b/spring-beans/spring-beans.gradle
+index e3f6f73b76..48c4d9e3fb 100644
+--- a/spring-beans/spring-beans.gradle
++++ b/spring-beans/spring-beans.gradle
+@@ -23,7 +23,7 @@ sourceSets {
+ }
+ 
+ compileGroovy {
+-	options.compilerArgs += "-Werror"
++//	options.compilerArgs += "-Werror"
+ }
+ 
+ // This module also builds Kotlin code and the compileKotlin task naturally depends on
+EOF
+) | patch --strip=1
+
+# Patch 3: Add task createSquishClasspath
+(cat <<EOF
+diff --git a/build.gradle b/build.gradle
+index 6021fa574d..15d29ed699 100644
+--- a/build.gradle
++++ b/build.gradle
+@@ -431,3 +431,19 @@ configure(rootProject) {
+ 		}
+ 	}
+ }
++
++// see https://stackoverflow.com/questions/28986968/generate-classpath-from-all-multiproject-gradle-build-dependencies
++task createSquishClasspath {
++  doLast {
++    def dependencies = new LinkedHashSet()
++    dependencies.addAll(moduleProjects.configurations.compileClasspath.resolvedConfiguration.resolvedArtifacts.file.flatten())
++    dependencies.addAll(moduleProjects.configurations.testCompileClasspath.resolvedConfiguration.resolvedArtifacts.file.flatten())
++
++    def paths = new ArrayList()
++    paths.addAll(moduleProjects.jar.outputs.files.asPath)
++    paths.addAll(moduleProjects.sourceSets.test.output.resourcesDir)
++    paths.addAll(moduleProjects.sourceSets.test.output.classesDirs.files.flatten())
++    paths.addAll(dependencies)
++    println paths.join(File.pathSeparator)
++  }
++}
+EOF
+) | patch --strip=1
+
+# Patch 4: Add https://maven.repository.redhat.com/ga/ as repository in order to resolve
+#   dependency com.ibm.websphere/uow/6.0.2.17
+#   See https://spring.io/blog/2020/10/29/notice-of-permissions-changes-to-repo-spring-io-fall-and-winter-2020
+(cat <<EOF
+diff --git a/build.gradle b/build.gradle
+index 6021fa57..8319ff76 100644
+--- a/build.gradle
++++ b/build.gradle
+@@ -291,6 +291,7 @@ configure(allprojects) { project ->
+ 		}
+ 		repositories {
+ 			mavenCentral()
++			maven { url "https://maven.repository.redhat.com/ga/" }
+ 			maven { url "https://repo.spring.io/libs-spring-framework-build" }
+ 		}
+ 	}
+EOF
+) | patch --strip=1
+
+./gradlew --console=plain --build-cache --no-daemon --max-workers=4 build testClasses -x test -x javadoc -x api -x asciidoctor -x asciidoctorPdf
+./gradlew --console=plain --build-cache --no-daemon --max-workers=4 createSquishClasspath -q > classpath.txt
+]]></build-command>
+    <auxclasspath-command>echo -n "${HOME}/openjdk11/lib/jrt-fs.jar:"; cat classpath.txt</auxclasspath-command>
+    <cpd-options>
+      <language>java</language>
+      <minimum-tokens>150</minimum-tokens>
+      <max-memory>4g</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>openjdk-11</name>
+    <type>git</type>
+    <connection>https://github.com/openjdk/jdk</connection>
+    <tag>jdk-11+28</tag>
+    <src-subpath>src/java.base</src-subpath>
+    <auxclasspath-command>echo -n "${HOME}/openjdk11/lib/jrt-fs.jar"</auxclasspath-command>
+    <cpd-options>
+      <language>java</language>
+      <minimum-tokens>150</minimum-tokens>
+      <max-memory>4g</max-memory>
+      <directories>
+        <directory>src/java.base</directory>
+      </directories>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>Schedul-o-matic-9000</name>
+    <type>git</type>
+    <connection>https://github.com/SalesforceLabs/Schedul-o-matic-9000</connection>
+    <tag>6b1229ba43b38931fbbab5924bc9b9611d19a786</tag>
+    <cpd-options>
+      <language>apex</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>512m</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>fflib-apex-common</name>
+    <type>git</type>
+    <connection>https://github.com/apex-enterprise-patterns/fflib-apex-common</connection>
+    <tag>7e0891efb86d23de62811af56d87d0959082a322</tag>
+    <cpd-options>
+      <language>apex</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>512m</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>apex-link</name>
+    <type>git</type>
+    <connection>https://github.com/nawforce/apex-link</connection>
+    <tag>v2.3.0</tag>
+    <src-subpath>samples</src-subpath>
+    <cpd-options>
+      <language>apex</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>512m</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>java-regression-tests</name>
+    <type>git</type>
+    <connection>https://github.com/pmd/java-regression-tests</connection>
+    <tag>main</tag>
+    <auxclasspath-command>realpath java-regression-tests-*.jar</auxclasspath-command>
+    <cpd-options>
+      <language>java</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>512m</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>OracleDBUtils</name>
+    <type>git</type>
+    <connection>https://github.com/Qualtagh/OracleDBUtils</connection>
+    <tag>0513fe6b053b31e6c09ac6f86eb2064733ecf32d</tag>
+    <cpd-options>
+      <language>plsql</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>512m</max-memory>
+    </cpd-options>
+  </project>
+
+  <project>
+    <name>alsa-firmware</name>
+    <type>git</type>
+    <connection>https://github.com/alsa-project/alsa-firmware</connection>
+    <tag>v1.2.1</tag>
+    <cpd-options>
+      <language>cpp</language>
+      <minimum-tokens>100</minimum-tokens>
+      <max-memory>4g</max-memory>
+    </cpd-options>
+  </project>
+</projectlist>

data/config/projectlist_1_2_0.xsd

@@ -1,5 +1,5 @@
 <?xml version="1.0" ?>
-<!-- version 1.1.0 -->
+<!-- version 1.2.0 -->
 <xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema">
   <xs:element name="projectlist">
     <xs:complexType>

data/config/projectlist_1_3_0.xsd

@@ -0,0 +1,53 @@
+<?xml version="1.0" ?>
+<!-- version 1.3.0 -->
+<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema">
+  <xs:element name="projectlist">
+    <xs:complexType>
+      <xs:sequence>
+        <xs:element name="description" type="xs:string" minOccurs="1" maxOccurs="1"/>
+        <xs:element name="project" type="project" minOccurs="1" maxOccurs="unbounded"/>
+      </xs:sequence>
+    </xs:complexType>
+  </xs:element>
+  <xs:complexType name="project">
+    <xs:sequence>
+      <xs:element name="name" type="xs:string" minOccurs="1" maxOccurs="1"/>
+      <xs:element name="type" minOccurs="1" maxOccurs="1">
+        <xs:simpleType>
+          <xs:restriction base="xs:string">
+            <xs:enumeration value="git"/>
+          </xs:restriction>
+        </xs:simpleType>
+      </xs:element>
+      <xs:element name="connection" type="xs:string" minOccurs="1" maxOccurs="1"/>
+      <xs:element name="webview-url" type="xs:string" minOccurs="0" maxOccurs="1"/>
+      <xs:element name="tag" type="xs:string" minOccurs="0" maxOccurs="1"/>
+      <xs:element name="src-subpath" type="xs:string" minOccurs="0" maxOccurs="1" default=".">
+        <xs:annotation>
+          <xs:documentation>
+            Value of the -dir option for the PMD run.
+            The value must be a directory path relative to the root directory of the clone.
+            Defaults to just '.', ie the entire directory.
+          </xs:documentation>
+        </xs:annotation>
+      </xs:element>
+      <xs:element name="exclude-pattern" type="xs:string" minOccurs="0" maxOccurs="unbounded"/>
+      <xs:element name="build-command" type="xs:string" minOccurs="0" maxOccurs="1"/>
+      <xs:element name="auxclasspath-command" type="xs:string" minOccurs="0" maxOccurs="1"/>
+      <xs:element name="cpd-options" type="cpd-options" minOccurs="0" maxOccurs="1"/>
+    </xs:sequence>
+  </xs:complexType>
+  <xs:complexType name="cpd-options">
+    <xs:sequence>
+      <xs:element name="language" type="xs:string" minOccurs="0" maxOccurs="1" default="java"/>
+      <xs:element name="minimum-tokens" type="xs:integer" minOccurs="0" maxOccurs="1" default="100"/>
+      <xs:element name="max-memory" type="xs:string" minOccurs="0" maxOccurs="1" default="512m"/>
+      <xs:element name="directories" type="directories" minOccurs="0" maxOccurs="1"/>
+    </xs:sequence>
+  </xs:complexType>
+  <xs:complexType name="directories">
+    <xs:sequence>
+      <xs:element name="directory" type="xs:string" minOccurs="1" maxOccurs="unbounded"/>
+    </xs:sequence>
+  </xs:complexType>
+</xs:schema>

data/lib/pmdtester/builders/cpd_project_hasher.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module PmdTester
+  # Turn a CPD project report into a hash that can be rendered somewhere else
+  module CpdProjectHasher
+    include PmdTester
+
+    def cpd_report_diff_to_h(cpd_rdiff)
+      {
+        'duplication_counts' => cpd_rdiff.duplication_counts.to_h.transform_keys(&:to_s),
+        'error_counts' => cpd_rdiff.error_counts.to_h.transform_keys(&:to_s),
+
+        'base_details' => cpd_rdiff.base_report.report_details.to_h,
+        'patch_details' => cpd_rdiff.patch_report.report_details.to_h,
+        'diff_execution_time' => PmdReportDetail.convert_seconds(cpd_rdiff.patch_report.report_details.execution_time -
+                                                                 cpd_rdiff.base_report.report_details.execution_time)
+      }
+    end
+
+    def duplications_to_hash(project, duplications, is_diff)
+      filename_index = {}
+      duplications.each do |d|
+        d.files.each do |f|
+          local_path = project.get_local_path(f.path)
+          filename_index[local_path] = filename_index.size unless filename_index.include?(local_path)
+        end
+      end
+
+      duplications_table = duplications.map do |d|
+        make_duplication_table(project, filename_index, d, is_diff)
+      end
+
+      {
+        'file_index' => filename_index.keys,
+        'duplications' => duplications_table
+      }
+    end
+
+    def cpd_errors_to_h(project)
+      errors = project.cpd_report_diff.error_diffs
+      errors.map { |e| error_to_hash(e, project) }
+    end
+
+    private
+
+    def make_duplication_table(project, filename_index, duplication, is_diff)
+      locations = duplication.files.map do |f|
+        [filename_index[project.get_local_path(f.path)], f.location.beginline, f.location.endline, f.location.to_s]
+      end
+      type = if !is_diff || duplication.added?
+               '+'
+             elsif duplication.changed?
+               '~'
+             else
+               '-'
+             end
+      [locations, duplication.lines, duplication.tokens, duplication.codefragment,
+       type] + generate_old_duplication_info(project, filename_index, duplication, is_diff)
+    end
+
+    def generate_old_duplication_info(project, filename_index, duplication, is_diff)
+      return [] unless is_diff && duplication.changed?
+
+      old_files = duplication.old_files.map do |f|
+        [filename_index[project.get_local_path(f.path)], f.location.beginline, f.location.endline, f.location.to_s]
+      end
+      [duplication.old_lines, duplication.old_tokens, old_files]
+    end
+  end
+end

data/lib/pmdtester/builders/liquid_renderer.rb

@@ -12,7 +12,7 @@ module PmdTester
     def render_liquid(template_path, env)
       to_render = File.read(ResourceLocator.resource(template_path))
       includes = Liquid::LocalFileSystem.new(ResourceLocator.resource('_includes'), '%s.html')
-      Liquid::Template.file_system = includes
+      Liquid::Environment.default.file_system = includes
       template = Liquid::Template.parse(to_render, error_mode: :strict)
       template.render!(env, { strict_variables: true })
     end
@@ -43,6 +43,7 @@ module PmdTester
   class LiquidProjectRenderer
     include PmdTester
     include ProjectHasher
+    include CpdProjectHasher
     include LiquidRenderer
 
     def write_project_index(project, root)
@@ -50,28 +51,65 @@ module PmdTester
         'diff' => report_diff_to_h(project.report_diff),
         'error_diffs' => errors_to_h(project),
         'configerror_diffs' => configerrors_to_h(project),
+        'cpd_diff' => cpd_report_diff_to_h(project.cpd_report_diff),
+        'cpd_error_diffs' => cpd_errors_to_h(project),
         'project_name' => project.name
       }
 
       # Renders index.html using liquid
       write_file("#{root}/index.html", render_liquid('project_diff_report.html', liquid_env))
+      write_pmd_diff_report(project, root)
+      write_pmd_full_report(project, root)
+      write_cpd_diff_report(project, root)
+      write_cpd_full_report(project, root)
+    end
+
+    private
+
+    def write_pmd_diff_report(project, root)
       # generate array of violations in json
-      write_file("#{root}/project_data.js", dump_violations_json(project))
+      write_file("#{root}/diff_pmd_data.js", dump_violations_json(project))
       # copy original pmd reports
       copy_file("#{root}/base_pmd_report.xml", project.report_diff.base_report.file)
       copy_file("#{root}/patch_pmd_report.xml", project.report_diff.patch_report.file)
-      # copy stdout and stderr outputs
-      copy_file("#{root}/base_stdout.txt", "#{project.report_diff.base_report.report_folder}/stdout.txt")
-      copy_file("#{root}/base_stderr.txt", "#{project.report_diff.base_report.report_folder}/stderr.txt")
-      copy_file("#{root}/patch_stdout.txt", "#{project.report_diff.patch_report.report_folder}/stdout.txt")
-      copy_file("#{root}/patch_stderr.txt", "#{project.report_diff.patch_report.report_folder}/stderr.txt")
+      write_pmd_stdout_stderr(root, project.report_diff)
+      copy_file("#{root}/base_pmd_recording.jfr",
+                project.report_diff.base_report.report_details.jfr_summary.recording_path)
+      copy_file("#{root}/patch_pmd_recording.jfr",
+                project.report_diff.patch_report.report_details.jfr_summary.recording_path)
+    end
+
+    def write_pmd_full_report(project, root)
       # render full pmd reports
       write_file("#{root}/base_pmd_report.html",
                  render_liquid('project_pmd_report.html', pmd_report_liquid_env(project, BASE)))
-      write_file("#{root}/base_data.js", dump_violations_json(project, BASE))
+      write_file("#{root}/base_pmd_data.js", dump_violations_json(project, BASE))
       write_file("#{root}/patch_pmd_report.html",
                  render_liquid('project_pmd_report.html', pmd_report_liquid_env(project, PATCH)))
-      write_file("#{root}/patch_data.js", dump_violations_json(project, PATCH))
+      write_file("#{root}/patch_pmd_data.js", dump_violations_json(project, PATCH))
+    end
+
+    def write_cpd_diff_report(project, root)
+      # generate array of cpd duplications in json
+      write_file("#{root}/diff_cpd_data.js", dump_cpd_duplications_json(project, 'diff'))
+      # copy original cpd reports
+      copy_file("#{root}/base_cpd_report.xml", project.cpd_report_diff.base_report.file)
+      copy_file("#{root}/patch_cpd_report.xml", project.cpd_report_diff.patch_report.file)
+      write_cpd_stdout_stderr(root, project.cpd_report_diff)
+      copy_file("#{root}/base_cpd_recording.jfr",
+                project.cpd_report_diff.base_report.report_details.jfr_summary.recording_path)
+      copy_file("#{root}/patch_cpd_recording.jfr",
+                project.cpd_report_diff.patch_report.report_details.jfr_summary.recording_path)
+    end
+
+    def write_cpd_full_report(project, root)
+      # render full cpd reports
+      write_file("#{root}/base_cpd_report.html",
+                 render_liquid('project_cpd_report.html', cpd_report_liquid_env(project, BASE)))
+      write_file("#{root}/base_cpd_data.js", dump_cpd_duplications_json(project, BASE))
+      write_file("#{root}/patch_cpd_report.html",
+                 render_liquid('project_cpd_report.html', cpd_report_liquid_env(project, PATCH)))
+      write_file("#{root}/patch_cpd_data.js", dump_cpd_duplications_json(project, PATCH))
     end
 
     def dump_violations_json(project, branch = 'diff')
@@ -89,18 +127,47 @@ module PmdTester
         **violations_to_hash(project, violations_by_file, branch == 'diff')
       }
 
-      project_data = JSON.fast_generate(h, indent: '    ', object_nl: "\n", array_nl: "\n")
-      "let project = #{project_data}"
+      project_data = JSON.generate(h, object_nl: "\n")
+      "let pmd_report = #{project_data}"
     end
 
-    private
+    def dump_cpd_duplications_json(project, branch)
+      duplications = if branch == BASE
+                       project.cpd_report_diff.base_report.duplications
+                     elsif branch == PATCH
+                       project.cpd_report_diff.patch_report.duplications
+                     else
+                       project.cpd_report_diff.duplication_diffs
+                     end
+      h = {
+        'source_link_base' => project.webview_url,
+        'source_link_template' => link_template(project),
+        **duplications_to_hash(project, duplications, branch == 'diff')
+      }
+
+      "let cpd_report = #{JSON.generate(h, object_nl: "\n")}"
+    end
+
+    def write_pmd_stdout_stderr(root, report_diff)
+      write_file("#{root}/base_pmd_stdout.txt", report_diff.base_report.report_details.stdout)
+      write_file("#{root}/base_pmd_stderr.txt", report_diff.base_report.report_details.stderr)
+      write_file("#{root}/patch_pmd_stdout.txt", report_diff.patch_report.report_details.stdout)
+      write_file("#{root}/patch_pmd_stderr.txt", report_diff.patch_report.report_details.stderr)
+    end
+
+    def write_cpd_stdout_stderr(root, cpd_report_diff)
+      write_file("#{root}/base_cpd_stdout.txt", cpd_report_diff.base_report.report_details.stdout)
+      write_file("#{root}/base_cpd_stderr.txt", cpd_report_diff.base_report.report_details.stderr)
+      write_file("#{root}/patch_cpd_stdout.txt", cpd_report_diff.patch_report.report_details.stdout)
+      write_file("#{root}/patch_cpd_stderr.txt", cpd_report_diff.patch_report.report_details.stderr)
+    end
 
     def copy_file(target_file, source_file)
       if File.exist? source_file
         FileUtils.cp(source_file, target_file)
         logger&.info "Written #{target_file}"
       else
-        logger&.warn "File #{source_file} not found"
+        logger&.warn "Cannot write #{target_file}. Source file #{source_file} not found"
       end
     end
 
@@ -117,20 +184,48 @@ module PmdTester
       }
     end
 
+    def cpd_report_liquid_env(project, branch)
+      report = if branch == BASE
+                 project.cpd_report_diff.base_report
+               else
+                 project.cpd_report_diff.patch_report
+               end
+      {
+        'project_name' => project.name,
+        'branch' => branch,
+        'cpd_report' => cpd_report_to_h(project, report)
+      }
+    end
+
     def report_to_h(project, report)
       {
         'violation_counts' => report.violations_by_file.total_size,
         'error_counts' => report.errors_by_file.total_size,
         'configerror_counts' => report.configerrors_by_rule.values.flatten.length,
 
-        'execution_time' => PmdReportDetail.convert_seconds(report.exec_time),
-        'timestamp' => report.timestamp,
-        'exit_code' => report.exit_code,
+        'execution_time' => report.report_details.execution_time_formatted,
+        'timestamp' => report.report_details.timestamp,
+        'exit_code' => report.report_details.exit_code,
+        'jfr_summary' => report.report_details.jfr_summary.to_h_for_liquid,
 
         'rules' => report.rule_summaries,
         'errors' => report.errors_by_file.all_values.map { |e| error_to_hash(e, project) },
         'configerrors' => report.configerrors_by_rule.values.flatten.map { |e| configerror_to_hash(e) }
       }
     end
+
+    def cpd_report_to_h(project, cpd_report)
+      {
+        'duplication_counts' => cpd_report.duplications.length,
+        'error_counts' => cpd_report.errors.length,
+
+        'execution_time' => cpd_report.report_details.execution_time_formatted,
+        'timestamp' => cpd_report.report_details.timestamp,
+        'exit_code' => cpd_report.report_details.exit_code,
+        'jfr_summary' => cpd_report.report_details.jfr_summary.to_h_for_liquid,
+
+        'errors' => cpd_report.errors.map { |e| error_to_hash(e, project) }
+      }
+    end
   end
 end