From 1ebbabb7e007b0fd9c0d1f44267a4e12b2104ac4 Mon Sep 17 00:00:00 2001
From: Svante Schubert <Svante.Schubert@gmail.com>
Date: Tue, 29 Sep 2026 13:59:17 +0200
Subject: [PATCH] Build: fixes #447: JaCoCo instrumentation exceptions on newer
 JDKs

Update JaCoCo to 0.8.15 and exclude JDK classes (java.*, javax.*, jdk.*, sun.*, com.sun.*) from the JaCoCo agent, so unsupported JDK class file versions no longer cause IllegalClassFormatException stack traces in the build log.
---
 odfdom/pom.xml |  2 +-
 pom.xml        | 11 ++++++++++-
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/odfdom/pom.xml b/odfdom/pom.xml
index 6319dab55..753dbdb0f 100644
--- a/odfdom/pom.xml
+++ b/odfdom/pom.xml
@@ -195,7 +195,7 @@
                 <artifactId>maven-surefire-plugin</artifactId>
                 <configuration>
                     <useModulePath>false</useModulePath>
-                    <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/coverage-reports/jacoco-ut.exec</argLine>
+                    <argLine>-javaagent:${settings.localRepository}/org/jacoco/org.jacoco.agent/${jacoco.version}/org.jacoco.agent-${jacoco.version}-runtime.jar=destfile=${project.build.directory}/coverage-reports/jacoco-ut.exec,excludes=${jacoco.agent.excludes}</argLine>
                     <systemPropertyVariables>
                         <org.odftoolkit.odfdom.validation>true</org.odftoolkit.odfdom.validation>
                     </systemPropertyVariables>
diff --git a/pom.xml b/pom.xml
index 6c79ba976..e405f2278 100644
--- a/pom.xml
+++ b/pom.xml
@@ -58,7 +58,10 @@
         <maven.javadoc.failOnError>false</maven.javadoc.failOnError>
         <maven.build.timestamp.format>yyyy-MM-dd'T'HH:mm:ss</maven.build.timestamp.format>
         <!-- JaCoCo Properties -->
-        <jacoco.version>0.8.14</jacoco.version>
+        <jacoco.version>0.8.15</jacoco.version>
+        <!-- JDK classes are never instrumented by the JaCoCo agent, as newer JDKs might use class file
+             versions unknown to JaCoCo resulting into exceptions in the build log -->
+        <jacoco.agent.excludes>java.*:javax.*:jdk.*:sun.*:com.sun.*</jacoco.agent.excludes>
         <sonar.java.coveragePlugin>jacoco</sonar.java.coveragePlugin>
         <sonar.dynamicAnalysis>reuseReports</sonar.dynamicAnalysis>
         <sonar.coverage.jacoco.xmlReportPaths>${project.reporting.outputDirectory}/jacoco-ut/jacoco.xml</sonar.coverage.jacoco.xmlReportPaths>
@@ -494,6 +497,9 @@
                             <configuration>
                                 <!-- Sets the path to the file which contains the execution data. -->
                                 <destFile>${project.build.directory}/coverage-reports/jacoco-ut.exec</destFile>
+                                <excludes>
+                                    <exclude>${jacoco.agent.excludes}</exclude>
+                                </excludes>
                                 <!--
                                     Sets the name of the property containing the settings
                                     for JaCoCo runtime agent.
@@ -535,6 +541,9 @@
                             <configuration>
                                 <!-- Sets the path to the file which contains the execution data. -->
                                 <destFile>${project.build.directory}/coverage-reports/jacoco-it.exec</destFile>
+                                <excludes>
+                                    <exclude>${jacoco.agent.excludes}</exclude>
+                                </excludes>
                                 <!--
                                     Sets the name of the property containing the settings
                                     for JaCoCo runtime agent.