Re: [PATCH v2 0/6] MR716: windows.devices.enumeration: Add DeviceWatcher class and IDeviceInformationStatics2 interface.

[Bernhard Kölbl (＠besentv)]

Bernhard K��lbl (@besentv) commented about dlls/windows.devices.enumeration/tests/devices.c:
> -
>        &stopped_token);
> - ok(hr == S_OK, "got hr %#lx\n", hr);
> -
> - hr = IDeviceWatcher_get_Status(device_watcher, &status);
> - todo_wine ok(hr == S_OK, "got hr %#lx\n", hr);
> - todo_wine ok(status == DeviceWatcherStatus_Created, "got status %u\n", status);
> -
> - hr = IDeviceWatcher_Start(device_watcher);
> - ok(hr == S_OK, "got hr %#lx\n", hr);
> - hr = IDeviceWatcher_get_Status(device_watcher, &status);
> - todo_wine ok(hr == S_OK, "got hr %#lx\n", hr);
> - todo_wine ok(status == DeviceWatcherStatus_Started, "got status %u\n", status);
> -
> - hr = IDeviceWatcher_Stop(device_watcher);
> - ok(hr == S_OK, "got hr %#lx\n", hr);
> - WaitForSingleObject(stopped_handler.event, INFINITE);

It's not good to block the test infinitely. Just put this to 1000 and add a test to check it didn't time out.