[PATCH] user32/tests: Mark newer Win10 behaviour as broken

[André Hentschel]

Signed-off-by: André Hentschel 
---
 dlls/user32/tests/monitor.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
index f4c1a4b17c..8ba6cc267b 100644
--- a/dlls/user32/tests/monitor.c
+++ b/dlls/user32/tests/monitor.c
@@ -585,7 +585,7 @@ static void test_display_config(void)
     paths = modes = 100;
     ret = pGetDisplayConfigBufferSizes(0, &paths, &modes);
     ok(ret == ERROR_INVALID_PARAMETER || ret == ERROR_NOT_SUPPORTED, "got %d\n", ret);
-    ok(modes == 0 && paths == 0, "got %u, %u\n", modes, paths);
+    ok((modes == 0 || broken(modes == 100)) && paths == 0, "got %u, %u\n", modes, paths);
 }

START_TEST(monitor)

[Alexandre Julliard]

André Hentschel  writes:
> Signed-off-by: André Hentschel
>
> dlls/user32/tests/monitor.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
> index f4c1a4b17c..8ba6cc267b 100644
> --- a/dlls/user32/tests/monitor.c
> +++ b/dlls/user32/tests/monitor.c
> @@ -585,7 +585,7 @@ static void test_display_config(void)
>      paths = modes = 100;
>      ret = pGetDisplayConfigBufferSizes(0, &paths, &modes);
>      ok(ret == ERROR_INVALID_PARAMETER || ret == ERROR_NOT_SUPPORTED, "got %d\n", ret);
> - ok(modes == 0 && paths == 0, "got %u, %u\n", modes, paths);
> - ok((modes == 0 || broken(modes == 100)) && paths == 0, "got %u, %u\n", modes, paths);

This looks like a reasonable result at least for the
ERROR_INVALID_PARAMETER case, I'm not sure why you'd consider it broken.

[André Hentschel]

On 17.09.2018 18:33, Alexandre Julliard wrote:
> André Hentschel  writes:
> > Signed-off-by: André Hentschel
> >
> > dlls/user32/tests/monitor.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/dlls/user32/tests/monitor.c b/dlls/user32/tests/monitor.c
> > index f4c1a4b17c..8ba6cc267b 100644
> > --- a/dlls/user32/tests/monitor.c
> > +++ b/dlls/user32/tests/monitor.c
> > @@ -585,7 +585,7 @@ static void test_display_config(void)
> >       paths = modes = 100;
> >       ret = pGetDisplayConfigBufferSizes(0, &paths, &modes);
> >       ok(ret == ERROR_INVALID_PARAMETER || ret == ERROR_NOT_SUPPORTED, "got %d\n", ret);
> > - ok(modes == 0 && paths == 0, "got %u, %u\n", modes, paths);
> > - ok((modes == 0 || broken(modes == 100)) && paths == 0, "got %u, %u\n", modes, paths);
>
> This looks like a reasonable result at least for the
> ERROR_INVALID_PARAMETER case, I'm not sure why you'd consider it broken.

Good point, I'll send an update later