New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds SITL test infrastructure. #197
Conversation
830ec43
to
4839424
Compare
2b5e469
to
92df258
Compare
92291cc
to
e12a558
Compare
(Rerunning one last time to remove some echo statements.) |
print "Read new value of param 'THR_MIN': %s" % v.parameters['THR_MIN'] | ||
|
||
# TODO | ||
# assert end - start < 1000, 'time to set parameter was %s, over 1s' % (end - start,) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the relevant Asserts here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #12 (my last comment)
great place to start, added comments but they are not limiting to merge, great work @tcr3dr, now we need to get better coverage and to start requiring tests on code contributions |
👍 |
Adds SITL test infrastructure.
Awesome!
For that you will need a brief guide of how the test infastructure works, how/if it can be set up locally, and information on how you set up your own tests. Is there an issue for that outstanding (I can't see one) and can you write it? |
@hamishwillee see #203 |
Right now, this only tests a very trivial property check and code around timing #12. This PR would serve as the basis for forthcoming tests.
The test rig itself is wonky, but it's designed to emulate a single-script test as much as possible. In a first-class DKPY module, we'd be using
nosetests
and these would be straightforward to port over.You can run it by running
cd tests; python -m sitl
.