From 11a0f65bae425baf09508e2d57c830f5cad69df5 Mon Sep 17 00:00:00 2001
From: gcontini <1121667+gcontini@users.noreply.github.com>
Date: 周六, 14 12月 2019 09:42:26 +0800
Subject: [PATCH] cleanup and change license generation parameters

---
 test/library/LicenseLocator_test.cpp |  122 +++++++++++++++-------------------------
 1 files changed, 46 insertions(+), 76 deletions(-)

diff --git a/test/library/LicenseLocator_test.cpp b/test/library/LicenseLocator_test.cpp
index 6110f34..6c83fc0 100644
--- a/test/library/LicenseLocator_test.cpp
+++ b/test/library/LicenseLocator_test.cpp
@@ -27,32 +27,30 @@
 using namespace std;
 using namespace boost::filesystem;
 
-static boost::optional<path> find_file(const path& dir_path, const path& file_name) {
+static boost::optional<path> find_file(const path &dir_path, const path &file_name) {
 	const recursive_directory_iterator end;
 	const auto it = find_if(recursive_directory_iterator(dir_path), end,
-		[&file_name](const directory_entry& e) {
-			return e.path().filename() == file_name;
-		});
+							[&file_name](const directory_entry &e) { return e.path().filename() == file_name; });
 	return it == end ? boost::optional<path>() : it->path();
 }
-
 
 /*****************************************************************************
  * Application Folder tests
  *****************************************************************************/
-BOOST_AUTO_TEST_CASE( read_license_near_module ) {
+BOOST_AUTO_TEST_CASE(read_license_near_module) {
 	const string testExeFolder = PROJECT_BINARY_DIR "/test/library";
 	bool exeFileFound = false;
 	string referenceExeFileName;
 	string referenceLicenseFileName;
-	//Verify we're pointing the correct executable, in windows isn't clear where it's built
+	// Verify we're pointing the correct executable, in windows isn't clear where it's built
 #ifdef _WIN32
 	boost::optional<path> exeLocation(find_file(path(testExeFolder), path(BOOST_TEST_MODULE ".exe")));
 	exeFileFound = exeLocation.has_value();
 	if (exeFileFound) {
 		referenceExeFileName = exeLocation.get().string();
-		referenceLicenseFileName = referenceExeFileName.replace(referenceExeFileName.find(BOOST_TEST_MODULE ".exe"),
-			string(BOOST_TEST_MODULE ".exe").size(), BOOST_TEST_MODULE ".lic");
+		referenceLicenseFileName =
+			referenceExeFileName.replace(referenceExeFileName.find(BOOST_TEST_MODULE ".exe"),
+										 string(BOOST_TEST_MODULE ".exe").size(), BOOST_TEST_MODULE ".lic");
 	}
 #else
 	referenceExeFileName = testExeFolder + "/" + BOOST_TEST_MODULE;
@@ -62,7 +60,7 @@
 #endif
 	BOOST_WARN_MESSAGE(!exeFileFound, "File [" + referenceExeFileName + "] NOT found");
 	if (exeFileFound) {
-		//copy test license near module
+		// copy test license near module
 		std::ifstream src(MOCK_LICENSE, std::ios::binary);
 		std::ofstream dst(referenceLicenseFileName, std::ios::binary);
 		dst << src.rdbuf();
@@ -74,15 +72,12 @@
 		BOOST_CHECK(registry.isGood());
 		BOOST_REQUIRE_EQUAL(1, licenseInfos.size());
 		string currentLocation = licenseInfos[0];
-		BOOST_CHECK_MESSAGE(equivalent(path(referenceLicenseFileName),path(currentLocation)),
-			"file " +currentLocation + "found at expected location");
-		string licenseRealContent = applicationFolder.retrieve_license_content(
-			currentLocation);
+		BOOST_CHECK_MESSAGE(equivalent(path(referenceLicenseFileName), path(currentLocation)),
+							"file " + currentLocation + "found at expected location");
+		string licenseRealContent = applicationFolder.retrieve_license_content(currentLocation);
 		src.seekg(0, ios::beg);
-		std::string referenceContent((std::istreambuf_iterator<char>(src)),
-			std::istreambuf_iterator<char>());
-		BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0,
-			"File content is same");
+		std::string referenceContent((std::istreambuf_iterator<char>(src)), std::istreambuf_iterator<char>());
+		BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0, "File content is same");
 		remove(referenceLicenseFileName.c_str());
 	}
 }
