Re: [PATCH v6 0/2] MR6186: comctl32/progress: Add states for progress bar.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/progress.c:
> -
>    {PBST_ERROR, PBST_ERROR, pbm_setstate_seq, 0, 1},
> -
>    {PBFS_PARTIAL, PBST_ERROR, pbm_setstate_seq, 1, 1}
> - };
> -
> - progress_bar = create_progress(0);
> -
> - old_proc = (WNDPROC)SetWindowLongPtrA(progress_bar, GWLP_WNDPROC, (LONG_PTR)test_pbm_setstate_proc);
> - flush_events();
> - flush_sequences(sequences, NUM_MSG_SEQUENCES);
> -
> - for (int i = 0; i < ARRAY_SIZE(tests); i++)
> - {
> -
>    state = SendMessageA(progress_bar, PBM_SETSTATE, tests[i].state, 0);
> -
>    flush_events();
> -
>    ok_sequence(sequences, CHILD_SEQ_INDEX, tests[i].expected_seq, "PBM_SETSTATE", tests[i].todo);
> -
>    todo_wine_if(tests[i].todo) ok(state == (tests[i].error ? 0 : tests[i].previous_state), "Expected %ld, but got %d.\n", tests[i].previous_state, state);

Since all tests[i].todo are TRUE, you can just use todo_wine and then remove it in the latter patch.