tedge-apt-plugin update list missing keep config options

[rina23q]

Proposed changes

Refactor tedge-apt-plugin to move logic to a common area. It solves the linked tickets.

The systems tests were updated to improve coverage for the keep config options which was previously erroneously passing due to using manual tedge-apt-plugin install commands rather than installing packages from the cloud which uses a different code path to install packages.

Types of changes

• 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

• tedge-apt-plugin has unused code and new features were put in the unused section #2669
• tedge-apt-plugin should disable debian frontends when using apt commands #2671
• tedge-apt-plugin should ignore installing recommended packages #2670

Checklist

• I have read the CONTRIBUTING doc
• I have signed the CLA (in all commits with git commit -s)
• I ran cargo fmt as mentioned in CODING_GUIDELINES
• 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]

Codecov Report

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

Comparison is base (ca154f9) 76.0% compared to head (c3eb394) 75.9%.
Report is 28 commits behind head on main.

Additional details and impacted files

Files	Coverage Δ
plugins/tedge_apt_plugin/src/lib.rs	24.4% <0.0%> (-1.3%)	⬇️

... and 7 files with indirect coverage changes

[github-actions]

Robot Results

✅ Passed	❌ Failed	⏭️ Skipped	Total	Pass %	⏱️ Duration
390	0	3	390	100	55m44.457s

[reubenmiller]

From an apt command point of view, I can confirm that the commands and test is good.

Though wait for a Rust review before merging.

[didier-wenzek]

Approved. Nicely done. This refactoring will help avoiding the same kind of issues.

That said, this PR description is not correct: it doesn't address #2666

[reubenmiller]

Approved. Nicely done. This refactoring will help avoiding the same kind of issues.

That said, this PR description is not correct: it doesn't address #2666

Sorry that was my fault, I linked the wrong ticket...updating now.