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
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
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
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
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
1464afe990
Add notes on Qukeys changes to UPGRADING
...
Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
4 years ago
6bc8c4e243
testing/makefiles: Respect the VERBOSE environment variable
...
Instead of unconditionally running the arduino-based build parts in verbose
mode, respect $VERBOSE. This cuts down on the amount of output we generate by
default, while still allowing us to have a verbose mode if need be.
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
4 years ago
478b91decb
Merge pull request #926 from keyboardio/docker/tuning
...
Get our docker testing timings to hurt less
4 years ago
ba7b496af5
Rework our Docker config for performance
...
* never try to write anything to the host's disks
* read as little as possible from the host's disks
* keep source in ram
* cache build artifacts and intermediate content persistently
Most of these hacks are only necessary because Docker disk performance on macOS is...not performant
Signed-off-by: Jesse Vincent <jesse@keyboard.io>
Signed-off-by: Gergely Nagy <algernon@keyboard.io>
4 years ago
5d59552403
Add a note about how we should offer at least a 60 day deprecation period for APIs
4 years ago
Jesse Vincent
Michael Richters
Gergely Nagy
Graeme Geldenhuys
Gergely Nagy