Alex Henrie : kernel32/tests: Split up checks in flush_proc function.
Alexandre Julliard
julliard at winehq.org
Mon Dec 13 15:59:53 CST 2021
Module: wine
Branch: master
Commit: d7161a857652d80dd413613de1e92e8952beaa1f
URL: https://source.winehq.org/git/wine.git/?a=commit;h=d7161a857652d80dd413613de1e92e8952beaa1f
Author: Alex Henrie <alexhenrie24 at gmail.com>
Date: Sun Dec 12 19:04:29 2021 -0700
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 at gmail.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/kernel32/tests/pipe.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/dlls/kernel32/tests/pipe.c b/dlls/kernel32/tests/pipe.c
index 93eeca16c75..5af92f8f8a5 100644
--- 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;
}
More information about the wine-cvs
mailing list