diff options
| author | Teddy Wing | 2017-11-11 22:55:15 +0100 |
|---|---|---|
| committer | Teddy Wing | 2017-11-11 22:55:58 +0100 |
| commit | 7f369fe35557fab2e9d5c5693e41d238842dcf17 (patch) | |
| tree | 992d108101620b938e64a062c37d137bf60e86de | |
| parent | b9d225974d9be1ded1ee0b47d8025916edd5736c (diff) | |
| download | kipper-7f369fe35557fab2e9d5c5693e41d238842dcf17.tar.bz2 | |
find_and_track_build_and_update_status(): Elaborate `expect` messages
Make these panic messages more meaningful by adding some contextual
information.
| -rw-r--r-- | src/jenkins.rs | 27 |
1 files changed, 24 insertions, 3 deletions
diff --git a/src/jenkins.rs b/src/jenkins.rs index 024f7f7..7ce7778 100644 --- a/src/jenkins.rs +++ b/src/jenkins.rs @@ -102,7 +102,14 @@ pub fn find_and_track_build_and_update_status(commit_ref: CommitRef) job_url.clone(), None, "continuous-integration/jenkins".to_string() - ).expect("GitHub pending status update failed."); + ).expect( + format!( + "GitHub pending status update failed for {}/{} {}.", + commit_ref.owner, + commit_ref.repo, + commit_ref.sha + ).as_ref() + ); while job.result == JobStatus::Pending { // loop @@ -122,7 +129,14 @@ pub fn find_and_track_build_and_update_status(commit_ref: CommitRef) job_url.clone(), Some("The status checker timed out.".to_string()), "continuous-integration/jenkins".to_string() - ).expect("GitHub timeout error status update failed."); + ).expect( + format!( + "GitHub timeout error status update failed for {}/{} {}.", + commit_ref.owner, + commit_ref.repo, + commit_ref.sha + ).as_ref() + ); return } @@ -142,7 +156,14 @@ pub fn find_and_track_build_and_update_status(commit_ref: CommitRef) job_url.clone(), None, "continuous-integration/jenkins".to_string() - ).expect("GitHub status update failed."); + ).expect( + format!( + "GitHub status update failed for {}/{} {}.", + commit_ref.owner, + commit_ref.repo, + commit_ref.sha + ).as_ref() + ); return } |