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

Define release acceptance tests #102

Open
Bckempa opened this issue Nov 2, 2023 · 5 comments
Open

Define release acceptance tests #102

Bckempa opened this issue Nov 2, 2023 · 5 comments
Assignees
Labels
release followup Issues discovered during release

Comments

@Bckempa
Copy link

Bckempa commented Nov 2, 2023

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.

@Bckempa Bckempa added the release followup Issues discovered during release label Nov 2, 2023
@EzraBrooks EzraBrooks added this to the humble-2024.02.0 milestone Nov 16, 2023
@EzraBrooks
Copy link
Member

document what we considered essential in October, then go from there

@quarkytale
Copy link

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.

@ivanperez-keera
Copy link

ivanperez-keera commented Jan 27, 2024

@quarkytale Are any of the ros packages installed currently as part of the build process of the spaceros docker image running any of the static analyzers included by default with spaceros as part of their test suite?

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.

@quarkytale
Copy link

quarkytale commented Feb 8, 2024

Are any of the ros packages installed currently as part of the build process of the spaceros docker image running any of the static analyzers included by default with spaceros as part of their test suite?

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.

@ivanperez-keera
Copy link

ivanperez-keera commented Apr 18, 2024

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.

This was referenced Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release followup Issues discovered during release
Projects
Status: In Progress
Development

No branches or pull requests

5 participants