Repository navigation
tests: add parallel-instances spread test for components - #17809
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Instance removal does not assert that its component blobs and mounts are cleaned up.
1 open finding
What changed in this PR
Adds spread coverage for component operations across parallel snap instances.
Changes:
- Tests installation, removal, refresh, revert, hooks, and
snapctloperations. - Adds local snap and component fixtures.
| File | Description |
|---|---|
tests/main/component-parallel-instances/task.yaml |
Adds parallel-instance component scenarios. |
tests/main/component-parallel-instances/snap-with-comps/test |
Verifies component visibility. |
tests/main/component-parallel-instances/snap-with-comps/meta/snap.yaml |
Defines the test snap. |
tests/main/component-parallel-instances/comp1/meta/component.yaml |
Defines its test component. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| snap run test-snap-with-components_key one | ||
|
|
||
| echo "Removing an instance removes only its components" | ||
| snap remove --purge test-snap-with-components_key |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #17809 +/- ##
==========================================
- Coverage 82.26% 82.22% -0.05%
==========================================
Files 1410 1411 +1
Lines 201153 201797 +644
Branches 2503 2626 +123
==========================================
+ Hits 165485 165934 +449
- Misses 33756 33855 +99
- Partials 1912 2008 +96
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Tue Oct 13 17:45:18 UTC 2026 |
github-actions
Bot
added
the
Auto rerun spread
No description provided.