mirrors/CLI11
1
0
Fork 0
mirror of https://github.com/CLIUtils/CLI11.git synced 2025-04-30 04:33:53 +00:00
Code Issues Releases Wiki Activity

Throw error if subcommand does not exist, better test

Browse Source
This commit is contained in:
Henry Fredrick Schreiner 2017-03-15 09:22:45 -04:00
parent c5cdace048
commit dbe03bea95
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/CLI/App.hpp
View File

@ -540,7 +540,10 @@ public:
for(const auto subcomptr : selected_subcommands_) for(const auto subcomptr : selected_subcommands_)
if(subcomptr->check_name(name)) if(subcomptr->check_name(name))
return true; return true;
return false; for(const App_p &subcomptr : subcommands_)
if(subcomptr->check_name(name))
return false;
throw CLI::OptionNotFound(name);
} }
///@} ///@}

9
tests/SubcommandTest.cpp
View File

@ -50,6 +50,15 @@ TEST_F(TApp, MultiSubFallthrough) {
app.require_subcommand(1); app.require_subcommand(1);
EXPECT_THROW(run(), CLI::RequiredError); EXPECT_THROW(run(), CLI::RequiredError);
app.reset();
args = {"sub1"};
run();
EXPECT_TRUE(app.got_subcommand("sub1"));
EXPECT_FALSE(app.got_subcommand("sub2"));
EXPECT_THROW(app.got_subcommand("sub3"), CLI::OptionNotFound);
} }