Mirrors/clap
2
0
Fork 0
mirror of https://github.com/clap-rs/clap synced 2024-11-10 14:54:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #2633 from epage/value_name

Browse source 
fix(derive): Don't mix implicit and explicit `value_name`s
This commit is contained in:
Pavan Kumar Sunkara 2021-07-30 10:40:33 +01:00 committed by GitHub
commit 75445b974e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 35 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
clap_derive/src/derives/args.rs
View file

@ -231,7 +231,15 @@ pub fn gen_augment(
_ => quote!(),
};
let value_name = attrs.value_name();
let value_name = if attrs.has_method("value_name") {
// `value_name` appends, so don't touch it if the user does.
quote!()
} else {
let value_name = attrs.value_name();
quote_spanned! { func.span()=>
.value_name(#value_name)
}
};
let modifier = match **ty {
Ty::Bool => quote!(),
@ -247,7 +255,7 @@ pub fn gen_augment(
quote_spanned! { ty.span()=>
.takes_value(true)
.value_name(#value_name)
#value_name
#possible_values
#validator
}
@ -255,7 +263,7 @@ pub fn gen_augment(
Ty::OptionOption => quote_spanned! { ty.span()=>
.takes_value(true)
.value_name(#value_name)
#value_name
.multiple_values(false)
.min_values(0)
.max_values(1)
@ -264,7 +272,7 @@ pub fn gen_augment(
Ty::OptionVec => quote_spanned! { ty.span()=>
.takes_value(true)
.value_name(#value_name)
#value_name
.multiple_values(true)
.min_values(0)
#validator
@ -281,7 +289,7 @@ pub fn gen_augment(
quote_spanned! { ty.span()=>
.takes_value(true)
.value_name(#value_name)
#value_name
.multiple_values(true)
#possible_values
#validator
@ -307,7 +315,7 @@ pub fn gen_augment(
quote_spanned! { ty.span()=>
.takes_value(true)
.value_name(#value_name)
#value_name
.required(#required)
#possible_values
#validator

22
clap_derive/tests/arguments.rs
View file

@ -86,7 +86,7 @@ fn arguments_safe() {
}
#[test]
fn value_name() {
fn auto_value_name() {
#[derive(Clap, PartialEq, Debug)]
struct Opt {
my_special_arg: i32,
@ -97,4 +97,24 @@ fn value_name() {
let help = String::from_utf8(help).unwrap();
assert!(help.contains("MY_SPECIAL_ARG"));
// Ensure the implicit `num_vals` is just 1
assert_eq!(Opt { my_special_arg: 10 }, Opt::parse_from(&["test", "10"]));
}
#[test]
fn explicit_value_name() {
#[derive(Clap, PartialEq, Debug)]
struct Opt {
#[clap(value_name = "BROWNIE_POINTS")]
my_special_arg: i32,
}
let mut help = Vec::new();
Opt::into_app().write_help(&mut help).unwrap();
let help = String::from_utf8(help).unwrap();
assert!(help.contains("BROWNIE_POINTS"));
assert!(!help.contains("MY_SPECIAL_ARG"));
// Ensure the implicit `num_vals` is just 1
assert_eq!(Opt { my_special_arg: 10 }, Opt::parse_from(&["test", "10"]));
}