Re: [PATCH v3 0/1] MR8859: windows.applicationmodel/tests: Add tests for exposing COM interfaces through the manifest.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.applicationmodel/tests/application.c:
> +}
> -
>
> +static HRESULT WINAPI test_inspectable_GetRuntimeClassName( IActivationFactory *iface, HSTRING *name )
> +{
> - static const WCHAR *nameW = L"Wine.Application.Class";
> - return WindowsCreateString( nameW, wcslen( nameW ), name );
>
> +}
> +static HRESULT WINAPI test_inspectable_GetIids( IActivationFactory *iface, ULONG *count, GUID **iids )
> +{
> - return E_NOTIMPL;
>
> +}
> -
>
> +static HRESULT WINAPI test_inspectable_GetTrustLevel( IActivationFactory *iface, TrustLevel *level )
> +{
> - return S_OK;
>
> +}

```suggestion:-13+0
static HRESULT WINAPI test_inspectable_GetRuntimeClassName( IActivationFactory *iface, HSTRING *name )
{
    static const WCHAR *nameW = L"Wine.Application.Class";
    return WindowsCreateString( nameW, wcslen( nameW ), name );
}

static HRESULT WINAPI test_inspectable_GetIids( IActivationFactory *iface, ULONG *count, GUID **iids )
{
    return E_NOTIMPL;
}

static HRESULT WINAPI test_inspectable_GetTrustLevel( IActivationFactory *iface, TrustLevel *level )
{
    return S_OK;
}
```

Missing line.

Also, might be interesting to add some tests in them, `ok( 0, "unexpected\n" );` if they aren't supposed to be called.