From 2b3daa690dd8edd51b25ac2ea70ccb2928000fca Mon Sep 17 00:00:00 2001 From: Yoshiya Hinosawa Date: Fri, 8 Dec 2023 17:35:42 +0900 Subject: [PATCH] feat(test): add default to --coverage option (#21510) --- cli/args/flags.rs | 23 +++++++++++++++++++++-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git a/cli/args/flags.rs b/cli/args/flags.rs index 2eed2a183c..9d7e513f72 100644 --- a/cli/args/flags.rs +++ b/cli/args/flags.rs @@ -2161,12 +2161,14 @@ Directory arguments are expanded to all contained files matching the glob .arg( Arg::new("coverage") .long("coverage") - .require_equals(true) .value_name("DIR") + .num_args(0..=1) + .require_equals(true) + .default_missing_value("coverage") .conflicts_with("inspect") .conflicts_with("inspect-wait") .conflicts_with("inspect-brk") - .help("Collect coverage profile data into DIR"), + .help("Collect coverage profile data into DIR. If DIR is not specified, it uses 'coverage/'."), ) .arg( Arg::new("parallel") @@ -7436,6 +7438,23 @@ mod tests { ); } + #[test] + fn test_coverage_default_dir() { + let r = flags_from_vec(svec!["deno", "test", "--coverage"]); + assert_eq!( + r.unwrap(), + Flags { + subcommand: DenoSubcommand::Test(TestFlags { + coverage_dir: Some("coverage".to_string()), + ..TestFlags::default() + }), + type_check_mode: TypeCheckMode::Local, + no_prompt: true, + ..Flags::default() + } + ); + } + #[test] fn bundle_with_cafile() { let r = flags_from_vec(svec![