From: Bjoern Schiessle Date: Mon, 30 Sep 2013 10:23:44 +0000 (+0200) Subject: Make sure that testDeleteShare() has some shares to delete X-Git-Tag: v6.0.0alpha2~95^2~6 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=9a2e82c3c54c0fa3661e8e5cb54acc4e3052b3e1;p=nextcloud-server.git Make sure that testDeleteShare() has some shares to delete --- 9a2e82c3c54c0fa3661e8e5cb54acc4e3052b3e1 diff --cc apps/files_sharing/tests/api.php index 04640ab726a,f5897a22d8f..271484f8d44 --- a/apps/files_sharing/tests/api.php +++ b/apps/files_sharing/tests/api.php @@@ -226,10 -226,9 +226,10 @@@ class Test_Files_Sharing_Api extends \P // share was successful? $this->assertTrue($result); + // get item to determine share ID $result = \OCP\Share::getItemShared('file', $fileInfo['fileid']); - $this->assertEquals(count($result), 1); + $this->assertEquals(1, count($result)); // get first element $share = reset($result); @@@ -354,8 -367,8 +368,19 @@@ * @depends testCreateShare */ function testDeleteShare() { ++ ++ $fileInfo = $this->view->getFileInfo($this->filename); ++ ++ \OCP\Share::shareItem('file', $fileInfo['fileid'], \OCP\Share::SHARE_TYPE_USER, ++ \Test_Files_Sharing_Api::TEST_FILES_SHARING_API_USER2, 31); ++ ++ \OCP\Share::shareItem('file', $fileInfo['fileid'], \OCP\Share::SHARE_TYPE_LINK, ++ null, 1); ++ $items = \OCP\Share::getItemShared('file', null); ++ $this->assertEquals(2, count($items)); ++ foreach ($items as $item) { $result = Share\Api::deleteShare(array('id' => $item['id'])); @@@ -365,6 -378,6 +390,7 @@@ $itemsAfterDelete = \OCP\Share::getItemShared('file', null); $this->assertTrue(empty($itemsAfterDelete)); ++ } /**