summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFred Sundvik <fsundvik@gmail.com>2017-07-08 18:23:39 +0200
committerJack Humbert <jack.humb@gmail.com>2017-07-09 03:59:51 +0200
commit60b1880a6248b8d94da0d8d0db638af130557416 (patch)
treeaac7fb12c00e0f11582d81a86a09d8e5ef9c0cb2
parent67f722c9c8cb077b946dfb2b6a3b538e37f3aa8c (diff)
downloadqmk_firmware-60b1880a6248b8d94da0d8d0db638af130557416.tar.gz
qmk_firmware-60b1880a6248b8d94da0d8d0db638af130557416.tar.xz
Reference issue #1476 from the unit tests
-rw-r--r--tests/basic/test_keypress.cpp4
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/basic/test_keypress.cpp b/tests/basic/test_keypress.cpp
index 460c38385..d85cba446 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)));