search

apache / celix

Apache Celix is a framework for C and C++14 to develop dynamic modular software applications using component and in-process service-oriented programming.
https://celix.apache.org/
Apache License 2.0
158 stars 85 forks source link

Add unit test for dfi #691

Closed xuzhenbao closed 7 months ago

xuzhenbao commented 7 months ago

Add unit test for DFI, and fix some bugs

codecov-commenter commented 7 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e9642b9) 83.14% compared to head (87a4b35) 83.46%.

:exclamation: Current head 87a4b35 differs from pull request most recent head bd931c0. Consider uploading reports for the commit bd931c0 to get more accurate results

Files Patch % Lines
libs/dfi/src/dyn_avpr_type.c 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #691 +/- ## ========================================== + Coverage 83.14% 83.46% +0.32% ========================================== Files 254 254 Lines 32935 32951 +16 ========================================== + Hits 27384 27504 +120 + Misses 5551 5447 -104 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

pnoltes commented 7 months ago

Good to see some additional test for libdfi. I will try to review this PR this week.