astarte-platform / astarte-device-sdk-rust

Apache License 2.0
6 stars 12 forks source link

feat(introspection): improve the api for checks #326

Closed joshuachp closed 7 months ago

joshuachp commented 7 months ago

Improve the DynamicIntrospection by adding a return value letting us know if and which interfaces have been added or removed from or to the introspection.

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 69.1%. Comparing base (d11142a) to head (a49bbd1). Report is 1 commits behind head on master.

:exclamation: Current head a49bbd1 differs from pull request most recent head 05b1a90. Consider uploading reports for the commit 05b1a90 to get more accurate results

Additional details and impacted files | [Files](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/326?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform) | Coverage Δ | | |---|---|---| | [src/client.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/326?src=pr&el=tree&filepath=src%2Fclient.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2NsaWVudC5ycw==) | `59.2% <100.0%> (-0.4%)` | :arrow_down: | | [src/introspection.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/326?src=pr&el=tree&filepath=src%2Fintrospection.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludHJvc3BlY3Rpb24ucnM=) | `100.0% <ø> (ø)` | | | [src/connection.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/326?src=pr&el=tree&filepath=src%2Fconnection.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2Nvbm5lY3Rpb24ucnM=) | `65.6% <83.3%> (+0.6%)` | :arrow_up: |