[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