← Back to issue list

spread: introduce appstream parse-info test

View original Github issue

Metadata

Project
snapcraft
Number
#2952
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged in #2952 via 969af6c98, adding a spread test for AppStream adopt-info/parse-info data and ${SNAP} rewriting of desktop icon paths.

Resolution: closed resolved

Reason: PR #2952 was approved and merged as commit 969af6c98. It added coverage for AppStream metadata adoption/parsing and desktop icon path rewriting.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- Ensure that adopt-info + parse-info is capturing the correct data. - Test the desktop icon path is rewritten using ${SNAP}. Signed-off-by: Chris Patterson <chris.patterson@canonical.com> There does some to be some unexplained difference when running this test against local LXD. My newlines are different in the description field... see https://github.com/snapcore/snapcraft/compare/4f3f58f0fcd40e3cc6383a278eefb7f61b1b90d3..221da1a606f2cf51d33489db0ac1fcada30519fa

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in #2952 via 969af6c98, adding a spread test for AppStream adopt-info/parse-info data and ${SNAP} rewriting of desktop icon paths.
stealth/space-bunny-alpha — closed resolved Merged PR #2952 as 969af6c98, adding spread coverage for AppStream parse-info/adopt-info data and ${SNAP} desktop-icon path rewriting.
stealth/space-bunny-alpha — closed resolved Merged #2952 as 969af6c98, adding appstream tests for adopt-info/parse-info data and ${SNAP} desktop-icon rewriting; LXD description-newline differences were noted as unexplained.
qwen/qwen3.6-35b-a3b — — Merged adding a spread test to verify appstream parse-info data capture and desktop icon path rewriting. Minor LXD newline discrepancies did not block the merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged a spread test for appstream parse-info to verify adopt-info data capture and desktop icon path rewriting using ${SNAP}. Local LXD newline discrepancies were noted and resolved.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1694 Develop appstream handler snapcraft closed AppStream handler feature request delivered by merged PR canonical/snapcraft#1826 (commit 9cb05f4c5), adding an AppStream metadata extractor for parts covering summary and description. Extended later by #1853 (icon/desktop) and #1861 (appdata.xml).
71%
#2435 Appstream desktop snapcraft merged Merged PR #2435 added AppStream-derived desktop-file support to Snapcraft, updating 10 files with 95.91% diff coverage.
71%
#3380 snap packaging: fix broken path check when copying local icon snapcraft merged Merged PR #3380 fixed copying local icons referenced by desktop files and added spread-test coverage for the previously missed AppStream path.
71%