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 | |
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')
-rw-r--r-- | src/github.rs | 2 | ||||
-rw-r--r-- | src/jenkins.rs | 7 | ||||
-rw-r--r-- | src/pull_request.rs | 3 |
3 files changed, 7 insertions, 5 deletions
diff --git a/src/github.rs b/src/github.rs index f24d1f0..731d261 100644 --- a/src/github.rs +++ b/src/github.rs @@ -104,7 +104,7 @@ mod tests { "https://jenkins.example.com/job/octocat/3".to_string(), None, "continuous-integration/jenkins".to_string() - ); + ).expect("Failed to update commit status"); mock.assert(); } 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(), diff --git a/src/pull_request.rs b/src/pull_request.rs index df4cd2a..c97df53 100644 --- a/src/pull_request.rs +++ b/src/pull_request.rs @@ -448,7 +448,8 @@ mod tests { } }"#; - let commit_ref = CommitRef::new(payload); + let commit_ref = CommitRef::new(payload) + .expect("Failed to create CommitRef from payload"); assert_eq!(commit_ref.owner, "baxterthehacker"); assert_eq!(commit_ref.repo, "public-repo"); |