Re: [PATCH v2 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/appxdeploymentclient/tests/appx.c:
> -
>
> ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
>
> check_interface( inspectable, &IID_IAgileObject, TRUE );
> -
> -
>
> hr = IPackageManager_FindPackagesByNamePublisher( package_manager, NULL, NULL, &packages );
> -
>
> ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> -
>
> hr = WindowsCreateString( L"Wine", wcslen( L"Wine" ), &str );
> -
>
> ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
>
> hr = IPackageManager_FindPackagesByNamePublisher( package_manager, str, NULL, &packages );
> -
>
> ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> -
>
> hr = WindowsCreateString( L"The Wine Project", wcslen( L"The Wine Project" ), &str2 );
> -
>
> ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
>
> hr = IPackageManager_FindPackagesByNamePublisher( package_manager, NULL, str2, &packages );
> -
>
> ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> -
>
> hr = IPackageManager_FindPackagesByNamePublisher( package_manager, str, str2, &packages );
> -
>
> todo_wine ok( hr == S_OK || broken ( hr == E_ACCESSDENIED /* Requires admin privileges */ ) , "got hr %#lx.\n", hr );

```suggestion:-0+0
    todo_wine ok( hr == S_OK || broken(hr == E_ACCESSDENIED) /* Requires admin privileges */, "got hr %#lx.\n", hr );
    if (hr == S_OK) IIterable_Package_Release( packages );
```