Zebediah Figura : winmm/tests: Test the mmioOpen() search path with various flags.

Alexandre Julliard julliard at winehq.org
Fri Dec 18 12:52:54 CST 2020


Module: wine
Branch: master
Commit: 45c98bf0b07d2291db87fd3b8906b144a3ca0703
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=45c98bf0b07d2291db87fd3b8906b144a3ca0703

Author: Zebediah Figura <z.figura12 at gmail.com>
Date:   Thu Dec 17 16:40:49 2020 -0600

winmm/tests: Test the mmioOpen() search path with various flags.

Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/winmm/tests/mmio.c | 60 ++++++++++++++++++++++++++++++++++++++++++-------
 1 file changed, 52 insertions(+), 8 deletions(-)

diff --git a/dlls/winmm/tests/mmio.c b/dlls/winmm/tests/mmio.c
index e3984fbaf3a..0236855f246 100644
--- a/dlls/winmm/tests/mmio.c
+++ b/dlls/winmm/tests/mmio.c
@@ -478,23 +478,34 @@ static void test_mmioOpen_create(void)
     WCHAR cwd[MAX_PATH], temp_dir[MAX_PATH];
     /* According to docs, filename must be no more than 128 bytes, but it will
      * actually allow longer than that. */
-    WCHAR filename[] = L"very_long_filename_"
+    WCHAR long_filename[] = L"very_long_filename_"
         L"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
         L"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
         L"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx";
+    WCHAR buffer[MAX_PATH], expect[MAX_PATH];
+    char exedir_filename[MAX_PATH];
+    MMIOINFO info = {0};
+    BOOL ret;
+    FILE *f;
+
+    GetModuleFileNameA(NULL, exedir_filename, ARRAY_SIZE(exedir_filename));
+    strcpy(strrchr(exedir_filename, '\\') + 1, "test_mmio_path");
+    f = fopen(exedir_filename, "w");
+    ok(!!f, "failed to create %s: %s\n", debugstr_a(exedir_filename), strerror(errno));
+    fclose(f);
 
     GetCurrentDirectoryW(ARRAY_SIZE(cwd), cwd);
     GetTempPathW(ARRAY_SIZE(temp_dir), temp_dir);
     SetCurrentDirectoryW(temp_dir);
 
-    DeleteFileW(filename);
+    DeleteFileW(long_filename);
 
     /* open with MMIO_DENYNONE */
-    hmmio = mmioOpenW(filename, NULL, MMIO_CREATE | MMIO_WRITE | MMIO_DENYNONE);
+    hmmio = mmioOpenW(long_filename, NULL, MMIO_CREATE | MMIO_WRITE | MMIO_DENYNONE);
     ok(hmmio != NULL, "mmioOpen failed\n");
 
     /* MMIO_DENYNONE lets us open it here, too */
-    handle = CreateFileW(filename, GENERIC_READ,
+    handle = CreateFileW(long_filename, GENERIC_READ,
             FILE_SHARE_WRITE | FILE_SHARE_READ | FILE_SHARE_DELETE, NULL,
             OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
     ok(handle != INVALID_HANDLE_VALUE, "Couldn't open non-exclusive file\n");
@@ -502,14 +513,14 @@ static void test_mmioOpen_create(void)
 
     mmioClose(hmmio, 0);
 
-    DeleteFileW(filename);
+    DeleteFileW(long_filename);
 
     /* open with MMIO_EXCLUSIVE */
-    hmmio = mmioOpenW(filename, NULL, MMIO_CREATE | MMIO_WRITE | MMIO_EXCLUSIVE);
+    hmmio = mmioOpenW(long_filename, NULL, MMIO_CREATE | MMIO_WRITE | MMIO_EXCLUSIVE);
     ok(hmmio != NULL, "mmioOpen failed\n");
 
     /* should fail due to MMIO_EXCLUSIVE */
-    handle = CreateFileW(filename, GENERIC_READ,
+    handle = CreateFileW(long_filename, GENERIC_READ,
             FILE_SHARE_WRITE | FILE_SHARE_READ | FILE_SHARE_DELETE, NULL,
             OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
     ok(handle == INVALID_HANDLE_VALUE, "Opening exclusive file should have failed\n");
@@ -518,9 +529,42 @@ static void test_mmioOpen_create(void)
 
     mmioClose(hmmio, 0);
 
-    DeleteFileW(filename);
+    DeleteFileW(long_filename);
+
+    wcscpy(buffer, L"test_mmio_path");
+    hmmio = mmioOpenW(buffer, &info, MMIO_WRITE);
+    todo_wine ok(!!hmmio, "failed to open file, error %#x\n", info.wErrorRet);
+    mmioClose(hmmio, 0);
+
+    wcscpy(buffer, L"test_mmio_path");
+    hmmio = mmioOpenW(buffer, &info, MMIO_PARSE);
+    ok(hmmio == (HMMIO)TRUE, "failed to parse file name, error %#x\n", info.wErrorRet);
+    wcscpy(expect, temp_dir);
+    wcscat(expect, L"test_mmio_path");
+    todo_wine ok(!wcscmp(buffer, expect), "expected %s, got %s\n", debugstr_w(expect), debugstr_w(buffer));
+
+    wcscpy(buffer, L"test_mmio_path");
+    info.wErrorRet = 0xdead;
+    hmmio = mmioOpenW(buffer, &info, MMIO_EXIST);
+    ok(hmmio == (HMMIO)FALSE, "file should exist\n");
+    todo_wine ok(info.wErrorRet == MMIOERR_FILENOTFOUND, "got error %#x\n", info.wErrorRet);
+
+    ret = DeleteFileA("test_mmio_path");
+    ok(!ret, "expected failure\n");
+    ok(GetLastError() == ERROR_FILE_NOT_FOUND, "got error %u\n", GetLastError());
+
+    wcscpy(buffer, L"test_mmio_path");
+    hmmio = mmioOpenW(buffer, &info, MMIO_WRITE | MMIO_CREATE);
+    ok(!!hmmio, "failed to open file, error %#x\n", info.wErrorRet);
+    mmioClose(hmmio, 0);
+
+    ret = DeleteFileA("test_mmio_path");
+    ok(ret, "got error %u\n", GetLastError());
 
     SetCurrentDirectoryW(cwd);
+
+    ret = DeleteFileA(exedir_filename);
+    ok(ret, "got error %u\n", GetLastError());
 }
 
 static void test_mmioSetBuffer(char *fname)




More information about the wine-cvs mailing list