Skip to content

Add JaCoCo test coverage reporting#14

Merged
brunoborges merged 3 commits intomainfrom
copilot/include-jacoco-for-test-coverage
Feb 5, 2026
Merged

Add JaCoCo test coverage reporting#14
brunoborges merged 3 commits intomainfrom
copilot/include-jacoco-for-test-coverage

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

Integrates JaCoCo 0.8.12 for test coverage analysis.

Configuration

  • Build plugin: Instruments bytecode during test execution, generates HTML/XML reports in target/site/jacoco-coverage/
  • Reporting plugin: Integrates coverage reports into Maven site generation
  • Scope: All com.github.copilot.sdk classes, excluding test infrastructure (E2ETestContext, CapiProxy)
  • Execution data: target/jacoco-test-results/sdk-tests.exec

Usage

mvn verify                    # Generate coverage reports
mvn verify site              # Include in Maven site

Reports accessible at target/site/jacoco-coverage/index.html

Current Metrics

  • 65% instruction coverage (5,347/8,116)
  • 57% branch coverage (219/382)
  • 158 classes analyzed

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/copilot_internal/user
    • Triggering command: /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot --server --log-level info --stdio --auth-token-env COPILOT_SDK_AUTH_TOKEN --no-auto-login (http block)
    • Triggering command: /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot --server --log-level info --stdio --auth-token-env COPILOT_SDK_AUTH_TOKEN --no-auto-login ame (http block)
    • Triggering command: /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot /home/REDACTED/work/copilot-sdk-java/copilot-sdk-java/target/copilot-sdk/test/harness/node_modules/@github/copilot-linux-x64/copilot --server --log-level info --stdio --auth-token-env COPILOT_SDK_AUTH_TOKEN --no-auto-login user.email (http block)
  • opensource.org
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.12/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.12/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.12 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.12/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/copilot-sdk-java/copilot-sdk-java org.codehaus.plexus.classworlds.launcher.Launcher site ava/target/copilot-sdk/test/harness/node_modules/.bin/sh bash --no�� --noprofile git /usr/sbin/which copilot/include-which (dns block)
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.12/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.12/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.12 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.12/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/copilot-sdk-java/copilot-sdk-java org.codehaus.plexus.classworlds.launcher.Launcher site:site rgo/bin/git which k/co�� (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Include JaCoCo for test coverage</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 5, 2026 21:03
Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>
Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>
@brunoborges brunoborges marked this pull request as ready for review February 5, 2026 21:11
Copilot AI changed the title [WIP] Add JaCoCo for test coverage Add JaCoCo test coverage reporting Feb 5, 2026
Copilot AI requested a review from brunoborges February 5, 2026 21:12
@brunoborges brunoborges merged commit 4d65f00 into main Feb 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Include JaCoCo for test coverage

2 participants