Simplify oof testing asserts

This commit is contained in:
João M. Bezerra 2021-05-26 12:58:52 -03:00
parent 22f13552f7
commit 5a2028b117

View File

@ -238,15 +238,12 @@ mod tests {
#[test] #[test]
fn test_unknown_flags() { fn test_unknown_flags() {
let result = setup_args_scenario("ouch a.zip -s b.tar.gz c.tar").unwrap_err(); let result = setup_args_scenario("ouch a.zip -s b.tar.gz c.tar").unwrap_err();
assert!(matches!(result, OofError::UnknownShortFlag('s'))); assert!(matches!(result, OofError::UnknownShortFlag(flag) if flag == 's'));
let unknown_long_flag = "foobar".to_string(); let unknown_long_flag = "foobar".to_string();
let result = setup_args_scenario("ouch a.zip --foobar b.tar.gz c.tar").unwrap_err(); let result = setup_args_scenario("ouch a.zip --foobar b.tar.gz c.tar").unwrap_err();
assert!(match result { assert!(matches!(result, OofError::UnknownLongFlag(flag) if flag == unknown_long_flag));
OofError::UnknownLongFlag(flag) if flag == unknown_long_flag => true,
_ => false,
})
} }
#[test] #[test]
@ -254,10 +251,7 @@ mod tests {
let incomplete_flag = ArgFlag::long("output_file").short('o'); let incomplete_flag = ArgFlag::long("output_file").short('o');
let result = setup_args_scenario("ouch a.zip b.tar.gz c.tar -o").unwrap_err(); let result = setup_args_scenario("ouch a.zip b.tar.gz c.tar -o").unwrap_err();
assert!(match result { assert!(matches!(result, OofError::MissingValueToFlag(flag) if flag == incomplete_flag));
OofError::MissingValueToFlag(flag) if flag == incomplete_flag => true,
_ => false,
})
} }
#[test] #[test]
@ -265,10 +259,7 @@ mod tests {
let duplicated_flag = ArgFlag::long("output_file").short('o'); let duplicated_flag = ArgFlag::long("output_file").short('o');
let result = setup_args_scenario("ouch a.zip b.tar.gz c.tar -o -o -o").unwrap_err(); let result = setup_args_scenario("ouch a.zip b.tar.gz c.tar -o -o -o").unwrap_err();
assert!(match result { assert!(matches!(result, OofError::DuplicatedFlag(flag) if flag == duplicated_flag));
OofError::DuplicatedFlag(flag) if flag == duplicated_flag => true,
_ => false,
})
} }
#[test] #[test]
@ -276,11 +267,9 @@ mod tests {
let misplaced_flag = ArgFlag::long("output_file").short('o'); let misplaced_flag = ArgFlag::long("output_file").short('o');
let result = setup_args_scenario("ouch -ov a.zip b.tar.gz c.tar").unwrap_err(); let result = setup_args_scenario("ouch -ov a.zip b.tar.gz c.tar").unwrap_err();
assert!(match result { assert!(
OofError::MisplacedShortArgFlagError(flag) if flag == misplaced_flag.short.unwrap() => matches!(result, OofError::MisplacedShortArgFlagError(flag) if flag == misplaced_flag.short.unwrap())
true, );
_ => false,
})
} }
#[test] #[test]
@ -289,10 +278,7 @@ mod tests {
let invalid_unicode_flag = OsString::from_vec(vec![45, 0, 0, 0, 255, 255, 255, 255]); let invalid_unicode_flag = OsString::from_vec(vec![45, 0, 0, 0, 255, 255, 255, 255]);
let result = filter_flags(vec![invalid_unicode_flag.clone()], &[]).unwrap_err(); let result = filter_flags(vec![invalid_unicode_flag.clone()], &[]).unwrap_err();
assert!(match result { assert!(matches!(result, OofError::InvalidUnicode(flag) if flag == invalid_unicode_flag));
OofError::InvalidUnicode(flag) if flag == invalid_unicode_flag => true,
_ => false,
})
} }
// asdasdsa // asdasdsa