Re: [PATCH v19 0/2] MR5909: comctl32/tests: Add LVM_GETNEXTITEM test.
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
participants (1)
-
Zhiyi Zhang (@zhiyi)