Re: [PATCH 0/5] MR2782: mf/tests: Test byte stream handler resolution time and media source presentation descriptor.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
> +    check_interface(handler, &IID_IUnknown, TRUE);
> +    check_interface(handler, &IID_IMFAsyncCallback, FALSE);
> +
> +    hr = IMFByteStreamHandler_GetMaxNumberOfBytesRequiredForResolution(handler, &bytes);
> +    todo_wine ok(hr == S_OK, "got hr %#lx\n", hr);
> +
> +
> +    load_resource_stream(resource, &stream);
> +
> +    time -= GetTickCount();
> +    hr = IMFByteStreamHandler_BeginCreateObject(handler, stream, NULL, MF_RESOLUTION_MEDIASOURCE, NULL, NULL, callback, NULL);
> +    ok(hr == S_OK, "got hr %#lx\n", hr);
> +    wait_async_result(callback, &result);
> +    time += GetTickCount();
> +    todo_wine ok(time <= 5 || broken(time <= 20) /* testbot may be very slow */, "source resolution took %lums\n", time);
> +    ok(!!result, "got result %p\n", result);
> Do we need such test in wine? I think we usually avoid testing things that rely on timing too much.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2782#note_32429