JENKINS-13798 Set the job status based on the Jenkins Result

jenkinsci · Jun 9, 2012 · f9d7cb1 · f9d7cb1
1 parent 9177a5f
commit f9d7cb1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/hudson/plugins/sauce_ondemand/SauceOnDemandBuildWrapper.java b/src/main/java/hudson/plugins/sauce_ondemand/SauceOnDemandBuildWrapper.java
@@ -223,6 +223,7 @@ public boolean tearDown(AbstractBuild build, BuildListener listener) throws IOEx
     private void processBuildOutput(AbstractBuild build) {
         JobFactory factory = new JobFactory(new Credential(getUserName(), getApiKey()));
         //todo we only want to iterate over lines if we're not using junit test results
+
         String[] array = logParser.getLines().toArray(new String[logParser.getLines().size()]);
         List<String[]> sessionIDs = SauceOnDemandReportFactory.findSessionIDs(null, array);
 
@@ -238,7 +239,7 @@ private void processBuildOutput(AbstractBuild build) {
                                     false,
                                     Collections.<String>emptyList(),
                                     Integer.toString(build.getNumber()),
-                                    true,//todo how can we tell if the build has passed or failed here?
+                                    build.getResult().equals(Result.SUCCESS),
                                     Collections.<String, Object>emptyMap()));
                 }
             } catch (IOException e) {