summaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/tests
diff options
context:
space:
mode:
Diffstat (limited to 'apps/files_sharing/tests')
-rw-r--r--apps/files_sharing/tests/ApiTest.php4
-rw-r--r--apps/files_sharing/tests/CacheTest.php4
-rw-r--r--apps/files_sharing/tests/Collaboration/ShareRecipientSorterTest.php2
-rw-r--r--apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php6
-rw-r--r--apps/files_sharing/tests/Controller/ExternalShareControllerTest.php2
-rw-r--r--apps/files_sharing/tests/Controller/PublicPreviewControllerTest.php2
-rw-r--r--apps/files_sharing/tests/Controller/ShareAPIControllerTest.php2
-rw-r--r--apps/files_sharing/tests/Controller/ShareControllerTest.php4
-rw-r--r--apps/files_sharing/tests/Controller/ShareInfoControllerTest.php2
-rw-r--r--apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php2
-rw-r--r--apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php8
-rw-r--r--apps/files_sharing/tests/ExpireSharesJobTest.php4
-rw-r--r--apps/files_sharing/tests/External/CacheTest.php4
-rw-r--r--apps/files_sharing/tests/External/ManagerTest.php2
-rw-r--r--apps/files_sharing/tests/External/ScannerTest.php2
-rw-r--r--apps/files_sharing/tests/LockingTest.php4
-rw-r--r--apps/files_sharing/tests/Middleware/OCSShareAPIMiddlewareTest.php2
-rw-r--r--apps/files_sharing/tests/Middleware/ShareInfoMiddlewareTest.php2
-rw-r--r--apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php2
-rw-r--r--apps/files_sharing/tests/Migration/SetPasswordColumnTest.php4
-rw-r--r--apps/files_sharing/tests/MountProviderTest.php2
-rw-r--r--apps/files_sharing/tests/PermissionsTest.php4
-rw-r--r--apps/files_sharing/tests/PropagationTestCase.php6
-rw-r--r--apps/files_sharing/tests/ShareTest.php4
-rw-r--r--apps/files_sharing/tests/SharedMountTest.php4
-rw-r--r--apps/files_sharing/tests/SharedStorageTest.php4
-rw-r--r--apps/files_sharing/tests/TestCase.php8
-rw-r--r--apps/files_sharing/tests/UnshareChildrenTest.php4
-rw-r--r--apps/files_sharing/tests/UpdaterTest.php6
-rw-r--r--apps/files_sharing/tests/WatcherTest.php4
30 files changed, 55 insertions, 55 deletions
diff --git a/apps/files_sharing/tests/ApiTest.php b/apps/files_sharing/tests/ApiTest.php
index 5cf3906e565..05acb9c18e2 100644
--- a/apps/files_sharing/tests/ApiTest.php
+++ b/apps/files_sharing/tests/ApiTest.php
@@ -63,7 +63,7 @@ class ApiTest extends TestCase {
/** @var string */
private $subsubfolder;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
\OC::$server->getConfig()->setAppValue('core', 'shareapi_exclude_groups', 'no');
@@ -88,7 +88,7 @@ class ApiTest extends TestCase {
$this->userFolder = \OC::$server->getUserFolder(self::TEST_FILES_SHARING_API_USER1);
}
- protected function tearDown() {
+ protected function tearDown(): void {
if($this->view instanceof \OC\Files\View) {
$this->view->unlink($this->filename);
$this->view->deleteAll($this->folder);
diff --git a/apps/files_sharing/tests/CacheTest.php b/apps/files_sharing/tests/CacheTest.php
index e8d8ae7f63b..d2f96d01eb8 100644
--- a/apps/files_sharing/tests/CacheTest.php
+++ b/apps/files_sharing/tests/CacheTest.php
@@ -62,7 +62,7 @@ class CacheTest extends TestCase {
/** @var \OCP\Share\IManager */
protected $shareManager;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->shareManager = \OC::$server->getShareManager();
@@ -128,7 +128,7 @@ class CacheTest extends TestCase {
$this->sharedCache = $this->sharedStorage->getCache();
}
- protected function tearDown() {
+ protected function tearDown(): void {
if($this->sharedCache) {
$this->sharedCache->clear();
}
diff --git a/apps/files_sharing/tests/Collaboration/ShareRecipientSorterTest.php b/apps/files_sharing/tests/Collaboration/ShareRecipientSorterTest.php
index 8f516788761..93bef4afbf6 100644
--- a/apps/files_sharing/tests/Collaboration/ShareRecipientSorterTest.php
+++ b/apps/files_sharing/tests/Collaboration/ShareRecipientSorterTest.php
@@ -43,7 +43,7 @@ class ShareRecipientSorterTest extends TestCase {
/** @var ShareRecipientSorter */
protected $sorter;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->shareManager = $this->createMock(IManager::class);
diff --git a/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php b/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php
index 257d9c35dff..4e100cf2099 100644
--- a/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php
+++ b/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php
@@ -57,7 +57,7 @@ class CleanupRemoteStoragesTest extends TestCase {
['notExistingId' => 'shared::c34568c143cdac7d2f06e0800b5280f9', 'share_token' => 'f2c69dad1dc0649f26976fd210fc62e7', 'remote' => 'https://hostname.tld/owncloud7', 'user' => 'user7'],
];
- protected function setup() {
+ protected function setUp(): void {
parent::setUp();
$this->connection = \OC::$server->getDatabaseConnection();
@@ -110,7 +110,7 @@ class CleanupRemoteStoragesTest extends TestCase {
$this->command = new CleanupRemoteStorages($this->connection);
}
- public function tearDown() {
+ public function tearDown(): void {
$storageQuery = \OC::$server->getDatabaseConnection()->getQueryBuilder();
$storageQuery->delete('storages')
->where($storageQuery->expr()->eq('id', $storageQuery->createParameter('id')));
@@ -133,7 +133,7 @@ class CleanupRemoteStoragesTest extends TestCase {
}
}
- return parent::tearDown();
+ parent::tearDown();
}
private function doesStorageExist($numericId) {
diff --git a/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php b/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php
index 71a4f4adeba..d2237acbd7f 100644
--- a/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php
@@ -43,7 +43,7 @@ class ExternalShareControllerTest extends \Test\TestCase {
/** @var IClientService */
private $clientService;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->request = $this->getMockBuilder('\\OCP\\IRequest')
->disableOriginalConstructor()->getMock();
diff --git a/apps/files_sharing/tests/Controller/PublicPreviewControllerTest.php b/apps/files_sharing/tests/Controller/PublicPreviewControllerTest.php
index ed556172a27..cbe3872bb67 100644
--- a/apps/files_sharing/tests/Controller/PublicPreviewControllerTest.php
+++ b/apps/files_sharing/tests/Controller/PublicPreviewControllerTest.php
@@ -54,7 +54,7 @@ class PublicPreviewControllerTest extends TestCase {
/** @var PublicPreviewController */
private $controller;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->previewManager = $this->createMock(IPreview::class);
diff --git a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
index 6452ad50019..32a2e88f781 100644
--- a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
@@ -100,7 +100,7 @@ class ShareAPIControllerTest extends TestCase {
/** @var IServerContainer|\PHPUnit_Framework_MockObject_MockObject */
private $serverContainer;
- protected function setUp() {
+ protected function setUp(): void {
$this->shareManager = $this->createMock(IManager::class);
$this->shareManager
->expects($this->any())
diff --git a/apps/files_sharing/tests/Controller/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php
index 01b20e58572..ec6fc9a598d 100644
--- a/apps/files_sharing/tests/Controller/ShareControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareControllerTest.php
@@ -93,7 +93,7 @@ class ShareControllerTest extends \Test\TestCase {
/** @var IL10N */
private $l10n;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->appName = 'files_sharing';
@@ -142,7 +142,7 @@ class ShareControllerTest extends \Test\TestCase {
$this->loginAsUser($this->user);
}
- protected function tearDown() {
+ protected function tearDown(): void {
\OC_Util::tearDownFS();
\OC_User::setUserId('');
Filesystem::tearDown();
diff --git a/apps/files_sharing/tests/Controller/ShareInfoControllerTest.php b/apps/files_sharing/tests/Controller/ShareInfoControllerTest.php
index 627990a2f90..b8d87d5cd68 100644
--- a/apps/files_sharing/tests/Controller/ShareInfoControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareInfoControllerTest.php
@@ -45,7 +45,7 @@ class ShareInfoControllerTest extends TestCase {
private $shareManager;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->shareManager = $this->createMock(ShareManager::class);
diff --git a/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php
index c5d31e3b59d..20790fc0bfa 100644
--- a/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php
@@ -62,7 +62,7 @@ class ShareesAPIControllerTest extends TestCase {
/** @var ISearch|\PHPUnit_Framework_MockObject_MockObject */
protected $collaboratorSearch;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->uid = 'test123';
diff --git a/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php b/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php
index 03f19525e0d..028878e3abd 100644
--- a/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php
+++ b/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php
@@ -61,7 +61,7 @@ class DeleteOrphanedSharesJobTest extends \Test\TestCase {
*/
private $user2;
- public static function setUpBeforeClass() {
+ public static function setUpBeforeClass(): void {
$appManager = \OC::$server->getAppManager();
self::$trashBinStatus = $appManager->isEnabledForUser('files_trashbin');
$appManager->disableApp('files_trashbin');
@@ -70,13 +70,13 @@ class DeleteOrphanedSharesJobTest extends \Test\TestCase {
\OC\Files\Filesystem::getLoader()->removeStorageWrapper('oc_trashbin');
}
- public static function tearDownAfterClass() {
+ public static function tearDownAfterClass(): void {
if (self::$trashBinStatus) {
\OC::$server->getAppManager()->enableApp('files_trashbin');
}
}
- protected function setup() {
+ protected function setUp(): void {
parent::setUp();
$this->connection = \OC::$server->getDatabaseConnection();
@@ -95,7 +95,7 @@ class DeleteOrphanedSharesJobTest extends \Test\TestCase {
$this->job = new DeleteOrphanedSharesJob();
}
- protected function tearDown() {
+ protected function tearDown(): void {
$this->connection->executeUpdate('DELETE FROM `*PREFIX*share`');
$userManager = \OC::$server->getUserManager();
diff --git a/apps/files_sharing/tests/ExpireSharesJobTest.php b/apps/files_sharing/tests/ExpireSharesJobTest.php
index e255f08ccf1..0f34f0785a7 100644
--- a/apps/files_sharing/tests/ExpireSharesJobTest.php
+++ b/apps/files_sharing/tests/ExpireSharesJobTest.php
@@ -55,7 +55,7 @@ class ExpireSharesJobTest extends \Test\TestCase {
*/
private $user2;
- protected function setup() {
+ protected function setUp(): void {
parent::setUp();
$this->connection = \OC::$server->getDatabaseConnection();
@@ -74,7 +74,7 @@ class ExpireSharesJobTest extends \Test\TestCase {
$this->job = new ExpireSharesJob();
}
- protected function tearDown() {
+ protected function tearDown(): void {
$this->connection->executeUpdate('DELETE FROM `*PREFIX*share`');
$userManager = \OC::$server->getUserManager();
diff --git a/apps/files_sharing/tests/External/CacheTest.php b/apps/files_sharing/tests/External/CacheTest.php
index 02109e35db1..39ff1ff24f1 100644
--- a/apps/files_sharing/tests/External/CacheTest.php
+++ b/apps/files_sharing/tests/External/CacheTest.php
@@ -56,7 +56,7 @@ class CacheTest extends TestCase {
/** @var ICloudIdManager */
private $cloudIdManager;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->cloudIdManager = new CloudIdManager();
@@ -83,7 +83,7 @@ class CacheTest extends TestCase {
);
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->cache) {
$this->cache->clear();
}
diff --git a/apps/files_sharing/tests/External/ManagerTest.php b/apps/files_sharing/tests/External/ManagerTest.php
index 93c17ca10cf..2224b767083 100644
--- a/apps/files_sharing/tests/External/ManagerTest.php
+++ b/apps/files_sharing/tests/External/ManagerTest.php
@@ -78,7 +78,7 @@ class ManagerTest extends TestCase {
private $user;
private $testMountProvider;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->uid = $this->getUniqueID('user');
diff --git a/apps/files_sharing/tests/External/ScannerTest.php b/apps/files_sharing/tests/External/ScannerTest.php
index fe77182e83f..b9bfb1a84a8 100644
--- a/apps/files_sharing/tests/External/ScannerTest.php
+++ b/apps/files_sharing/tests/External/ScannerTest.php
@@ -33,7 +33,7 @@ class ScannerTest extends TestCase {
/** @var \OC\Files\Cache\Cache|\PHPUnit_Framework_MockObject_MockObject */
protected $cache;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->storage = $this->getMockBuilder('\OCA\Files_Sharing\External\Storage')
diff --git a/apps/files_sharing/tests/LockingTest.php b/apps/files_sharing/tests/LockingTest.php
index 414e8790819..47394fc588f 100644
--- a/apps/files_sharing/tests/LockingTest.php
+++ b/apps/files_sharing/tests/LockingTest.php
@@ -46,7 +46,7 @@ class LockingTest extends TestCase {
private $ownerUid;
private $recipientUid;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->userBackend = new \Test\Util\User\Dummy();
@@ -74,7 +74,7 @@ class LockingTest extends TestCase {
$this->assertTrue(Filesystem::file_exists('bar.txt'));
}
- public function tearDown() {
+ public function tearDown(): void {
\OC::$server->getUserManager()->removeBackend($this->userBackend);
parent::tearDown();
}
diff --git a/apps/files_sharing/tests/Middleware/OCSShareAPIMiddlewareTest.php b/apps/files_sharing/tests/Middleware/OCSShareAPIMiddlewareTest.php
index a1d1f1bf551..06d33859a28 100644
--- a/apps/files_sharing/tests/Middleware/OCSShareAPIMiddlewareTest.php
+++ b/apps/files_sharing/tests/Middleware/OCSShareAPIMiddlewareTest.php
@@ -44,7 +44,7 @@ class OCSShareAPIMiddlewareTest extends \Test\TestCase {
/** @var OCSShareAPIMiddleware */
private $middleware;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->shareManager = $this->createMock(IManager::class);
diff --git a/apps/files_sharing/tests/Middleware/ShareInfoMiddlewareTest.php b/apps/files_sharing/tests/Middleware/ShareInfoMiddlewareTest.php
index 781861aa392..e6744d5fd1d 100644
--- a/apps/files_sharing/tests/Middleware/ShareInfoMiddlewareTest.php
+++ b/apps/files_sharing/tests/Middleware/ShareInfoMiddlewareTest.php
@@ -41,7 +41,7 @@ class ShareInfoMiddlewareTest extends TestCase {
/** @var ShareInfoMiddleware */
private $middleware;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->shareManager = $this->createMock(ShareManager::class);
diff --git a/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php b/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
index f49b472a35c..e3fb602acd6 100644
--- a/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
+++ b/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
@@ -59,7 +59,7 @@ class SharingCheckMiddlewareTest extends \Test\TestCase {
/** @var IRequest | \PHPUnit_Framework_MockObject_MockObject */
private $request;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->config = $this->createMock(IConfig::class);
diff --git a/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php b/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php
index c4c78b04a23..418ede9179a 100644
--- a/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php
+++ b/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php
@@ -47,7 +47,7 @@ class SetPasswordColumnTest extends TestCase {
private $table = 'share';
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->connection = \OC::$server->getDatabaseConnection();
@@ -57,7 +57,7 @@ class SetPasswordColumnTest extends TestCase {
$this->cleanDB();
}
- public function tearDown() {
+ public function tearDown(): void {
parent::tearDown();
$this->cleanDB();
}
diff --git a/apps/files_sharing/tests/MountProviderTest.php b/apps/files_sharing/tests/MountProviderTest.php
index f4acfdfafe7..5b20524da47 100644
--- a/apps/files_sharing/tests/MountProviderTest.php
+++ b/apps/files_sharing/tests/MountProviderTest.php
@@ -59,7 +59,7 @@ class MountProviderTest extends \Test\TestCase {
/** @var ILogger | \PHPUnit_Framework_MockObject_MockObject */
private $logger;
- public function setUp() {
+ public function setUp(): void {
parent::setUp();
$this->config = $this->getMockBuilder(IConfig::class)->getMock();
diff --git a/apps/files_sharing/tests/PermissionsTest.php b/apps/files_sharing/tests/PermissionsTest.php
index a64d0b25e17..e47cfff68dd 100644
--- a/apps/files_sharing/tests/PermissionsTest.php
+++ b/apps/files_sharing/tests/PermissionsTest.php
@@ -60,7 +60,7 @@ class PermissionsTest extends TestCase {
/** @var Cache */
private $ownerCache;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
@@ -115,7 +115,7 @@ class PermissionsTest extends TestCase {
$this->sharedCacheRestrictedShare = $this->sharedStorageRestrictedShare->getCache();
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->sharedCache) {
$this->sharedCache->clear();
}
diff --git a/apps/files_sharing/tests/PropagationTestCase.php b/apps/files_sharing/tests/PropagationTestCase.php
index 21cea3033af..5f16173725e 100644
--- a/apps/files_sharing/tests/PropagationTestCase.php
+++ b/apps/files_sharing/tests/PropagationTestCase.php
@@ -31,17 +31,17 @@ abstract class PropagationTestCase extends TestCase {
protected $fileIds = []; // [$user=>[$path=>$id]]
protected $fileEtags = []; // [$id=>$etag]
- public static function setUpBeforeClass() {
+ public static function setUpBeforeClass(): void {
parent::setUpBeforeClass();
\OCA\Files_Sharing\Helper::registerHooks();
}
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->setUpShares();
}
- protected function tearDown() {
+ protected function tearDown(): void {
\OC_Hook::clear('OC_Filesystem', 'post_write');
\OC_Hook::clear('OC_Filesystem', 'post_delete');
\OC_Hook::clear('OC_Filesystem', 'post_rename');
diff --git a/apps/files_sharing/tests/ShareTest.php b/apps/files_sharing/tests/ShareTest.php
index 61bec2caea3..f0614d5c984 100644
--- a/apps/files_sharing/tests/ShareTest.php
+++ b/apps/files_sharing/tests/ShareTest.php
@@ -38,7 +38,7 @@ class ShareTest extends TestCase {
private static $tempStorage;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->folder = self::TEST_FOLDER_NAME;
@@ -56,7 +56,7 @@ class ShareTest extends TestCase {
$this->view->file_put_contents($this->folder . $this->subfolder . $this->filename, $this->data);
}
- protected function tearDown() {
+ protected function tearDown(): void {
self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
$this->view->unlink($this->filename);
$this->view->deleteAll($this->folder);
diff --git a/apps/files_sharing/tests/SharedMountTest.php b/apps/files_sharing/tests/SharedMountTest.php
index f01e9896efb..17b2abf3fce 100644
--- a/apps/files_sharing/tests/SharedMountTest.php
+++ b/apps/files_sharing/tests/SharedMountTest.php
@@ -44,7 +44,7 @@ class SharedMountTest extends TestCase {
/** @var IUserManager */
private $userManager;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->folder = '/folder_share_storage_test';
@@ -62,7 +62,7 @@ class SharedMountTest extends TestCase {
$this->userManager = \OC::$server->getUserManager();
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->view) {
if ($this->view->file_exists($this->folder)) {
$this->view->unlink($this->folder);
diff --git a/apps/files_sharing/tests/SharedStorageTest.php b/apps/files_sharing/tests/SharedStorageTest.php
index e2cb65d9acc..c4cefee4a34 100644
--- a/apps/files_sharing/tests/SharedStorageTest.php
+++ b/apps/files_sharing/tests/SharedStorageTest.php
@@ -40,7 +40,7 @@ use OCP\Share\IShare;
*/
class SharedStorageTest extends TestCase {
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
\OCA\Files_Trashbin\Trashbin::registerHooks();
$this->folder = '/folder_share_storage_test';
@@ -55,7 +55,7 @@ class SharedStorageTest extends TestCase {
$this->view->file_put_contents($this->folder . $this->filename, "file in subfolder");
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->view) {
if ($this->view->file_exists($this->folder)) {
$this->view->unlink($this->folder);
diff --git a/apps/files_sharing/tests/TestCase.php b/apps/files_sharing/tests/TestCase.php
index 6c5f078f9e7..ce5ceb1f642 100644
--- a/apps/files_sharing/tests/TestCase.php
+++ b/apps/files_sharing/tests/TestCase.php
@@ -68,7 +68,7 @@ abstract class TestCase extends \Test\TestCase {
/** @var \OCP\Files\IRootFolder */
protected $rootFolder;
- public static function setUpBeforeClass() {
+ public static function setUpBeforeClass(): void {
parent::setUpBeforeClass();
$application = new Application();
@@ -107,7 +107,7 @@ abstract class TestCase extends \Test\TestCase {
\OC::$server->getGroupManager()->addBackend($groupBackend);
}
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
//login as user1
@@ -120,7 +120,7 @@ abstract class TestCase extends \Test\TestCase {
$this->rootFolder = \OC::$server->getRootFolder();
}
- protected function tearDown() {
+ protected function tearDown(): void {
$qb = \OC::$server->getDatabaseConnection()->getQueryBuilder();
$qb->delete('share');
$qb->execute();
@@ -128,7 +128,7 @@ abstract class TestCase extends \Test\TestCase {
parent::tearDown();
}
- public static function tearDownAfterClass() {
+ public static function tearDownAfterClass(): void {
// cleanup users
$user = \OC::$server->getUserManager()->get(self::TEST_FILES_SHARING_API_USER1);
if ($user !== null) { $user->delete(); }
diff --git a/apps/files_sharing/tests/UnshareChildrenTest.php b/apps/files_sharing/tests/UnshareChildrenTest.php
index ce7767d1920..92ff48eb3be 100644
--- a/apps/files_sharing/tests/UnshareChildrenTest.php
+++ b/apps/files_sharing/tests/UnshareChildrenTest.php
@@ -42,7 +42,7 @@ class UnshareChildrenTest extends TestCase {
private static $tempStorage;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
\OCP\Util::connectHook('OC_Filesystem', 'post_delete', '\OCA\Files_Sharing\Hooks', 'unshareChildren');
@@ -61,7 +61,7 @@ class UnshareChildrenTest extends TestCase {
$this->view->file_put_contents($this->folder . $this->subfolder . $this->filename, $this->data);
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->view) {
$this->view->deleteAll($this->folder);
}
diff --git a/apps/files_sharing/tests/UpdaterTest.php b/apps/files_sharing/tests/UpdaterTest.php
index 2e044716d07..d20f794883a 100644
--- a/apps/files_sharing/tests/UpdaterTest.php
+++ b/apps/files_sharing/tests/UpdaterTest.php
@@ -38,12 +38,12 @@ class UpdaterTest extends TestCase {
const TEST_FOLDER_NAME = '/folder_share_updater_test';
- public static function setUpBeforeClass() {
+ public static function setUpBeforeClass(): void {
parent::setUpBeforeClass();
\OCA\Files_Sharing\Helper::registerHooks();
}
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
$this->folder = self::TEST_FOLDER_NAME;
@@ -56,7 +56,7 @@ class UpdaterTest extends TestCase {
$this->view->file_put_contents($this->folder . '/' . $this->filename, $this->data);
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->view) {
$this->view->unlink($this->filename);
$this->view->deleteAll($this->folder);
diff --git a/apps/files_sharing/tests/WatcherTest.php b/apps/files_sharing/tests/WatcherTest.php
index 59f84057a14..e332327a917 100644
--- a/apps/files_sharing/tests/WatcherTest.php
+++ b/apps/files_sharing/tests/WatcherTest.php
@@ -53,7 +53,7 @@ class WatcherTest extends TestCase {
/** @var \OCP\Share\IShare */
private $_share;
- protected function setUp() {
+ protected function setUp(): void {
parent::setUp();
self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
@@ -88,7 +88,7 @@ class WatcherTest extends TestCase {
$this->sharedCache = $this->sharedStorage->getCache();
}
- protected function tearDown() {
+ protected function tearDown(): void {
if ($this->sharedCache) {
$this->sharedCache->clear();
}