Test result of delete_invitation_key

Signed-off-by: Florian Pritz <bluewind@xinu.at>

1 files changed, 10 insertions, 6 deletions

diff --git a/application/test/tests/test_service_user.php b/application/test/tests/test_service_user.php
index 4cbbdfac0..d7e34a71b 100644
--- a/application/test/tests/test_service_user.php
+++ b/application/test/tests/test_service_user.php
@@ -34,24 +34,28 @@ class test_service_user extends \test\Test {
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([['user' => "".$userid, 'key' => $key, 'action' => 'invitation']], $result, "database contains new key");
 
-		\service\user::delete_invitation_key($userid+1, $key);
+		$ret = \service\user::delete_invitation_key($userid+1, $key);
+		$this->t->is(0, $ret, "Should have removed no keys because incorrect user/key");
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([['user' => "".$userid, 'key' => $key, 'action' => 'invitation']], $result, "database contains new key after incorrect deletion");
 
-		\service\user::delete_invitation_key($userid+1, "foobar-");
+		$ret = \service\user::delete_invitation_key($userid+1, "foobar-");
+		$this->t->is(0, $ret, "Should have removed no keys because incorrect user/key");
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([['user' => "".$userid, 'key' => $key, 'action' => 'invitation']], $result, "database contains new key after incorrect deletion");
 
-		\service\user::delete_invitation_key($userid+1, "");
+		$ret = \service\user::delete_invitation_key($userid+1, "");
+		$this->t->is(0, $ret, "Should have removed no keys because incorrect user/key");
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([['user' => "".$userid, 'key' => $key, 'action' => 'invitation']], $result, "database contains new key after incorrect deletion");
 
-		\service\user::delete_invitation_key($userid, "");
+		$ret = \service\user::delete_invitation_key($userid, "");
+		$this->t->is(0, $ret, "Should have removed no keys because incorrect user/key");
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([['user' => "".$userid, 'key' => $key, 'action' => 'invitation']], $result, "database contains new key");
 
-		\service\user::delete_invitation_key($userid, $key);
-
+		$ret = \service\user::delete_invitation_key($userid, $key);
+		$this->t->is(1, $ret, "One key should be removed");
 		$result = $CI->db->select('user, key, action')->from('actions')->get()->result_array();
 		$this->t->is_deeply([], $result, "key has been deleted");