Improve coverage and clarify InputParser behaviour

egecetin · Dec 31, 2024 · f913c59 · f913c59
1 parent 0dc3684
commit f913c59
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/include/utils/InputParser.hpp b/include/utils/InputParser.hpp
@@ -44,7 +44,7 @@ class InputParser {
 	}
 
 	/**
-	 * Gets all command line options
+	 * Gets all command line options. All options should start with '-'
 	 * @return std::vector<std::pair<std::string, std::string>> All command line options
 	 */
 	[[nodiscard]] std::vector<std::pair<std::string, std::string>> getCmdOptions() const

diff --git a/tests/unittests/Utils_UnitTests.cpp b/tests/unittests/Utils_UnitTests.cpp
@@ -112,4 +112,9 @@ TEST(Utils_Tests, InputParserUnitTests)
 
 	ASSERT_EQ("", parser.getCmdOption("--argument2"));
 	ASSERT_TRUE(parser.cmdOptionExists("--argument2"));
+
+	auto options = parser.getCmdOptions();
+	ASSERT_EQ(options.size(), 1);
+	ASSERT_EQ(options[1].first, "--argument2");
+	ASSERT_EQ(options[1].second, "");
 }