Stefan Dösinger : d3d8/tests: Skip test_cursor_pos if we can't move the cursor.
Module: wine Branch: master Commit: 414d225bc346646b81cca2d6d1870d7fb2708c5b URL: https://source.winehq.org/git/wine.git/?a=commit;h=414d225bc346646b81cca2d6d... Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Jan 31 13:19:45 2022 +0300 d3d8/tests: Skip test_cursor_pos if we can't move the cursor. Signed-off-by: Stefan Dösinger <stefan(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d8/tests/device.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index 7fdb7909db2..68b7fd00e70 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -1210,6 +1210,7 @@ static void test_cursor_pos(void) HWND window; HRESULT hr; BOOL ret; + POINT pt; /* Note that we don't check for movement we're not supposed to receive. * That's because it's hard to distinguish from the user accidentally @@ -1241,6 +1242,18 @@ static void test_cursor_pos(void) ok(ret, "Failed to set cursor position.\n"); flush_events(); + /* Check if we can move the cursor. If we're running in a virtual desktop + * that does not have focus or the mouse is outside the desktop window, some + * window managers (e.g. kwin) will refuse to let us steal the pointer. That + * is reasonable, but breaks the test. */ + ret = GetCursorPos(&pt); + ok(ret, "Failed to get cursor position.\n"); + if (pt.x != 99 || pt.y != 99) + { + skip("Could not warp the cursor (cur pos %ux%u), skipping test.\n", pt.x, pt.y); + return; + } + wc.lpfnWndProc = test_cursor_proc; wc.lpszClassName = "d3d8_test_cursor_wc"; ok(RegisterClassA(&wc), "Failed to register window class.\n");
participants (1)
-
Alexandre Julliard