1
0
Fork 0
forked from forks/qmk_firmware
qmk_firmware/tests/test_common/test_fixture.cpp
fredizzimo 9e8767917d
Fix pressing two keys with the same keycode but different modifiers (#2710)
* Fix extra keyboard report during test_fixture teardown

* Add tests for pressing two keys with only different modifers

* Fix #1708

When two keys that use the same keycode, but different modifiers were
pressed at the same time, the second keypress wasn't registered. This is
fixed by forcing a key release when we detect a new press for the same
keycode.

* Fix the NKRO version of is_key_pressed

* Fix uninitalized loop variable

Co-authored-by: Jack Humbert <jack.humb@gmail.com>
2020-03-13 14:09:38 -04:00

56 lines
1.2 KiB
C++

#include "test_fixture.hpp"
#include "gmock/gmock.h"
#include "test_driver.hpp"
#include "test_matrix.h"
#include "keyboard.h"
#include "action.h"
#include "action_tapping.h"
extern "C" {
#include "action_layer.h"
}
extern "C" {
void set_time(uint32_t t);
void advance_time(uint32_t ms);
}
using testing::_;
using testing::AnyNumber;
using testing::Between;
using testing::Return;
void TestFixture::SetUpTestCase() {
TestDriver driver;
EXPECT_CALL(driver, send_keyboard_mock(_));
keyboard_init();
}
void TestFixture::TearDownTestCase() {}
TestFixture::TestFixture() {}
TestFixture::~TestFixture() {
TestDriver driver;
// Run for a while to make sure all keys are completely released
EXPECT_CALL(driver, send_keyboard_mock(_)).Times(AnyNumber());
layer_clear();
clear_all_keys();
idle_for(TAPPING_TERM + 10);
testing::Mock::VerifyAndClearExpectations(&driver);
// Verify that the matrix really is cleared
EXPECT_CALL(driver, send_keyboard_mock(KeyboardReport())).Times(0);
idle_for(TAPPING_TERM + 10);
}
void TestFixture::run_one_scan_loop() {
keyboard_task();
advance_time(1);
}
void TestFixture::idle_for(unsigned time) {
for (unsigned i = 0; i < time; i++) {
run_one_scan_loop();
}
}