[SHLWAPI_APITEST]
authorHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Fri, 13 Jan 2017 21:55:25 +0000 (21:55 +0000)
committerHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Fri, 13 Jan 2017 21:55:25 +0000 (21:55 +0000)
- Add tests for PathIsUNCServer and PathIsUNCServerShare, by Jared Smudde. Thanks! ROSTESTS-258 #resolve
- Simplify the code: Since those functions (as well as PathIsUNC) are public and exported by name, we can directly load the apitest with shlwapi and call the APIs directly (no need for explicit LoadLibrary & GetProcAddress).

svn path=/trunk/; revision=73544

rostests/apitests/shlwapi/CMakeLists.txt
rostests/apitests/shlwapi/PathIsUNC.c
rostests/apitests/shlwapi/PathIsUNCServer.c [new file with mode: 0644]
rostests/apitests/shlwapi/PathIsUNCServerShare.c [new file with mode: 0644]
rostests/apitests/shlwapi/testlist.c

index c0d9f64..8975ec5 100644 (file)
@@ -1,6 +1,8 @@
 
 list(APPEND SOURCE
     PathIsUNC.c
+    PathIsUNCServer.c
+    PathIsUNCServerShare.c
     PathUnExpandEnvStrings.c
     testlist.c)
 
index ef37c56..3b9d786 100644 (file)
 /* Documentation: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773712(v=vs.85).aspx */
 
 #include <apitest.h>
+#include <shlwapi.h>
 
-static BOOL (WINAPI *pPathIsUNC)(PCWSTR);
-
-#define CALL_ISUNC(exp, str) \
+#define DO_TEST(exp, str) \
 do { \
-    BOOL ret = pPathIsUNC((str)); \
+    BOOL ret = PathIsUNCW((str)); \
     ok(ret == (exp), "Expected %s to be %d, was %d\n", wine_dbgstr_w((str)), (exp), ret); \
 } while (0)
 
 START_TEST(isuncpath)
 {
-    HMODULE hDll = LoadLibraryA("shlwapi.dll");
-
-    pPathIsUNC = (void*)GetProcAddress(hDll, "PathIsUNCW");
-    if (!hDll || !pPathIsUNC)
-    {
-        skip("shlwapi.dll or export PathIsUNCW not found! Tests will be skipped\n");
-        return;
-    }
-
-    CALL_ISUNC(TRUE, L"\\\\path1\\path2");
-    CALL_ISUNC(TRUE, L"\\\\path1");
-    CALL_ISUNC(FALSE, L"reactos\\path4\\path5");
-    CALL_ISUNC(TRUE, L"\\\\");
-    CALL_ISUNC(TRUE, L"\\\\?\\UNC\\path1\\path2");
-    CALL_ISUNC(TRUE, L"\\\\?\\UNC\\path1");
-    CALL_ISUNC(TRUE, L"\\\\?\\UNC\\");
-    CALL_ISUNC(FALSE, L"\\path1");
-    CALL_ISUNC(FALSE, L"path1");
-    CALL_ISUNC(FALSE, L"c:\\path1");
+    DO_TEST(TRUE, L"\\\\path1\\path2");
+    DO_TEST(TRUE, L"\\\\path1");
+    DO_TEST(FALSE, L"reactos\\path4\\path5");
+    DO_TEST(TRUE, L"\\\\");
+    DO_TEST(TRUE, L"\\\\?\\UNC\\path1\\path2");
+    DO_TEST(TRUE, L"\\\\?\\UNC\\path1");
+    DO_TEST(TRUE, L"\\\\?\\UNC\\");
+    DO_TEST(FALSE, L"\\path1");
+    DO_TEST(FALSE, L"path1");
+    DO_TEST(FALSE, L"c:\\path1");
 
     /* MSDN says FALSE but the test shows TRUE on Windows 2003, but returns FALSE on Windows 7 */
-    CALL_ISUNC(TRUE, L"\\\\?\\c:\\path1");
-
-    CALL_ISUNC(TRUE, L"\\\\path1\\");
-    CALL_ISUNC(FALSE, L"//");
-    CALL_ISUNC(FALSE, L"////path1");
-    CALL_ISUNC(FALSE, L"////path1//path2");
-    CALL_ISUNC(FALSE, L"reactos//path3//path4");
-    CALL_ISUNC(TRUE, L"\\\\reactos\\?");
-    CALL_ISUNC(TRUE, L"\\\\reactos\\\\");
-    CALL_ISUNC(FALSE, (wchar_t*)NULL);
-    CALL_ISUNC(FALSE, L" ");
+    DO_TEST(TRUE, L"\\\\?\\c:\\path1");
+
+    DO_TEST(TRUE, L"\\\\path1\\");
+    DO_TEST(FALSE, L"//");
+    DO_TEST(FALSE, L"////path1");
+    DO_TEST(FALSE, L"////path1//path2");
+    DO_TEST(FALSE, L"reactos//path3//path4");
+    DO_TEST(TRUE, L"\\\\reactos\\?");
+    DO_TEST(TRUE, L"\\\\reactos\\\\");
+    DO_TEST(FALSE, (wchar_t*)NULL);
+    DO_TEST(FALSE, L" ");
 
     /* The test shows TRUE on Windows 2003, but returns FALSE on Windows 7 */
-    CALL_ISUNC(TRUE, L"\\\\?\\");
+    DO_TEST(TRUE, L"\\\\?\\");
 }
