You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@streams.apache.org by Stanton Sievers <si...@gmail.com> on 2014/02/25 18:08:00 UTC

Review Request 18473: Enabling site building and cobertura reporting

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18473/
-----------------------------------------------------------

Review request for streams.


Repository: streams


Description
-------

Having cobertura reports are handy for a project to allow developers to focus unit testing efforts.

As part of this, I updated the poms to allow for running "mvn site".  As part of site, cobertura reports will be generated.  I've also configured cobertura so it can be executed via "mvn cobertura:cobertura" which will be useful for the builds.apache.org environment.

The changes to wrappers/pom.xml were necessary otherwise the maven site plugin threw errors when generating anchors for those elements.

WARNING: Running mvn site for this project with maven 3.1.x will fail.  Only Maven 3.0.x can be used.  If we wish to support 3.1.x we'll need to update several plugin versions.


Diffs
-----

  http://svn.apache.org/repos/asf/incubator/streams/trunk/pom.xml 1571357 
  http://svn.apache.org/repos/asf/incubator/streams/trunk/poms/wrappers/pom.xml 1571357 

Diff: https://reviews.apache.org/r/18473/diff/


Testing
-------

Ran "mvn site" locally and verified that cobertura reports were present.

Ran "mvn clean verify" locally and verified that cobertura reports were not present.

Ran "mvn clean install cobertura:cobertura" on my personal Jenkins system and verified cobertura reports were present in a format that Jenkins could report.


Thanks,

Stanton Sievers