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/groups.rs | 207 +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 207 insertions(+) create mode 100644 clap/tests/groups.rs (limited to 'clap/tests/groups.rs') diff --git a/clap/tests/groups.rs b/clap/tests/groups.rs new file mode 100644 index 0000000..bb108ea --- /dev/null +++ b/clap/tests/groups.rs @@ -0,0 +1,207 @@ +extern crate clap; +extern crate regex; + +include!("../clap-test.rs"); + +use clap::{App, Arg, ArgGroup, ErrorKind, SubCommand}; + +static REQ_GROUP_USAGE: &'static str = "error: The following required arguments were not provided: + + +USAGE: + clap-test + +For more information try --help"; + +static REQ_GROUP_CONFLICT_USAGE: &'static str = "error: The argument '' cannot be used with '--delete' + +USAGE: + clap-test + +For more information try --help"; + +static REQ_GROUP_CONFLICT_REV: &'static str = "error: The argument '--delete' cannot be used with 'base' + +USAGE: + clap-test + +For more information try --help"; + +#[test] +fn required_group_missing_arg() { + let result = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color 'some other flag'") + .group(ArgGroup::with_name("req") + .args(&["flag", "color"]) + .required(true)) + .get_matches_from_safe(vec![""]); + assert!(result.is_err()); + let err = result.err().unwrap(); + assert_eq!(err.kind, ErrorKind::MissingRequiredArgument); +} + +#[test] +#[should_panic] +fn non_existing_arg() { + let _ = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color 'some other flag'") + .group(ArgGroup::with_name("req") + .args(&["flg", "color"]) + .required(true)) + .get_matches_from_safe(vec![""]); +} + +#[test] +#[should_panic(expected = "The group 'c' contains the arg 'd' that doesn't actually exist.")] +fn non_existing_arg_in_subcommand_help() { + let _ = App::new("a") + .subcommand( + SubCommand::with_name("b") + .group( + ArgGroup::with_name("c") + .args(&["d"]) + .required(true), + ) + ).get_matches_from_safe(vec!["a", "help", "b"]); +} + +#[test] +fn group_single_value() { + let res = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color [color] 'some option'") + .group(ArgGroup::with_name("grp") + .args(&["flag", "color"])) + .get_matches_from_safe(vec!["", "-c", "blue"]); + assert!(res.is_ok()); + + let m = res.unwrap(); + assert!(m.is_present("grp")); + assert_eq!(m.value_of("grp").unwrap(), "blue"); +} + +#[test] +fn group_single_flag() { + let res = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color [color] 'some option'") + .group(ArgGroup::with_name("grp") + .args(&["flag", "color"])) + .get_matches_from_safe(vec!["", "-f"]); + assert!(res.is_ok()); + + let m = res.unwrap(); + assert!(m.is_present("grp")); + assert!(m.value_of("grp").is_none()); +} + +#[test] +fn group_empty() { + let res = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color [color] 'some option'") + .group(ArgGroup::with_name("grp") + .args(&["flag", "color"])) + .get_matches_from_safe(vec![""]); + assert!(res.is_ok()); + + let m = res.unwrap(); + assert!(!m.is_present("grp")); + assert!(m.value_of("grp").is_none()); +} + +#[test] +fn group_reqired_flags_empty() { + let result = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color 'some option'") + .group(ArgGroup::with_name("grp") + .required(true) + .args(&["flag", "color"])) + .get_matches_from_safe(vec![""]); + assert!(result.is_err()); + let err = result.err().unwrap(); + assert_eq!(err.kind, ErrorKind::MissingRequiredArgument); +} + +#[test] +fn group_multi_value_single_arg() { + let res = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color [color]... 'some option'") + .group(ArgGroup::with_name("grp") + .args(&["flag", "color"])) + .get_matches_from_safe(vec!["", "-c", "blue", "red", "green"]); + assert!(res.is_ok(), "{:?}", res.unwrap_err().kind); + + let m = res.unwrap(); + assert!(m.is_present("grp")); + assert_eq!(&*m.values_of("grp").unwrap().collect::>(), &["blue", "red", "green"]); +} + +#[test] +fn empty_group() { + let r = App::new("empty_group") + .arg(Arg::from_usage("-f, --flag 'some flag'")) + .group(ArgGroup::with_name("vers") + .required(true)) + .get_matches_from_safe(vec!["empty_prog"]); + assert!(r.is_err()); + let err = r.err().unwrap(); + assert_eq!(err.kind, ErrorKind::MissingRequiredArgument); +} + +#[test] +fn req_group_usage_string() { + let app = App::new("req_group") + .args_from_usage("[base] 'Base commit' + -d, --delete 'Remove the base commit information'") + .group(ArgGroup::with_name("base_or_delete") + .args(&["base", "delete"]) + .required(true)); + + assert!(test::compare_output(app, "clap-test", REQ_GROUP_USAGE, true)); +} + +#[test] +fn req_group_with_conflict_usage_string() { + let app = App::new("req_group") + .arg(Arg::from_usage("[base] 'Base commit'").conflicts_with("delete")) + .arg(Arg::from_usage("-d, --delete 'Remove the base commit information'")) + .group(ArgGroup::with_name("base_or_delete") + .args(&["base", "delete"]) + .required(true)); + + assert!(test::compare_output2(app, "clap-test --delete base", REQ_GROUP_CONFLICT_REV, REQ_GROUP_CONFLICT_USAGE, true)); +} + +#[test] +fn required_group_multiple_args() { + let result = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color 'some other flag'") + .group(ArgGroup::with_name("req") + .args(&["flag", "color"]) + .required(true) + .multiple(true)) + .get_matches_from_safe(vec!["group", "-f", "-c"]); + assert!(result.is_ok()); + let m = result.unwrap(); + assert!(m.is_present("flag")); + assert!(m.is_present("color")); +} + +#[test] +fn group_multiple_args_error() { + let result = App::new("group") + .args_from_usage("-f, --flag 'some flag' + -c, --color 'some other flag'") + .group(ArgGroup::with_name("req") + .args(&["flag", "color"])) + .get_matches_from_safe(vec!["group", "-f", "-c"]); + assert!(result.is_err()); + let err = result.unwrap_err(); + assert_eq!(err.kind, ErrorKind::ArgumentConflict); +} -- cgit v1.2.1