From 60b1880a6248b8d94da0d8d0db638af130557416 Mon Sep 17 00:00:00 2001 From: Fred Sundvik Date: Sat, 8 Jul 2017 19:23:39 +0300 Subject: [PATCH] Reference issue #1476 from the unit tests --- tests/basic/test_keypress.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/basic/test_keypress.cpp b/tests/basic/test_keypress.cpp index 460c3838..d85cba44 100644 --- a/tests/basic/test_keypress.cpp +++ b/tests/basic/test_keypress.cpp @@ -42,6 +42,7 @@ TEST_F(KeyPress, CorrectKeysAreReportedWhenTwoKeysArePressed) { press_key(1, 0); press_key(0, 3); //Note that QMK only processes one key at a time + //See issue #1476 for more information EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B))); keyboard_task(); EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B, KC_C))); @@ -68,6 +69,7 @@ TEST_F(KeyPress, LeftShiftIsReportedCorrectly) { press_key(3, 0); press_key(0, 0); // Unfortunately modifiers are also processed in the wrong order + // See issue #1476 for more information EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A))); keyboard_task(); EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A, KC_LSFT))); @@ -85,6 +87,7 @@ TEST_F(KeyPress, PressLeftShiftAndControl) { press_key(3, 0); press_key(5, 0); // Unfortunately modifiers are also processed in the wrong order + // See issue #1476 for more information EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); keyboard_task(); EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_LCTRL))); @@ -96,6 +99,7 @@ TEST_F(KeyPress, LeftAndRightShiftCanBePressedAtTheSameTime) { press_key(3, 0); press_key(4, 0); // Unfortunately modifiers are also processed in the wrong order + // See issue #1476 for more information EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); keyboard_task(); EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_RSFT)));