diff --git a/src/flags/recursion.rs b/src/flags/recursion.rs index e8659359f..54c8b1ec7 100644 --- a/src/flags/recursion.rs +++ b/src/flags/recursion.rs @@ -133,7 +133,7 @@ mod test { use clap::ErrorKind; #[test] - fn test_enabled_from_arg_matches_none() { + fn test_enabled_from_arg_matches_empty() { let argv = vec!["lsd"]; let matches = app::build().get_matches_from_safe(argv).unwrap(); assert_eq!(None, Recursion::enabled_from_arg_matches(&matches)); @@ -147,7 +147,7 @@ mod test { } #[test] - fn test_enabled_from_config_empty() { + fn test_enabled_from_empty_matches_and_config() { let argv = vec!["lsd"]; assert_eq!( false, @@ -159,7 +159,7 @@ mod test { } #[test] - fn test_enabled_from_config_true() { + fn test_enabled_from_matches_empty_and_config_true() { let argv = vec!["lsd"]; let mut c = Config::with_none(); c.recursion = Some(config_file::Recursion { @@ -173,7 +173,7 @@ mod test { } #[test] - fn test_enabled_from_config_false() { + fn test_enabled_from_matches_empty_and_config_false() { let argv = vec!["lsd"]; let mut c = Config::with_none(); c.recursion = Some(config_file::Recursion { @@ -190,7 +190,7 @@ mod test { // of the assert_eq macro, because clap::Error does not implement PartialEq. #[test] - fn test_depth_from_arg_matches_none() { + fn test_depth_from_arg_matches_empty() { let argv = vec!["lsd"]; let matches = app::build().get_matches_from_safe(argv).unwrap(); assert!(match Recursion::depth_from_arg_matches(&matches) { diff --git a/src/flags/sorting.rs b/src/flags/sorting.rs index 0618a7fa1..4f4239ef4 100644 --- a/src/flags/sorting.rs +++ b/src/flags/sorting.rs @@ -420,7 +420,7 @@ mod test_sort_order { } #[test] - fn test_from_config_reverse() { + fn test_from_config_reverse_true() { let mut c = Config::with_none(); c.sorting = Some(Sorting { column: None, @@ -429,6 +429,17 @@ mod test_sort_order { }); assert_eq!(Some(SortOrder::Reverse), SortOrder::from_config(&c)); } + + #[test] + fn test_from_config_reverse_false() { + let mut c = Config::with_none(); + c.sorting = Some(Sorting { + column: None, + reverse: Some(false), + dir_grouping: None, + }); + assert_eq!(Some(SortOrder::Default), SortOrder::from_config(&c)); + } } #[cfg(test)]