clean up comments in tests; add error case test
@@ -323,7 +323,6 @@ fn required_unless_one_2() {
#[test]
fn required_unless_one_works_with_short() {
// GitHub issue: https://github.com/kbknapp/clap-rs/issues/1135
- // removing `short("x")` will make this test pass
let res = App::new("unlessone")
.arg(Arg::with_name("a").conflicts_with("b").short("a"))
.arg(Arg::with_name("b").short("b"))
@@ -336,10 +335,22 @@ fn required_unless_one_works_with_short() {
assert!(res.is_ok());
}
+#[test]
+fn required_unless_one_works_with_short_err() {
+ let res = App::new("unlessone")
+ .arg(Arg::with_name("a").conflicts_with("b").short("a"))
+ .arg(Arg::with_name("b").short("b"))
+ .arg(
+ Arg::with_name("x")
+ .short("x")
+ .required_unless_one(&["a", "b"])
+ ).get_matches_from_safe(vec!["unlessone"]);
+
+ assert!(!res.is_ok());
+}
+
#[test]
fn required_unless_one_works_without() {
- // GitHub issue: https://github.com/kbknapp/clap-rs/issues/1135
- // removing `short("x")` will make this test pass
let res = App::new("unlessone")
.arg(Arg::with_name("a").conflicts_with("b").short("a"))
.arg(Arg::with_name("b").short("b"))
