mirror of
https://github.com/clap-rs/clap
synced 2024-12-14 14:52:33 +00:00
4b7ed54d7e
`Parser::parse_from` will call `exit` on failure and we don't just lose backtrace information but we don't even know which of the tests running in parallel panicked. I ran into this when experimenting with `clap_derive` and I couldn't tell what actually failed. So let's switch to `Parse::try_parse_from`. Errors went from: ``` test option_option ... ok error: Found argument 'bar' which wasn't expected, or isn't valid in this context USAGE: clap_derive [OPTIONS] For more information try --help error: test failed, to rerun pass '--test arg_enum' ``` To: ``` test option_option ... ok test variant_with_defined_casing ... ok test skip_variant ... ok test default_value ... ok test vector ... FAILED test option_vector ... ok failures: ---- vector stdout ---- thread 'vector' panicked at 'called `Result::unwrap()` on an `Err` value: Error { message: Formatted(Colorizer { use_stderr: true, color_when: Auto , pieces: [("error:", Some(Red)), (" ", None), ("Found argument '", None), ("bar", Some(Yellow)), ("' which wasn't expected, or isn't valid in this context", None), ("\n\n", None), ("USAGE:\n clap_derive [OPTIONS]", None), ("\n\nFor more information try ", None), ("--help", Some(Green)), (" \n", None)] }), kind: UnknownArgument, info: ["bar"], source: None, backtrace: Backtrace }', clap_derive/tests/arg_enum.rs:388:56 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace failures: vector test result: FAILED. 15 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s error: test failed, to rerun pass '--test arg_enum' ```
156 lines
4 KiB
Rust
156 lines
4 KiB
Rust
// Copyright 2018 Guillaume Pinot (@TeXitoi) <texitoi@texitoi.eu>,
|
|
// Kevin Knapp (@kbknapp) <kbknapp@gmail.com>, and
|
|
// Andrew Hobden (@hoverbear) <andrew@hoverbear.org>
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
|
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
|
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
|
// option. This file may not be copied, modified, or distributed
|
|
// except according to those terms.
|
|
//
|
|
// This work was derived from Structopt (https://github.com/TeXitoi/structopt)
|
|
// commit#ea76fa1b1b273e65e3b0b1046643715b49bec51f which is licensed under the
|
|
// MIT/Apache 2.0 license.
|
|
|
|
use clap::{AppSettings, ErrorKind, Parser};
|
|
use std::num::ParseIntError;
|
|
|
|
pub const DISPLAY_ORDER: usize = 2;
|
|
|
|
// Check if the global settings compile
|
|
#[derive(Parser, Debug, PartialEq, Eq)]
|
|
#[clap(global_setting = AppSettings::AllowLeadingHyphen)]
|
|
struct Opt {
|
|
#[clap(
|
|
long = "x",
|
|
display_order = DISPLAY_ORDER,
|
|
next_line_help = true,
|
|
default_value = "0",
|
|
require_equals = true
|
|
)]
|
|
x: i32,
|
|
|
|
#[clap(short = 'l', long = "level", aliases = &["set-level", "lvl"])]
|
|
level: String,
|
|
|
|
#[clap(long("values"))]
|
|
values: Vec<i32>,
|
|
|
|
#[clap(name = "FILE", requires_if("FILE", "values"))]
|
|
files: Vec<String>,
|
|
}
|
|
|
|
#[test]
|
|
fn test_slice() {
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: Vec::new(),
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "-l", "1"]).unwrap()
|
|
);
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: Vec::new(),
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "--level", "1"]).unwrap()
|
|
);
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: Vec::new(),
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "--set-level", "1"]).unwrap()
|
|
);
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: Vec::new(),
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "--lvl", "1"]).unwrap()
|
|
);
|
|
}
|
|
|
|
#[test]
|
|
fn test_multi_args() {
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: vec!["file".to_string()],
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "-l", "1", "file"]).unwrap()
|
|
);
|
|
assert_eq!(
|
|
Opt {
|
|
x: 0,
|
|
level: "1".to_string(),
|
|
files: vec!["FILE".to_string()],
|
|
values: vec![1],
|
|
},
|
|
Opt::try_parse_from(&["test", "-l", "1", "--values", "1", "--", "FILE"]).unwrap()
|
|
);
|
|
}
|
|
|
|
#[test]
|
|
fn test_multi_args_fail() {
|
|
let result = Opt::try_parse_from(&["test", "-l", "1", "--", "FILE"]);
|
|
assert!(result.is_err());
|
|
}
|
|
|
|
#[test]
|
|
fn test_bool() {
|
|
assert_eq!(
|
|
Opt {
|
|
x: 1,
|
|
level: "1".to_string(),
|
|
files: vec![],
|
|
values: vec![],
|
|
},
|
|
Opt::try_parse_from(&["test", "-l", "1", "--x=1"]).unwrap()
|
|
);
|
|
let result = Opt::try_parse_from(&["test", "-l", "1", "--x", "1"]);
|
|
assert!(result.is_err());
|
|
assert_eq!(result.unwrap_err().kind, ErrorKind::NoEquals);
|
|
}
|
|
|
|
fn parse_hex(input: &str) -> Result<u64, ParseIntError> {
|
|
u64::from_str_radix(input, 16)
|
|
}
|
|
|
|
#[derive(Parser, PartialEq, Debug)]
|
|
struct HexOpt {
|
|
#[clap(short, parse(try_from_str = parse_hex))]
|
|
number: u64,
|
|
}
|
|
|
|
#[test]
|
|
fn test_parse_hex_function_path() {
|
|
assert_eq!(
|
|
HexOpt { number: 5 },
|
|
HexOpt::try_parse_from(&["test", "-n", "5"]).unwrap()
|
|
);
|
|
assert_eq!(
|
|
HexOpt {
|
|
number: 0x00ab_cdef
|
|
},
|
|
HexOpt::try_parse_from(&["test", "-n", "abcdef"]).unwrap()
|
|
);
|
|
|
|
let err = HexOpt::try_parse_from(&["test", "-n", "gg"]).unwrap_err();
|
|
assert!(
|
|
err.to_string().contains("invalid digit found in string"),
|
|
"{}",
|
|
err
|
|
);
|
|
}
|