-
Notifications
You must be signed in to change notification settings - Fork 4
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
Python tests migration #16
Comments
Actually that is not that easy since all the setup scripts and CI scripts have to change. I propose to revisit this in the next release (0.7). |
If this is an API test (unit test) it should be in there, if its a platform test, then let's move them all to the platform? |
Didn't get you. What you mean by "python tests"? If you mean these, they are integration tests, as they test the API, the web services and the ROS background nodes. |
I thought we had python unit tests, never mind them, nothing to move to the api for python. |
@klpanagi could you please migrate the Python tests under the API? AFAIK they are currently under the platform.
The text was updated successfully, but these errors were encountered: