Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional report-integration and prepare-agent-integration goals for integration tests. #152

Merged
merged 13 commits into from
Nov 17, 2013
Merged

Additional report-integration and prepare-agent-integration goals for integration tests. #152

merged 13 commits into from
Nov 17, 2013

Conversation

mfriedenhagen
Copy link
Member

This is the second take of #105 as discussed in https://groups.google.com/d/msg/jacoco/I6Z43tfzoQc/DuSb_oZNYxYJ:

  • To differentiate between the coverage of unit tests and integration tests, define additional goals prepare-agent-integration and report-integration. It is crucial to use maven-site-plugin version 3.3+
  • See jacoco-maven-plugin.test/it/it-site-failsafe/pom.xml for an example how to use this.

@ghost ghost assigned mfriedenhagen Nov 17, 2013
mfriedenhagen added a commit that referenced this pull request Nov 17, 2013
Additional report-integration and prepare-agent-integration goals for integration tests.
@mfriedenhagen mfriedenhagen merged commit 5754253 into jacoco:master Nov 17, 2013
@mfriedenhagen mfriedenhagen deleted the multiple-maven-reports-new branch November 17, 2013 21:19
@marchof
Copy link
Member

marchof commented Nov 18, 2013

@mfriedenhagen The new goals should be linked in documentation ./org.jacoco.doc/docroot/doc/maven.html Also I wonder whether we should add a section here about unit testing vs. integration testing?

@jacoco jacoco locked and limited conversation to collaborators Jan 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants