From bbd71d4a8c95092deb9cbf4aef86722a1e9628db Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Tue, 7 Mar 2017 22:02:59 +0300
Subject: Fix unit tests on windows.

---
 src/utils/stringutils_unittest.cc | 53 +++++++++++++++++++++------------------
 1 file changed, 29 insertions(+), 24 deletions(-)

(limited to 'src/utils/stringutils_unittest.cc')

diff --git a/src/utils/stringutils_unittest.cc b/src/utils/stringutils_unittest.cc
index 31e09e4ae..cf0e880aa 100644
--- a/src/utils/stringutils_unittest.cc
+++ b/src/utils/stringutils_unittest.cc
@@ -1256,84 +1256,87 @@ TEST_CASE("stringuntils escapeString")
 TEST_CASE("stringuntils sanitizePath")
 {
     std::string path;
+    const std::string sep = dirSeparator;
     path = "";
     sanitizePath(path);
     REQUIRE(path == "");
     path = "/";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "/\\";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "\\/";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "//";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "///";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "//\\/";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "///\\";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "\\";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "\\\\";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "\\/\\";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "\\\\/";
     sanitizePath(path);
-    REQUIRE(path == "/");
+    REQUIRE(path == dirSeparator);
     path = "test";
     sanitizePath(path);
     REQUIRE(path == "test");
     path = "./test";
     sanitizePath(path);
-    REQUIRE(path == "./test");
+    REQUIRE(path == "." + sep + "test");
     path = "test line";
     sanitizePath(path);
     REQUIRE(path == "test line");
     path = "dir/test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "/dir/test";
     sanitizePath(path);
-    REQUIRE(path == "/dir/test");
+    REQUIRE(path == sep + "dir" + sep + "test");
     path = "dir//test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "dir///test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "dir///\\test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "dir/\\//test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "dir\\test";
     sanitizePath(path);
-    REQUIRE(path == "dir/test");
+    REQUIRE(path == "dir" + sep + "test");
     path = "dir/test/";
     sanitizePath(path);
-    REQUIRE(path == "dir/test/");
+    REQUIRE(path == "dir" + sep + "test" + sep);
     path = "dir/test\\";
     sanitizePath(path);
-    REQUIRE(path == "dir/test/");
+    REQUIRE(path == "dir" + sep + "test" + sep);
     path = "/very\\long/dir\\with\\sepa/ra/tors";
     sanitizePath(path);
-    REQUIRE(path == "/very/long/dir/with/sepa/ra/tors");
+    REQUIRE(path == sep + "very" + sep + "long" + sep + \
+        "dir" + sep + "with" + sep + "sepa" + sep + "ra" + sep + "tors");
     path = "/very\\long/dir\\\\with\\sepa//ra/tors";
     sanitizePath(path);
-    REQUIRE(path == "/very/long/dir/with/sepa/ra/tors");
+    REQUIRE(path == sep + "very" + sep + "long" + sep + \
+        "dir" + sep + "with" + sep + "sepa" + sep + "ra" + sep + "tors");
 }
 
 TEST_CASE("stringuntils secureChatCommand")
@@ -1358,6 +1361,8 @@ TEST_CASE("stringuntils secureChatCommand")
     REQUIRE(str == "_#test");
 }
 
+#ifndef WIN32
+// disabled on windows for now, because no gettext
 TEST_CASE("stringuntils timeDiffToString")
 {
     REQUIRE(timeDiffToString(60 * 60 * 24 * 7) == "1 week");
@@ -1388,10 +1393,10 @@ TEST_CASE("stringuntils timeDiffToString")
     REQUIRE(timeDiffToString(60 * 7
         ) == "7 minutes");
 }
+#endif  // WIN32
 
 TEST_CASE("stringuntils replaceItemLinks")
 {
-    dirSeparator = "/";
     logger = new Logger();
     ResourceManager::init();
     VirtFs::addDirToSearchPathSilent("data", Append_false);
-- 
cgit v1.2.3-70-g09d2