-
-
Notifications
You must be signed in to change notification settings - Fork 130
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? #22
Comments
Hi @leg0 , |
Is this a request to unit test the library itself, or to provide an ability to mock this library for the purposes of unit testing 3rd-party Arduino libraries? Asking as the maintainer of the |
In short, it's a request to have a place to put unit tests for a code in this repo. When I created this question, i had a PR that I had some tests, but I did not see any unit testing framework being used in this project. So including a set of unit tests that nobody would run seemed like a waste, and as a result that test code would probably rot and not be able to run anyway after a while. If there are plans to add unit testing to this project, it would be helpful to know which testing framework is going to be used. And once that is available, running the tests as part of CI pipeline would then become a possiblity. |
Hi @leg0. Some nice advancements have been made in this area since the time you created the issue. You can see the unit tests here: They are using the Catch2 test framework. These test are run by the GitHub Actions-based CI system: |
I was wondering if there are any plans to start using unit tests in this project? Any plans to run them in a continuous integration server? There are some free CI servers, for example Travis CI, Azure Devops, etc.
As part of a pull request I would like to provide a set of unit tests to demonstrate that my changes actually work (at least the way I intended them to).
The text was updated successfully, but these errors were encountered: