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

[Hans Leidekker]

Signed-off-by: Hans Leidekker 
---
 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);

[Zebediah Figura]

On 20/02/18 05:06, Hans Leidekker wrote:
> Signed-off-by: Hans Leidekker
>
> 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.