diff options
author | Teddy Wing | 2017-11-11 23:01:53 +0100 |
---|---|---|
committer | Teddy Wing | 2017-11-11 23:01:53 +0100 |
commit | dd76f4de394da3d9ce8cd19b853da10d56caf8cd (patch) | |
tree | eb0c9d6fb372423bb3106b959ad3e8265a5b249e /src/jenkins.rs | |
parent | 7f369fe35557fab2e9d5c5693e41d238842dcf17 (diff) | |
download | kipper-dd76f4de394da3d9ce8cd19b853da10d56caf8cd.tar.bz2 |
Handle `Result` return values in tests
Update our failing test suite to work with our new return types now that
many of our functions return `Result`s. Since these are tests, we handle
them by panicking with error messages.
Diffstat (limited to 'src/jenkins.rs')
-rw-r--r-- | src/jenkins.rs | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/jenkins.rs b/src/jenkins.rs index 7ce7778..3269653 100644 --- a/src/jenkins.rs +++ b/src/jenkins.rs @@ -263,7 +263,7 @@ mod tests { "result": "SUCCESS" }"#.to_string(); - let job = Job::new(payload); + let job = Job::new(payload).expect("Failed to create job from payload"); assert_eq!(job.display_name, "3296-fix-typo-700d0"); assert_eq!(job.result, JobStatus::Success); @@ -293,7 +293,7 @@ mod tests { "#) .create(); - let jobs = get_jobs("changes"); + let jobs = get_jobs("changes").expect("Failed to request jobs"); assert_eq!( jobs, @@ -317,7 +317,8 @@ mod tests { "#) .create(); - let job = request_job("http://jenkins.example.com/job/changes-branches/15"); + let job = request_job("http://jenkins.example.com/job/changes-branches/15") + .expect("Failed to request job"); let expected = Job { display_name: "2388-delete-the-codes-391af".to_string(), |