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

Add test for non-const version of App::get_options (#337)

Browse Source 
After https://github.com/CLIUtils/CLI11/pull/329 the get_options version being used in the test was the non-const.
Forced testing on non-const by using a const-ref to the app.
Also added testing to ensure the list of options of both const and non-const are the same as expected.
This commit is contained in:
Jesus Briales 2019-10-27 17:53:54 +00:00 committed by Henry Schreiner
parent 2bea3983c0
commit 41d3c967d7
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tests/CreationTest.cpp
View File

@ -564,11 +564,17 @@ TEST_F(TApp, GetOptionList) {
auto flag = app.add_flag("--one"); auto flag = app.add_flag("--one");
auto opt = app.add_option("--two", two); auto opt = app.add_option("--two", two);
auto opt_list = app.get_options(); const CLI::App &const_app = app; // const alias to force use of const-methods
std::vector<const CLI::Option *> opt_list = const_app.get_options();
ASSERT_EQ(opt_list.size(), static_cast<size_t>(3)); ASSERT_EQ(opt_list.size(), static_cast<size_t>(3));
EXPECT_EQ(opt_list.at(1), flag); EXPECT_EQ(opt_list.at(1), flag);
EXPECT_EQ(opt_list.at(2), opt); EXPECT_EQ(opt_list.at(2), opt);
std::vector<CLI::Option *> nonconst_opt_list = app.get_options();
for(size_t i = 0; i < opt_list.size(); ++i) {
EXPECT_EQ(nonconst_opt_list.at(i), opt_list.at(i));
}
} }
TEST(ValidatorTests, TestValidatorCreation) { TEST(ValidatorTests, TestValidatorCreation) {