Add --option=value test for #90, as well

CLIUtils · Mar 26, 2018 · 2b4780d · 2b4780d
1 parent aa7c01f
commit 2b4780d
Showing 1 changed file with 46 additions and 24 deletions.
diff --git a/tests/AppTest.cpp b/tests/AppTest.cpp
@@ -415,6 +415,52 @@ TEST_F(TApp, RequiredOptsDoubleNeg) {
     EXPECT_EQ(strs, std::vector<std::string>({"one", "two"}));
 }
 
+// This makes sure unlimited option priority is
+// correct for space vs. no space #90
+TEST_F(TApp, PositionalNoSpace) {
+    std::vector<std::string> options;
+    std::string foo, bar;
+
+    app.add_option("-O", options);
+    app.add_option("foo", foo)->required();
+    app.add_option("bar", bar)->required();
+
+    args = {"-O", "Test", "param1", "param2"};
+    run();
+
+    EXPECT_EQ(options.size(), (size_t)1);
+    EXPECT_EQ(options.at(0), "Test");
+
+    app.reset();
+    args = {"-OTest", "param1", "param2"};
+    run();
+
+    EXPECT_EQ(options.size(), (size_t)1);
+    EXPECT_EQ(options.at(0), "Test");
+}
+
+TEST_F(TApp, PositionalNoSpaceLong) {
+    std::vector<std::string> options;
+    std::string foo, bar;
+
+    app.add_option("--option", options);
+    app.add_option("foo", foo)->required();
+    app.add_option("bar", bar)->required();
+
+    args = {"--option", "Test", "param1", "param2"};
+    run();
+
+    EXPECT_EQ(options.size(), (size_t)1);
+    EXPECT_EQ(options.at(0), "Test");
+
+    app.reset();
+    args = {"--option=Test", "param1", "param2"};
+    run();
+
+    EXPECT_EQ(options.size(), (size_t)1);
+    EXPECT_EQ(options.at(0), "Test");
+}
+
 TEST_F(TApp, RequiredOptsUnlimited) {
 
     std::vector<std::string> strs;
@@ -707,30 +753,6 @@ TEST_F(TApp, BigPositional) {
     EXPECT_EQ(args, vec);
 }
 
-// This makes sure unlimited option priority is
-// correct for space vs. no space #90
-TEST_F(TApp, PositionalNoSpace) {
-    std::vector<std::string> options;
-    std::string foo, bar;
-
-    app.add_option("-O", options);
-    app.add_option("foo", foo)->required();
-    app.add_option("bar", bar)->required();
-
-    args = {"-O", "Test", "param1", "param2"};
-    run();
-
-    EXPECT_EQ(options.size(), (size_t)1);
-    EXPECT_EQ(options.at(0), "Test");
-
-    app.reset();
-    args = {"-OTest", "param1", "param2"};
-    run();
-
-    EXPECT_EQ(options.size(), (size_t)1);
-    EXPECT_EQ(options.at(0), "Test");
-}
-
 TEST_F(TApp, Reset) {
 
     app.add_flag("--simple");