Re: [PATCH v4 0/8] MR6874: windows.devices.enumeration: Add a basic implementation for DeviceInformationStatics::FindAllAsync().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.enumeration/tests/devices.c:
> -
>            hr = IVectorView_DeviceInformation_GetAt( info_collection, idx, &info);
> -
>            ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> -
>            if (SUCCEEDED( hr ))
> -
>            {
> -
>                hr = IDeviceInformation_get_Id( info, &id2 );
> -
>                ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> -
>            }
> -
>            if (id1 && id2)
> -
>            {
> -
>                INT32 order = 1;
> -
>                WindowsCompareStringOrdinal( id1, id2, &order );
> -
>                ok( !order, "%s != %s\n", debugstr_hstring( id1 ), debugstr_hstring( id2 ) );
> -
>            }
> -
>            WindowsDeleteString( id1 );
> -
>            WindowsDeleteString( id2 );
> -
>            if (info)

This also doesn't look useful, info shouldn't be NULL unless tests have failed already.