Log Message
JENKINS-27593: Do not try to export test results for canceled or broken builds
Task-Url: https://issues.jenkins-ci.org/browse/JENKINS-27593
Modified Paths
Diff
Modified: trunk/hudson/plugins/klaros-testmanagement/src/main/java/hudson/plugins/klaros/KlarosTestResultPublisher.java (41268 => 41269)
--- trunk/hudson/plugins/klaros-testmanagement/src/main/java/hudson/plugins/klaros/KlarosTestResultPublisher.java 2015-03-25 15:53:00 UTC (rev 41268)
+++ trunk/hudson/plugins/klaros-testmanagement/src/main/java/hudson/plugins/klaros/KlarosTestResultPublisher.java 2015-03-25 16:09:20 UTC (rev 41269)
@@ -360,47 +360,59 @@
final BuildListener listener) {
boolean result = false;
+ if (Result.SUCCESS.equals(build.getResult())
+ || Result.UNSTABLE.equals(build.getResult())) {
- FilePath ws = build.getWorkspace();
- if (ws == null) {
- listener.error("No workspace defined!");
- build.setResult(Result.FAILURE);
- result = false;
- } else {
- for (ResultSet resultSet : getResultSets()) {
- if (StringUtils.isEmpty(resultSet.getSpec())) {
- listener.getLogger().println(
- "Empty result spec implementation detected");
- } else {
- listener.getLogger().println(
- "The test result(s) contained in target "
- + resultSet.getSpec() + " will be exported to the "
- + "Klaros-Testmanagement Server at " + getUrl(url)
- + ".");
- listener.getLogger().println(
- "With parameters Project[" + config + "], Environment[" + env
- + "], SUT[" + sut + "] and Type[" + type + "].");
+ FilePath ws = build.getWorkspace();
+ if (ws == null) {
+ listener.error("No workspace defined!");
+ build.setResult(Result.FAILURE);
+ result = false;
+ } else {
+ for (ResultSet resultSet : getResultSets()) {
+ if (StringUtils.isEmpty(resultSet.getSpec())) {
+ listener.getLogger().println(
+ "Empty result spec implementation detected");
+ } else {
+ listener.getLogger().println(
+ "The test result(s) contained in target "
+ + resultSet.getSpec()
+ + " will be exported to the "
+ + "Klaros-Testmanagement Server at "
+ + getUrl(url) + ".");
+ listener.getLogger().println(
+ "With parameters Project[" + config + "], Environment["
+ + env + "], SUT[" + sut + "] and Type[" + type
+ + "].");
- try {
- FileCallableImplementation exporter = new FileCallableImplementation(
- listener);
- exporter.setKlarosUrl(getKlarosUrl(url));
- exporter.setResultSet(resultSet);
- ws.act(exporter);
+ try {
+ FileCallableImplementation exporter = new FileCallableImplementation(
+ listener);
+ exporter.setKlarosUrl(getKlarosUrl(url));
+ exporter.setResultSet(resultSet);
+ ws.act(exporter);
- } catch (IOException e) {
- listener.getLogger().println("Failure to export test result(s).");
- e.printStackTrace(listener.getLogger());
- } catch (InterruptedException e) {
- listener.getLogger().println("Failure to export test result(s).");
- e.printStackTrace(listener.getLogger());
- }
+ } catch (IOException e) {
+ listener.getLogger().println(
+ "Failure to export test result(s).");
+ e.printStackTrace(listener.getLogger());
+ } catch (InterruptedException e) {
+ listener.getLogger().println(
+ "Failure to export test result(s).");
+ e.printStackTrace(listener.getLogger());
+ }
- listener.getLogger().println("Test result(s) successfully exported.");
+ listener.getLogger().println(
+ "Test result(s) successfully exported.");
- result = true;
+ result = true;
+ }
}
}
+ } else {
+ listener.getLogger().println(
+ "Skipping export of test results to Klaros-Testmangement due to build status");
+ result = true;
}
return result;
}
You received this message because you are subscribed to the Google Groups "Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit https://groups.google.com/d/optout.
