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

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/progress.c:
> -
>        state = SendMessageA(progress_bar, PBM_SETSTATE, test_states[i], 0);
> -
>        flush_events();
> -
>        /* No paint message if new and old state are the same */
> -
>        if (i == 3)
> -
>            ok_sequence(sequences, CHILD_SEQ_INDEX, pbm_setstate_seq, "PBM_SETSTATE", TRUE);
> -
>        else
> -
>            ok_sequence(sequences, CHILD_SEQ_INDEX, paint_pbm_setstate_seq, "PBM_SETSTATE", TRUE);
> -
> -
>        todo_wine ok(state == test_states[i-1], "Expected %d, but got %d.\n", test_states[i-1], state);
> -
> -
>    }
> -
>    state = SendMessageA(progress_bar, PBM_GETSTATE, 0, 0);
> -
>    todo_wine ok(state == test_states[i], "Expected %d, but got %d.\n", test_states[i], state);
> - }
> -
> - /* Check bad state */

These are not interesting. Having tests for PBFS_PARTIAL is enough.