add build results base on tests (fixes #13, via #167)

Author: vbragin

src/main/java/ru/yandex/qatools/allure/jenkins/AllureReportPublisher.java

@@ -1,5 +1,6 @@
 package ru.yandex.qatools.allure.jenkins;
 
+import com.google.common.base.Optional;
 import hudson.EnvVars;
 import hudson.FilePath;
 import hudson.Launcher;
@@ -11,6 +12,7 @@
 import hudson.model.Action;
 import hudson.model.BuildListener;
 import hudson.model.JDK;
+import hudson.model.Result;
 import hudson.model.Run;
 import hudson.model.TaskListener;
 import hudson.tasks.BuildStepMonitor;
@@ -28,6 +30,7 @@
 import ru.yandex.qatools.allure.jenkins.config.ResultsConfig;
 import ru.yandex.qatools.allure.jenkins.exception.AllurePluginException;
 import ru.yandex.qatools.allure.jenkins.tools.AllureCommandlineInstallation;
+import ru.yandex.qatools.allure.jenkins.utils.BuildSummary;
 import ru.yandex.qatools.allure.jenkins.utils.BuildUtils;
 import ru.yandex.qatools.allure.jenkins.utils.FilePathUtils;
 import ru.yandex.qatools.allure.jenkins.utils.TrueZipArchiver;
@@ -193,7 +196,13 @@ public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnul
         final List<FilePath> results = new ArrayList<>();
 
         final EnvVars buildEnvVars = BuildUtils.getBuildEnvVars(run, listener);
-        for (final ResultsConfig resultsConfig : getResults()) {
+
+        List<ResultsConfig> resultsConfigs = getResults();
+        if (resultsConfigs == null) {
+            throw new AllurePluginException("The property 'Results' have to be specified!" +
+                    " Check your job's configuration.");
+        }
+        for (final ResultsConfig resultsConfig : resultsConfigs) {
             String expandedPath = buildEnvVars.expand(resultsConfig.getPath());
             results.addAll(workspace.act(new FindByGlob(expandedPath)));
         }
@@ -281,7 +290,9 @@ private void generateReport(@Nonnull List<FilePath> resultsPaths, @Nonnull Run<?
             }
             listener.getLogger().println("Allure report was successfully generated.");
             saveAllureArtifact(run, workspace, reportPath, listener);
-            run.addAction(new AllureReportBuildAction(FilePathUtils.extractSummary(run)));
+            BuildSummary buildSummary = FilePathUtils.extractSummary(run);
+            run.addAction(new AllureReportBuildAction(buildSummary));
+            run.setResult(buildSummary.getResult());
         } finally {
             FilePathUtils.deleteRecursive(reportPath, listener.getLogger());
         }

src/main/java/ru/yandex/qatools/allure/jenkins/utils/BuildSummary.java

@@ -1,5 +1,7 @@
 package ru.yandex.qatools.allure.jenkins.utils;
 
+import hudson.model.Result;
+
 import java.util.Map;
 
 /**
@@ -37,4 +39,11 @@ public long getBrokenCount() {
     public long getUnknownCount() {
         return getStatistic("unknown");
     }
+
+    public Result getResult() {
+        if (getFailedCount() > 0 || getBrokenCount() > 0) {
+            return Result.UNSTABLE;
+        }
+        return Result.SUCCESS;
+    }
 }

src/test/java/ru/yandex/qatools/allure/jenkins/ReportGenerateIT.java

@@ -91,7 +91,6 @@ public void shouldGenerateReportForParameters() throws Exception {
         FreeStyleBuild build = jRule.buildAndAssertSuccess(project);
 
         assertThat(build.getActions(AllureReportBuildAction.class)).hasSize(1);
-
     }
 
     @Test
@@ -106,7 +105,16 @@ public void shouldGenerateReportForWrappedParameters() throws Exception {
         FreeStyleBuild build = jRule.buildAndAssertSuccess(project);
 
         assertThat(build.getActions(AllureReportBuildAction.class)).hasSize(1);
+    }
 
+    @Test
+    public void shouldGenerateReportWithUnstableResult() throws Exception {
+        FreeStyleProject project = jRule.createFreeStyleProject();
+        project.setScm(getSimpleFileScm("sample-testsuite-with-failed.xml", ALLURE_RESULTS));
+        project.getPublishersList().add(createAllurePublisher("allure-results"));
+        FreeStyleBuild build = jRule.assertBuildStatus(Result.UNSTABLE, project.scheduleBuild2(0));
+
+        assertThat(build.getActions(AllureReportBuildAction.class)).hasSize(1);
     }
 
     @Test

src/test/resources/sample-testsuite-with-failed.xml

@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<ns2:test-suite xmlns:ns2="urn:model.allure.qatools.yandex.ru" start="1412949538848" stop="1412949560045"
+                version="1.4.4-SNAPSHOT">
+    <name>my.company.Sample</name>
+    <test-cases>
+        <test-case start="1412949539363" stop="1412949539730" status="passed">
+            <name>sampleTestCase</name>
+        </test-case>
+        <test-case start="1412949539800" stop="1412949539900" status="failed">
+            <name>sampleFailedTestCase</name>
+            <failure>
+                <message>Failed test!</message>
+                <stack-trace>
+                    Failed stack trace!
+                </stack-trace>
+            </failure>
+        </test-case>
+    </test-cases>
+</ns2:test-suite>