Fix remaining test failures from built-in bridge

jarhodes314 commented 1 month ago

Proposed changes

This PR fixes some remaining issues with the built-in bridge so that the tests work with the built-in bridge enabled.

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

2794

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 77.8%. Comparing base (d52bbdf) to head (299627a). Report is 3 commits behind head on main.

Additional details and impacted files

| [Files](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge) | Coverage Δ | | |---|---|---| | [...core/tedge/src/cli/disconnect/disconnect\_bridge.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge%2Fsrc%2Fcli%2Fdisconnect%2Fdisconnect_bridge.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2Uvc3JjL2NsaS9kaXNjb25uZWN0L2Rpc2Nvbm5lY3RfYnJpZGdlLnJz) | `32.8% <100.0%> (-0.5%)` | :arrow_down: | | [crates/extensions/tedge\_mqtt\_bridge/src/config.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fextensions%2Ftedge_mqtt_bridge%2Fsrc%2Fconfig.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvdGVkZ2VfbXF0dF9icmlkZ2Uvc3JjL2NvbmZpZy5ycw==) | `96.1% <100.0%> (ø)` | | | [crates/extensions/tedge\_mqtt\_bridge/src/health.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fextensions%2Ftedge_mqtt_bridge%2Fsrc%2Fhealth.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvdGVkZ2VfbXF0dF9icmlkZ2Uvc3JjL2hlYWx0aC5ycw==) | `95.0% <100.0%> (+0.1%)` | :arrow_up: | | [crates/core/c8y\_api/src/smartrest/inventory.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Fc8y_api%2Fsrc%2Fsmartrest%2Finventory.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvYzh5X2FwaS9zcmMvc21hcnRyZXN0L2ludmVudG9yeS5ycw==) | `58.8% <94.1%> (+1.9%)` | :arrow_up: | | [crates/core/tedge/src/cli/common.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge%2Fsrc%2Fcli%2Fcommon.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2Uvc3JjL2NsaS9jb21tb24ucnM=) | `0.0% <0.0%> (ø)` | | | [crates/extensions/tedge\_mqtt\_bridge/src/lib.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fextensions%2Ftedge_mqtt_bridge%2Fsrc%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvdGVkZ2VfbXF0dF9icmlkZ2Uvc3JjL2xpYi5ycw==) | `89.1% <96.0%> (-0.1%)` | :arrow_down: | | [crates/extensions/tedge\_mqtt\_bridge/src/topics.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fextensions%2Ftedge_mqtt_bridge%2Fsrc%2Ftopics.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvdGVkZ2VfbXF0dF9icmlkZ2Uvc3JjL3RvcGljcy5ycw==) | `71.4% <20.0%> (-20.3%)` | :arrow_down: | | [crates/core/tedge/src/cli/refresh\_bridges.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge%2Fsrc%2Fcli%2Frefresh_bridges.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2Uvc3JjL2NsaS9yZWZyZXNoX2JyaWRnZXMucnM=) | `0.0% <0.0%> (ø)` | | | [crates/core/tedge/src/cli/connect/command.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge%2Fsrc%2Fcli%2Fconnect%2Fcommand.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2Uvc3JjL2NsaS9jb25uZWN0L2NvbW1hbmQucnM=) | `0.0% <0.0%> (ø)` | | | [crates/core/tedge\_mapper/src/c8y/mapper.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge_mapper%2Fsrc%2Fc8y%2Fmapper.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2VfbWFwcGVyL3NyYy9jOHkvbWFwcGVyLnJz) | `0.0% <0.0%> (ø)` | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2879/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge)

github-actions[bot] commented 1 month ago

Robot Results

:white_check_mark: Passed	:x: Failed	:next_track_button: Skipped	Total	Pass %	:stopwatch: Duration
444	0	3	444	100	1h1m32.791408999s

thin-edge / thin-edge.io