tests/main/system-usernames-hooks: a snap using system-usernames and hooks - #17405
tests/main/system-usernames-hooks: a snap using system-usernames and hooks#17405bboozzoo wants to merge 1 commit into
Conversation
…hooks Add a spread tests which executes a refresh like scenario with a snap using system-usernames and a post-refresh hook. Related: SNAPDENG-37266 Signed-off-by: Maciej Borzecki <maciej.borzecki@canonical.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #17405 +/- ##
==========================================
+ Coverage 78.84% 78.89% +0.05%
==========================================
Files 1411 1396 -15
Lines 196727 196606 -121
Branches 2462 2462
==========================================
+ Hits 155111 155122 +11
+ Misses 32280 32153 -127
+ Partials 9336 9331 -5
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:
|
|
Fri Jul 24 11:24:44 UTC 2026 Failures:Preparing:
Executing:
Restoring:
Skipped tests from snapd-testing-skipIf you wish to have any of the below tests run in your PR, in your PR description, add 'unskip:' followed by a copy-and-pasted list of the below tests you wish to run (unskip plus test list must be valid yaml) |
ZeyadYasser
left a comment
There was a problem hiding this comment.
LGTM, Thank you! nitpick comment
| tests.cleanup defer "$TESTSTOOLS"/user-state remove-with-group snap_daemon | ||
|
|
||
| execute: | | ||
| "$TESTSTOOLS"/snaps-state install-local test-snapd-sh |
There was a problem hiding this comment.
maybe add some assertions for files (e.g. "$SNAP_COMMON"/hook.log and $SNAP_COMMON/data/foo) after install and refresh?
Add a spread tests which executes a refresh like scenario with a snap using system-usernames and a post-refresh hook.
Related: SNAPDENG-37266
Thanks for helping us make a better snapd!
Have you signed the license agreement and read the contribution guide?