test: Add tests for ArgParser::splitInputOptions

* Integrate Google testing framework into the project.
* Add simple unit tests for ArgParser::splitInputOptions.

Signed-off-by: Alexander Kurbatov <Alexander.Kurbatov@acronis.com>

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.1)
+cmake_minimum_required(VERSION 3.10)
 
 project(Observer)
 
@@ -15,5 +15,10 @@ set(BUILD_API_EXAMPLES OFF CACHE INTERNAL "" FORCE)
 # Disable building of tests in the cpp-sc2 submodule.
 set(BUILD_API_TESTS OFF CACHE INTERNAL "" FORCE)
 
+# Search for Google test suite.
+find_package(GTest REQUIRED)
+include_directories(${GTEST_INCLUDE_DIRS})
+
 add_subdirectory("cpp-sc2")
 add_subdirectory("src")
+add_subdirectory("test")

README.md

@@ -11,10 +11,12 @@ Starcraft 2 replays auto observer inspired by the [SSCAIT-ObserverModule](https:
   * Windows: C:\Program Files\StarCraft II\Maps
   * OS X: /Applications/StarCraft II/Maps
 
-3. Download and install [CMake](https://cmake.org/download/).
+3. Download and install [CMake >= 3.10](https://cmake.org/download/).
 
 4. A compiler with C++17 support.
 
+5. Install [Google test >= 1.10.0](https://github.com/google/googletest)
+
 5. Windows: Download and install [Visual Studio](https://www.visualstudio.com/downloads/)
 
 6. OS X: Install XCode.

test/ArgParser.test.cpp

@@ -0,0 +1,163 @@
+#include "src/ArgParser.h"
+
+#include <gmock/gmock.h>
+#include <gtest/gtest.h>
+
+#include <vector>
+
+TEST(splitInputOptions, ProperlySplitsObserverAndSC2Args) {
+	std::vector<char*> observer_options;
+	std::vector<char*> sc2_options;
+
+	const char* argv [] = {
+		"./bin/Observer",
+		"--Path",
+		"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay",
+		"--",
+		"-d",
+		"B89B5D6FA7CBF6452E721311BFBC6CB2",
+		"-e",
+		"/Applications/StarCraft II/Versions/Base75689/SC2.app/Contents/MacOS/SC2"
+	};
+	int argc = sizeof argv / sizeof argv[0];
+
+	splitInputOptions(argc, const_cast<char**>(argv), &observer_options, &sc2_options);
+
+	ASSERT_THAT(
+		observer_options,
+		testing::ElementsAre(
+			"./bin/Observer",
+			"--Path",
+			"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay"
+		)
+	);
+
+	ASSERT_THAT(
+		sc2_options,
+		testing::ElementsAre(
+			"./bin/Observer",
+			"-d",
+			"B89B5D6FA7CBF6452E721311BFBC6CB2",
+			"-e",
+			"/Applications/StarCraft II/Versions/Base75689/SC2.app/Contents/MacOS/SC2"
+		)
+	);
+}
+
+TEST(splitInputOptions, DoesntSplitAnythingIfDelimiterNotProvided) {
+	std::vector<char*> observer_options;
+	std::vector<char*> sc2_options;
+
+	const char* argv [] = {
+		"./bin/Observer",
+		"--Path",
+		"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay"
+	};
+	int argc = sizeof argv / sizeof argv[0];
+
+	splitInputOptions(argc, const_cast<char**>(argv), &observer_options, &sc2_options);
+
+	ASSERT_THAT(
+		observer_options,
+		testing::ElementsAre(
+			"./bin/Observer",
+			"--Path",
+			"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay"
+		)
+	);
+
+	ASSERT_THAT(
+		sc2_options,
+		testing::ElementsAre(
+			"./bin/Observer"
+		)
+	);
+}
+
+TEST(splitInputOptions, ReturnsObserverArgsIfDelimiterPassedWithoutSC2Options) {
+	std::vector<char*> observer_options;
+	std::vector<char*> sc2_options;
+
+	const char* argv [] = {
+		"./bin/Observer",
+		"--Path",
+		"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay",
+		"--"
+	};
+	int argc = sizeof argv / sizeof argv[0];
+
+	splitInputOptions(argc, const_cast<char**>(argv), &observer_options, &sc2_options);
+
+	ASSERT_THAT(
+		observer_options,
+		testing::ElementsAre(
+			"./bin/Observer",
+			"--Path",
+			"/Users/alkurbatov/Downloads/358809_TyrZ_DoogieHowitzer_IceandChromeLE.SC2Replay"
+		)
+	);
+
+	ASSERT_THAT(
+		sc2_options,
+		testing::ElementsAre(
+			"./bin/Observer"
+		)
+	);
+}
+
+TEST(splitInputOptions, ReturnsSC2ArgsIfDelimiterPassedWithoutObserverOptions) {
+	std::vector<char*> observer_options;
+	std::vector<char*> sc2_options;
+
+	const char* argv [] = {
+		"./bin/Observer",
+		"--",
+		"-d",
+		"B89B5D6FA7CBF6452E721311BFBC6CB2"
+	};
+	int argc = sizeof argv / sizeof argv[0];
+
+	splitInputOptions(argc, const_cast<char**>(argv), &observer_options, &sc2_options);
+
+	ASSERT_THAT(
+		observer_options,
+		testing::ElementsAre(
+			"./bin/Observer"
+		)
+	);
+
+	ASSERT_THAT(
+		sc2_options,
+		testing::ElementsAre(
+			"./bin/Observer",
+			"-d",
+			"B89B5D6FA7CBF6452E721311BFBC6CB2"
+		)
+	);
+}
+
+TEST(splitInputOptions, DoesntFailIfNoOptionsSpecified) {
+	std::vector<char*> observer_options;
+	std::vector<char*> sc2_options;
+
+	const char* argv [] = {
+		"./bin/Observer"
+	};
+	int argc = sizeof argv / sizeof argv[0];
+
+	splitInputOptions(argc, const_cast<char**>(argv), &observer_options, &sc2_options);
+
+	ASSERT_THAT(
+		observer_options,
+		testing::ElementsAre(
+			"./bin/Observer"
+		)
+	);
+
+	ASSERT_THAT(
+		sc2_options,
+		testing::ElementsAre(
+			"./bin/Observer"
+		)
+	);
+}

test/CMakeLists.txt

@@ -0,0 +1,11 @@
+include(GoogleTest)
+
+include_directories(${PROJECT_SOURCE_DIR})
+
+add_executable(
+    TestArgParser
+    ${PROJECT_SOURCE_DIR}/src/ArgParser.h
+    ${PROJECT_SOURCE_DIR}/src/ArgParser.cpp
+    ArgParser.test.cpp)
+target_link_libraries(TestArgParser GTest::Main)
+gtest_discover_tests(TestArgParser)