diff options
Diffstat (limited to 'tests/basic/test_keypress.cpp')
| -rw-r--r-- | tests/basic/test_keypress.cpp | 22 |
1 files changed, 0 insertions, 22 deletions
diff --git a/tests/basic/test_keypress.cpp b/tests/basic/test_keypress.cpp index bb68ced557..6d5b502a00 100644 --- a/tests/basic/test_keypress.cpp +++ b/tests/basic/test_keypress.cpp | |||
| @@ -64,11 +64,7 @@ TEST_F(KeyPress, CorrectKeysAreReportedWhenTwoKeysArePressed) { | |||
| 64 | 64 | ||
| 65 | key_b.press(); | 65 | key_b.press(); |
| 66 | key_c.press(); | 66 | key_c.press(); |
| 67 | // Note that QMK only processes one key at a time | ||
| 68 | // See issue #1476 for more information | ||
| 69 | EXPECT_REPORT(driver, (key_b.report_code)); | 67 | EXPECT_REPORT(driver, (key_b.report_code)); |
| 70 | keyboard_task(); | ||
| 71 | |||
| 72 | EXPECT_REPORT(driver, (key_b.report_code, key_c.report_code)); | 68 | EXPECT_REPORT(driver, (key_b.report_code, key_c.report_code)); |
| 73 | keyboard_task(); | 69 | keyboard_task(); |
| 74 | 70 | ||
| @@ -76,8 +72,6 @@ TEST_F(KeyPress, CorrectKeysAreReportedWhenTwoKeysArePressed) { | |||
| 76 | key_c.release(); | 72 | key_c.release(); |
| 77 | // Note that the first key released is the first one in the matrix order | 73 | // Note that the first key released is the first one in the matrix order |
| 78 | EXPECT_REPORT(driver, (key_c.report_code)); | 74 | EXPECT_REPORT(driver, (key_c.report_code)); |
| 79 | keyboard_task(); | ||
| 80 | |||
| 81 | EXPECT_EMPTY_REPORT(driver); | 75 | EXPECT_EMPTY_REPORT(driver); |
| 82 | keyboard_task(); | 76 | keyboard_task(); |
| 83 | } | 77 | } |
| @@ -92,10 +86,7 @@ TEST_F(KeyPress, LeftShiftIsReportedCorrectly) { | |||
| 92 | key_lsft.press(); | 86 | key_lsft.press(); |
| 93 | key_a.press(); | 87 | key_a.press(); |
| 94 | 88 | ||
| 95 | // Unfortunately modifiers are also processed in the wrong order | ||
| 96 | // See issue #1476 for more information | ||
| 97 | EXPECT_REPORT(driver, (key_a.report_code)); | 89 | EXPECT_REPORT(driver, (key_a.report_code)); |
| 98 | keyboard_task(); | ||
| 99 | EXPECT_REPORT(driver, (key_a.report_code, key_lsft.report_code)); | 90 | EXPECT_REPORT(driver, (key_a.report_code, key_lsft.report_code)); |
| 100 | keyboard_task(); | 91 | keyboard_task(); |
| 101 | 92 | ||
| @@ -118,11 +109,7 @@ TEST_F(KeyPress, PressLeftShiftAndControl) { | |||
| 118 | key_lsft.press(); | 109 | key_lsft.press(); |
| 119 | key_lctrl.press(); | 110 | key_lctrl.press(); |
| 120 | 111 | ||
| 121 | // Unfortunately modifiers are also processed in the wrong order | ||
| 122 | // See issue #1476 for more information | ||
| 123 | EXPECT_REPORT(driver, (key_lsft.report_code)); | 112 | EXPECT_REPORT(driver, (key_lsft.report_code)); |
| 124 | keyboard_task(); | ||
| 125 | |||
| 126 | EXPECT_REPORT(driver, (key_lsft.report_code, key_lctrl.report_code)); | 113 | EXPECT_REPORT(driver, (key_lsft.report_code, key_lctrl.report_code)); |
| 127 | keyboard_task(); | 114 | keyboard_task(); |
| 128 | 115 | ||
| @@ -130,8 +117,6 @@ TEST_F(KeyPress, PressLeftShiftAndControl) { | |||
| 130 | key_lctrl.release(); | 117 | key_lctrl.release(); |
| 131 | 118 | ||
| 132 | EXPECT_REPORT(driver, (key_lctrl.report_code)); | 119 | EXPECT_REPORT(driver, (key_lctrl.report_code)); |
| 133 | keyboard_task(); | ||
| 134 | |||
| 135 | EXPECT_EMPTY_REPORT(driver); | 120 | EXPECT_EMPTY_REPORT(driver); |
| 136 | keyboard_task(); | 121 | keyboard_task(); |
| 137 | } | 122 | } |
| @@ -145,20 +130,13 @@ TEST_F(KeyPress, LeftAndRightShiftCanBePressedAtTheSameTime) { | |||
| 145 | 130 | ||
| 146 | key_lsft.press(); | 131 | key_lsft.press(); |
| 147 | key_rsft.press(); | 132 | key_rsft.press(); |
| 148 | // Unfortunately modifiers are also processed in the wrong order | ||
| 149 | // See issue #1476 for more information | ||
| 150 | EXPECT_REPORT(driver, (key_lsft.report_code)); | 133 | EXPECT_REPORT(driver, (key_lsft.report_code)); |
| 151 | keyboard_task(); | ||
| 152 | |||
| 153 | EXPECT_REPORT(driver, (key_lsft.report_code, key_rsft.report_code)); | 134 | EXPECT_REPORT(driver, (key_lsft.report_code, key_rsft.report_code)); |
| 154 | keyboard_task(); | 135 | keyboard_task(); |
| 155 | 136 | ||
| 156 | key_lsft.release(); | 137 | key_lsft.release(); |
| 157 | key_rsft.release(); | 138 | key_rsft.release(); |
| 158 | |||
| 159 | EXPECT_REPORT(driver, (key_rsft.report_code)); | 139 | EXPECT_REPORT(driver, (key_rsft.report_code)); |
| 160 | keyboard_task(); | ||
| 161 | |||
| 162 | EXPECT_EMPTY_REPORT(driver); | 140 | EXPECT_EMPTY_REPORT(driver); |
| 163 | keyboard_task(); | 141 | keyboard_task(); |
| 164 | } | 142 | } |
