[Bug 52648] New: conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52648

Bug ID: 52648
           Summary: conhost.exe:tty test_read_console_control() fails in
                    Hindi and Portuguese
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: programs
          Assignee: [e-mail]
          Reporter: [e-mail]

conhost.exe:tty test_read_console_control() fails on Windows in Hindi and
Portuguese:

tty.c:1365: Test failed: keyboard state: got 8, expected 18

https://test.winehq.org/data/patterns.html#conhost.exe:tty

This can be reproduced on w10pro64-hi, w10pro64-pt-BR and w10pro64-pt-PT but
does not fail in the other locales.

This failure started when these tests were added:

commit 89a8b32d7a976504ee98ba1a7d08574bc9bc00e6
Author:     Eric Pouech 
AuthorDate: Tue Mar 1 09:26:08 2022 +0100

conhost/tests: Add tests for ReadConsoleW with control.

Signed-off-by: Eric Pouech 
    Signed-off-by: Jacek Caban 
    Signed-off-by: Alexandre Julliard

[Bug 52648] conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52648

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
    Regression SHA1|                            |89a8b32d7a976504ee98ba1a7d0
                   |                            |8574bc9bc00e6
           Keywords|                            |regression, source,
                   |                            |testcase

[Bug 52648] conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52648

Eric Pouech  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[e-mail]

--- Comment #1 from Eric Pouech  ---
well, keyboard mapping is a real pain ;-)
the error is not directly linked to the locale, but rather to the keyboard
installed on the VM...

NB: don't try to run the conhost:tty tests with a French keyboard

I did consider also using MapVirtualKeyEx (which would be real clean fix, ie
not having the tests being dependant on the installed keyboard). but this API
is only half implemented (and the half we'd need is the one missing <g>)

I'll loosen up the test

[Bug 52648] conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52648

Eric Pouech  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |c19f3b620acda9125cc975c0299
                   |                            |2a544b27e5929

--- Comment #2 from Eric Pouech  ---
looks like its ok now (tty tests still failing on some other parts though)

[Bug 52648] conhost.exe:tty test_read_console_control() fails in Hindi and Portuguese

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52648

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #3 from Alexandre Julliard  ---
Closing bugs fixed in 7.20.