Re: [PATCH v19 0/2] MR5909: comctl32/tests: Add LVM_GETNEXTITEM test.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/listview.c:
> +    DWORD r;
> +    UINT uFlagsAbove = MAKELPARAM(LVNI_ABOVE, 0);
> +    UINT uFlagsBelow = MAKELPARAM(LVNI_BELOW, 0);
> +
> +    hwnd = create_listview_control(LVS_REPORT);
> +    insert_item(hwnd, 0);
> +    insert_item(hwnd, 1);
> +
> +    r = SendMessageA(hwnd, LVM_GETNEXTITEM, 0, uFlagsAbove);
> +    expect(-1, r);
> +    r = SendMessageA(hwnd, LVM_GETNEXTITEM, 0, uFlagsBelow);
> +    expect(1, r);
> +    r = SendMessageA(hwnd, LVM_GETNEXTITEM, 1, uFlagsAbove);
> +    expect(0, r);
> +    r = SendMessageA(hwnd, LVM_GETNEXTITEM, 1, uFlagsBelow);
> +    expect(-1, r);
> There should be a test failing on Wine in this patch. Then you add a todo_wine to make the test pass in this commit. Then in the next patch after you've fixed the bug, the todo_wine should get removed.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5909#note_79085