Replace panics with exits using the expect-exit library

Author: GJKrupa

src/main.rs

@@ -52,7 +52,7 @@ fn main() {
         get_rules_file(&args).or_exit_("Must specify -f or have a ~/.sdk-rules.yaml file");
     let rules = parse_rules(fs::read_to_string(rules_file).or_exit_("Failed to read rules file"));
 
-    for (name, candidate) in rules.expect("Rules file could not be parsed").candidates {
+    for (name, candidate) in rules.or_exit_("Rules file could not be parsed").candidates {
         let installed: HashSet<_> = sdkman
             .installed_versions(name.clone())
             .into_iter()

src/rules/mod.rs

@@ -1,3 +1,4 @@
+use expect_exit::Expected;
 use regex::Regex;
 use std::collections::HashMap;
 
@@ -32,7 +33,7 @@ pub trait VersionMatch {
 impl VersionMatch for Version {
     fn get_matching(&self, name: String, available: Vec<String>) -> Option<String> {
         let pattern = Regex::new(self.pattern.as_str())
-            .unwrap_or_else(|_| panic!("Invalid regex for {}: {}", name, self.pattern));
+            .or_exit_(format!("Invalid regex for {}: {}", name, self.pattern).as_str());
         let mut matches: Vec<String> = available
             .iter()
             .filter(|it| pattern.is_match(it))
@@ -41,7 +42,7 @@ impl VersionMatch for Version {
 
         if let Some(exclude) = self.exclude.as_ref() {
             let exclude_pattern = Regex::new(exclude.join("|").as_str())
-                .unwrap_or_else(|_| panic!("Invalid regex for {}: {}", name, exclude.join("|")));
+                .or_exit_(format!("Invalid regex for {}: {}", name, exclude.join("|")).as_str());
             matches.retain(|it| !exclude_pattern.is_match(it));
         }
 

src/sdkman/mod.rs

@@ -1,4 +1,5 @@
 use crate::sdkman::candidate::{Candidate, SdkManCandidate};
+use expect_exit::Expected;
 use std::fs::read_dir;
 use std::io::Write;
 use std::path::Path;
@@ -27,7 +28,7 @@ impl ToolManager for SdkMan {
                 let base = Path::new(dir.as_str()).join("candidates").join(candidate);
                 if base.is_dir() {
                     read_dir(base)
-                        .expect("Failed to read $SDKMAN_DIR")
+                        .or_exit_("Failed to read $SDKMAN_DIR")
                         .filter(|entry| entry.as_ref().unwrap().file_type().unwrap().is_dir())
                         .map(|entry| entry.unwrap().file_name().into_string().unwrap())
                         .collect()
@@ -49,7 +50,7 @@ impl ToolManager for SdkMan {
             .arg("-c")
             .arg(format!("sdk list {}", candidate))
             .output()
-            .expect("Failed to run the sdk list command. Is SDKMAN installed?");
+            .or_exit_("Failed to run the sdk list command. Is SDKMAN installed?");
         if output.status.success() {
             SdkManCandidate {
                 name: candidate,
@@ -77,7 +78,7 @@ impl ToolManager for SdkMan {
                 .arg("-c")
                 .arg(cmd)
                 .output()
-                .expect("Error running sdk install command");
+                .or_exit_("Error running sdk install command");
             if output.status.success() {
                 println!("OK");
             } else {
@@ -106,7 +107,7 @@ impl ToolManager for SdkMan {
                 .arg("-c")
                 .arg(cmd)
                 .output()
-                .expect("Error running sdk install command");
+                .or_exit_("Error running sdk install command");
             if output.status.success() {
                 println!("OK");
             } else {
@@ -133,7 +134,7 @@ impl ToolManager for SdkMan {
                 .arg("-c")
                 .arg(cmd)
                 .output()
-                .expect("Error running sdk install command");
+                .or_exit_("Error running sdk install command");
             if output.status.success() {
                 println!("OK");
             } else {