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

Anton Baskanov baskanov at gmail.com
Thu Apr 8 09:33:53 CDT 2021


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.






More information about the wine-devel mailing list