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

Unit Tests and CI #8

Open
soraxas opened this issue Jul 18, 2024 · 6 comments
Open

Unit Tests and CI #8

soraxas opened this issue Jul 18, 2024 · 6 comments

Comments

@soraxas
Copy link
Contributor

soraxas commented Jul 18, 2024

Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?

In https://github.com/dfki-ric/ugv_nav4d/blob/main/src/test/test_ugv_nav4d.cpp there contains a few google test function.
But I wouldn't call it unit testing as it's only setting up 1 scenario and tested:

  • setting up grid resolution
  • valid / invalid start/end states
  • tested it can finishes (finding solution) for 1 particular planning request

Can we have some more structured unit testing in the traditional sense?

(part of openjournals/joss-reviews#6983)

@soraxas
Copy link
Contributor Author

soraxas commented Jul 18, 2024

The CI file in https://github.com/dfki-ric/ugv_nav4d/blob/main/.gitlab-ci.yml is using Gitlab format.

Presumably, it's from dfki-ric's private gitlab instance? If this public instance will be in Github, you should use github action instead

https://docs.github.com/en/actions

@haider8645
Copy link
Collaborator

Build CI: Added github actions based build CI

Unit Testing: The current tests test basic functionality of the planner. If you want something else then can you point me to an example implementation on another library? Thanks!

@soraxas
Copy link
Contributor Author

soraxas commented Sep 17, 2024

Some C++ examples:

https://github.com/tbrown122387/pf/tree/master/test

https://github.com/kdm9/libqcpp/tree/master/src/tests

Both of these are JOSS C++ libraries

@planthaber
Copy link
Member

The tests are located here: https://github.com/dfki-ric/ugv_nav4d/tree/main/src/test

@haider8645: Add the TESTS_ENABLED=ON cmake option, ply download and running the tests in the github action.

@soraxas
Copy link
Contributor Author

soraxas commented Sep 17, 2024

Thanks @planthaber but that's what I've already linked in the OP.

As mentioned, some form of unit test would be better (there's only one file in the folder, contains only integration test, i.e., end-to-end test).

E.g. From a quick glance, (non-exhaustive list) your repo has things like Dijkstra algorithm, EnvironmentXYZTheta, compute motion, etc. These are individual item can be use some form tests.

@haider8645
Copy link
Collaborator

haider8645 commented Sep 17, 2024

ply download

uploaded the ply files to zenodo. They just need to be added to our organization by the curators and then can remove the test files from repo and update docu and action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants