user32/tests: Move test_mdi early in the test list.

The test gets spurious GetScrollInfo SB_HORZ failures if some other windows were created before using CW_USEDEFAULT positions, for instance the window from test_capture_4. This happened on some Win10 VMs, and was thought to be fixed with 9b5346df86bf1a85336ed03a67a630970ce08e9f, but it then broke the test on some other VMs. This is hopefully a better fix, and it reverts the previous one too. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=52507Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

user32/tests: Move test_mdi early in the test list.
fb9237cf · Rémi Bernon · Alexandre Julliard · 49c76f99 · fb9237cf
Commit fb9237cf authored Feb 07, 2022 by Rémi Bernon Committed by Alexandre Julliard Feb 07, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

win.c dlls/user32/tests/win.c +5 -2

No files found.
--- a/dlls/user32/tests/win.c
+++ b/dlls/user32/tests/win.c
@@ -2256,7 +2256,7 @@ static void test_mdi(void)
    mdi_hwndMain = CreateWindowExA(0, "MDI_parent_Class", "MDI parent window",
                                   WS_CAPTION | WS_SYSMENU | WS_MINIMIZEBOX |
                                   WS_MAXIMIZEBOX /*| WS_VISIBLE*/,
-                                   200, 100, CW_USEDEFAULT, CW_USEDEFAULT,
+                                   100, 100, CW_USEDEFAULT, CW_USEDEFAULT,
                                   GetDesktopWindow(), 0,
                                   GetModuleHandleA(NULL), NULL);
    assert(mdi_hwndMain);
@@ -12644,6 +12644,10 @@ START_TEST(win)

    our_pid = GetWindowThreadProcessId(hwndMain, NULL);

+    /* This test is sensitive to other windows created with
+     * CW_USEDEFAULT position, execute it early too */
+    test_mdi();
+
    /* Add the tests below this line */
    test_child_window_from_point();
    test_window_from_point(argv[0]);
@@ -12665,7 +12669,6 @@ START_TEST(win)
    test_parent_owner();
    test_enum_thread_windows();

-    test_mdi();
    test_icons();
    test_SetWindowPos(hwndMain, hwndMain2);
    test_SetMenu(hwndMain);