Added unittest for recursive aliases.

Author: Alexhuszagh

src/cli.rs

@@ -277,6 +277,8 @@ pub fn parse(target_list: &TargetList) -> Result<Args> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::rustc;
+    use crate::shell::Verbosity;
 
     #[test]
     fn is_verbose_test() {
@@ -289,4 +291,20 @@ mod tests {
         assert!(is_verbose("-vvvv"));
         assert!(!is_verbose("-version"));
     }
+
+    #[test]
+    #[should_panic]
+    fn test_recursive_alias() {
+        let mut args = Args::default();
+        let target_list =
+            rustc::target_list(&mut Verbosity::Quiet.into()).expect("failed to get target list");
+
+        parse_subcommand(
+            "recursive".to_owned(),
+            &mut args,
+            &target_list,
+            Some("recursive"),
+        )
+        .ok();
+    }
 }

src/shell.rs

@@ -208,7 +208,14 @@ impl MessageInfo {
     pub fn fatal<T: fmt::Display>(&mut self, message: T, code: i32) -> ! {
         self.error(message)
             .expect("could not display fatal message");
-        std::process::exit(code);
+
+        // need to catch panics in unittests, otherwise
+        // want the custom styled error message
+        if cfg!(test) {
+            panic!("");
+        } else {
+            std::process::exit(code);
+        }
     }
 
     /// prints a red 'error' message.