Zebediah Figura : user32/tests: Do not stop checking the message list when skipping an optional or unsupported message.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: a8c56b19397893c40d11692b73b3acd75c67a738
URL:    https://gitlab.winehq.org/wine/wine/-/commit/a8c56b19397893c40d11692b73b3acd...

Author: Zebediah Figura 
Date:   Thu Mar  2 19:56:08 2023 -0600

user32/tests: Do not stop checking the message list when skipping an optional or unsupported message.

This was either added by accident, or in an attempt to stop checking the message
list once we hit a winevent hook todo [and accidentally given too wide of a
scope]. However, the same commit also uses a global counter to ensure that only
one winevent todo is printed, so we don't need to break in the first hunk
anyway.

For trailing winevent todos, there is no global counter, so instead move the
goto inside the if block.

Fixes: a72bffe768c9fe462010d7e40e20226322fb82c7

The test marked todo here was broken by 33617af8145b6fadfe41d48968a728ff57b3064b, which was hidden due to the above commit.

---

dlls/user32/tests/msg.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index 303d6dbf012..624d86899b3 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -2853,7 +2853,6 @@ static void ok_sequence_(const struct message *expected_list, const char *contex
                 }
             }
        expected++;
-            goto done;
         }
    else if (todo)
    {
@@ -2888,9 +2887,9 @@ static void ok_sequence_(const struct message *expected_list, const char *contex
                 ok_( file, line) (FALSE, "%s: %u: the msg sequence is not complete: expected 0x%04x - actual 0x%04x\n",
                                 context, count, expected->message, actual->message);
             }
+            goto done;
         }
    expected++;
-        goto done;
     }

if (todo)
@@ -5794,7 +5793,7 @@ static void test_messages(void)
     test_WM_SETREDRAW(hchild);

ShowWindow(hchild, SW_SHOW);
-    ok_sequence(WmShowChildSeq, "ShowWindow(SW_SHOW):child", FALSE);
+    ok_sequence(WmShowChildSeq, "ShowWindow(SW_SHOW):child", TRUE);

/* check parent messages too */
     log_all_parent_messages++;