@@ -91,126 +86,101 @@
  * External_Definition tests
  *****************************************************************************/
 
-BOOST_AUTO_TEST_CASE( external_definition ) {
-	//an application can define multiple license locations separated by ';'
-	const char *applicationDefinedString =
-	MOCK_LICENSE ";/this/one/doesnt/exist";
+BOOST_AUTO_TEST_CASE(external_definition) {
+	// an application can define multiple license locations separated by ';'
+	const char *applicationDefinedString = MOCK_LICENSE ";/this/one/doesnt/exist";
 
-	//read test license
+	// read test license
 	std::ifstream src(MOCK_LICENSE, std::ios::binary);
-	std::string referenceContent((std::istreambuf_iterator<char>(src)),
-			std::istreambuf_iterator<char>());
+	std::string referenceContent((std::istreambuf_iterator<char>(src)), std::istreambuf_iterator<char>());
 	license::EventRegistry registry;
-	const LicenseLocation licLocation={applicationDefinedString,nullptr};
+	const LicenseLocation licLocation = {applicationDefinedString, nullptr};
 	ExternalDefinition externalDefinition(&licLocation);
 	vector<string> licenseInfos = externalDefinition.license_locations(registry);
 	BOOST_CHECK(registry.isGood());
 	BOOST_CHECK_EQUAL(1, licenseInfos.size());
 	string currentLocation = licenseInfos[0];
-	BOOST_CHECK_MESSAGE(string(MOCK_LICENSE).compare(currentLocation) == 0,
-			"file found at expected location");
-	string licenseRealContent = externalDefinition.retrieve_license_content(
-			currentLocation);
-	BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0,
-			"File content is same");
+	BOOST_CHECK_MESSAGE(string(MOCK_LICENSE).compare(currentLocation) == 0, "file found at expected location");
+	string licenseRealContent = externalDefinition.retrieve_license_content(currentLocation);
+	BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0, "File content is same");
 }
 
 /**
  * The license file doesn't exist. Check that the locator reports the right error
  */
