[PATCH 0/3] MR987: d3d9/tests: Mark tests failing randomly on Windows as flaky.

[PATCH 1/3] d3d9/tests: Mark the test_cursor_pos() tests failing randomly on Windows as flaky.

[Francois Gouget]

From: Francois Gouget

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53194
---
 dlls/d3d9/tests/device.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
index dbdce1e5871..9fe05625190 100644
--- a/dlls/d3d9/tests/device.c
+++ b/dlls/d3d9/tests/device.c
@@ -5287,6 +5287,7 @@ static void test_cursor_pos(void)
     ok(ret, "Failed to set cursor position.\n");
     flush_events();

+    flaky
     ok((!expect_pos->x && !expect_pos->y) || broken(expect_pos - points == 7),
         "Didn't receive MOUSEMOVE %u (%ld, %ld).\n",
         (unsigned)(expect_pos - points), expect_pos->x, expect_pos->y);

[PATCH 2/3] d3d9/tests: Mark the test_wndproc_windowed() tests failing randomly on Windows as flaky.

[Francois Gouget]

From: Francois Gouget

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53283
---
 dlls/d3d9/tests/d3d9ex.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dlls/d3d9/tests/d3d9ex.c b/dlls/d3d9/tests/d3d9ex.c
index 19c4eec7921..b9c5ecd2e05 100644
--- a/dlls/d3d9/tests/d3d9ex.c
+++ b/dlls/d3d9/tests/d3d9ex.c
@@ -3297,6 +3297,7 @@ static void test_wndproc_windowed(void)
     if (thread_params.running_in_foreground)
     {
         tmp = GetForegroundWindow();
+        flaky
         ok(tmp == thread_params.dummy_window, "Expected foreground window %p, got %p.\n",
                 thread_params.dummy_window, tmp);
     }

[PATCH 3/3] d3d9/tests: Mark the test_wndproc() tests failing randomly on Windows as flaky.

[Francois Gouget]

From: Francois Gouget

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53281
---
 dlls/d3d9/tests/d3d9ex.c | 1 +
 dlls/d3d9/tests/device.c | 1 +
 2 files changed, 2 insertions(+)

diff --git a/dlls/d3d9/tests/d3d9ex.c b/dlls/d3d9/tests/d3d9ex.c
index b9c5ecd2e05..d1ea5587f3a 100644
--- a/dlls/d3d9/tests/d3d9ex.c
+++ b/dlls/d3d9/tests/d3d9ex.c
@@ -3096,6 +3096,7 @@ static void test_wndproc(void)
         SetForegroundWindow(GetDesktopWindow());
         ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
                 expect_messages->message, expect_messages->window, i);
+        flaky_if(i == 0 || i == 1)
         ok(!windowposchanged_received, "Received WM_WINDOWPOSCHANGED but did not expect it, i=%u.\n", i);
         expect_messages = NULL;

diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
index 9fe05625190..c824edec9df 100644
--- a/dlls/d3d9/tests/device.c
+++ b/dlls/d3d9/tests/device.c
@@ -4342,6 +4342,7 @@ static void test_wndproc(void)
         SetForegroundWindow(GetDesktopWindow());
         ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
                 expect_messages->message, expect_messages->window, i);
+        flaky_if(i == 0)
         ok(!windowposchanged_received, "Received WM_WINDOWPOSCHANGED but did not expect it, i=%u.\n", i);
         expect_messages = NULL;

[PATCH 3/3] d3d9/tests: Mark the test_wndproc() tests failing randomly on Windows as flaky.

[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.

The tests also ran into some preexisting test failures. If you know how
to fix them that would be helpful. See the TestBot job for the details:

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

Your paranoid android.

=== w1064_2qxl (64 bit report) ===

d3d9:
device.c:5292: Test marked flaky: Didn't receive MOUSEMOVE 0 (50, 50).

Report validation errors:
d3d9:device returned success despite having failures

[PATCH 0/3] MR987: d3d9/tests: Mark tests failing randomly on Windows as flaky. - approved

[Jan Sikorski (＠jsikorski)]

This merge request was approved by Jan Sikorski.