diff --git a/src/concourse.rs b/src/concourse.rs index 76c6fdd..f2a1456 100644 --- a/src/concourse.rs +++ b/src/concourse.rs @@ -76,7 +76,7 @@ impl Source { } // return unwrapped value with default false for ease of use pub(crate) fn skip_check(&self) -> bool { - return self.skip_check.unwrap_or(false) + return self.skip_check.unwrap_or(false); } } diff --git a/src/github_issue.rs b/src/github_issue.rs index 0e53d8c..c4bb62d 100644 --- a/src/github_issue.rs +++ b/src/github_issue.rs @@ -361,9 +361,12 @@ mod tests { #[test] fn test_str_to_params_state() { // octocrab::params::State does not implement Eq - str_to_params_state("Open").unwrap(); - str_to_params_state("Closed").unwrap(); - str_to_params_state("All").unwrap(); + str_to_params_state("Open") + .expect("could not convert \"Open\" to octocrab::params::State::Open"); + str_to_params_state("Closed") + .expect("could not convert \"Closed\" to octocrab::params::State::Closed"); + str_to_params_state("All") + .expect("could not convert \"All\" to octocrab::params::State::All"); } #[test]