Julian Klemann : httpapi/tests: Move sleep statement.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 39502adc6c2578c4279e3e6f125673ed6c658a74
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=39502adc6c2578c4279e3e6f125673ed6c658a74

Author: Julian Klemann <jklemann at codeweavers.com>
Date:   Thu Jun 16 12:55:07 2022 -0500

httpapi/tests: Move sleep statement.

The mentioned bug randomly occurred after some subsequent patches, so
the sleep should happen before GetOverlappedResult is called.

Signed-off-by: Julian Klemann <jklemann at codeweavers.com>
Signed-off-by: Zebediah Figura <zfigura at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/httpapi/tests/httpapi.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/httpapi/tests/httpapi.c b/dlls/httpapi/tests/httpapi.c
index be1d31c12ad..9d344f26d02 100644
--- a/dlls/httpapi/tests/httpapi.c
+++ b/dlls/httpapi/tests/httpapi.c
@@ -228,10 +228,6 @@ static void test_v1_server(void)
     ret = send(s, req_text, strlen(req_text), 0);
     ok(ret == strlen(req_text), "send() returned %d.\n", ret);
 
-    ret = GetOverlappedResult(queue, &ovl, &ret_size, TRUE);
-    ok(ret, "Got error %lu.\n", GetLastError());
-    ok(ret_size > sizeof(*req), "Got size %lu.\n", ret_size);
-
     /* Various versions of Windows (observed on 64-bit Windows 8 and Windows 10
      * version 1507, but probably affecting others) suffer from a bug where the
      * kernel will report success before completely filling the buffer or
@@ -239,6 +235,10 @@ static void test_v1_server(void)
      * around this. */
     Sleep(100);
 
+    ret = GetOverlappedResult(queue, &ovl, &ret_size, TRUE);
+    ok(ret, "Got error %lu.\n", GetLastError());
+    ok(ret_size > sizeof(*req), "Got size %lu.\n", ret_size);
+
     ok(!req->Flags, "Got flags %#lx.\n", req->Flags);
     ok(req->ConnectionId, "Expected nonzero connection ID.\n");
     ok(req->RequestId, "Expected nonzero connection ID.\n");