adding STATUS_SUCCESS constant instead of hardcoded string

diff --git a/jenkinsapi/build.py b/jenkinsapi/build.py
index 254fc54..930eb83 100644 (file)
--- a/jenkinsapi/build.py
+++ b/jenkinsapi/build.py
@@ -2,7 +2,7 @@ from jenkinsapi.artifact import Artifact
 from jenkinsapi import config
 from jenkinsapi.jenkinsbase import JenkinsBase
 from jenkinsapi.exceptions import NoResults, FailedNoResults
-from jenkinsapi.constants import STATUS_FAIL, STATUS_ABORTED, RESULTSTATUS_FAILURE
+from jenkinsapi.constants import STATUS_FAIL, STATUS_ABORTED, RESULTSTATUS_FAILURE, STATUS_SUCCESS
 from jenkinsapi.result_set import ResultSet
 
 from time import sleep
@@ -211,7 +211,7 @@ class Build(JenkinsBase):
         Return a bool, true if the build was good.
         If the build is still running, return False.
         """
-        return ( not self.is_running() ) and self._data["result"] == 'SUCCESS'
+        return ( not self.is_running() ) and self._data["result"] == STATUS_SUCCESS
 
     def block_until_complete(self, delay=15):
         assert isinstance( delay, int )

diff --git a/jenkinsapi/constants.py b/jenkinsapi/constants.py
index 570148c..9489e20 100644 (file)
--- a/jenkinsapi/constants.py
+++ b/jenkinsapi/constants.py
@@ -4,6 +4,7 @@ STATUS_FAIL = "FAIL"
 STATUS_ERROR = "ERROR"
 STATUS_ABORTED = "ABORTED"
 STATUS_REGRESSION = "REGRESSION"
+STATUS_SUCCESS = "SUCCESS"
 
 STATUS_FIXED = "FIXED"
 STATUS_PASSED = "PASSED"