Re: [PATCH v6 0/6] MR3548: windows.applicationmodel: Implement some functions.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.applicationmodel/tests/model.c:
> if (SUCCEEDED(test_register_package( manager, &package )))
>      {
>          test_execute_package( package );
> +        test_PackageStatics();

This won't work like that. At this point the test has indeed installed a package (and executed its application), but this code still executes in the context of the normal test executable.

You should add the tests in `application.c` if you want to the behavior from within an packaged application. For instance you could have a `test_PackageStatics` there too, which would expect the current package to be always valid.

Of course, we don't support running this on Wine yet, because we don't support running packaged applications, so the tests are entirely skipped. But at least it will show us native's behavior.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3548#note_46048