fix: enable and update the apt release test
Metadata
Current evaluation
Merged in canonical/fetch-service#71; follow-up PR #75 (commit b9d7ce0) added missing APT release package-verification coverage after lint skipped the test run.
Resolution: closed resolved
Reason: PR #71 re-enabled and modernized the skipped APT release test and was merged. Follow-up PR #75, commit b9d7ce0, added the missing package-verification test after the maintainer identified a missing test file as the cause of the skipped run.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Apt release tests were disable and outdated. Enable the previously skipped test file,
and update the test to match the current artefact inspector implementation.
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/fetch-service#71; follow-up PR #75 (commit b9d7ce0) added missing APT release package-verification coverage after lint skipped the test run. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/fetch-service#75, re-enabling the skipped APT release test and updating it for the current artefact inspector; the missing test file was identified as the linting cause. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after enabling and updating a skipped apt release test to match the artifact inspector implementation. A missing test file causing linting to skip execution was fixed, allowing the changes to pass review and CI. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/fetch-service#75
(confidence 98%)
Follow-up PR committed as b9d7ce0, adding package verification coverage to the APT release tests.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #75 inspectors/apt: add test for packages verification in release | fetch-service | merged | Merged PR #75 added APT release package-verification test coverage, landing as commit b9d7ce0. | |
| #12 metadata: better apt release file detection | fetch-service | merged | Merged in canonical/fetch-service#12 via commit 21b2d02, tightening APT release-file detection to prevent false positives for legacy Release files. | |
| #239 fix: properly set FETCH_APT_RELEASE_PUBLIC_KEY | fetch-service | merged | Merged in canonical/fetch-service#239: corrected ${SNAP} expansion and added fail-fast validation for FETCH_APT_RELEASE_PUBLIC_KEY; shipped in v0.2.3. | |
| #345 tests: reformat apt inspector test parameters | fetch-service | closed | Closed unmerged as superseded by canonical/fetch-service#365, which merged as commit 3f46e20. | |
| #422 fix(i/apt): prevent false positives in deb inspection | fetch-service | merged | Merged in canonical/fetch-service#422 (0a4cc2a), making deb inspection the final approval and enforcing Packages/Release verification dependencies to prevent false positives. |