diff options
author | Fred Sundvik <fsundvik@gmail.com> | 2017-07-08 19:23:39 +0300 |
---|---|---|
committer | Jack Humbert <jack.humb@gmail.com> | 2017-07-08 21:59:51 -0400 |
commit | 60b1880a6248b8d94da0d8d0db638af130557416 (patch) | |
tree | aac7fb12c00e0f11582d81a86a09d8e5ef9c0cb2 /tests/basic/test_keypress.cpp | |
parent | 67f722c9c8cb077b946dfb2b6a3b538e37f3aa8c (diff) | |
download | qmk_firmware-60b1880a6248b8d94da0d8d0db638af130557416.tar.gz qmk_firmware-60b1880a6248b8d94da0d8d0db638af130557416.zip |
Reference issue #1476 from the unit tests
Diffstat (limited to 'tests/basic/test_keypress.cpp')
-rw-r--r-- | tests/basic/test_keypress.cpp | 4 |
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) { | |||
42 | press_key(1, 0); | 42 | press_key(1, 0); |
43 | press_key(0, 3); | 43 | press_key(0, 3); |
44 | //Note that QMK only processes one key at a time | 44 | //Note that QMK only processes one key at a time |
45 | //See issue #1476 for more information | ||
45 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B))); | 46 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B))); |
46 | keyboard_task(); | 47 | keyboard_task(); |
47 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B, KC_C))); | 48 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_B, KC_C))); |
@@ -68,6 +69,7 @@ TEST_F(KeyPress, LeftShiftIsReportedCorrectly) { | |||
68 | press_key(3, 0); | 69 | press_key(3, 0); |
69 | press_key(0, 0); | 70 | press_key(0, 0); |
70 | // Unfortunately modifiers are also processed in the wrong order | 71 | // Unfortunately modifiers are also processed in the wrong order |
72 | // See issue #1476 for more information | ||
71 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A))); | 73 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A))); |
72 | keyboard_task(); | 74 | keyboard_task(); |
73 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A, KC_LSFT))); | 75 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_A, KC_LSFT))); |
@@ -85,6 +87,7 @@ TEST_F(KeyPress, PressLeftShiftAndControl) { | |||
85 | press_key(3, 0); | 87 | press_key(3, 0); |
86 | press_key(5, 0); | 88 | press_key(5, 0); |
87 | // Unfortunately modifiers are also processed in the wrong order | 89 | // Unfortunately modifiers are also processed in the wrong order |
90 | // See issue #1476 for more information | ||
88 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); | 91 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); |
89 | keyboard_task(); | 92 | keyboard_task(); |
90 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_LCTRL))); | 93 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_LCTRL))); |
@@ -96,6 +99,7 @@ TEST_F(KeyPress, LeftAndRightShiftCanBePressedAtTheSameTime) { | |||
96 | press_key(3, 0); | 99 | press_key(3, 0); |
97 | press_key(4, 0); | 100 | press_key(4, 0); |
98 | // Unfortunately modifiers are also processed in the wrong order | 101 | // Unfortunately modifiers are also processed in the wrong order |
102 | // See issue #1476 for more information | ||
99 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); | 103 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT))); |
100 | keyboard_task(); | 104 | keyboard_task(); |
101 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_RSFT))); | 105 | EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport(KC_LSFT, KC_RSFT))); |