820560f77c
Change the target the test suite uses when compiling sketches to avoid duplicate compilation
4 years ago
e96d06893c
Make it possible to only run a subset of the simulator tests using the TEST_PATH variable
...
Suggested by @gendankenexperimenter
4 years ago
2d47d64961
Add testcase for basic modifier keycodes
...
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
2207c3f4f8
Add modifier information to KeyboardReport class
...
This change adds the keycodes for keyboard modifier keys to the vector returned
by KeyboardReport::ActiveKeycodes(). Also, a simple method for getting the
modifiers byte from a keyboard hid report.
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
9ec11f1b4b
Force compilation of sketches in the simulator.
...
Because the Arduino build system doesn't -really- interface well with Makefiles, there's no other way to pick up Kaleidoscope or sketch changes from Make. This -does- slow down sim builds / test runs a bit.
Fixes #942
4 years ago
5c0edd9b69
"make clean" needs to clear out build and output paths
...
It previously did not. Also, try -not- to clean out google test if the build dir doesn't even exist
4 years ago
240f141538
Reduce the number of cycles the "long" test runs for, for perf reasons
4 years ago
895f5f1e00
Add some tests for SetCycleTime
4 years ago
cfcfd3b75e
Add diag strings for some tests to help debug
4 years ago
a646261982
Add SetCycleTime and CycleTime to control how many ms each cycle take.
4 years ago
b5c744a7cf
Move RunForMillis.cpp to the .cpp file.
...
Remove RunMillis and RunForCycles. I know this gets rid of parallelism in naming, but the alternate versions read wrong to me
4 years ago
1c24228f80
Add testcases for `RunForMillis()` simulator function
...
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
76a48e0a12
Add `RunFor*` variants for `Cycles` and `Millis` functions
...
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
69fc1dc0e2
Add `SimHarness.RunMillis(t)` function
...
For now, this is just a wrapper around RunCycles(n), but the two could diverge
in the future. RunMillis(t) should always cause `t` milliseconds to elapse in
the simulator, regardless of how many cycles that takes.
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
3a16fc95fb
Cut down the sketch for the test to remove extraneous stuff
4 years ago
23a78e119b
Testcase for simultaneous keypresses
...
This test runs multiple keypresses in the same cycle, and checks to see that the
last report generated contains the correct active keycodes (and no others).
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
39f0bbe159
Add KeyAddr versions of Press() & Release() to the simulator
...
This commit adds versions of `SimHarness::Press()` and `SimHarness::Release()`
functions that use a `KeyAddr` parameter instead of row & column integers.
to build the test.
4 years ago
51d48245ad
Switch ConsumerControlReport and SystemControlReport to use ActiveKeycodes and ActiveKeycode respectively, to match Keyboard.
4 years ago
7a12be25bc
Fix an embarassing typo.
...
This is why we do PRs, even for quick fixes and don't just push to master because we're the project lead and yolo.
4 years ago
de45646c9d
extract out a variable for ARDUINO_BUILDER_TOOLS_PATH to make it easier to
...
get up and running on arch
4 years ago
1d0d2d43d0
Merge pull request #934 from keyboardio/build-cleanup
...
Some more builder cleanups.
4 years ago
34b428c4a9
Simplify conditional building
4 years ago
c98b513636
Revert "missed one"
...
This reverts commit de062ea897
4 years ago
5bea41f630
reduce a bit of code duplication
4 years ago
de062ea897
missed one
4 years ago
6e8d1d020a
Switch to a less verbose way to set defaults for environment variables
...
By reducing duplication, we improve readability and reduce errors
4 years ago
a91c6a556e
refactor config file list to all be in one place
4 years ago
dbb4f3b1ae
migrate check-docs from generic makefile
4 years ago
fed3a6d6dc
Merge pull request #932 from keyboardio/builder-cleanup
...
Additional cleanup of kaleidoscope-builder
4 years ago
25c8a8a180
Refactor the "SKETCH" environment variable and friends.
...
Functionality should be the same, but code should be clearer.
4 years ago
e96902a9df
Switch from locally defining a "DIR" for every bin, to setting a global Kaleidoscope bin directory var
4 years ago
660747c916
Rename ROOT to KALEIDOSCOPE_DIR, which is what it appears to actually be.
4 years ago
ea0abc8833
inline a single-use cunction
4 years ago
19f728ca56
lift find_bootloader_path' out of the config file into the program.
...
as it isn't something that's platform specific
'
4 years ago
52a5265b9b
ROOT is defined automatically and not overridable. Remove the lines in the comments and docs
4 years ago
9f846c4905
Lift a bunch more project-specific stuff out of the generic makefile for sketches
4 years ago
12e59b8684
lift up BUILD_FILE_PATH to somewhere reusable.
4 years ago
45bafe9bf5
SOURCEDIR was not actually used or honored. It was always set to pwd
4 years ago
dfadd32400
Removeo outdated and unused 'LIBRARY' env variable
4 years ago
a278efc21e
Absolute path to sized elf
4 years ago
d43f6adddf
Merge pull request #931 from keyboardio/builder-cleanup
...
Doing some cleanup to the makefile and builder
4 years ago
afc8388cd1
Move a bunch of makefile targets from the generic sketch makefile
4 years ago
383a29d3e3
move run-tests out to the Makefile
4 years ago
bbc84f6f33
move build-gtest-gmock into the Makefile
4 years ago
1865e33426
Lift duplication of command to a var
4 years ago
b2a1528419
Merge pull request #930 from graemeg/patch-1
...
Update core-plugins.md - fixes a small spelling mistake
4 years ago
41e11bf473
Update core-plugins.md
...
Fixes a small spelling mistake.
4 years ago
d5e4cd3ba0
Prevent rollover to a qukey from causing unintended repeats
...
When rollover occurs from a non-modifier key to a qukey, if we delay the release
event of that key until after the qukey's state is resolved, and if the hold
timeout is set to a fairly large value (on the order of 500ms), unintended
repeats would occur for a key that was actually only tapped. To prevent this, if
there's only one event in the queue (the press of the qukey), and we see a
release of a non-modifier key that's not the qukey, it's okay to allow that
release event to skip the queue and simply proceed as if it had been released
before the qukey was pressed.
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
e20357174b
Merge pull request #927 from gedankenexperimenter/qukeys-deprecation-dates
...
Add deprecation dates for old Qukeys functions
4 years ago
fcd7f25998
Add removal dates to Qukeys deprecation warnings
...
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
Jesse Vincent
Michael Richters
Gergely Nagy
Graeme Geldenhuys