Re: [PATCH v8 0/4] MR5759: comctl32 & user32: Mark radio buttons at selected when focused

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/button.c:
> - {
> -
>    { WM_SETFOCUS, sent },
> -
>    { WM_COMMAND, sent|parent|wparam, ID_BUTTON },
> -
>    { WM_PAINT, sent },
> -
>    { 0 }
> - };
> -
> - hwnd = CreateWindowExA(0, "TestParentClass", "Test parent", WS_OVERLAPPEDWINDOW | WS_VISIBLE,
> -
>                       100, 100, 200, 200, 0, 0, 0, NULL);
> - ok(hwnd != 0, "Failed to create parent window\n");
> -
> - for (i = 0; i < ARRAY_SIZE(types); i++)
> - {
> -
>    /* Test default button */
> -
>    button = create_button(types[i] | WS_VISIBLE, hwnd);
> -
>    while (PeekMessageA(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessageA(&msg);

I would prefer flush_events() like you did with user32 tests. That will also reduce the difference for user32 and comctl32 tests.