From 5e20a29b4fdc8a2d442d1093681b396dcb4b816b Mon Sep 17 00:00:00 2001 From: Robin Krahl Date: Tue, 7 Jan 2020 11:18:04 +0000 Subject: Add structopt dependency in version 0.3.7 This patch series replaces argparse with structopt in the argument handling code. As a first step, we need structopt as a dependency. Import subrepo structopt/:structopt at efbdda4753592e27bc430fb01f7b9650b2f3174d Import subrepo bitflags/:bitflags at 30668016aca6bd3b02c766e8347e0b4080d4c296 Import subrepo clap/:clap at 784524f7eb193e35f81082cc69454c8c21b948f7 Import subrepo heck/:heck at 093d56fbf001e1506e56dbfa38631d99b1066df1 Import subrepo proc-macro-error/:proc-macro-error at 6c4cfe79a622c5de8ae68557993542be46eacae2 Import subrepo proc-macro2/:proc-macro2 at d5d48eddca4566e5438e8a2cbed4a74e049544de Import subrepo quote/:quote at 727436c6c137b20f0f34dde5d8fda2679b9747ad Import subrepo rustversion/:rustversion at 0c5663313516263059ce9059ef81fc7a1cf655ca Import subrepo syn-mid/:syn-mid at 5d3d85414a9e6674e1857ec22a87b96e04a6851a Import subrepo syn/:syn at e87c27e87f6f4ef8919d0372bdb056d53ef0d8f3 Import subrepo textwrap/:textwrap at abcd618beae3f74841032aa5b53c1086b0a57ca2 Import subrepo unicode-segmentation/:unicode-segmentation at 637c9874c4fe0c205ff27787faf150a40295c6c3 Import subrepo unicode-width/:unicode-width at 3033826f8bf05e82724140a981d5941e48fce393 Import subrepo unicode-xid/:unicode-xid at 4baae9fffb156ba229665b972a9cd5991787ceb7 --- clap/tests/opts.rs | 461 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 461 insertions(+) create mode 100644 clap/tests/opts.rs (limited to 'clap/tests/opts.rs') diff --git a/clap/tests/opts.rs b/clap/tests/opts.rs new file mode 100644 index 0000000..a9029ac --- /dev/null +++ b/clap/tests/opts.rs @@ -0,0 +1,461 @@ +extern crate clap; +extern crate regex; + +include!("../clap-test.rs"); + +use clap::{App, ArgMatches, Arg, ErrorKind}; + +#[cfg(feature = "suggestions")] +static DYM: &'static str = "error: Found argument '--optio' which wasn't expected, or isn't valid in this context +\tDid you mean --option? + +USAGE: + clap-test --option ... + +For more information try --help"; + +#[test] +fn require_equals_fail() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .require_equals(true) + .takes_value(true) + .long("config")) + .get_matches_from_safe(vec![ + "prog", "--config", "file.conf" + ]); + assert!(res.is_err()); + assert_eq!(res.unwrap_err().kind, ErrorKind::EmptyValue); +} + +#[test] +fn require_equals_min_values_zero() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .require_equals(true) + .takes_value(true) + .min_values(0) + .long("config")) + .arg(Arg::with_name("cmd")) + .get_matches_from_safe(vec![ + "prog", "--config", "cmd" + ]); + assert!(res.is_ok()); + let m = res.unwrap(); + assert!(m.is_present("cfg")); + assert_eq!(m.value_of("cmd"), Some("cmd")); +} + +#[test] +fn double_hyphen_as_value() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .takes_value(true) + .allow_hyphen_values(true) + .long("config")) + .get_matches_from_safe(vec![ + "prog", "--config", "--" + ]); + assert!(res.is_ok(), "{:?}", res); + assert_eq!(res.unwrap().value_of("cfg"), Some("--")); +} + +#[test] +fn require_equals_no_empty_values_fail() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .require_equals(true) + .takes_value(true) + .long("config")) + .arg(Arg::with_name("some")) + .get_matches_from_safe(vec![ + "prog", "--config=", "file.conf" + ]); + assert!(res.is_err()); + assert_eq!(res.unwrap_err().kind, ErrorKind::EmptyValue); +} + +#[test] +fn require_equals_empty_vals_pass() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .require_equals(true) + .takes_value(true) + .empty_values(true) + .long("config")) + .get_matches_from_safe(vec![ + "prog", "--config=" + ]); + assert!(res.is_ok()); +} + +#[test] +fn require_equals_pass() { + let res = App::new("prog") + .arg(Arg::with_name("cfg") + .require_equals(true) + .takes_value(true) + .long("config")) + .get_matches_from_safe(vec![ + "prog", "--config=file.conf" + ]); + assert!(res.is_ok()); +} + +#[test] +fn stdin_char() { + let r = App::new("opts") + .arg(Arg::from_usage("-f [flag] 'some flag'")) + .get_matches_from_safe(vec!["", "-f", "-"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("f")); + assert_eq!(m.value_of("f").unwrap(), "-"); +} + +#[test] +fn opts_using_short() { + let r = App::new("opts") + .args(&[Arg::from_usage("-f [flag] 'some flag'"), + Arg::from_usage("-c [color] 'some other flag'")]) + .get_matches_from_safe(vec!["", "-f", "some", "-c", "other"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("f")); + assert_eq!(m.value_of("f").unwrap(), "some"); + assert!(m.is_present("c")); + assert_eq!(m.value_of("c").unwrap(), "other"); +} + +#[test] +fn lots_o_vals() { + let r = App::new("opts") + .arg(Arg::from_usage("-o [opt]... 'some opt'")) + .get_matches_from_safe(vec!["", "-o", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some", "some", "some", "some", + "some", "some", "some", "some"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>().len(), 297); // i.e. more than u8 +} + +#[test] +fn opts_using_long_space() { + let r = App::new("opts") + .args(&[Arg::from_usage("--flag [flag] 'some flag'"), + Arg::from_usage("--color [color] 'some other flag'")]) + .get_matches_from_safe(vec!["", "--flag", "some", "--color", "other"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("flag")); + assert_eq!(m.value_of("flag").unwrap(), "some"); + assert!(m.is_present("color")); + assert_eq!(m.value_of("color").unwrap(), "other"); +} + +#[test] +fn opts_using_long_equals() { + let r = App::new("opts") + .args(&[Arg::from_usage("--flag [flag] 'some flag'"), + Arg::from_usage("--color [color] 'some other flag'")]) + .get_matches_from_safe(vec!["", "--flag=some", "--color=other"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("flag")); + assert_eq!(m.value_of("flag").unwrap(), "some"); + assert!(m.is_present("color")); + assert_eq!(m.value_of("color").unwrap(), "other"); +} + +#[test] +fn opts_using_mixed() { + let r = App::new("opts") + .args(&[Arg::from_usage("-f, --flag [flag] 'some flag'"), + Arg::from_usage("-c, --color [color] 'some other flag'")]) + .get_matches_from_safe(vec!["", "-f", "some", "--color", "other"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("flag")); + assert_eq!(m.value_of("flag").unwrap(), "some"); + assert!(m.is_present("color")); + assert_eq!(m.value_of("color").unwrap(), "other"); +} + +#[test] +fn opts_using_mixed2() { + let r = App::new("opts") + .args(&[Arg::from_usage("-f, --flag [flag] 'some flag'"), + Arg::from_usage("-c, --color [color] 'some other flag'")]) + .get_matches_from_safe(vec!["", "--flag=some", "-c", "other"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("flag")); + assert_eq!(m.value_of("flag").unwrap(), "some"); + assert!(m.is_present("color")); + assert_eq!(m.value_of("color").unwrap(), "other"); +} + +#[test] +fn default_values_user_value() { + let r = App::new("df") + .arg(Arg::from_usage("-o [opt] 'some opt'").default_value("default")) + .get_matches_from_safe(vec!["", "-o", "value"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.value_of("o").unwrap(), "value"); +} + +#[test] +fn multiple_vals_pos_arg_equals() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'")) + .arg(Arg::from_usage("[file] 'some file'")) + .get_matches_from_safe(vec!["", "-o=1", "some"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.value_of("o").unwrap(), "1"); + assert!(m.is_present("file")); + assert_eq!(m.value_of("file").unwrap(), "some"); +} + +#[test] +fn multiple_vals_pos_arg_delim() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'")) + .arg(Arg::from_usage("[file] 'some file'")) + .get_matches_from_safe(vec!["", "-o", "1,2", "some"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["1", "2"]); + assert!(m.is_present("file")); + assert_eq!(m.value_of("file").unwrap(), "some"); +} + +#[test] +fn require_delims_no_delim() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").require_delimiter(true)) + .arg(Arg::from_usage("[file] 'some file'")) + .get_matches_from_safe(vec!["mvae", "-o", "1", "2", "some"]); + assert!(r.is_err()); + let err = r.unwrap_err(); + assert_eq!(err.kind, ErrorKind::UnknownArgument); +} + +#[test] +fn require_delims() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").require_delimiter(true)) + .arg(Arg::from_usage("[file] 'some file'")) + .get_matches_from_safe(vec!["", "-o", "1,2", "some"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["1", "2"]); + assert!(m.is_present("file")); + assert_eq!(m.value_of("file").unwrap(), "some"); +} + +#[test] +fn leading_hyphen_pass() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").allow_hyphen_values(true)) + .get_matches_from_safe(vec!["", "-o", "-2", "3"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["-2", "3"]); +} + +#[test] +fn leading_hyphen_fail() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt] 'some opt'")) + .get_matches_from_safe(vec!["", "-o", "-2"]); + assert!(r.is_err()); + let m = r.unwrap_err(); + assert_eq!(m.kind, ErrorKind::UnknownArgument); +} + +#[test] +fn leading_hyphen_with_flag_after() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").allow_hyphen_values(true)) + .arg_from_usage("-f 'some flag'") + .get_matches_from_safe(vec!["", "-o", "-2", "-f"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["-2", "-f"]); + assert!(!m.is_present("f")); +} + +#[test] +fn leading_hyphen_with_flag_before() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").allow_hyphen_values(true)) + .arg_from_usage("-f 'some flag'") + .get_matches_from_safe(vec!["", "-f", "-o", "-2"]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["-2"]); + assert!(m.is_present("f")); +} + +#[test] +fn leading_hyphen_with_only_pos_follows() { + let r = App::new("mvae") + .arg(Arg::from_usage("-o [opt]... 'some opt'").number_of_values(1).allow_hyphen_values(true)) + .arg_from_usage("[arg] 'some arg'") + .get_matches_from_safe(vec!["", "-o", "-2", "--", "val"]); + assert!(r.is_ok(), "{:?}", r); + let m = r.unwrap(); + assert!(m.is_present("o")); + assert_eq!(m.values_of("o").unwrap().collect::>(), &["-2"]); + assert_eq!(m.value_of("arg"), Some("val")); +} + +#[test] +#[cfg(feature="suggestions")] +fn did_you_mean() { + assert!(test::compare_output(test::complex_app(), + "clap-test --optio=foo", + DYM, + true)); +} + +#[test] +fn issue_665() { + let res = App::new("tester") + .arg_from_usage("-v, --reroll-count=[N] 'Mark the patch series as PATCH vN'") + .arg(Arg::from_usage( +"--subject-prefix [Subject-Prefix] 'Use [Subject-Prefix] instead of the standard [PATCH] prefix'") + .empty_values(false)) + .get_matches_from_safe(vec!["test", "--subject-prefix", "-v", "2"]); + + assert!(res.is_err()); + assert_eq!(res.unwrap_err().kind, ErrorKind::EmptyValue); +} + +#[test] +fn issue_1047_min_zero_vals_default_val() { + let m = App::new("foo") + .arg( + Arg::with_name("del") + .short("d") + .long("del") + .takes_value(true) + .require_equals(true) + .min_values(0) + .default_value("default"), + ) + .get_matches_from(vec!["foo", "-d"]); + assert_eq!(m.occurrences_of("del"), 1); + assert_eq!(m.value_of("del"), Some("default")); +} + +fn issue_1105_setup(argv: Vec<&'static str>) -> Result, clap::Error> { + App::new("opts") + .arg_from_usage("-o, --option [opt] 'some option'") + .arg_from_usage("--flag 'some flag'") + .get_matches_from_safe(argv) +} + +#[test] +fn issue_1105_empty_value_long_fail() { + let r = issue_1105_setup(vec!["app", "--option", "--flag"]); + assert!(r.is_err()); + assert_eq!(r.unwrap_err().kind, ErrorKind::EmptyValue); +} + +#[test] +fn issue_1105_empty_value_long_explicit() { + let r = issue_1105_setup(vec!["app", "--option", ""]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert_eq!(m.value_of("option"), Some("")); +} + +#[test] +fn issue_1105_empty_value_long_equals() { + let r = issue_1105_setup(vec!["app", "--option="]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert_eq!(m.value_of("option"), Some("")); +} + +#[test] +fn issue_1105_empty_value_short_fail() { + let r = issue_1105_setup(vec!["app", "-o", "--flag"]); + assert!(r.is_err()); + assert_eq!(r.unwrap_err().kind, ErrorKind::EmptyValue); +} + +#[test] +fn issue_1105_empty_value_short_explicit() { + let r = issue_1105_setup(vec!["app", "-o", ""]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert_eq!(m.value_of("option"), Some("")); +} + +#[test] +fn issue_1105_empty_value_short_equals() { + let r = issue_1105_setup(vec!["app", "-o="]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert_eq!(m.value_of("option"), Some("")); +} + +#[test] +fn issue_1105_empty_value_short_explicit_no_space() { + let r = issue_1105_setup(vec!["app", "-o", ""]); + assert!(r.is_ok()); + let m = r.unwrap(); + assert_eq!(m.value_of("option"), Some("")); +} -- cgit v1.2.1