Dmitry Timoshkov : kernel32/tests: Make ClearCommError very first test since it depends on device queue being empty .

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b288f0a8790dbcb79332de4e2af9f6674c5d3484
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=b288f0a8790dbcb79332de4e2a...

Author: Dmitry Timoshkov <dmitry(a)baikal.ru>
Date:   Thu Aug 15 16:38:15 2013 +0900

kernel32/tests: Make ClearCommError very first test since it depends on device queue being empty.

---

 dlls/kernel32/tests/comm.c |    8 +-------
 1 files changed, 1 insertions(+), 7 deletions(-)

diff --git a/dlls/kernel32/tests/comm.c b/dlls/kernel32/tests/comm.c
index e9a8690..38a8bbc 100644
--- a/dlls/kernel32/tests/comm.c
+++ b/dlls/kernel32/tests/comm.c
@@ -1671,6 +1671,7 @@ START_TEST(comm)
     hcom = test_OpenComm(FALSE);
     if (hcom != INVALID_HANDLE_VALUE)
     {
+	test_ClearCommErrors(hcom); /* keep it the very first test */
 	test_GetModemStatus(hcom);
 	test_ReadTimeOut(hcom);
 	test_waittxempty(hcom);
@@ -1679,13 +1680,6 @@ START_TEST(comm)
     hcom = test_OpenComm(FALSE);
     if (hcom != INVALID_HANDLE_VALUE)
     {
-	Sleep(200); /* Give the laster character of test_waittxempty to drop into the receiver */
-	test_ClearCommErrors(hcom);
-	CloseHandle(hcom);
-    }
-    hcom = test_OpenComm(FALSE);
-    if (hcom != INVALID_HANDLE_VALUE)
-    {
         test_non_pending_errors(hcom);
 	CloseHandle(hcom);
     }