edgehog-device-manager / edgehog-device-runtime

Edgehog Device Runtime is a portable middleware written in Rust, that enables remote device management on Linux-based systems.
Apache License 2.0
11 stars 10 forks source link

feat(ota): improve tests and make the progress a stream #255

Closed joshuachp closed 10 months ago

joshuachp commented 1 year ago

Remove the delays and channels from the tests. Replace the test that require to receive a channel as "expect" and make it return a progress stream directly.

codecov[bot] commented 1 year ago

Codecov Report

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

Comparison is base (1b9fd2a) 74.73% compared to head (39cd1a5) 72.51%. Report is 11 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #255 +/- ## ========================================== - Coverage 74.73% 72.51% -2.23% ========================================== Files 26 26 Lines 1338 1386 +48 ========================================== + Hits 1000 1005 +5 - Misses 338 381 +43 ``` | [Files](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager) | Coverage Δ | | |---|---|---| | [src/ota/mod.rs](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager#diff-c3JjL290YS9tb2QucnM=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: | | [src/power\_management.rs](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager#diff-c3JjL3Bvd2VyX21hbmFnZW1lbnQucnM=) | `0.00% <0.00%> (ø)` | | | [src/ota/ota\_handle.rs](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager#diff-c3JjL290YS9vdGFfaGFuZGxlLnJz) | `82.08% <73.80%> (-1.71%)` | :arrow_down: | | [src/ota/ota\_handler.rs](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager#diff-c3JjL290YS9vdGFfaGFuZGxlci5ycw==) | `86.22% <72.13%> (+1.53%)` | :arrow_up: | | [src/ota/rauc.rs](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager#diff-c3JjL290YS9yYXVjLnJz) | `7.05% <0.00%> (-4.71%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/edgehog-device-manager/edgehog-device-runtime/pull/255/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edgehog-device-manager)

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