Skip to content
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

Integration tests for rosserial #243

Merged
merged 20 commits into from Sep 8, 2016
Merged

Integration tests for rosserial #243

merged 20 commits into from Sep 8, 2016

Conversation

@mikepurvis
Copy link
Member

mikepurvis commented Sep 3, 2016

The idea here is to have some integration tests which exercise basic pathways of the rosserial server and client. As the test suite gets built up further, we can validate functions of the server and client portions, and be more confident merging improvements and refinements to the components.

This change has been a long time coming; I'm pleased to finally have it in what I believe to be a mergeable state (pending some style and other fixups, no doubt).

@PaulBouchier @mikeodr @vmatos @garyservin

@mikeodr

This comment has been minimized.

Copy link
Contributor

mikeodr commented Sep 8, 2016

LGTM as long as all tests are passing.

@mikepurvis mikepurvis merged commit 56dbe8c into jade-devel Sep 8, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@mikepurvis mikepurvis deleted the rosserial-tests branch Sep 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.