getappmap / getappmap/appmap-java

Coverage data is generated when the agent is tested

Open
#262 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Java
Stars
88
Forks
17
Avg merge
2d 2h
Merged PRs (30d)
1

Description

It seems like we might be able to generate some coverage with JaCoCo, but only for the tests run by gradle (i.e. with ./gradlew check integrationTest). This is only a small subset of our tests, of course -- nothing run by bats will generate any coverage.

We may be able to instrument the agent jar using OpenClover, and then configure it when bats runs.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by running the mentioned Gradle checks and integration tests, then inspect how the agent jar is exercised by bats. Compare JaCoCo coverage from ./gradlew check integrationTest with the proposed OpenClover instrumentation. Done means coverage is generated for the intended test paths, including bats, without breaking the existing test commands.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, testing
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.