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.