[snowpatch] [PATCH 3/5] jenkins: Rename JenkinsBuildStatus to BuildStatus

Andrew Donnellan andrew.donnellan at au1.ibm.com
Fri Feb 1 17:17:12 AEDT 2019


There's no reason JenkinsBuildStatus needs to be Jenkins-specific at the
moment.

Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>
---
 src/jenkins.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/jenkins.rs b/src/jenkins.rs
index 547c396c3513..c76561f1fda2 100644
--- a/src/jenkins.rs
+++ b/src/jenkins.rs
@@ -86,7 +86,7 @@ impl CIBackend for JenkinsBackend {
 }
 
 #[derive(Eq, PartialEq)]
-pub enum JenkinsBuildStatus {
+pub enum BuildStatus {
     Running,
     Done,
 }
@@ -166,10 +166,10 @@ impl JenkinsBackend {
         }
     }
 
-    pub fn get_build_status(&self, build_handle: &str) -> Result<JenkinsBuildStatus, Box<Error>> {
+    pub fn get_build_status(&self, build_handle: &str) -> Result<BuildStatus, Box<Error>> {
         match self.get_api_json_object(build_handle)?["building"].as_bool() {
-            Some(true) => Ok(JenkinsBuildStatus::Running),
-            Some(false) => Ok(JenkinsBuildStatus::Done),
+            Some(true) => Ok(BuildStatus::Running),
+            Some(false) => Ok(BuildStatus::Done),
             None => Err("Error getting build status".into()),
         }
     }
@@ -230,11 +230,11 @@ impl JenkinsBackend {
         }
     }
 
-    pub fn wait_build(&self, build_handle: &str) -> Result<JenkinsBuildStatus, Box<Error>> {
+    pub fn wait_build(&self, build_handle: &str) -> Result<BuildStatus, Box<Error>> {
         // TODO: Implement a timeout?
-        while self.get_build_status(build_handle)? != JenkinsBuildStatus::Done {
+        while self.get_build_status(build_handle)? != BuildStatus::Done {
             sleep(Duration::from_millis(JENKINS_POLLING_INTERVAL));
         }
-        Ok(JenkinsBuildStatus::Done)
+        Ok(BuildStatus::Done)
     }
 }
-- 
2.11.0



More information about the snowpatch mailing list