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"); | 
