From 0973f348d2f652f04f26e90fe7a2657ec60c476f Mon Sep 17 00:00:00 2001
From: Henry Fredrick Schreiner <henry.fredrick.schreiner@cern.ch>
Date: Fri, 9 Jun 2017 13:36:43 -0400
Subject: [PATCH] Adding new tests for red lines

---
 tests/AppTest.cpp | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/tests/AppTest.cpp b/tests/AppTest.cpp
index 5990e99b..dd9a3684 100644
--- a/tests/AppTest.cpp
+++ b/tests/AppTest.cpp
@@ -892,3 +892,43 @@ TEST_F(TApp, CheckSubcomFail) {
 
     EXPECT_THROW(CLI::detail::AppFriend::parse_subcommand(&app, args), CLI::HorribleError);
 }
+
+// Added to test defaults on dual method
+TEST_F(TApp, OptionWithDefaults) {
+    int someint=2;
+    app.add_option("-a", someint, "", true);
+    
+    args = {"-a1", "-a2"};
+    
+    EXPECT_THROW(run(), CLI::ConversionError);
+}
+
+// Added to test defaults on dual method
+TEST_F(TApp, SetWithDefaults) {
+    int someint=2;
+    app.add_set("-a", someint, {1,2,3,4}, "", true);
+    
+    args = {"-a1", "-a2"};
+    
+    EXPECT_THROW(run(), CLI::ConversionError);
+}
+
+// Added to test defaults on dual method
+TEST_F(TApp, SetWithDefaultsConversion) {
+    int someint=2;
+    app.add_set("-a", someint, {1,2,3,4}, "", true);
+    
+    args = {"-a", "hi"};
+    
+    EXPECT_THROW(run(), CLI::ConversionError);
+}
+
+// Added to test defaults on dual method
+TEST_F(TApp, SetWithDefaultsIC) {
+    std::string someint="ho";
+    app.add_set_ignore_case("-a", someint, {"Hi", "Ho"}, "", true);
+    
+    args = {"-aHi", "-aHo"};
+    
+    EXPECT_THROW(run(), CLI::ConversionError);
+}