summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFred Sundvik <fsundvik@gmail.com>2017-07-08 19:26:47 +0300
committerJack Humbert <jack.humb@gmail.com>2017-07-08 21:59:51 -0400
commit41efcd6d73ca08774e680daa39c42b0437133387 (patch)
tree6a377db579fde69cf1a618756ad68912b71d1551
parent60b1880a6248b8d94da0d8d0db638af130557416 (diff)
downloadqmk_firmware-41efcd6d73ca08774e680daa39c42b0437133387.tar.gz
qmk_firmware-41efcd6d73ca08774e680daa39c42b0437133387.zip
Reference issue #524 from the unit tests
-rw-r--r--tests/basic/test_keypress.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/basic/test_keypress.cpp b/tests/basic/test_keypress.cpp
index d85cba4465..194b1745b1 100644
--- a/tests/basic/test_keypress.cpp
+++ b/tests/basic/test_keypress.cpp
@@ -111,6 +111,9 @@ TEST_F(KeyPress, RightShiftLeftControlAndCharWithTheSameKey) {
press_key(6, 0);
// BUG: The press is split into two reports
// BUG: It reports RSFT instead of LSFT
+ // See issue #524 for more information
+ // The underlying cause is that we use only one bit to represent the right hand
+ // modifiers.
EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_RSFT, KC_RCTRL)));
EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_RSFT, KC_RCTRL, KC_O)));
keyboard_task();