From 8e650c3873d2ec2d957f4c216ba2702cd66d0609 Mon Sep 17 00:00:00 2001
From: Henry Fredrick Schreiner <henry.fredrick.schreiner@cern.ch>
Date: Wed, 11 Apr 2018 12:28:57 +0200
Subject: [PATCH] Adding two required functions

---
 CHANGELOG.md           |  2 ++
 include/CLI/App.hpp    | 12 ++++++++++++
 tests/CreationTest.cpp | 12 ++++++++++++
 tests/HelpTest.cpp     |  6 ++++++
 4 files changed, 32 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0262b0e0..6df6c71e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,8 +1,10 @@
 ## In progress
 
 * Make unlimited positionals vs. unlimited options more intuitive [#102]
+* Add missing getters `get_options` and `get_description` to App [#105]
 
 [#102]: https://github.com/CLIUtils/CLI11/issues/102
+[#105]: https://github.com/CLIUtils/CLI11/issues/105
 
 
 ## Version 1.5: Optionals
diff --git a/include/CLI/App.hpp b/include/CLI/App.hpp
index e2c23919..2d2c8f42 100644
--- a/include/CLI/App.hpp
+++ b/include/CLI/App.hpp
@@ -977,6 +977,18 @@ class App {
     /// @name Getters
     ///@{
 
+    /// Get the app or subcommand description
+    std::string get_description() const { return description_; }
+
+    /// Get the list of options (user facing function, so returns raw pointers)
+    std::vector<Option *> get_options() const {
+        std::vector<Option *> options(options_.size());
+        std::transform(std::begin(options_), std::end(options_), std::begin(options), [](const Option_p &val) {
+            return val.get();
+        });
+        return options;
+    }
+
     /// Check the status of ignore_case
     bool get_ignore_case() const { return ignore_case_; }
 
diff --git a/tests/CreationTest.cpp b/tests/CreationTest.cpp
index cee6d3bd..0603837e 100644
--- a/tests/CreationTest.cpp
+++ b/tests/CreationTest.cpp
@@ -404,3 +404,15 @@ TEST_F(TApp, SubcommandMinMax) {
     EXPECT_EQ(app.get_require_subcommand_min(), (size_t)3);
     EXPECT_EQ(app.get_require_subcommand_max(), (size_t)7);
 }
+
+TEST_F(TApp, GetOptionList) {
+    int two;
+    auto flag = app.add_flag("--one");
+    auto opt = app.add_option("--two", two);
+
+    auto opt_list = app.get_options();
+
+    ASSERT_EQ(opt_list.size(), static_cast<size_t>(3));
+    EXPECT_EQ(opt_list.at(1), flag);
+    EXPECT_EQ(opt_list.at(2), opt);
+}
diff --git a/tests/HelpTest.cpp b/tests/HelpTest.cpp
index 8ac69a83..4402a202 100644
--- a/tests/HelpTest.cpp
+++ b/tests/HelpTest.cpp
@@ -477,3 +477,9 @@ TEST(THelp, CustomDoubleOption) {
 
     EXPECT_THAT(app.help(), Not(HasSubstr("x 2")));
 }
+
+TEST(THelp, AccessDescription) {
+    CLI::App app{"My description goes here"};
+
+    EXPECT_EQ(app.get_description(), "My description goes here");
+}