Skip to content

Commit

Permalink
tests: adds tests to protect against panics when using globals and ca…
Browse files Browse the repository at this point in the history
…lling App::get_matches_from_safe_borrow multiple times
  • Loading branch information
kbknapp committed Oct 24, 2017
1 parent d86ec79 commit 2c3f7f6
Showing 1 changed file with 37 additions and 0 deletions.
37 changes: 37 additions & 0 deletions tests/global_args.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
extern crate clap;
extern crate regex;

#[cfg(test)]
mod tests {
include!("../clap-test.rs");
use clap::{App, Arg, SubCommand, ArgMatches};

fn get_app() -> App<'static, 'static> {
App::new("myprog")
.arg(Arg::with_name("GLOBAL_ARG")
.long("global-arg")
.help(
"Specifies something needed by the subcommands",
)
.global(true)
.takes_value(true)
.default_value("default_value"))
.arg(Arg::with_name("GLOBAL_FLAG")
.long("global-flag")
.help(
"Specifies something needed by the subcommands",
)
.multiple(true)
.global(true))
.subcommand(SubCommand::with_name("outer")
.subcommand(SubCommand::with_name("inner")))
}

#[test]
fn issue_1076() {
let mut app = get_app();
app.get_matches_from_safe_borrow(vec!["myprog"]);
app.get_matches_from_safe_borrow(vec!["myprog"]);
app.get_matches_from_safe_borrow(vec!["myprog"]);
}
}

0 comments on commit 2c3f7f6

Please sign in to comment.