Re: [PATCH v2 3/4] mf: Generate missing MEStreamSinkPaused event in the sample grabber.

[Nikolay Sivov]

On 5/28/21 12:13 PM, Giovanni Mascellani wrote:
> Il 28/05/21 10:29, Nikolay Sivov ha scritto:
> > This is incomplete, for several reasons. Pause when stopped is an
> > invalid transition; pause when already paused should not generate an
> > event.
>
> Ok. Should I write additional tests for these cases? Last time you
> suggested that I was writing too many tests, so I would like to
> understand what is the right amount.

That's easily observable with tests from 2/4 and a couple more calls. We
do need more tests in general, but not potentially fragile ones, and
event collection appears to be that for media foundation.
> Thanks again, Giovanni.