From 5b225c751d60d79916da4a7db761f823e12148de Mon Sep 17 00:00:00 2001
From: Florian Pritz <bluewind@xinu.at>
Date: Fri, 6 Feb 2015 12:42:08 +0100
Subject: Add more tests

Signed-off-by: Florian Pritz <bluewind@xinu.at>
---
 application/tests/Test.php        | 14 +++++++++++--
 application/tests/test_api_v1.php | 44 +++++++++++++++++++++++++++++++++++++++
 2 files changed, 56 insertions(+), 2 deletions(-)

(limited to 'application/tests')

diff --git a/application/tests/Test.php b/application/tests/Test.php
index 81225b312..192061db9 100644
--- a/application/tests/Test.php
+++ b/application/tests/Test.php
@@ -66,9 +66,9 @@ abstract class Test {
 		return $json;
 	}
 
-	protected function expectSuccess($testname, $reply)
+	protected function excpectStatus($testname, $reply, $status)
 	{
-		if (!isset($reply["status"]) || $reply["status"] != "success") {
+		if (!isset($reply["status"]) || $reply["status"] != $status) {
 			$this->t->fail($testname);
 			$this->diagReply($reply);
 		} else {
@@ -77,6 +77,16 @@ abstract class Test {
 		return $reply;
 	}
 
+	protected function expectSuccess($testname, $reply)
+	{
+		return $this->excpectStatus($testname, $reply, "success");
+	}
+
+	protected function expectError($testname, $reply)
+	{
+		return $this->excpectStatus($testname, $reply, "error");
+	}
+
 	protected function diagReply($reply)
 	{
 		$this->t->diag("Request got unexpected response:");
diff --git a/application/tests/test_api_v1.php b/application/tests/test_api_v1.php
index 387e3fe6c..9f415abbd 100644
--- a/application/tests/test_api_v1.php
+++ b/application/tests/test_api_v1.php
@@ -115,4 +115,48 @@ class test_api_v1 extends Test {
 		$this->t->ok(empty($ret["data"]["multipaste_items"]), "multipaste_items key exists and empty");
 		$this->t->is($ret["data"]["total_size"], 0, "total_size = 0 since no uploads");
 	}
+
+	public function test_delete_canDeleteUploaded()
+	{
+		$ret = $this->CallAPI("POST", "$this->server/api/1.0.0/file/upload", array(
+			"apikey" => $this->apikeys[2],
+			"file[1]" => curl_file_create("data/tests/small-file"),
+		));
+		$this->expectSuccess("upload file", $ret);
+
+		$id = $ret["data"]["ids"][0];
+
+		$ret = $this->CallAPI("POST", "$this->server/api/1.0.0/file/delete", array(
+			"apikey" => $this->apikeys[2],
+			"ids[1]" => $id,
+		));
+		$this->expectSuccess("delete uploaded file", $ret);
+
+		$this->t->ok(empty($ret["data"]["errors"]), "no errors");
+		$this->t->is_deeply(array($id => array("id" => $id)), $ret["data"]["deleted"], "deleted wanted ID");
+		$this->t->is($ret["data"]["total_count"], 1, "total_count correct");
+		$this->t->is($ret["data"]["deleted_count"], 1, "deleted_count correct");
+	}
+
+	public function test_delete_errorIfNotOwner()
+	{
+		$ret = $this->CallAPI("POST", "$this->server/api/1.0.0/file/upload", array(
+			"apikey" => $this->apikeys[2],
+			"file[1]" => curl_file_create("data/tests/small-file"),
+		));
+		$this->expectSuccess("upload file", $ret);
+
+		$id = $ret["data"]["ids"][0];
+
+		$ret = $this->CallAPI("POST", "$this->server/api/1.0.0/file/delete", array(
+			"apikey" => $this->apikeys[1],
+			"ids[1]" => $id,
+		));
+		$this->expectSuccess("delete file of someone else", $ret);
+
+		$this->t->ok(empty($ret["data"]["deleted"]), "not deleted");
+		$this->t->is_deeply(array($id => array("id" => $id, "reason" => "wrong owner")), $ret["data"]["errors"], "error wanted ID");
+		$this->t->is($ret["data"]["total_count"], 1, "total_count correct");
+		$this->t->is($ret["data"]["deleted_count"], 0, "deleted_count correct");
+	}
 }
-- 
cgit v1.2.3-24-g4f1b