[hbelusca] 73544: [SHLWAPI_APITEST] - Add tests for PathIsUNCServer and PathIsUNCServerShare, by Jared Smudde. Thanks! ROSTESTS-258 #resolve - Simplify the code: Since those functions (as well as P...
Author: hbelusca Date: Fri Jan 13 21:55:25 2017 New Revision: 73544
URL: http://svn.reactos.org/svn/reactos?rev=73544&view=rev Log: [SHLWAPI_APITEST] - 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).
Added: trunk/rostests/apitests/shlwapi/PathIsUNCServer.c (with props) trunk/rostests/apitests/shlwapi/PathIsUNCServerShare.c (with props) Modified: trunk/rostests/apitests/shlwapi/CMakeLists.txt trunk/rostests/apitests/shlwapi/PathIsUNC.c trunk/rostests/apitests/shlwapi/testlist.c
Modified: trunk/rostests/apitests/shlwapi/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/shlwapi/CMakeList... ============================================================================== --- trunk/rostests/apitests/shlwapi/CMakeLists.txt [iso-8859-1] (original) +++ trunk/rostests/apitests/shlwapi/CMakeLists.txt [iso-8859-1] Fri Jan 13 21:55:25 2017 @@ -1,6 +1,8 @@
list(APPEND SOURCE PathIsUNC.c + PathIsUNCServer.c + PathIsUNCServerShare.c PathUnExpandEnvStrings.c testlist.c)
Modified: trunk/rostests/apitests/shlwapi/PathIsUNC.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/shlwapi/PathIsUNC... ============================================================================== --- trunk/rostests/apitests/shlwapi/PathIsUNC.c [iso-8859-1] (original) +++ trunk/rostests/apitests/shlwapi/PathIsUNC.c [iso-8859-1] Fri Jan 13 21:55:25 2017 @@ -19,50 +19,40 @@ /* Documentation: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773712(v=vs.85).a... */
#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"); + DO_TEST(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"\\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"\\?\"); }
Added: trunk/rostests/apitests/shlwapi/PathIsUNCServer.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/shlwapi/PathIsUNC... ============================================================================== --- trunk/rostests/apitests/shlwapi/PathIsUNCServer.c (added) +++ trunk/rostests/apitests/shlwapi/PathIsUNCServer.c [iso-8859-1] Fri Jan 13 21:55:25 2017 @@ -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).a... */ + +#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"\\?"); +}
Propchange: trunk/rostests/apitests/shlwapi/PathIsUNCServer.c ------------------------------------------------------------------------------ svn:eol-style = native
Added: trunk/rostests/apitests/shlwapi/PathIsUNCServerShare.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/shlwapi/PathIsUNC... ============================================================================== --- trunk/rostests/apitests/shlwapi/PathIsUNCServerShare.c (added) +++ trunk/rostests/apitests/shlwapi/PathIsUNCServerShare.c [iso-8859-1] Fri Jan 13 21:55:25 2017 @@ -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).a... */ + +#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"\\?"); +}
Propchange: trunk/rostests/apitests/shlwapi/PathIsUNCServerShare.c ------------------------------------------------------------------------------ svn:eol-style = native
Modified: trunk/rostests/apitests/shlwapi/testlist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/shlwapi/testlist.... ============================================================================== --- trunk/rostests/apitests/shlwapi/testlist.c [iso-8859-1] (original) +++ trunk/rostests/apitests/shlwapi/testlist.c [iso-8859-1] Fri Jan 13 21:55:25 2017 @@ -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 } };
-
hbelusca＠svn.reactos.org