From 3873b647d123f427b519662cb8d7714727dcf3c8 Mon Sep 17 00:00:00 2001 From: ldm0 Date: Tue, 16 Feb 2021 03:58:58 +0000 Subject: [PATCH] Add tests for 2229 --- tests/multiple_values.rs | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/tests/multiple_values.rs b/tests/multiple_values.rs index 1285fa58..f1f0f54b 100644 --- a/tests/multiple_values.rs +++ b/tests/multiple_values.rs @@ -1192,3 +1192,19 @@ fn issue_1480_max_values_consumes_extra_arg_3() { assert!(res.is_err()); assert_eq!(res.unwrap_err().kind, ErrorKind::UnknownArgument); } + +#[test] +fn issue_2229() { + let m = App::new("multiple_values") + .arg( + Arg::new("pos") + .about("multiple positionals") + .number_of_values(3), + ) + .try_get_matches_from(vec![ + "myprog", "val1", "val2", "val3", "val4", "val5", "val6", + ]); + + assert!(m.is_err()); // This panics, because `m.is_err() == false`. + assert_eq!(m.unwrap_err().kind, ErrorKind::WrongNumberOfValues); +}