[PATCH v2] user32/tests: Skip some tests with inconsistent results with non-us kbd.

[Rémi Bernon]

See the same binary being tested twice here:
* https://testbot.winehq.org/JobDetails.pl?Key=58277
* https://testbot.winehq.org/JobDetails.pl?Key=58278

Signed-off-by: Rémi Bernon 
---
v2: This was PATCH 4/4 of the previous series for user32/tests, it's now
doing what it was supposed to do.

It would be better to fix the inconsistencies, but in the meantime this
creates noise on testbot.

dlls/user32/tests/input.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c
index 130ce49bb57..c5cf9556ede 100644
--- a/dlls/user32/tests/input.c
+++ b/dlls/user32/tests/input.c
@@ -284,6 +284,13 @@ static BOOL TestASet( HWND hWnd, int nrkev, const KEV kevdwn[], const KEV kevup[
     int i,j,k,l,m,n;
     static int count=0;
     KEV kbuf[MAXKEYEVENTS];
+    BOOL us_kbd = (GetKeyboardLayout(0) == (HKL)(ULONG_PTR)0x04090409);
+    if (!us_kbd)
+    {
+        win_skip( "skipping test with inconsistent results on non-us keyboard\n" );
+        return TRUE;
+    }
+
     assert( nrkev==2 || nrkev==3);
     for(i=0;i<MAXKEYEVENTS;i++) kbuf[i]=0;
     /* two keys involved gives 4 test cases */
@@ -1184,6 +1191,12 @@ static void test_Input_unicode(void)
     HHOOK hook;
     HMODULE hModuleImm32;
     BOOL (WINAPI *pImmDisableIME)(DWORD);
+    BOOL us_kbd = (GetKeyboardLayout(0) == (HKL)(ULONG_PTR)0x04090409);
+    if (!us_kbd)
+    {
+        win_skip( "skipping test with inconsistent results on non-us keyboard\n" );
+        return;
+    }

wclass.lpszClassName = classNameW;
     wclass.style         = CS_HREDRAW | CS_VREDRAW;
--
2.24.0.rc0

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=58285

Your paranoid android.

=== w1064v1809_ar (32 bit report) ===

user32:
input.c:1870: Test failed: 0: got L"\0634" expected L"a"
input.c:1868: Test failed: 8: got 1 expected 0
input.c:1909: Test failed: ToAscii for character 'A' was 212 (expected 97)

=== w1064v1809_he (32 bit report) ===

user32:
input.c:1870: Test failed: 0: got L"\05e9" expected L"a"
input.c:1868: Test failed: 8: got 1 expected 0
input.c:1909: Test failed: ToAscii for character 'A' was 249 (expected 97)

=== w1064v1809_zh_CN (32 bit report) ===

user32:
input.c:1425: Test failed: Wrong new pos: (149,149)

=== w1064v1809 (64 bit report) ===

user32:
input.c:2148: Test failed: expected WM_LBUTTONDOWN message
input.c:2149: Test failed: expected WM_LBUTTONUP message