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

[TASK] Add Action for automatic Unittesting #144

Open
3 tasks
alekskl01 opened this issue Mar 18, 2024 · 3 comments
Open
3 tasks

[TASK] Add Action for automatic Unittesting #144

alekskl01 opened this issue Mar 18, 2024 · 3 comments
Labels
feature New feature or request

Comments

@alekskl01
Copy link
Contributor

Description of task

Add automatic running of unittests for vortex-asv.

Suggested Workflow

No response

Specifications

No response

Contacts

No response

Code Quality

  • Every function in header files are documented (inputs/returns/exceptions)
  • The project has automated tests that cover MOST of the functions and branches in functions (pytest/gtest)
  • The code is documented on the wiki (provide link)
@alekskl01 alekskl01 added the feature New feature or request label Mar 18, 2024
@alekskl01 alekskl01 self-assigned this Mar 18, 2024
@alekskl01
Copy link
Contributor Author

Crashes due to jbdtool symlink, this should be added as a git submodule.

@alekskl01
Copy link
Contributor Author

Need to figure out some way of running unittests with vortex-msgs without adding it as a submodule.

@alekskl01 alekskl01 removed their assignment Apr 28, 2024
@chrstrom
Copy link
Member

chrstrom commented Jul 16, 2024

See #192 for pipeline that just builds the workspace @alekskl01 (there is a test step there but thats just boiler plate that should be changed/removed). Building and running tests and generating coverage should probably be parts of a multi-stage pipeline, so we avoid two build stages on every pr

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

No branches or pull requests

2 participants