[PATCH] msi/tests: Skip the deferred action test if the process is limited.

[Hans Leidekker]

Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
---
 dlls/msi/tests/install.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c
index acbf1629a5..7612a00785 100644
--- a/dlls/msi/tests/install.c
+++ b/dlls/msi/tests/install.c
@@ -5924,6 +5924,12 @@ static void test_deferred_action(void)
 {
     UINT r;
 
+    if (is_process_limited())
+    {
+        skip( "process is limited\n" );
+        return;
+    }
+
     create_database(msifile, da_tables, sizeof(da_tables) / sizeof(da_tables[0]));
 
     MsiSetInternalUI(INSTALLUILEVEL_NONE, NULL);
-- 
2.11.0

[Zebediah Figura]

On 20/02/18 05:06, Hans Leidekker wrote:
> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
> ---
>  dlls/msi/tests/install.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c
> index acbf1629a5..7612a00785 100644
> --- a/dlls/msi/tests/install.c
> +++ b/dlls/msi/tests/install.c
> @@ -5924,6 +5924,12 @@ static void test_deferred_action(void)
>  {
>      UINT r;
>
> +    if (is_process_limited())
> +    {
> +        skip( "process is limited\n" );
> +        return;
> +    }
> +
>      create_database(msifile, da_tables, sizeof(da_tables) / sizeof(da_tables[0]));
>
> MsiSetInternalUI(INSTALLUILEVEL_NONE, NULL);

I've sent a patch that fixes this so it works while limited instead.
Hopefully that will be preferable.

[Hans Leidekker]

On Tue, 2018-02-20 at 09:03 -0600, Zebediah Figura wrote:
> diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c
> > index acbf1629a5..7612a00785 100644
> > --- a/dlls/msi/tests/install.c
> > +++ b/dlls/msi/tests/install.c
> > @@ -5924,6 +5924,12 @@ static void test_deferred_action(void)
> >  {
> >      UINT r;
> >
> > +    if (is_process_limited())
> > +    {
> > +        skip( "process is limited\n" );
> > +        return;
> > +    }
> > +
> >      create_database(msifile, da_tables, sizeof(da_tables) / sizeof(da_tables[0]));
> >
> > MsiSetInternalUI(INSTALLUILEVEL_NONE, NULL);
>
> I've sent a patch that fixes this so it works while limited instead.
> Hopefully that will be preferable.

Yes, that's better. Thanks.