From 44de4de118f21a6a56368a9a91a9a1215e1913b7 Mon Sep 17 00:00:00 2001
From: Henry Fredrick Schreiner <henry.fredrick.schreiner@cern.ch>
Date: Fri, 24 Nov 2017 20:35:26 -0500
Subject: [PATCH] Adding test, fixing option name to single name in message

---
 include/CLI/Option.hpp |  2 +-
 tests/AppTest.cpp      | 25 +++++++++++++++++++++++--
 2 files changed, 24 insertions(+), 3 deletions(-)

diff --git a/include/CLI/Option.hpp b/include/CLI/Option.hpp
index bec77ad3..a71f5fac 100644
--- a/include/CLI/Option.hpp
+++ b/include/CLI/Option.hpp
@@ -437,7 +437,7 @@ class Option : public OptionBase<Option> {
                 for(const std::function<std::string(std::string &)> &vali : validators_) {
                     std::string err_msg = vali(result);
                     if(!err_msg.empty())
-                        throw ValidationError(get_name() + ": " + err_msg);
+                        throw ValidationError(single_name() + ": " + err_msg);
                 }
         }
 
diff --git a/tests/AppTest.cpp b/tests/AppTest.cpp
index 6c8649ab..01105682 100644
--- a/tests/AppTest.cpp
+++ b/tests/AppTest.cpp
@@ -1172,8 +1172,8 @@ TEST_F(TApp, SetWithDefaultsIC) {
     EXPECT_THROW(run(), CLI::ConversionError);
 }
 
-// Added to test defaults on dual method
-TEST_F(TApp, OrderedModifingValidators) {
+// Added to test ->transform
+TEST_F(TApp, OrderedModifingTransforms) {
     std::vector<std::string> val;
     auto m = app.add_option("-m", val);
     m->transform([](std::string x) { return x + "1"; });
@@ -1185,3 +1185,24 @@ TEST_F(TApp, OrderedModifingValidators) {
 
     EXPECT_EQ(val, std::vector<std::string>({"one12", "two12"}));
 }
+
+TEST_F(TApp, ThrowingTransform) {
+    std::string val;
+    auto m = app.add_option("-m,--mess", val);
+    m->transform([](std::string x) -> std::string { throw CLI::ValidationError("My Message"); });
+
+    EXPECT_NO_THROW(run());
+    app.reset();
+
+    args = {"-mone"};
+
+    ASSERT_THROW(run(), CLI::ValidationError);
+
+    app.reset();
+
+    try {
+        run();
+    } catch(const CLI::ValidationError &e) {
+        EXPECT_EQ(e.what(), std::string("--mess: My Message"));
+    }
+}