[JENKINS-26276] Implemented JUnitStep ("archiveTestResults")

[akbertram]

Here is a first attempt at a workflow step that returns test summary results to the caller.
Can be used as follows:

def tests = archiveTestResults()
if(tests.failCount > 0) {
  // take action!
}

Feedback welcome!

[jenkinsadmin]

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

[jglick]

The problem with doing this in junit is it means we cannot then bundle an update to this plugin in jenkins.war unless workflow-step-api is also bundled, which seems a big step. On the other hand the junit plugin is the most logical place for it. A compromise could be a new plugin inside the workflow-plugin repo depending on junit.

[jglick]

Delete this. If location is defined in a @DataBoundConstructor, it should be final and there should be no @DataBoundSetter.

[akbertram]

Thanks for the feedback! Will try to submit fixes by the end of next week.

[omehegan]

Any plans to continue work on this?

[miguelaferreira]

+1 on this feature

[oleg-nenashev]

@jglick Are you fine with the current impl? If yes, I'll try to integrate it

[olivergondza]

The problem with doing this in junit is it means we cannot then bundle an update to this plugin in jenkins.war unless workflow-step-api is also bundled, which seems a big step.

How about make it an optional dependency? I would vote for that even if junit were not bundled...

[olivergondza]

Also, the feature is untested.

[abayer]

+1 for resurrecting this - it'd be a handy feature.

[oleg-nenashev]

@abayer Are you working on it in #47?

[ziru]

Any update on this PR? No go?

[oleg-nenashev]

@ziru IIRC @abayer tried to implement something in #47, then @jglick -1'ed the approach. I suppose there is not much progress now

[abayer]

Closing - I added a junit step (replacing the junit @Symbol) in #76