From 7f369fe35557fab2e9d5c5693e41d238842dcf17 Mon Sep 17 00:00:00 2001 From: Teddy Wing Date: Sat, 11 Nov 2017 22:55:15 +0100 Subject: find_and_track_build_and_update_status(): Elaborate `expect` messages Make these panic messages more meaningful by adding some contextual information. --- src/jenkins.rs | 27 ++++++++++++++++++++++++--- 1 file changed, 24 insertions(+), 3 deletions(-) (limited to 'src') 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 } -- cgit v1.2.3