Re: [PATCH 0/2] MR9299: ole32, combase: Add tests for COM context switching and delayed marshaling.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/combase/tests/roapi.c:
> -
>    ok(hr == S_OK, "got hr %#lx\n", hr);
> -
> -
>    unk_ctx_impl = impl_unk_ctx_impl_from_IUnknown(unknown);
> -
>    hr = RoGetAgileReference(AGILEREFERENCE_DELAYEDMARSHAL, &IID_IUnknown, unknown, &agile_reference);
> -
>    ok(hr == S_OK, "got hr %#lx\n", hr);
> -
>    EXPECT_REF(unknown, 2);
> -
> -
>    for (to_type = RO_INIT_SINGLETHREADED; to_type <= RO_INIT_MULTITHREADED; to_type++)
> -
>    {
> -
>        struct test_agile_resolve_context_params params = {from_type, to_type, agile_reference};
> -
> -
>        winetest_push_context("to_type=%d", to_type);
> -
>        unk_ctx_impl->todo = TRUE;
> -
>        thread = CreateThread(NULL, 0, test_agile_resolve_context, &params, 0, NULL);
> -
>        flush_events();
> -
>        ret = WaitForSingleObject(thread, 100);

This timeout is too short to defend against flakiness, especially since this wait is always supposed to complete.

```suggestion:-0+0
            ret = WaitForSingleObject(thread, INFINITE);
```