-BOOST_AUTO_TEST_CASE( external_definition_not_found ) {
+BOOST_AUTO_TEST_CASE(external_definition_not_found) {
 	const char *applicationDefinedString = PROJECT_TEST_SRC_DIR "/this/file/doesnt/exist";
 	license::EventRegistry registry;
-	const LicenseLocation licLocation = { applicationDefinedString,nullptr };
+	const LicenseLocation licLocation = {applicationDefinedString, nullptr};
 	ExternalDefinition externalDefinition(&licLocation);
 	vector<string> licenseInfos = externalDefinition.license_locations(registry);
 
-	BOOST_CHECK_MESSAGE(registry.isGood(),
-			"No fatal error for now, only warnings");
+	BOOST_CHECK_MESSAGE(registry.isGood(), "No fatal error for now, only warnings");
 	registry.turnWarningsIntoErrors();
 	BOOST_REQUIRE_MESSAGE(!registry.isGood(), "Error detected");
 	BOOST_CHECK_EQUAL(0, licenseInfos.size());
-	BOOST_CHECK_MESSAGE(
-			registry.getLastFailure()->event_type == LICENSE_FILE_NOT_FOUND,
-			"Error detected");
-
+	BOOST_CHECK_MESSAGE(registry.getLastFailure()->event_type == LICENSE_FILE_NOT_FOUND, "Error detected");
 }
 
 /*****************************************************************************
  * EnvironmentVarLocation tests
  *****************************************************************************/
-BOOST_AUTO_TEST_CASE( environment_var_location ) {
-	//an application can define multiple license locations separated by ';'
-	const char *environment_variable_value =
-	MOCK_LICENSE ";/this/one/doesnt/exist";
+BOOST_AUTO_TEST_CASE(environment_var_location) {
+	// an application can define multiple license locations separated by ';'
+	const char *environment_variable_value = MOCK_LICENSE ";/this/one/doesnt/exist";
 #ifdef _WIN32
 	_putenv_s(LICENSE_LOCATION_ENV_VAR, environment_variable_value);
 #else
 	setenv(LICENSE_LOCATION_ENV_VAR, environment_variable_value, 1);
 #endif
-	//read test license
+	// read test license
 	std::ifstream src(MOCK_LICENSE, std::ios::binary);
-	std::string referenceContent((std::istreambuf_iterator<char>(src)),
-			std::istreambuf_iterator<char>());
+	std::string referenceContent((std::istreambuf_iterator<char>(src)), std::istreambuf_iterator<char>());
 	license::EventRegistry registry;
 
 	EnvironmentVarLocation envVarLocationStrategy;
-	vector<string> licenseInfos = envVarLocationStrategy.license_locations(
-			registry);
+	vector<string> licenseInfos = envVarLocationStrategy.license_locations(registry);
 	BOOST_CHECK(registry.isGood());
 	BOOST_CHECK_EQUAL(1, licenseInfos.size());
 	string currentLocation = licenseInfos[0];
-	BOOST_CHECK_MESSAGE(string(MOCK_LICENSE).compare(currentLocation) == 0,
-			"file found at expected location");
-	string licenseRealContent = envVarLocationStrategy.retrieve_license_content(
-			currentLocation);
-	BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0,
-			"File content is same");
+	BOOST_CHECK_MESSAGE(string(MOCK_LICENSE).compare(currentLocation) == 0, "file found at expected location");
+	string licenseRealContent = envVarLocationStrategy.retrieve_license_content(currentLocation);
+	BOOST_CHECK_MESSAGE(referenceContent.compare(licenseRealContent) == 0, "File content is same");
 	UNSETENV(LICENSE_LOCATION_ENV_VAR);
 }
 
 /**
  * The license file doesn't exist. Check that the locator reports the right error
  */
-BOOST_AUTO_TEST_CASE( environment_var_location_not_found ) {
-	const char *environment_variable_value =
-	PROJECT_TEST_SRC_DIR "/this/file/doesnt/exist";
-    SETENV(LICENSE_LOCATION_ENV_VAR, environment_variable_value);
+BOOST_AUTO_TEST_CASE(environment_var_location_not_found) {
+	const char *environment_variable_value = PROJECT_TEST_SRC_DIR "/this/file/doesnt/exist";
+	SETENV(LICENSE_LOCATION_ENV_VAR, environment_variable_value);
 
 	license::EventRegistry registry;
 	EnvironmentVarLocation envVarLocationStrategy;
-	vector<string> licenseInfos = envVarLocationStrategy.license_locations(
-			registry);
-	BOOST_CHECK_MESSAGE(registry.isGood(),
-			"No fatal error for now, only warnings");
+	vector<string> licenseInfos = envVarLocationStrategy.license_locations(registry);
+	BOOST_CHECK_MESSAGE(registry.isGood(), "No fatal error for now, only warnings");
 	registry.turnWarningsIntoErrors();
 	BOOST_REQUIRE_MESSAGE(!registry.isGood(), "Error detected");
 	BOOST_CHECK_EQUAL(0, licenseInfos.size());
-	BOOST_CHECK_MESSAGE(
-			registry.getLastFailure()->event_type == LICENSE_FILE_NOT_FOUND,
-			"Error detected");
+	BOOST_CHECK_MESSAGE(registry.getLastFailure()->event_type == LICENSE_FILE_NOT_FOUND, "Error detected");
 	UNSETENV(LICENSE_LOCATION_ENV_VAR);
 }
 
 /**
  * The license file doesn't exist. Check that the locator reports the right error
  */
-BOOST_AUTO_TEST_CASE( environment_var_location_not_defined ) {
+BOOST_AUTO_TEST_CASE(environment_var_location_not_defined) {
 	UNSETENV(LICENSE_LOCATION_ENV_VAR);
 	license::EventRegistry registry;
 	EnvironmentVarLocation environmentVarLocation;
-	vector<string> licenseInfos = environmentVarLocation.license_locations(
-			registry);
+	vector<string> licenseInfos = environmentVarLocation.license_locations(registry);
 
-	BOOST_CHECK_MESSAGE(registry.isGood(),
-			"No fatal error for now, only warnings");
+	BOOST_CHECK_MESSAGE(registry.isGood(), "No fatal error for now, only warnings");
 	registry.turnWarningsIntoErrors();
 	BOOST_REQUIRE_MESSAGE(!registry.isGood(), "Error detected");
 	BOOST_CHECK_EQUAL(0, licenseInfos.size());
-	BOOST_CHECK_MESSAGE(
-			registry.getLastFailure()->event_type
-					== ENVIRONMENT_VARIABLE_NOT_DEFINED, "Error detected");
-
+	BOOST_CHECK_MESSAGE(registry.getLastFailure()->event_type == ENVIRONMENT_VARIABLE_NOT_DEFINED, "Error detected");
 }
 
-}  //namespace test
+}  // namespace test

--
Gitblit v1.9.1