Skip to content

Aggregation does not generate scoverage.xml #45

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

Closed
Quizzie opened this issue Jun 24, 2015 · 6 comments
Closed

Aggregation does not generate scoverage.xml #45

Quizzie opened this issue Jun 24, 2015 · 6 comments

Comments

@Quizzie
Copy link

Quizzie commented Jun 24, 2015

The aggregateScoverage task generates a cobertura.xml file and the HTML files, but not the scoverage.xml file in the build/scoverage-aggregate directory.

It should be fixed by adding a ScoverageXmlWriter to the AggregateReportApp.java file.

@Kwestor
Copy link
Contributor

Kwestor commented Jun 24, 2015

There is a PR with a lot of fixes to aggregate task: #42, but it stuck.

@sksamuel
Copy link
Member

I've ping the author of the plugin.

On 24 June 2015 at 14:14, Jerzy Müller [email protected] wrote:

There is a PR with a lot of fixes to aggregate task:
https://github.com/scoverage/gradle-scoverage/pull/42/commits, but it
stuck.


Reply to this email directly or view it on GitHub
#45 (comment)
.

@maiflai
Copy link
Contributor

maiflai commented Jun 25, 2015

Sorry for the delay, I'll take a look this weekend.

@maiflai
Copy link
Contributor

maiflai commented Jul 5, 2015

I've released 1.0.9 which should publish the scoverage file by default - please could you check?

Thanks,
Stu

@Quizzie
Copy link
Author

Quizzie commented Jul 10, 2015

Yes, it works. Thank you.

You might want to add this note to the readme:
For checking coverage of the aggregated result, configure the checkScoverage task:

 checkScoverage {
     reportDir = file("$buildDir/scoverage-aggregate/")
 }

@Quizzie Quizzie closed this as completed Jul 10, 2015
@maiflai
Copy link
Contributor

maiflai commented Jul 12, 2015

I've updated the docs, thanks

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

No branches or pull requests

4 participants