-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add unit tests #2
Comments
It's unclear to me how to run tests on an actual pi. Does GitHub support docker easily? |
Looks like we would need to setup our own GitHub Actions runner to enable this, or have the C API mocked out like #12 was trying to do. In the meantime, the existing unit tests do work, as long as you specify |
@TDHolmes Github does support Docker easily as a part of workflows if you believe that is beneficial |
surprisingly, the C++ library compiles in normal ubuntu, so I think we could do jelly bean GitHub actions compilation. However, the actual runtime behavior might be weird :p Lets wait and see for #1 since before then, linking properly will be... tricky. |
I tried to get this going with #27 without success |
Tests using the stub functions would work though, however most of the library relies on the FFI so not sure if that's beneficial |
We've added CI to ensure it builds & we don't get clippy regressions, but this doesn't prevent runtime issues
The text was updated successfully, but these errors were encountered: