astarte-platform / astarte-device-sdk-rust

Apache License 2.0
6 stars 12 forks source link

fix(store): remove interface's properties #313

Closed joshuachp closed 7 months ago

joshuachp commented 8 months ago

Implement in the store a way to remove all the properties for an interface. This fixes the deletion of the properties when an interface is removed, since before it was using the mapping without the parameter.

Closes #312

codecov[bot] commented 8 months ago

Codecov Report

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

Project coverage is 64.0%. Comparing base (ea692a8) to head (fcc8bcf). Report is 2 commits behind head on release-0.6.

Additional details and impacted files | [Files](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?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/interface/mod.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Finterface%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9tb2QucnM=) | `72.4% <ø> (ø)` | | | [src/lib.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2xpYi5ycw==) | `55.4% <100.0%> (+3.8%)` | :arrow_up: | | [src/payload.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Fpayload.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3BheWxvYWQucnM=) | `64.2% <ø> (ø)` | | | [src/store/sqlite.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Fstore%2Fsqlite.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3N0b3JlL3NxbGl0ZS5ycw==) | `94.9% <100.0%> (+0.2%)` | :arrow_up: | | [src/store/memory.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Fstore%2Fmemory.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3N0b3JlL21lbW9yeS5ycw==) | `93.7% <80.0%> (-1.6%)` | :arrow_down: | | [src/store/wrapper.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Fstore%2Fwrapper.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3N0b3JlL3dyYXBwZXIucnM=) | `81.4% <80.0%> (-0.4%)` | :arrow_down: | | [src/store/error.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313?src=pr&el=tree&filepath=src%2Fstore%2Ferror.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3N0b3JlL2Vycm9yLnJz) | `0.0% <0.0%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/313/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform)