Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/listview.c:
> + res = SendMessageA(hwnd, WM_KEYDOWN, VK_DOWN, 0);
> + expect(0, res);
> + ok_sequence(sequences, PARENT_ODSTATECHANGED_SEQ_INDEX,
> + ownerdata_multiselect_select_3_odstatechanged_seq,
> + "ownerdata multiselect: deselect all, select item 3 via DOWN", FALSE);
> + res = SendMessageA(hwnd, WM_KEYUP, VK_DOWN, 0);
> + expect(0, res);
> + res = SendMessageA(hwnd, LVM_GETSELECTEDCOUNT, 0, 0);
> + expect(1, res);
> +
> + hold_key(VK_SHIFT);
> +
> + flush_sequences(sequences, NUM_MSG_SEQUENCES);
> +
> + /* First up then down */
> + /* Select multiple items via SHIFT+UP */
For consistency, let's move hold_key() to the start of each test. Right now some of them are at the start and some are at the up.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/550#note_5316