/snap-confine

Ensure that snap-confine is dead after each test terminates #144

Merged
merged 2 commits into from Sep 14, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@zyga @jdstrand
@zyga
Collaborator

zyga commented Sep 14, 2016

This is just a sanity check so that no copy of snap-confine is around
after test terminates. If something ever breaks where the child process
may end up still hanging we will know.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com
@zyga
Ensure that snap-confine is dead after each test terminates 
This is just a sanity check so that no copy of snap-confine is around
after test terminates. If something ever breaks where the child process
may end up still hanging we will know.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
3f641f5
spread.yaml
@@ -34,6 +34,9 @@ suites:
spread-tests/main/:
summary: Full-system tests for snap-confine
restore-each: |
+ # Ensure that we don't have stranded processes around
+ test "$(pgrep ubuntu-core-launcher | wc -l)" -eq 0
+ test "$(pgrep snap-confine | wc -l)" -eq 0
@jdstrand

jdstrand Sep 14, 2016

Contributor

You can use 'pgrep -c' instead of piping to wc -l.

+1 with that change.

@zyga

zyga Sep 14, 2016

Collaborator

Done, thanks for the suggestion. I didn't know about it

@zyga zyga merged commit 76b27a4 into master Sep 14, 2016
1 check was pending

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@zyga zyga deleted the extra-sanity-tests branch Sep 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment