-
Notifications
You must be signed in to change notification settings - Fork 11
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
Define release acceptance tests #32
Comments
document what we considered essential in October, then go from there |
Currently, there is only manual testing, where I ensure the robots (mars_rover and canadaram) are responding to control inputs. A better way, in my mind, is a gtest setup/integration test which will spawn the gzserver, apply a world force and validate the motion using positions of the model, see this for reference. I'm not sure how or if it'll work with the ROS framework here. But I can certainly start looking into it from this direction. |
@quarkytale Are any of the ros packages installed currently as part of the build process of the Would it make sense to have a minimal demo app, which could be based on a hello world for ros, that only exercises the features we provide? I'm trying to think of how to accomplish a small, incremental step that provides some value but is also so small that we'll have very little risk of not being able to complete it if the next release development period is busy at our respective jobs. EDIT: Note. |
I'm not sure about that will check and get back to you. As per discussion during the technical meeting, the first action item for the next milestone is to have a test or a CI job checking all the demos are running, robots are spawning etc. No need to test Gazebo integration. |
The following is what I did to complete this work during the January release: Identify release name. For all the commands below, it is imperative that the right branch/tag be identified.
|
If Ivan doesn't get a chance to do so, I will collate his and Brian's notes during the release process. |
For the just released version of Space ROS, we did this: #33 (comment), which includes some tests. |
While full automated acceptance testing is under development, we need a
minimum equipment list
of tests (automated or manual) that must pass for a release to be tagged. Presumably these will be defined per component.The text was updated successfully, but these errors were encountered: