Re: [PATCH 2/5] amstream/tests: Add tests for MediaStreamFilter::EndOfStream.

[Anton Baskanov]

On среда, 7 апреля 2021 г. 22:43:02 +07 you wrote:
> On 4/6/21 1:04 PM, Anton Baskanov wrote:
> > - /* Flush with cancel_eos=TRUE decrements EOS count. */
> > - hr = IAMMultiMediaStream_SetState(mmstream, STREAMSTATE_RUN);
> > - ok(hr == S_OK, "Got hr %#x.\n", hr);
> > -
> > - graph.got_notify = 0;
> > -
> > - hr = IMediaStreamFilter_EndOfStream(filter);
> > - todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr);
> > -
> > - hr = IMediaStreamFilter_Flush(filter, TRUE);
> > -
> > - hr = IMediaStreamFilter_EndOfStream(filter);
> > - todo_wine ok(hr == S_OK, "Got hr %#x.\n", hr);
> > -
> > - ok(graph.got_notify == 0, "Got %d calls to
> >
> > IMediaEventSink::Notify().\n", graph.got_notify); +
> > - hr = IAMMultiMediaStream_SetState(mmstream, STREAMSTATE_STOP);
> > - ok(hr == S_OK, "Got hr %#x.\n", hr);
> > -
>
> It may be worth expanding this to check whether e.g. {EOS, EOS, flush,
> EOS} generates a notification.

I've added the test and it passes both in Windows and Wine. I'll send it in 
the next batch so that it applies cleanly if you don't mind.