kernel32/tests: Split up checks in flush_proc function.

This test is failing intermittently on the testbot but not enough information is output to tell why. Signed-off-by: Alex Henrie <alexhenrie24@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

kernel32/tests: Split up checks in flush_proc function.
d7161a85 · Alex Henrie · Alexandre Julliard · 56a6a039 · d7161a85
Commit d7161a85 authored Dec 12, 2021 by Alex Henrie Committed by Alexandre Julliard Dec 13, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

pipe.c dlls/kernel32/tests/pipe.c +7 -1

No files found.
--- a/dlls/kernel32/tests/pipe.c
+++ b/dlls/kernel32/tests/pipe.c
@@ -2858,9 +2858,15 @@ static DWORD CALLBACK flush_proc(HANDLE pipe)

    res = FlushFileBuffers(pipe);
    if (expected_flush_error == ERROR_SUCCESS)
+    {
        ok(res, "FlushFileBuffers failed: %u\n", GetLastError());
+    }
    else
-        ok(!res && GetLastError() == expected_flush_error, "FlushFileBuffers failed: %u\n", GetLastError());
+    {
+        ok(!res, "FlushFileBuffers should have failed\n");
+        ok(GetLastError() == expected_flush_error,
+           "FlushFileBuffers set error %u, expected %u\n", GetLastError(), expected_flush_error);
+    }
    return 0;
 }