Re: [PATCH v3 0/2] MR10451: comctl32/treeview: Implemented TVN_ITEMCHANGING & TVN_ITEMCHANGED

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/treeview.c:
> ok_sequence(sequences, TREEVIEW_SEQ_INDEX, rootchild_select_seq,
>                  "root-child select seq", FALSE);
>
> +    if (g_v6)
> +    {
> +        ok_sequence(sequences, PARENT_SEQ_INDEX, parent_rootchild_select_seq_v6, "root-child parent select seq v6", TRUE);

Could you make these tests simpler? For example, I would expect that after getting an item to change, there is a sequence of TVN_ITEMCHANGING and TVN_ITEMCHANGED. Having a long sequence of messages after many operations is not exactly easy to understand. Keep the test as simple as possible.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10451#note_135908