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 Testing #65

Open
wants to merge 1 commit into
base: rolling
Choose a base branch
from

Conversation

atticusrussell
Copy link
Contributor

Establish a framework for unit testing and implement some examples:

  • Use PlatformIO testing with the Unity framework

  • Add detailed comments to files in firmware/lib/motor and firmware/lib/pid

  • Create native unit tests for PID

  • Create embedded unit tests for motors (2 of 4 types so far - generic ones)

  • Add running native (desktop) unit tests to GitHub Actions CI

  • Describe Unit Testing in README

Other unit tests should still be added in future PRs, but this should serve as good example for developers writing unit tests for features.

* Add detailed comments to files in `firmware/lib/motor` and `firmware/lib/pid`

* Create native unit tests for PID

* Create embedded unit tests for motors (2 of 4 types so far - generic ones)

* Add running native (desktop) unit tests to GitHub Actions CI

* Describe Unit Testing in README
@atticusrussell
Copy link
Contributor Author

@grassjelly thoughts?

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

Successfully merging this pull request may close these issues.

1 participant