Re: [v5 1/3] evr: Add tests

[Alexandre Julliard]

Fabian Maurer  writes:
> +static void test_evr_filter_aggregations(void)
> +{
> - const IID * iids[] = {
> -
>    &IID_IAMCertifiedOutputProtection, &IID_IAMFilterMiscFlags, &IID_IBaseFilter,
> -
>    &IID_IKsPropertySet, &IID_IMediaEventSink, &IID_IMediaSeeking, &IID_IQualityControl,
> -
>    &IID_IQualProp, &IID_IEVRFilterConfig, &IID_IMFGetService, &IID_IMFVideoPositionMapper,
> -
>    &IID_IMFVideoRenderer, &IID_IQualityControl
> - };
> - int i;
> -
> - if(!strcmp(winetest_platform, "wine"))
> - {
> -
>    skip("Not supported yet.\n");
> -
>    return;
> - }

I don't think it's useful to have a test that doesn't test anything. You
should mark the tests as todo but still run them. If that's not
possible, it should wait until we have something to test.