Make compatible with Jenkins Workflow/Pipeline plugin for report publishing about cobertura-plugin HOT 80 CLOSED

Any update on this? Seems like a great shame that this isn't getting implemented. Seems like #55 is mostly there, but has some issues right now. Anyone who is a contributor on this project able to give an update regarding whether this will be implemented soon?

from cobertura-plugin.

I've been waiting for this support for a while now, unfortunately I do not know enough Java to do it myself or I would.

Are there any plans to implement this? Besides eventually of course.

from cobertura-plugin.

Any news?

from cobertura-plugin.

I also cannot get the Coverage metrics target generated by the snippet generator. Anyone knows how to code my pipeline to include it?

from cobertura-plugin.

Fixed in version 1.10 which has been uploaded to the repo. Waiting until it shows up in update center before closing this issue.

from cobertura-plugin.

@mbarrien mind to let us know what the state of this ticket is?

from cobertura-plugin.

Pull Request #55 appears to be to add this feature request.

from cobertura-plugin.

+1 for 2017!!!!

from cobertura-plugin.

I can see that the plugin has now been published. Does anyone know the pipeline syntax for using it?

Other jenkins plugins support the pipeline syntax generator but I can't see cobertura listed under sample steps.

from cobertura-plugin.

Sorry, I've not got around to working on it recently; I would need to find the time to do some more digging to understand the refactoring needed.

So no, it's not likely to be soon, sorry. Feel free to work on it if you find time to do so before I do!

On 14 March 2016 13:29:06 GMT+00:00, Stijn Van Nieuwenhuyse [email protected] wrote:

@leth, any chance to get your changes merged in the near feature?

---

Reply to this email directly or view it on GitHub:
#50 (comment)

Sent from my phone. Please excuse my brevity.

from cobertura-plugin.

@SuperBarracuda: If you select the step: General Build Step sample step, you'll see an option to Publish Cobertura Coverage Report.

I haven't used it yet, but seems like it is all there.

from cobertura-plugin.

I have a line like this:

   step([$class: 'CoberturaPublisher', coberturaReportFile: 'coverage.xml'])

(My coverage output is in coverage.xml). Works like a charm again.

from cobertura-plugin.

Would it make sense to add @ Symbol("cobertura") or a different one so it's easier to use/find?

from cobertura-plugin.

I'm working a changeset for this, but I could do with some advice from someone more familiar with how Jenkins plugins work, as I'm mostly guessing and copying from the junit and xunit plugins.
I'll post a PR with more info in the next couple of days.

from cobertura-plugin.

These are the areas I could use some advice on: https://github.com/leth/cobertura-plugin/commit/52a2ce9b2ec9dde01ff9054e1184c2e833e00a60

from cobertura-plugin.

@leth, any chance to get your changes merged in the near feature?

from cobertura-plugin.

Keep us updated @leth

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

I think what's needed is to implement the SimpleBuildStep interface. similar to what the xunit plugin does: https://github.com/jenkinsci/xunit-plugin/blob/master/src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java#L63

from cobertura-plugin.

I brought this issue up here https://groups.google.com/forum/#!msg/jenkinsci-users/UnAK_1YASa0/0TPbYN7HBQAJ

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

Can we have a release (and docs) for this?

from cobertura-plugin.

I don't think it's been fully implemented yet.

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

This is a feature killer, is there a way we can help here? I see there's currently 6 commits referencing this. Anyone with experience with Jenkins pipeline and cobertura-plugin could provide us with details on what's left to do and or what could be done to finish this faster?

I'm not very familiar with Java nor jenkins plugin development, but I really want this feature. If I can do something to help here, I'd be very happy to contribute.

Oh I'll start by testing #55 🍸
Edit: while trying to setup the plugin, I found a few other jenkinfiles on github and they seem to use this instead:
https://github.com/cwacek/cobertura-clover-transform

// install cobertura -> clover transform (until cobertura is supported in pipelines)
// requires python-pip libxslt-devel
sh """
  pip install cobertura-clover-transform"  // needs lxml
  cobertura-clover-transform coverage.xml > clover.xml
"""

step([$class: 'CloverPublisher', cloverReportDir: 'reports', cloverReportFileName: 'clover.xml'])

But I'm personally getting the following stacktrace with this:

java.lang.NullPointerException
	at hudson.plugins.clover.CloverPublisher.processCloverXml(CloverPublisher.java:250)
	at hudson.plugins.clover.CloverPublisher.performImpl(CloverPublisher.java:202)
	at hudson.plugins.clover.CloverPublisher.perform(CloverPublisher.java:167)
	at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:69)
	at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:59)
	at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1$1.call(AbstractSynchronousNonBlockingStepExecution.java:52)
	at hudson.security.ACL.impersonate(ACL.java:221)
	at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1.run(AbstractSynchronousNonBlockingStepExecution.java:49)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
	at java.util.concurrent.FutureTask.run(FutureTask.java:262)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	at java.lang.Thread.run(Thread.java:745)

and Clover is actually 300$ for a single machine

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

Any news on this issue?

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

👍

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1 +10 +100 +1000 +10000

from cobertura-plugin.

Any news on it ?

from cobertura-plugin.

See progress in #55 Someone just requested a merge on master.

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

+1

from cobertura-plugin.

@reinout @lsglick Thanks I have it working now. However the snippet doesn't seem to generate " Coverage Metric Targets" for lines. Do you know how this would be specified?

from cobertura-plugin.

@lsglick I'm not seeing Step: General Build Step in my snippet generator. Is there possibly another plugin I'm missing? Using the snippet that @reinout mentioned works, though.

from cobertura-plugin.

The default output from the snippet generator I get is:
step([$class: 'CoberturaPublisher',
autoUpdateHealth: false,
autoUpdateStability: false,
coberturaReportFile: 'coverage.xml',
failUnhealthy: false,
failUnstable: false,
maxNumberOfBuilds: 0,
onlyStable: false,
sourceEncoding: 'ASCII',
zoomCoverageChart: false])

from cobertura-plugin.

the "coverage metrics target" cannot be generated, need this to fail the build if the coverage is below certain threshold

from cobertura-plugin.

Released last week. Please file new issues for any bug found in the pipeline code.

from cobertura-plugin.

@lsglick is this the pipeline groovy plugin you are using?

https://cloud.githubusercontent.com/assets/5394693/25432882/48cc30f8-2a54-11e7-9efb-47e30cd6751b.png

from cobertura-plugin.

+1 for Coverage metrics target from pipeline step

from cobertura-plugin.