ppkarwasz opened a new pull request, #82: URL: https://github.com/apache/commons-secure-xml/pull/82
Three related fixes to how the build reports on its own tests. **The site's Surefire report counted zero tests.** The eight surefire executions write into `target/surefire-reports/<flavor>`, while the report plugin scans `target/surefire-reports` itself. It also used the forking `report` goal, and in the forked lifecycle the first execution is configured with a mixture of the other executions' settings (Xerces added, Saxon excluded, failures ignored), so `mvn test site` ran a ninth, misconfigured test round that overwrote the real Saxon reports. Now `report-only` over the eight directories; `failsafe-report-only` keeps its own default. **The Android instrumented run never reached JaCoCo.** The Gradle build now enables the Android Gradle plugin's androidTest coverage, pinned to the JaCoCo version the Maven build reports with, and the `jacoco` profile merges the resulting execution data into `target/jacoco.exec` before the site report. The merge sits in the site lifecycle only, so `jacoco:check` keeps measuring the JVM run alone and the minimums mean the same thing on every machine. **The coverage minimums were unreachable.** `mvn -Pjacoco clean verify` violated five of the six rules on every JDK, because much of the library only runs on Java 8 or only on a device. They are now a point below what the weaker of a JDK 8 and a JDK 21 run reaches on its own. Verified: - `mvn -Pjacoco clean verify` passes the coverage check on JDK 8, 11, 17, 21 and 25. - `mvn clean test site` runs each surefire execution exactly once; the report shows 1159 tests and the failsafe page its own 2. - The documented round trip (JAR on JDK 17, `connectedDebugAndroidTest`, JDK 8 pass, JDK 21 pass) clears every rule: instructions 97.99%, branches 96.74%, lines 97.94%, complexity 98.51%, methods 99.74%, classes 100%. - The Android suite is unchanged by the instrumentation: 259 tests, 0 failures. Build and documentation only, no main code changes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