diff --git a/rostests/apitests/shlwapi/PathIsUNCServer.c b/rostests/apitests/shlwapi/PathIsUNCServer.c
new file mode 100644 (file)
index 0000000..8827d7d
--- /dev/null
@@ -0,0 +1,44 @@
+/*
+ * Copyright 2017 Jared Smudde
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+/* Documentation: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773722(v=vs.85).aspx */
+
+#include <apitest.h>
+#include <shlwapi.h>
+
+#define DO_TEST(exp, str) \
+do { \
+    BOOL ret = PathIsUNCServerW((str)); \
+    ok(ret == (exp), "Expected %s to be %d, was %d\n", wine_dbgstr_w((str)), (exp), ret); \
+} while (0)
+
+START_TEST(isuncpathserver)
+{
+    DO_TEST(TRUE, L"\\\\server");
+    DO_TEST(TRUE, L"\\\\");
+    DO_TEST(FALSE, L"\\\\server\\folder");
+    DO_TEST(FALSE, L"reactos\\some\\folder");
+    DO_TEST(FALSE, L"////server//share");
+    DO_TEST(FALSE, L"c:\\path1");
+    DO_TEST(FALSE, (wchar_t*)NULL);
+    DO_TEST(FALSE, L"");
+    DO_TEST(FALSE, L" ");
+
+    /* The test shows TRUE on Windows 2003, but returns FALSE on Windows 7 */
+    DO_TEST(TRUE, L"\\\\?");
+}
diff --git a/rostests/apitests/shlwapi/PathIsUNCServerShare.c b/rostests/apitests/shlwapi/PathIsUNCServerShare.c
new file mode 100644 (file)
index 0000000..f4f1f86
--- /dev/null
@@ -0,0 +1,42 @@
+/*
+ * Copyright 2017 Jared Smudde
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+/* Documentation: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773723(v=vs.85).aspx */
+
+#include <apitest.h>
+#include <shlwapi.h>
+
+#define DO_TEST(exp, str) \
+do { \
+    BOOL ret = PathIsUNCServerShareW((str)); \
+    ok(ret == (exp), "Expected %s to be %d, was %d\n", wine_dbgstr_w((str)), (exp), ret); \
+} while (0)
+
+START_TEST(isuncpathservershare)
+{
+    DO_TEST(TRUE, L"\\\\server\\share");
+    DO_TEST(TRUE, L"\\\\reactos\\folder9");
+    DO_TEST(FALSE, L"\\\\");
+    DO_TEST(FALSE, L"reactos\\some\\folder");
+    DO_TEST(FALSE, L"////server//share");
+    DO_TEST(FALSE, L"c:\\path1");
+    DO_TEST(FALSE, (wchar_t*)NULL);
+    DO_TEST(FALSE, L"");
+    DO_TEST(FALSE, L" ");
+    DO_TEST(FALSE, L"\\\\?");
+}
index fc4559c..7a7015f 100644 (file)
@@ -2,11 +2,15 @@
 #include <apitest.h>
 
 extern void func_isuncpath(void);
+extern void func_isuncpathserver(void);
+extern void func_isuncpathservershare(void);
 extern void func_PathUnExpandEnvStrings(void);
 
 const struct test winetest_testlist[] =
 {
     { "PathIsUNC", func_isuncpath },
+    { "PathIsUNCServer", func_isuncpathserver },
+    { "PathIsUNCServerShare", func_isuncpathservershare },
     { "PathUnExpandEnvStrings", func_PathUnExpandEnvStrings },
     { 0, 0 }
 };