-
Notifications
You must be signed in to change notification settings - Fork 28
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
88 add GitHub automated build integration tests for sadb mariadb interface 1 #109
88 add GitHub automated build integration tests for sadb mariadb interface 1 #109
Conversation
Fixed CMAKE issue causing builds to fail. |
Codecov Report
@@ Coverage Diff @@
## dev #109 +/- ##
==========================================
- Coverage 80.27% 76.78% -3.50%
==========================================
Files 20 21 +1
Lines 5644 5901 +257
==========================================
Hits 4531 4531
- Misses 1113 1370 +257
Continue to review full report at Codecov.
|
Code coverage falls down a little with this because there's no code coverage module in my docker container with the mariadb installed. Didn't think about this, and I can work on it later if that's not a big deal to anyone else (don't know if there is a solution yet - there should be somewhere though). Should be good to merge w/o conflicts now. |
Excellent 👍 |
Added Docker Container to allow for SQL testing (MARIADB)
Added several unit tests, and modified yml to work within main/dev.
Additional tests could be written in the future when things are flushed out more. We could probably also use some additional tests now that I've not written, but I think the basis is covered.