Hi guys,

There is a new change that includes the "jacoco-maven-plugin" in all our
modules. The goal is to help us to have a good test code coverage. We will
do it by breaking the build in case it doesn't reach the minimum coverage
ratio.

Now, the build creates a "target/site/jacoco/index.html" file in each
module. This file shows the coverage ratio for that particular module. I
need hands to help me to make it ">85%" in our main modules.

How to contribute?

* Checkout the source code [http://tomee.apache.org/dev/source-code.html].
* Choose what you want to ensure test coverage.
* Create a JIRA issue stating what you are working on - Just trying to
avoid duplicated work.
* Run "mvn clean install", update the code and create unit tests as needed.
* Repeat last step until you are satisfied with the coverage you managed to
get.
* Add or update the following property in that module:

  <properties>
    <!-- Put the coverage shown in the target/site/jacoco/index.html file.
 -->

<test.complexity.coverage.minimun>0.68</test.complexity.coverage.minimun>
    <test.branch.coverage.minimun>0.4</test.branch.coverage.minimun>
  </properties>

* run "mvn clean install" again to ensure that "jacoco-maven-plugin" does
not complain.
* Submit a patch.

Note that you can create a JIRA task to cover only part of a module and not
the entire thing. For Example, a nice JIRA task would be something like
this https://issues.apache.org/jira/browse/TOMEE-1231.

Sample report: http://www.eclemma.org/jacoco/trunk/coverage/

Committers, any concern about it? I know that a few of our unit tests won't
work due to obscure instrumentation issues. That's why some of the modules
aren't covered. The most important modules (container, server and tomee)
aren't affected by this problem.

[]s,
Thiago.

Reply via email to