-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
snap: add tests for multiple commands #5488
snap: add tests for multiple commands #5488
Conversation
26b3b22
to
59b23b7
Compare
@russoz what's the state on this? |
This is out there in the cold, crying alone in the dark. I am working on a couple of other things first, then will come back to this. |
Added |
@felixfontein I am trying to get the vagrant-based test-runs going here, and for this one (snap) I had it working by running the test straight as Couple of thoughts here:
|
It's probably not set because it also works without it in CI... But yes, it should be set there as well.
I'm not sure whether that works well with ansible-test. The |
Backport to stable-6: 💚 backport PR created✅ Backport PR branch: Backported as #6340 🤖 @patchback |
@russoz thanks for improving the tests! |
* snap: add tests for multiple commands * snap: add tests + become * remove packages again for idempotency * roll back become=true in tests (cherry picked from commit d734094)
…ommands (#6340) snap: add tests for multiple commands (#5488) * snap: add tests for multiple commands * snap: add tests + become * remove packages again for idempotency * roll back become=true in tests (cherry picked from commit d734094) Co-authored-by: Alexei Znamensky <[email protected]>
SUMMARY
Adds tests for handling multiple snaps in the same command.
ISSUE TYPE
COMPONENT NAME
snap