Merge lp:~jamesodhunt/snappy/selftest-add-pkg-removal-tests into lp:snappy/selftest

[James Hunt]

* Add 2 package removal tests:
  - tests/12_test_remove_hello
  - tests/13_test_remove_invalid

[John Lenton (chipaca)]

+1, with a comment.

review: Approve

[Michael Vogt (mvo)]

Thanks for the branch. Unfortunately it does work when testing on snappy:
"""
...
Running /apps/snappy-selftest/0.1/tests/12_test_remove_hello
Can not find '^Removing hello' in 'need root'
ERROR
FAILED: /apps/snappy-selftest/0.1/tests/12_test_remove_hello
"""

Looks like a missing "sudo" in the tests.

review: Needs Fixing

[lp:~jamesodhunt/snappy/selftest-add-pkg-removal-tests]

31. By James Hunt on 2015-04-09

* tests/12_test_remove_hello:
  - Added missing sudo call.
* tests/13_test_remove_invalid:
  - Added missing sudo call.
  - Updated for changed message.

32. By James Hunt on 2015-04-09

* Sync with lp:snappy/selftest.

[James Hunt (jamesodhunt)]

Branch updated.

=== added file 'tests/12_test_remove_hello'
--- tests/12_test_remove_hello	1970-01-01 00:00:00 +0000
+++ tests/12_test_remove_hello	2015-04-09 08:34:44 +0000
@@ -0,0 +1,3 @@
+test() {
+    test_regexp "^Removing hello" sudo $SNAPPY remove hello
+}

=== added file 'tests/13_test_remove_invalid'
--- tests/13_test_remove_invalid	1970-01-01 00:00:00 +0000
+++ tests/13_test_remove_invalid	2015-04-09 08:34:44 +0000
@@ -0,0 +1,5 @@
+test() {
+    pkg=foobarbazinvalid
+    test_regexp "^Removing $pkg" sudo $SNAPPY remove $pkg
+    test_regexp "^snappy package not found" sudo $SNAPPY remove $pkg
+}