Re: [PATCH v3 0/4] MR3983: windows.storage.applicationdata: Implement IApplicationDataStatics::get_Current().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.storage.applicationdata/tests/data.c:
>  hr = IActivationFactory_QueryInterface( factory, &IID_IApplicationDataStatics, (void **)&application_data_statics );
>  ok( hr == S_OK, "got hr %#lx.\n", hr );
>
> - hr = IApplicationDataStatics_get_Current( application_data_statics, NULL );
> - todo_wine ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> - hr = IApplicationDataStatics_get_Current( application_data_statics, &application_data );
> - todo_wine ok( hr == 0x80073d54, "got hr %#lx.\n", hr );
> - todo_wine ok( !application_data, "got application_data %p.\n", application_data );

Nit: In commit 3/4, as the function is still a pure stub, the `!application_data` test succeeds on Wine. You should probably either remove that check (as the call is normally supposed to fail, or remove the todo_wine to add it back in 4/4).