[PATCH 0/2] MR3735: user32/tests: Wait for GetMessage call to complete.

[Esme Povirk (＠madewokherd)]

It's possible otherwise for the second message to be sent before
GetMessage removes a posted message from the queue, in which case
GetMessage processes both sent messages.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54407

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3735

[PATCH 1/2] user32/tests: Trace run number in test_SendMessage_other_thread.

[Esme Povirk]

From: Esme Povirk <esme(a)codeweavers.com>

---
 dlls/user32/tests/msg.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index 53f8953902a..912e54921c5 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -19543,6 +19543,8 @@ static void test_SendMessage_other_thread(int thread_n)
     DWORD tid, ret;
     MSG msg;
 
+    winetest_push_context("thread_%i", thread_n);
+
     wnd_event.start_event = CreateEventA(NULL, 0, 0, NULL);
     wnd_event.stop_event = CreateEventA(NULL, 0, 0, NULL);
 
@@ -19620,6 +19622,8 @@ static void test_SendMessage_other_thread(int thread_n)
     flush_events();
     flush_sequence();
 
+    winetest_pop_context();
+
     CloseHandle(wnd_event.start_event);
     CloseHandle(wnd_event.stop_event);
 }
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/3735

[PATCH 2/2] user32/tests: Wait for GetMessage call to complete.

[Esme Povirk]

From: Esme Povirk <esme(a)codeweavers.com>

It's possible otherwise for the second message to be sent before
GetMessage removes a posted message from the queue, in which case
GetMessage processes both sent messages.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54407
---
 dlls/user32/tests/msg.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index 912e54921c5..c1f950e5cb9 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -9861,6 +9861,7 @@ struct wnd_event
     HANDLE grand_child;
     HANDLE start_event;
     HANDLE stop_event;
+    HANDLE getmessage_complete;
 };
 
 static DWORD WINAPI thread_proc(void *param)
@@ -19488,6 +19489,7 @@ static const struct message send_message_3[] = {
 static DWORD WINAPI SendMessage_thread_1(void *param)
 {
     struct wnd_event *wnd_event = param;
+    DWORD ret;
 
     if (winetest_debug > 1) trace("thread: starting\n");
     WaitForSingleObject(wnd_event->start_event, INFINITE);
@@ -19501,6 +19503,8 @@ static DWORD WINAPI SendMessage_thread_1(void *param)
     if (winetest_debug > 1) trace("thread: call SendMessage\n");
     SendMessageA(wnd_event->hwnd, WM_USER+2, 0, 0);
     SetEvent(wnd_event->stop_event);
+    ret = WaitForSingleObject(wnd_event->getmessage_complete, 100);
+    ok(ret == WAIT_OBJECT_0, "WaitForSingleObject failed, ret:%lx\n", ret);
 
     if (winetest_debug > 1) trace("thread: call SendMessage\n");
     SendMessageA(wnd_event->hwnd, WM_USER+3, 0, 0);
@@ -19511,6 +19515,7 @@ static DWORD WINAPI SendMessage_thread_1(void *param)
 static DWORD WINAPI SendMessage_thread_2(void *param)
 {
     struct wnd_event *wnd_event = param;
+    DWORD ret;
 
     if (winetest_debug > 1) trace("thread: starting\n");
     WaitForSingleObject(wnd_event->start_event, INFINITE);
@@ -19528,6 +19533,8 @@ static DWORD WINAPI SendMessage_thread_2(void *param)
     if (winetest_debug > 1) trace("thread: call SendMessage\n");
     SendMessageA(wnd_event->hwnd, WM_USER+2, 0, 0);
     SetEvent(wnd_event->stop_event);
+    ret = WaitForSingleObject(wnd_event->getmessage_complete, 100);
+    ok(ret == WAIT_OBJECT_0, "WaitForSingleObject failed, ret:%lx\n", ret);
 
     if (winetest_debug > 1) trace("thread: call SendMessage\n");
     SendMessageA(wnd_event->hwnd, WM_USER+3, 0, 0);
@@ -19547,6 +19554,7 @@ static void test_SendMessage_other_thread(int thread_n)
 
     wnd_event.start_event = CreateEventA(NULL, 0, 0, NULL);
     wnd_event.stop_event = CreateEventA(NULL, 0, 0, NULL);
+    wnd_event.getmessage_complete = CreateEventA(NULL, 0, 0, NULL);
 
     wnd_event.hwnd = CreateWindowExA(0, "TestWindowClass", NULL, WS_OVERLAPPEDWINDOW,
                                      100, 100, 200, 200, 0, 0, 0, NULL);
@@ -19581,6 +19589,8 @@ static void test_SendMessage_other_thread(int thread_n)
     DispatchMessageA(&msg);
     ok_sequence(send_message_1, "SendMessage from other thread 1", thread_n == 2);
 
+    SetEvent(wnd_event.getmessage_complete);
+
     ret = WaitForSingleObject(wnd_event.stop_event, 100);
     todo_wine_if (thread_n == 2)
     ok(ret == WAIT_OBJECT_0, "WaitForSingleObject failed, ret:%lx\n", ret);
@@ -19626,6 +19636,7 @@ static void test_SendMessage_other_thread(int thread_n)
 
     CloseHandle(wnd_event.start_event);
     CloseHandle(wnd_event.stop_event);
+    CloseHandle(wnd_event.getmessage_complete);
 }
 
 static LRESULT CALLBACK insendmessage_wnd_proc( HWND hwnd, UINT msg, WPARAM wp, LPARAM lp )
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/3735

[PATCH 2/2] user32/tests: Wait for GetMessage call to complete.

[Marvin]

Hi,

It looks like your patch introduced the new failures shown below.
Please investigate and fix them before resubmitting your patch.
If they are not new, fixing them anyway would help a lot. Otherwise
please ask for the known failures list to be updated.

3 TestBot errors prevented a full analysis of your patch.
If the test caused the operating system (e.g. Windows) to crash or
reboot you will probably have to modify it to avoid that.
Other issues should be reported to the TestBot administrators.

The full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=151774

Your paranoid android.


=== build (build log) ===

WineRunBuild.pl:error: Could not copy the patch to the VM: network read timed out (sendfile/connect:AgentVersion.h:0/9)
WineRunBuild.pl:error: BotError: The test VM has crashed, rebooted or lost connectivity (or the TestAgent server died)

=== debian11 (build log) ===

WineRunWineTest.pl:error: Could not send 'patch.diff' to the VM: network read timed out (sendfile/connect:AgentVersion.h:0/9)
WineRunWineTest.pl:error: BotError: The test VM has crashed, rebooted or lost connectivity (or the TestAgent server died)
WineRunWineTest.pl:error: Giving up after 3 run(s)

=== debian11b (build log) ===

WineRunWineTest.pl:error: Could not send 'patch.diff' to the VM: network read timed out (sendfile/connect:AgentVersion.h:0/9)
WineRunWineTest.pl:error: BotError: The test VM has crashed, rebooted or lost connectivity (or the TestAgent server died)
WineRunWineTest.pl:error: Giving up after 3 run(s)