summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorPascal Getreuer <50221757+getreuer@users.noreply.github.com>2023-05-20 05:35:06 -0700
committerGitHub <noreply@github.com>2023-05-20 22:35:06 +1000
commit3993b15f054265071730cdb450f43457dcf4c64a (patch)
tree61c5b980ed14428bae3c0278c27937dbb5d33627 /tests
parente1766df185869d8a591228d37f3f7b6d5b4049b4 (diff)
[Core] Add Repeat Key ("repeat last key") as a core feature. (#19700)
Co-authored-by: casuanoob <96005765+casuanoob@users.noreply.github.com> Co-authored-by: Sergey Vlasov <sigprof@gmail.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/repeat_key/alt_repeat_key/config.h18
-rw-r--r--tests/repeat_key/alt_repeat_key/test.mk18
-rw-r--r--tests/repeat_key/alt_repeat_key/test_alt_repeat_key.cpp523
-rw-r--r--tests/repeat_key/config.h20
-rw-r--r--tests/repeat_key/repeat_key_combo/config.h18
-rw-r--r--tests/repeat_key/repeat_key_combo/test.mk18
-rw-r--r--tests/repeat_key/repeat_key_combo/test_repeat_key_combo.cpp67
-rw-r--r--tests/repeat_key/test.mk18
-rw-r--r--tests/repeat_key/test_repeat_key.cpp754
-rw-r--r--tests/test_common/keycode_table.cpp2
-rw-r--r--tests/test_common/test_driver.hpp12
11 files changed, 1468 insertions, 0 deletions
diff --git a/tests/repeat_key/alt_repeat_key/config.h b/tests/repeat_key/alt_repeat_key/config.h
new file mode 100644
index 0000000000..d0c4ddadbd
--- /dev/null
+++ b/tests/repeat_key/alt_repeat_key/config.h
@@ -0,0 +1,18 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#pragma once
+
+#include "test_common.h"
diff --git a/tests/repeat_key/alt_repeat_key/test.mk b/tests/repeat_key/alt_repeat_key/test.mk
new file mode 100644
index 0000000000..080871c816
--- /dev/null
+++ b/tests/repeat_key/alt_repeat_key/test.mk
@@ -0,0 +1,18 @@
+# Copyright 2023 Google LLC
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+REPEAT_KEY_ENABLE = yes
+
+EXTRAKEY_ENABLE = yes
diff --git a/tests/repeat_key/alt_repeat_key/test_alt_repeat_key.cpp b/tests/repeat_key/alt_repeat_key/test_alt_repeat_key.cpp
new file mode 100644
index 0000000000..ae525acb45
--- /dev/null
+++ b/tests/repeat_key/alt_repeat_key/test_alt_repeat_key.cpp
@@ -0,0 +1,523 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#include <functional>
+
+#include "keyboard_report_util.hpp"
+#include "keycode.h"
+#include "test_common.hpp"
+#include "test_fixture.hpp"
+#include "test_keymap_key.hpp"
+
+using ::testing::AnyNumber;
+using ::testing::InSequence;
+
+namespace {
+
+bool process_record_user_default(uint16_t keycode, keyrecord_t* record) {
+ return true;
+}
+
+bool remember_last_key_user_default(uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ return true;
+}
+
+uint16_t get_alt_repeat_key_keycode_user_default(uint16_t keycode, uint8_t mods) {
+ return KC_TRNS;
+}
+
+// Indirections so that process_record_user() can be replaced with other
+// functions in the test cases below.
+std::function<bool(uint16_t, keyrecord_t*)> process_record_user_fun = process_record_user_default;
+std::function<bool(uint16_t, keyrecord_t*, uint8_t*)> remember_last_key_user_fun = remember_last_key_user_default;
+std::function<uint16_t(uint16_t, uint8_t)> get_alt_repeat_key_keycode_user_fun = get_alt_repeat_key_keycode_user_default;
+
+extern "C" bool process_record_user(uint16_t keycode, keyrecord_t* record) {
+ return process_record_user_fun(keycode, record);
+}
+
+extern "C" bool remember_last_key_user(uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ return remember_last_key_user_fun(keycode, record, remembered_mods);
+}
+
+extern "C" uint16_t get_alt_repeat_key_keycode_user(uint16_t keycode, uint8_t mods) {
+ return get_alt_repeat_key_keycode_user_fun(keycode, mods);
+}
+
+class AltRepeatKey : public TestFixture {
+ public:
+ bool process_record_user_was_called_;
+
+ void SetUp() override {
+ process_record_user_fun = process_record_user_default;
+ remember_last_key_user_fun = remember_last_key_user_default;
+ get_alt_repeat_key_keycode_user_fun = get_alt_repeat_key_keycode_user_default;
+ }
+
+ void ExpectProcessRecordUserCalledWith(bool expected_press, uint16_t expected_keycode, int8_t expected_repeat_key_count) {
+ process_record_user_was_called_ = false;
+ process_record_user_fun = [=](uint16_t keycode, keyrecord_t* record) {
+ EXPECT_EQ(record->event.pressed, expected_press);
+ EXPECT_KEYCODE_EQ(keycode, expected_keycode);
+ EXPECT_EQ(get_repeat_key_count(), expected_repeat_key_count);
+ // Tests below use this to verify process_record_user() was called.
+ process_record_user_was_called_ = true;
+ return true;
+ };
+ }
+
+ // Expects that the characters of `s` are sent.
+ // NOTE: This implementation is limited to chars a-z, A-Z.
+ void ExpectString(TestDriver& driver, const std::string& s) {
+ InSequence seq;
+ for (int c : s) {
+ switch (c) {
+ case 'a' ... 'z': { // Lowercase letter.
+ uint16_t keycode = c - ('a' - KC_A);
+ EXPECT_REPORT(driver, (keycode));
+ } break;
+
+ case 'A' ... 'Z': { // Capital letter = KC_LSFT + letter key.
+ uint16_t keycode = c - ('A' - KC_A);
+ EXPECT_REPORT(driver, (KC_LSFT, keycode));
+ } break;
+ }
+ }
+ }
+};
+
+TEST_F(AltRepeatKey, AlternateBasic) {
+ TestDriver driver;
+ KeymapKey key_bspc(0, 0, 0, KC_BSPC);
+ KeymapKey key_pgdn(0, 1, 0, KC_PGDN);
+ KeymapKey key_pgup(0, 2, 0, KC_PGUP);
+ KeymapKey key_repeat(0, 4, 0, QK_REP);
+ KeymapKey key_alt_repeat(0, 5, 0, QK_AREP);
+ set_keymap({key_bspc, key_pgdn, key_pgup, key_repeat, key_alt_repeat});
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_BSPC));
+ EXPECT_REPORT(driver, (KC_DEL));
+ EXPECT_REPORT(driver, (KC_DEL));
+ EXPECT_REPORT(driver, (KC_BSPC));
+ EXPECT_REPORT(driver, (KC_DEL));
+ EXPECT_REPORT(driver, (KC_PGDN));
+ EXPECT_REPORT(driver, (KC_PGUP));
+ EXPECT_REPORT(driver, (KC_PGUP));
+ EXPECT_REPORT(driver, (KC_PGDN));
+ }
+
+ tap_key(key_bspc);
+
+ for (int n = 1; n <= 2; ++n) { // Tap the Alternate Repeat Key twice.
+ ExpectProcessRecordUserCalledWith(true, KC_DEL, -n);
+ key_alt_repeat.press(); // Press the Alternate Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ // Expect the corresponding release event.
+ ExpectProcessRecordUserCalledWith(false, KC_DEL, -n);
+ key_alt_repeat.release(); // Release the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+ }
+
+ process_record_user_fun = process_record_user_default;
+ tap_keys(key_repeat, key_alt_repeat);
+ tap_keys(key_pgdn, key_alt_repeat);
+ tap_keys(key_pgup, key_alt_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+struct TestParamsAlternateKeyCodes {
+ uint16_t keycode;
+ uint8_t mods;
+ uint16_t expected_alt_keycode;
+};
+
+// Tests `get_alt_repeat_key_keycode()` for various keycodes.
+TEST_F(AltRepeatKey, GetAltRepeatKeyKeycode) {
+ for (const auto& params : std::vector<TestParamsAlternateKeyCodes>({
+ // clang-format off
+ // Each line tests one call to `get_alt_repeat_key_keycode()`:
+ // {keycode, mods, expected_alt_keycode}.
+ // Arrows.
+ {KC_LEFT, 0, KC_RGHT},
+ {KC_RGHT, 0, KC_LEFT},
+ {KC_LEFT, MOD_BIT(KC_LSFT), LSFT(KC_RGHT)},
+ {KC_LEFT, MOD_BIT(KC_RSFT), RSFT(KC_RGHT)},
+ {KC_LEFT, MOD_BIT(KC_LCTL) | MOD_BIT(KC_LSFT), C(S(KC_RGHT))},
+ {KC_LEFT, MOD_BIT(KC_LGUI), LGUI(KC_RGHT)},
+ {C(KC_LEFT), MOD_BIT(KC_LSFT), C(S(KC_RGHT))},
+ {KC_UP, 0, KC_DOWN},
+ // Navigation keys.
+ {KC_PGUP, 0, KC_PGDN},
+ {KC_HOME, 0, KC_END },
+ // Media keys.
+ {KC_WBAK, 0, KC_WFWD},
+ {KC_MNXT, 0, KC_MPRV},
+ {KC_MRWD, 0, KC_MFFD},
+ {KC_VOLU, 0, KC_VOLD},
+ {KC_BRIU, 0, KC_BRID},
+ // Emacs navigation.
+ {KC_N, MOD_BIT(KC_LCTL), C(KC_P)},
+ {KC_B, MOD_BIT(KC_LCTL), LCTL(KC_F)},
+ {KC_B, MOD_BIT(KC_RCTL), RCTL(KC_F)},
+ {KC_B, MOD_BIT(KC_LALT), LALT(KC_F)},
+ {KC_F, MOD_BIT(KC_LCTL), C(KC_B)},
+ {KC_A, MOD_BIT(KC_LCTL), C(KC_E)},
+ {KC_D, MOD_BIT(KC_LCTL), C(KC_U)},
+ // Vim navigation.
+ {KC_J, 0, KC_K},
+ {KC_K, 0, KC_J},
+ {KC_H, 0, KC_L},
+ {KC_B, 0, KC_W},
+ {KC_W, 0, KC_B},
+ {KC_E, 0, KC_B},
+ {KC_B, MOD_BIT(KC_LSFT), S(KC_W)},
+ {KC_W, MOD_BIT(KC_LSFT), S(KC_B)},
+ {KC_E, MOD_BIT(KC_LSFT), S(KC_B)},
+ {KC_O, MOD_BIT(KC_LCTL), C(KC_I)},
+ {KC_I, MOD_BIT(KC_LCTL), C(KC_O)},
+ // Other.
+ {KC_DEL, 0, KC_BSPC},
+ {KC_LBRC, 0, KC_RBRC},
+ {KC_LCBR, 0, KC_RCBR},
+ // Some keys where the last key is a tap-hold key.
+ {LSFT_T(KC_F), MOD_BIT(KC_RCTL), RCTL(KC_B)},
+ {LT(1, KC_A), MOD_BIT(KC_RGUI), RGUI(KC_E)},
+ {RALT_T(KC_J), 0, KC_K},
+ // Some keys where no alternate is defined.
+ {KC_A, 0, KC_NO},
+ {KC_F1, 0, KC_NO},
+ {QK_LEAD, 0, KC_NO},
+ {MO(1), 0, KC_NO},
+ // clang-format on
+ })) {
+ SCOPED_TRACE(std::string("Input keycode: ") + get_keycode_identifier_or_default(params.keycode));
+ set_last_keycode(params.keycode);
+ set_last_mods(params.mods);
+
+ const uint16_t actual = get_alt_repeat_key_keycode();
+
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), params.expected_alt_keycode);
+ }
+}
+
+// Test adding to and overriding the above through the
+// `get_alt_repeat_key_keycode_user()` callback.
+TEST_F(AltRepeatKey, GetAltRepeatKeyKeycodeUser) {
+ get_alt_repeat_key_keycode_user_fun = [](uint16_t keycode, uint8_t mods) -> uint16_t {
+ bool shifted = (mods & MOD_MASK_SHIFT);
+ switch (keycode) {
+ case KC_LEFT:
+ return KC_ENT;
+ case MO(1):
+ return TG(1);
+ case KC_TAB: // Tab <-> Shift + Tab example.
+ if (shifted) {
+ return KC_TAB;
+ } else {
+ return S(KC_TAB);
+ }
+ }
+
+ // Ctrl + Y <-> Ctrl + Z example.
+ if ((mods & MOD_MASK_CTRL)) {
+ switch (keycode) {
+ case KC_Y:
+ return C(KC_Z);
+ case KC_Z:
+ return C(KC_Y);
+ }
+ }
+
+ return KC_NO;
+ };
+
+ set_last_keycode(KC_LEFT);
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), KC_ENT);
+
+ set_last_keycode(MO(1));
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), TG(1));
+
+ set_last_keycode(KC_TAB);
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), S(KC_TAB));
+
+ set_last_keycode(KC_TAB);
+ set_last_mods(MOD_BIT(KC_LSFT));
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), KC_TAB);
+
+ set_last_keycode(KC_Z);
+ set_last_mods(MOD_BIT(KC_LCTL));
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), C(KC_Y));
+
+ set_last_keycode(KC_Y);
+ set_last_mods(MOD_BIT(KC_LCTL));
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), C(KC_Z));
+}
+
+// Tests rolling from a key to Alternate Repeat.
+TEST_F(AltRepeatKey, RollingToAltRepeat) {
+ TestDriver driver;
+ KeymapKey key_left(0, 0, 0, KC_LEFT);
+ KeymapKey key_alt_repeat(0, 1, 0, QK_AREP);
+ set_keymap({key_left, key_alt_repeat});
+
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LEFT, KC_RGHT));
+ EXPECT_REPORT(driver, (KC_RGHT));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_RGHT));
+ EXPECT_EMPTY_REPORT(driver);
+ }
+
+ // Perform a rolled press from Left to Alternate Repeat.
+
+ ExpectProcessRecordUserCalledWith(true, KC_LEFT, 0);
+ key_left.press();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(true, KC_RGHT, -1);
+ key_alt_repeat.press(); // Press the Alternate Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_LEFT, 0);
+ key_left.release();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_RGHT, -1);
+ key_alt_repeat.release(); // Release the Alternate Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ process_record_user_fun = process_record_user_default;
+ tap_key(key_alt_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests rolling from Alternate Repeat to another key.
+TEST_F(AltRepeatKey, RollingFromAltRepeat) {
+ TestDriver driver;
+ KeymapKey key_left(0, 0, 0, KC_LEFT);
+ KeymapKey key_up(0, 1, 0, KC_UP);
+ KeymapKey key_alt_repeat(0, 2, 0, QK_AREP);
+ set_keymap({key_left, key_up, key_alt_repeat});
+
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_LEFT));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_RGHT));
+ EXPECT_REPORT(driver, (KC_RGHT, KC_UP));
+ EXPECT_REPORT(driver, (KC_UP));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_DOWN));
+ EXPECT_EMPTY_REPORT(driver);
+ }
+
+ tap_key(key_left);
+
+ // Perform a rolled press from Alternate Repeat to Up.
+
+ ExpectProcessRecordUserCalledWith(true, KC_RGHT, -1);
+ key_alt_repeat.press(); // Press the Alternate Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(true, KC_UP, 0);
+ key_up.press();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_UP);
+
+ ExpectProcessRecordUserCalledWith(false, KC_RGHT, -1);
+ key_alt_repeat.release(); // Release the Alternate Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_UP, 0);
+ key_up.release();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ process_record_user_fun = process_record_user_default;
+ tap_key(key_alt_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests using the Alternate Repeat Key on a macro that doesn't have an
+// alternate keycode defined.
+TEST_F(AltRepeatKey, AlternateUnsupportedMacro) {
+ TestDriver driver;
+ KeymapKey key_foo(0, 0, 0, QK_USER_0);
+ KeymapKey key_alt_repeat(0, 1, 0, QK_AREP);
+ set_keymap({key_foo, key_alt_repeat});
+
+ process_record_user_fun = [=](uint16_t keycode, keyrecord_t* record) {
+ process_record_user_was_called_ = true;
+ switch (keycode) {
+ case QK_USER_0:
+ if (record->event.pressed) {
+ SEND_STRING("foo");
+ }
+ break;
+ }
+ return true;
+ };
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "foofoo");
+
+ process_record_user_was_called_ = false;
+ tap_key(key_foo);
+
+ EXPECT_TRUE(process_record_user_was_called_);
+ EXPECT_KEYCODE_EQ(get_last_keycode(), QK_USER_0);
+ EXPECT_KEYCODE_EQ(get_alt_repeat_key_keycode(), KC_NO);
+
+ process_record_user_was_called_ = false;
+ key_alt_repeat.press(); // Press Alternate Repeat.
+ run_one_scan_loop();
+
+ EXPECT_FALSE(process_record_user_was_called_);
+
+ process_record_user_was_called_ = false;
+ key_alt_repeat.release(); // Release Alternate Repeat.
+ run_one_scan_loop();
+
+ EXPECT_FALSE(process_record_user_was_called_);
+
+ process_record_user_was_called_ = false;
+ tap_key(key_foo);
+
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests a macro with custom alternate behavior.
+TEST_F(AltRepeatKey, MacroCustomAlternate) {
+ TestDriver driver;
+ KeymapKey key_foo(0, 0, 0, QK_USER_0);
+ KeymapKey key_alt_repeat(0, 1, 0, QK_AREP);
+ set_keymap({key_foo, key_alt_repeat});
+
+ get_alt_repeat_key_keycode_user_fun = [](uint16_t keycode, uint8_t mods) -> uint16_t {
+ switch (keycode) {
+ case QK_USER_0:
+ return QK_USER_0; // QK_USER_0 handles its own alternate.
+ default:
+ return KC_NO; // No key by default.
+ }
+ };
+ process_record_user_fun = [=](uint16_t keycode, keyrecord_t* record) {
+ process_record_user_was_called_ = true;
+ switch (keycode) {
+ case QK_USER_0:
+ if (record->event.pressed) {
+ if (get_repeat_key_count() >= 0) {
+ SEND_STRING("foo");
+ } else { // Key is being alternate repeated.
+ SEND_STRING("bar");
+ }
+ }
+ break;
+ }
+ return true;
+ };
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "foobarbar");
+
+ tap_keys(key_foo, key_alt_repeat, key_alt_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests the Additional "Alternate" keys example from the documentation page.
+TEST_F(AltRepeatKey, AdditionalAlternateKeysExample) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_w(0, 1, 0, KC_W);
+ KeymapKey key_altrep2(0, 2, 0, QK_USER_0);
+ KeymapKey key_altrep3(0, 3, 0, QK_USER_1);
+ set_keymap({key_a, key_w, key_altrep2, key_altrep3});
+
+ remember_last_key_user_fun = [](uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ switch (keycode) {
+ case QK_USER_0:
+ case QK_USER_1:
+ return false; // Ignore ALTREP keys.
+ }
+ return true; // Other keys can be repeated.
+ };
+ process_record_user_fun = [=](uint16_t keycode, keyrecord_t* record) {
+ switch (keycode) {
+ case QK_USER_0:
+ if (record->event.pressed) {
+ const uint16_t last_key = get_last_keycode();
+ switch (last_key) {
+ case KC_A:
+ SEND_STRING(/*a*/ "tion");
+ break;
+ case KC_W:
+ SEND_STRING(/*w*/ "hich");
+ break;
+ }
+ }
+ return false;
+ case QK_USER_1:
+ if (record->event.pressed) {
+ const uint16_t last_key = get_last_keycode();
+ switch (last_key) {
+ case KC_A:
+ SEND_STRING(/*a*/ "bout");
+ break;
+ case KC_W:
+ SEND_STRING(/*w*/ "ould");
+ break;
+ }
+ }
+ return false;
+ }
+ return true;
+ };
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "ationwhichaboutwould");
+
+ tap_keys(key_a, key_altrep2, key_w, key_altrep2);
+ tap_keys(key_a, key_altrep3, key_w, key_altrep3);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+} // namespace
diff --git a/tests/repeat_key/config.h b/tests/repeat_key/config.h
new file mode 100644
index 0000000000..003d980c82
--- /dev/null
+++ b/tests/repeat_key/config.h
@@ -0,0 +1,20 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#pragma once
+
+#include "test_common.h"
+
+#define NO_ALT_REPEAT_KEY
diff --git a/tests/repeat_key/repeat_key_combo/config.h b/tests/repeat_key/repeat_key_combo/config.h
new file mode 100644
index 0000000000..d0c4ddadbd
--- /dev/null
+++ b/tests/repeat_key/repeat_key_combo/config.h
@@ -0,0 +1,18 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#pragma once
+
+#include "test_common.h"
diff --git a/tests/repeat_key/repeat_key_combo/test.mk b/tests/repeat_key/repeat_key_combo/test.mk
new file mode 100644
index 0000000000..db6ea7789a
--- /dev/null
+++ b/tests/repeat_key/repeat_key_combo/test.mk
@@ -0,0 +1,18 @@
+# Copyright 2023 Google LLC
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+REPEAT_KEY_ENABLE = yes
+
+COMBO_ENABLE = yes
diff --git a/tests/repeat_key/repeat_key_combo/test_repeat_key_combo.cpp b/tests/repeat_key/repeat_key_combo/test_repeat_key_combo.cpp
new file mode 100644
index 0000000000..2d2fbaa966
--- /dev/null
+++ b/tests/repeat_key/repeat_key_combo/test_repeat_key_combo.cpp
@@ -0,0 +1,67 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#include "keyboard_report_util.hpp"
+#include "keycode.h"
+#include "test_common.hpp"
+#include "test_fixture.hpp"
+#include "test_keymap_key.hpp"
+
+using ::testing::AnyNumber;
+using ::testing::InSequence;
+
+namespace {
+
+extern "C" {
+// Define a combo: KC_X + KC_Y = KC_Q.
+const uint16_t xy_combo[] PROGMEM = {KC_X, KC_Y, COMBO_END};
+combo_t key_combos[] = {COMBO(xy_combo, KC_Q)};
+uint16_t COMBO_LEN = sizeof(key_combos) / sizeof(*key_combos);
+} // extern "C"
+
+class RepeatKey : public TestFixture {};
+
+// Tests repeating a combo, KC_X + KC_Y = KC_Q, by typing
+// "X, Repeat, Repeat, {X Y}, Repeat, Repeat". This produces "xxxqqq".
+TEST_F(RepeatKey, Combo) {
+ TestDriver driver;
+ KeymapKey key_x(0, 0, 0, KC_X);
+ KeymapKey key_y(0, 1, 0, KC_Y);
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_x, key_y, key_repeat});
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_X));
+ EXPECT_REPORT(driver, (KC_X));
+ EXPECT_REPORT(driver, (KC_X));
+ EXPECT_REPORT(driver, (KC_Q));
+ EXPECT_REPORT(driver, (KC_Q));
+ EXPECT_REPORT(driver, (KC_Q));
+ }
+
+ tap_keys(key_x, key_repeat, key_repeat);
+ tap_combo({key_x, key_y});
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_Q);
+
+ tap_keys(key_repeat, key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+} // namespace
diff --git a/tests/repeat_key/test.mk b/tests/repeat_key/test.mk
new file mode 100644
index 0000000000..aec8ff3bfb
--- /dev/null
+++ b/tests/repeat_key/test.mk
@@ -0,0 +1,18 @@
+# Copyright 2023 Google LLC
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+REPEAT_KEY_ENABLE = yes
+
+AUTO_SHIFT_ENABLE = yes
diff --git a/tests/repeat_key/test_repeat_key.cpp b/tests/repeat_key/test_repeat_key.cpp
new file mode 100644
index 0000000000..eee44fc104
--- /dev/null
+++ b/tests/repeat_key/test_repeat_key.cpp
@@ -0,0 +1,754 @@
+// Copyright 2023 Google LLC
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 2 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+#include <functional>
+
+#include "keyboard_report_util.hpp"
+#include "keycode.h"
+#include "test_common.hpp"
+#include "test_fixture.hpp"
+#include "test_keymap_key.hpp"
+
+using ::testing::AnyNumber;
+using ::testing::AnyOf;
+using ::testing::InSequence;
+
+#define FOO_MACRO SAFE_RANGE
+
+namespace {
+
+bool process_record_user_default(uint16_t keycode, keyrecord_t* record) {
+ return true;
+}
+
+bool remember_last_key_user_default(uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ return true;
+}
+
+// Indirection so that process_record_user() and remember_last_key_user()
+// can be replaced with other functions in the test cases below.
+std::function<bool(uint16_t, keyrecord_t*)> process_record_user_fun = process_record_user_default;
+std::function<bool(uint16_t, keyrecord_t*, uint8_t*)> remember_last_key_user_fun = remember_last_key_user_default;
+
+extern "C" bool process_record_user(uint16_t keycode, keyrecord_t* record) {
+ return process_record_user_fun(keycode, record);
+}
+extern "C" bool remember_last_key_user(uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ return remember_last_key_user_fun(keycode, record, remembered_mods);
+}
+
+class RepeatKey : public TestFixture {
+ public:
+ bool process_record_user_was_called_;
+
+ void SetUp() override {
+ autoshift_disable();
+ process_record_user_fun = process_record_user_default;
+ remember_last_key_user_fun = remember_last_key_user_default;
+ }
+
+ void ExpectProcessRecordUserCalledWith(bool expected_press, uint16_t expected_keycode, int8_t expected_repeat_key_count) {
+ process_record_user_was_called_ = false;
+ process_record_user_fun = [=](uint16_t keycode, keyrecord_t* record) {
+ EXPECT_EQ(record->event.pressed, expected_press);
+ EXPECT_KEYCODE_EQ(keycode, expected_keycode);
+ EXPECT_EQ(get_repeat_key_count(), expected_repeat_key_count);
+ // Tests below use this to verify process_record_user() was called.
+ process_record_user_was_called_ = true;
+ return true;
+ };
+ }
+
+ // Expects that the characters of `s` are sent.
+ // NOTE: This implementation is limited to chars a-z, A-Z.
+ void ExpectString(TestDriver& driver, const std::string& s) {
+ InSequence seq;
+ for (int c : s) {
+ switch (c) {
+ case 'a' ... 'z': { // Lowercase letter.
+ uint16_t keycode = c - ('a' - KC_A);
+ EXPECT_REPORT(driver, (keycode));
+ } break;
+
+ case 'A' ... 'Z': { // Capital letter = KC_LSFT + letter key.
+ uint16_t keycode = c - ('A' - KC_A);
+ EXPECT_REPORT(driver, (KC_LSFT, keycode));
+ } break;
+ }
+ }
+ }
+};
+
+// Tests that "A, Repeat, Repeat, B, Repeat" produces "aaabb".
+TEST_F(RepeatKey, Basic) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_b(0, 1, 0, KC_B);
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_a, key_b, key_repeat});
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "aaabb");
+
+ // When KC_A is pressed, process_record_user() should be called
+ // with a press event with keycode == KC_A and repeat_key_count() == 0.
+ ExpectProcessRecordUserCalledWith(true, KC_A, 0);
+ key_a.press();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ // After pressing A, the keycode of the key to be repeated is KC_A.
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_A);
+ EXPECT_EQ(get_last_mods(), 0);
+
+ // Expect the corresponding release event when A is released.
+ ExpectProcessRecordUserCalledWith(false, KC_A, 0);
+ key_a.release();
+ run_one_scan_loop();
+
+ for (int n = 1; n <= 2; ++n) { // Tap the Repeat Key twice.
+ // When Repeat is pressed, process_record_user() should be called with a
+ // press event with keycode == KC_A and repeat_key_count() == n.
+ ExpectProcessRecordUserCalledWith(true, KC_A, n);
+ key_repeat.press(); // Press the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ // Expect the corresponding release event.
+ ExpectProcessRecordUserCalledWith(false, KC_A, n);
+ key_repeat.release(); // Release the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+ }
+
+ process_record_user_fun = process_record_user_default;
+ tap_key(key_b);
+ // Then after tapping key_b, the keycode to be repeated becomes KC_B.
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_B);
+
+ tap_key(key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests repeating a macro. The keycode FOO_MACRO sends "foo" when pressed. The
+// test taps "FOO_MACRO, Repeat, Repeat", producing "foofoofoo".
+TEST_F(RepeatKey, Macro) {
+ TestDriver driver;
+ KeymapKey key_foo(0, 0, 0, FOO_MACRO);
+ KeymapKey key_repeat(0, 1, 0, QK_REP);
+ set_keymap({key_foo, key_repeat});
+
+ // Define process_record_user() to handle FOO_MACRO.
+ process_record_user_fun = [](uint16_t keycode, keyrecord_t* record) {
+ switch (keycode) {
+ case FOO_MACRO:
+ if (record->event.pressed) {
+ SEND_STRING("foo");
+ }
+ break;
+ }
+ return true;
+ };
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "foofoofoo");
+
+ tap_key(key_foo);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), FOO_MACRO);
+
+ tap_keys(key_repeat, key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests a macro with customized repeat behavior: "foo" is sent normally, "bar"
+// on the first repeat, and "baz" on subsequent repeats. The test taps
+// "FOO_MACRO, Repeat, Repeat, FOO_MACRO, Repeat", producing "foobarbazfoobar".
+TEST_F(RepeatKey, MacroCustomRepeat) {
+ TestDriver driver;
+ KeymapKey key_foo(0, 0, 0, FOO_MACRO);
+ KeymapKey key_repeat(0, 1, 0, QK_REP);
+ set_keymap({key_foo, key_repeat});
+
+ process_record_user_fun = [](uint16_t keycode, keyrecord_t* record) {
+ switch (keycode) {
+ case FOO_MACRO:
+ if (record->event.pressed) {
+ switch (get_repeat_key_count()) {
+ case 0: // When pressed normally.
+ SEND_STRING("foo");
+ break;
+ case 1: // On first repeat.
+ SEND_STRING("bar");
+ break;
+ default: // On subsequent repeats.
+ SEND_STRING("baz");
+ break;
+ }
+ }
+ break;
+ }
+ return true;
+ };
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "foobarbazfoobar");
+
+ tap_key(key_foo);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), FOO_MACRO);
+
+ tap_keys(key_repeat, key_repeat, key_foo, key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests repeating keys on different layers. A 2-layer keymap is defined:
+// Layer 0: QK_REP , MO(1) , KC_A
+// Layer 1: KC_TRNS, KC_TRNS, KC_B
+// The test does the following, which should produce "bbbaaa":
+// 1. Hold MO(1), switching to layer 1.
+// 2. Tap KC_B on layer 1.
+// 3. Release MO(1), switching back to layer 0.
+// 4. Tap Repeat twice.
+// 5. Tap KC_A on layer 0.
+// 6. Hold MO(1), switching to layer 1.
+// 7. Tap Repeat twice.
+TEST_F(RepeatKey, AcrossLayers) {
+ TestDriver driver;
+ KeymapKey key_repeat(0, 0, 0, QK_REP);
+ KeymapKey key_mo_1(0, 1, 0, MO(1));
+ KeymapKey regular_key(0, 2, 0, KC_A);
+ set_keymap({// Layer 0.
+ key_repeat, key_mo_1, regular_key,
+ // Layer 1.
+ KeymapKey{1, 0, 0, KC_TRNS}, KeymapKey{1, 1, 0, KC_TRNS}, KeymapKey{1, 2, 0, KC_B}});
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "bbbaaa");
+
+ key_mo_1.press(); // Hold the MO(1) layer key.
+ run_one_scan_loop();
+ tap_key(regular_key); // Taps the KC_B key on layer 1.
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_B);
+
+ key_mo_1.release(); // Release the layer key.
+ run_one_scan_loop();
+ tap_keys(key_repeat, key_repeat);
+ tap_key(regular_key); // Taps the KC_A key on layer 0.
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_A);
+
+ key_mo_1.press(); // Hold the layer key.
+ run_one_scan_loop();
+ tap_keys(key_repeat, key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests "A(down), Repeat(down), A(up), Repeat(up), Repeat" produces "aaa".
+TEST_F(RepeatKey, RollingToRepeat) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_repeat(0, 1, 0, QK_REP);
+ set_keymap({key_a, key_repeat});
+
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_EMPTY_REPORT(driver);
+ }
+
+ // Perform a rolled press from A to Repeat.
+
+ ExpectProcessRecordUserCalledWith(true, KC_A, 0);
+ key_a.press();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(true, KC_A, 1);
+ key_repeat.press(); // Press the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_A, 0);
+ key_a.release();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_A, 1);
+ key_repeat.release(); // Release the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ process_record_user_fun = process_record_user_default;
+ tap_key(key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests "A, Repeat(down), B(down), Repeat(up), B(up), Repeat" produces "aabb".
+TEST_F(RepeatKey, RollingFromRepeat) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_b(0, 1, 0, KC_B);
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_a, key_b, key_repeat});
+
+ {
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_REPORT(driver, (KC_A, KC_B));
+ EXPECT_REPORT(driver, (KC_B));
+ EXPECT_EMPTY_REPORT(driver);
+ EXPECT_REPORT(driver, (KC_B));
+ EXPECT_EMPTY_REPORT(driver);
+ }
+
+ tap_key(key_a);
+
+ // Perform a rolled press from Repeat to B.
+
+ ExpectProcessRecordUserCalledWith(true, KC_A, 1);
+ key_repeat.press(); // Press the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(true, KC_B, 0);
+ key_b.press();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_B);
+
+ ExpectProcessRecordUserCalledWith(false, KC_A, 1);
+ key_repeat.release(); // Release the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ ExpectProcessRecordUserCalledWith(false, KC_B, 0);
+ key_b.release();
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ process_record_user_fun = process_record_user_default;
+ tap_key(key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests Repeat Key with a modifier, types "AltGr+C, Repeat, Repeat, C".
+TEST_F(RepeatKey, RecallMods) {
+ TestDriver driver;
+ KeymapKey key_c(0, 0, 0, KC_C);
+ KeymapKey key_altgr(0, 1, 0, KC_RALT);
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_c, key_altgr, key_repeat});
+
+ // Allow any number of reports with no keys or only KC_RALT.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_RALT))))
+ .Times(AnyNumber());
+ // clang-format on
+
+ { // Expect: "AltGr+C, AltGr+C, AltGr+C, C".
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_RALT, KC_C));
+ EXPECT_REPORT(driver, (KC_RALT, KC_C));
+ EXPECT_REPORT(driver, (KC_RALT, KC_C));
+ EXPECT_REPORT(driver, (KC_C));
+ }
+
+ key_altgr.press();
+ run_one_scan_loop();
+ tap_key(key_c);
+ key_altgr.release();
+ run_one_scan_loop();
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_C);
+ EXPECT_EQ(get_last_mods(), MOD_BIT(KC_RALT));
+
+ tap_keys(key_repeat, key_repeat, key_c);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests that Repeat Key stacks mods, types
+// "Ctrl+Left, Repeat, Shift+Repeat, Shift+Repeat, Repeat, Left".
+TEST_F(RepeatKey, StackMods) {
+ TestDriver driver;
+ KeymapKey key_left(0, 0, 0, KC_LEFT);
+ KeymapKey key_shift(0, 1, 0, KC_LSFT);
+ KeymapKey key_ctrl(0, 2, 0, KC_LCTL);
+ KeymapKey key_repeat(0, 3, 0, QK_REP);
+ set_keymap({key_left, key_shift, key_ctrl, key_repeat});
+
+ // Allow any number of reports with no keys or only mods.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LCTL),
+ KeyboardReport(KC_LSFT),
+ KeyboardReport(KC_LCTL, KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+
+ { // Expect: "Ctrl+Left, Ctrl+Shift+Left".
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LSFT, KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LSFT, KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LEFT));
+ EXPECT_REPORT(driver, (KC_LEFT));
+ }
+
+ key_ctrl.press();
+ run_one_scan_loop();
+ tap_key(key_left);
+ run_one_scan_loop();
+ key_ctrl.release();
+ run_one_scan_loop();
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_LEFT);
+ EXPECT_EQ(get_last_mods(), MOD_BIT(KC_LCTL));
+
+ tap_key(key_repeat);
+
+ key_shift.press();
+ run_one_scan_loop();
+ tap_keys(key_repeat, key_repeat);
+ key_shift.release();
+ run_one_scan_loop();
+
+ EXPECT_EQ(get_last_mods(), MOD_BIT(KC_LCTL));
+
+ tap_keys(key_repeat, key_left);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Types: "S(KC_1), Repeat, Ctrl+Repeat, Ctrl+Repeat, Repeat, KC_2".
+TEST_F(RepeatKey, ShiftedKeycode) {
+ TestDriver driver;
+ KeymapKey key_exlm(0, 0, 0, S(KC_1));
+ KeymapKey key_2(0, 1, 0, KC_2);
+ KeymapKey key_ctrl(0, 2, 0, KC_LCTL);
+ KeymapKey key_repeat(0, 3, 0, QK_REP);
+ set_keymap({key_exlm, key_2, key_ctrl, key_repeat});
+
+ // Allow any number of reports with no keys or only mods.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LCTL),
+ KeyboardReport(KC_LSFT),
+ KeyboardReport(KC_LCTL, KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+
+ { // Expect: "Shift+1, Shift+1, Ctrl+Shift+1, Ctrl+Shift+1, Shift+1, 2".
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_LSFT, KC_1));
+ EXPECT_REPORT(driver, (KC_LSFT, KC_1));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LSFT, KC_1));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_LSFT, KC_1));
+ EXPECT_REPORT(driver, (KC_LSFT, KC_1));
+ EXPECT_REPORT(driver, (KC_2));
+ }
+
+ tap_key(key_exlm);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), S(KC_1));
+
+ tap_key(key_repeat);
+
+ key_ctrl.press();
+ run_one_scan_loop();
+ tap_keys(key_repeat, key_repeat);
+ key_ctrl.release();
+ run_one_scan_loop();
+
+ tap_keys(key_repeat, key_2);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests Repeat Key with a one-shot Shift, types
+// "A, OSM(MOD_LSFT), Repeat, Repeat".
+TEST_F(RepeatKey, WithOneShotShift) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_oneshot_shift(0, 1, 0, OSM(MOD_LSFT));
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_a, key_oneshot_shift, key_repeat});
+
+ // Allow any number of reports with no keys or only KC_RALT.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+ ExpectString(driver, "aAa");
+
+ tap_keys(key_a, key_oneshot_shift, key_repeat, key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests Repeat Key with a mod-tap key, types
+// "A, Repeat, Repeat, A(down), Repeat, Repeat, A(up), Repeat".
+TEST_F(RepeatKey, ModTap) {
+ TestDriver driver;
+ KeymapKey key_mt_a(0, 0, 0, LSFT_T(KC_A));
+ KeymapKey key_repeat(0, 1, 0, QK_REP);
+ set_keymap({key_mt_a, key_repeat});
+
+ // Allow any number of reports with no keys or only KC_LSFT.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+ ExpectString(driver, "aaaAAa");
+
+ tap_key(key_mt_a);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), LSFT_T(KC_A));
+
+ tap_keys(key_repeat, key_repeat);
+ key_mt_a.press();
+ run_one_scan_loop();
+ tap_key(key_repeat, TAPPING_TERM + 1);
+ tap_key(key_repeat);
+ key_mt_a.release();
+ run_one_scan_loop();
+ tap_key(key_repeat);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests with Auto Shift. When repeating an autoshiftable key, it does not
+// matter how long the original key was held, rather, quickly tapping vs.
+// long-pressing the Repeat Key determines whether the shifted key is repeated.
+//
+// The test does the following, which should produce "aaABbB":
+// 1. Tap KC_A quickly.
+// 2. Tap Repeat Key quickly.
+// 3. Long-press Repeat Key.
+// 4. Long-press KC_B.
+// 5. Tap Repeat Key quickly.
+// 6. Long-press Repeat Key.
+TEST_F(RepeatKey, AutoShift) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_b(0, 1, 0, KC_B);
+ KeymapKey key_repeat(0, 2, 0, QK_REP);
+ set_keymap({key_a, key_b, key_repeat});
+
+ autoshift_enable();
+
+ // Allow any number of reports with no keys or only KC_LSFT.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+ ExpectString(driver, "aaABbB");
+
+ tap_key(key_a); // Tap A quickly.
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_A);
+ EXPECT_EQ(get_last_mods(), 0);
+
+ tap_key(key_repeat);
+ tap_key(key_repeat, AUTO_SHIFT_TIMEOUT + 1);
+
+ tap_key(key_b, AUTO_SHIFT_TIMEOUT + 1); // Long press B.
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_B);
+ EXPECT_EQ(get_last_mods(), 0);
+
+ tap_key(key_repeat);
+ tap_key(key_repeat, AUTO_SHIFT_TIMEOUT + 1);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Defines `remember_last_key_user()` to forget the Shift mod and types:
+// "Ctrl+A, Repeat, Shift+A, Repeat, Shift+Repeat".
+TEST_F(RepeatKey, FilterRememberedMods) {
+ TestDriver driver;
+ KeymapKey key_a(0, 0, 0, KC_A);
+ KeymapKey key_ctrl(0, 1, 0, KC_LCTL);
+ KeymapKey key_shift(0, 2, 0, KC_LSFT);
+ KeymapKey key_repeat(0, 3, 0, QK_REP);
+ set_keymap({key_a, key_ctrl, key_shift, key_repeat});
+
+ remember_last_key_user_fun = [](uint16_t keycode, keyrecord_t* record, uint8_t* remembered_mods) {
+ *remembered_mods &= ~MOD_MASK_SHIFT;
+ return true;
+ };
+
+ // Allow any number of reports with no keys or only mods.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LCTL),
+ KeyboardReport(KC_LSFT),
+ KeyboardReport(KC_LCTL, KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+
+ { // Expect: "Ctrl+A, Ctrl+A, Shift+A, A, Shift+A".
+ InSequence seq;
+ EXPECT_REPORT(driver, (KC_LCTL, KC_A));
+ EXPECT_REPORT(driver, (KC_LCTL, KC_A));
+ EXPECT_REPORT(driver, (KC_LSFT, KC_A));
+ EXPECT_REPORT(driver, (KC_A));
+ EXPECT_REPORT(driver, (KC_LSFT, KC_A));
+ }
+
+ key_ctrl.press();
+ run_one_scan_loop();
+ tap_key(key_a);
+
+ EXPECT_EQ(get_last_mods(), MOD_BIT(KC_LCTL));
+
+ key_ctrl.release();
+ run_one_scan_loop();
+
+ tap_key(key_repeat);
+ key_shift.press();
+ run_one_scan_loop();
+ tap_key(key_a);
+
+ EXPECT_EQ(get_last_mods(), 0); // Shift should be forgotten.
+
+ key_shift.release();
+ run_one_scan_loop();
+
+ tap_key(key_repeat);
+
+ key_shift.press();
+ run_one_scan_loop();
+ tap_key(key_repeat);
+ key_shift.release();
+ run_one_scan_loop();
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests set_last_keycode() and set_last_mods().
+TEST_F(RepeatKey, SetRepeatKeyKeycode) {
+ TestDriver driver;
+ KeymapKey key_repeat(0, 0, 0, QK_REP);
+ set_keymap({key_repeat});
+
+ // Allow any number of reports with no keys or only KC_LSFT.
+ // clang-format off
+ EXPECT_CALL(driver, send_keyboard_mock(AnyOf(
+ KeyboardReport(),
+ KeyboardReport(KC_LSFT))))
+ .Times(AnyNumber());
+ // clang-format on
+ ExpectString(driver, "aaBB");
+
+ set_last_keycode(KC_A);
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_A);
+
+ for (int n = 1; n <= 2; ++n) { // Tap the Repeat Key twice.
+ // When Repeat is pressed, process_record_user() should be called with a
+ // press event with keycode == KC_A and repeat_key_count() == n.
+ ExpectProcessRecordUserCalledWith(true, KC_A, n);
+ key_repeat.press(); // Press the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ // Expect the corresponding release event.
+ ExpectProcessRecordUserCalledWith(false, KC_A, n);
+ key_repeat.release(); // Release the Repeat Key.
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+ }
+
+ process_record_user_fun = process_record_user_default;
+ set_last_keycode(KC_B);
+ set_last_mods(MOD_BIT(KC_LSFT));
+
+ tap_keys(key_repeat, key_repeat);
+
+ set_last_keycode(KC_NO);
+ tap_keys(key_repeat, key_repeat); // Has no effect.
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+// Tests the `repeat_key_invoke()` function.
+TEST_F(RepeatKey, RepeatKeyInvoke) {
+ TestDriver driver;
+ KeymapKey key_s(0, 0, 0, KC_S);
+ set_keymap({key_s});
+
+ // Allow any number of empty reports.
+ EXPECT_EMPTY_REPORT(driver).Times(AnyNumber());
+ ExpectString(driver, "ss");
+
+ tap_key(key_s);
+
+ EXPECT_KEYCODE_EQ(get_last_keycode(), KC_S);
+
+ // Calling repeat_key_invoke() should result in process_record_user()
+ // getting a press event with keycode KC_S.
+ ExpectProcessRecordUserCalledWith(true, KC_S, 1);
+ keyevent_t event;
+ event.key = {0, 0};
+ event.pressed = true;
+ event.time = timer_read();
+ event.type = KEY_EVENT;
+ repeat_key_invoke(&event);
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ // Make the release event.
+ ExpectProcessRecordUserCalledWith(false, KC_S, 1);
+ event.pressed = false;
+ event.time = timer_read();
+ repeat_key_invoke(&event);
+ run_one_scan_loop();
+ EXPECT_TRUE(process_record_user_was_called_);
+
+ testing::Mock::VerifyAndClearExpectations(&driver);
+}
+
+} // namespace
diff --git a/tests/test_common/keycode_table.cpp b/tests/test_common/keycode_table.cpp
index d21630c01b..9ed80cdbcf 100644
--- a/tests/test_common/keycode_table.cpp
+++ b/tests/test_common/keycode_table.cpp
@@ -663,6 +663,8 @@ std::map<uint16_t, std::string> KEYCODE_ID_TABLE = {
{QK_AUTOCORRECT_TOGGLE, "QK_AUTOCORRECT_TOGGLE"},
{QK_TRI_LAYER_LOWER, "QK_TRI_LAYER_LOWER"},
{QK_TRI_LAYER_UPPER, "QK_TRI_LAYER_UPPER"},
+ {QK_REPEAT_KEY, "QK_REPEAT_KEY"},
+ {QK_ALT_REPEAT_KEY, "QK_ALT_REPEAT_KEY"},
{QK_KB_0, "QK_KB_0"},
{QK_KB_1, "QK_KB_1"},
{QK_KB_2, "QK_KB_2"},
diff --git a/tests/test_common/test_driver.hpp b/tests/test_common/test_driver.hpp
index 8d09e44840..d8a6885d0f 100644
--- a/tests/test_common/test_driver.hpp
+++ b/tests/test_common/test_driver.hpp
@@ -20,6 +20,7 @@
#include <stdint.h>
#include "host.h"
#include "keyboard_report_util.hpp"
+#include "keycode_util.hpp"
#include "test_logger.hpp"
class TestDriver {
@@ -98,6 +99,17 @@ class TestDriver {
*/
#define EXPECT_NO_REPORT(driver) EXPECT_ANY_REPORT(driver).Times(0)
+/** @brief Tests whether keycode `actual` is equal to `expected`. */
+#define EXPECT_KEYCODE_EQ(actual, expected) EXPECT_THAT((actual), KeycodeEq((expected)))
+
+MATCHER_P(KeycodeEq, expected_keycode, "is equal to " + testing::PrintToString(expected_keycode) + ", keycode " + get_keycode_identifier_or_default(expected_keycode)) {
+ if (arg == expected_keycode) {
+ return true;
+ }
+ *result_listener << "keycode " << get_keycode_identifier_or_default(arg);
+ return false;
+}
+
/**
* @brief Verify and clear all gmock expectations that have been setup until
* this point.