aboutsummaryrefslogtreecommitdiffstats
path: root/tests/lib
diff options
context:
space:
mode:
Diffstat (limited to 'tests/lib')
-rw-r--r--tests/lib/Collaboration/Collaborators/RemotePluginTest.php2
-rw-r--r--tests/lib/Collaboration/Collaborators/SearchTest.php2
-rw-r--r--tests/lib/DB/MigratorTest.php10
-rw-r--r--tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php36
-rw-r--r--tests/lib/DB/QueryBuilder/QuoteHelperTest.php2
-rw-r--r--tests/lib/Files/Cache/QuerySearchHelperTest.php2
-rw-r--r--tests/lib/Files/Config/UserMountCacheTest.php34
-rw-r--r--tests/lib/Files/FilesystemTest.php10
-rw-r--r--tests/lib/Files/Node/HookConnectorTest.php2
-rw-r--r--tests/lib/Files/Storage/Wrapper/QuotaTest.php4
-rw-r--r--tests/lib/Group/MetaDataTest.php4
-rw-r--r--tests/lib/LegacyHelperTest.php2
-rw-r--r--tests/lib/Security/IdentityProof/ManagerTest.php2
-rw-r--r--tests/lib/TestCase.php2
14 files changed, 57 insertions, 57 deletions
diff --git a/tests/lib/Collaboration/Collaborators/RemotePluginTest.php b/tests/lib/Collaboration/Collaborators/RemotePluginTest.php
index 1345df13379..981260a80dd 100644
--- a/tests/lib/Collaboration/Collaborators/RemotePluginTest.php
+++ b/tests/lib/Collaboration/Collaborators/RemotePluginTest.php
@@ -133,7 +133,7 @@ class RemotePluginTest extends TestCase {
->method('search')
->willReturn([]);
- list($remoteUser, $remoteUrl) = $this->plugin->splitUserRemote($remote);
+ [$remoteUser, $remoteUrl] = $this->plugin->splitUserRemote($remote);
$this->assertSame($expectedUser, $remoteUser);
$this->assertSame($expectedUrl, $remoteUrl);
}
diff --git a/tests/lib/Collaboration/Collaborators/SearchTest.php b/tests/lib/Collaboration/Collaborators/SearchTest.php
index 0aebef72de0..17f5c39aca7 100644
--- a/tests/lib/Collaboration/Collaborators/SearchTest.php
+++ b/tests/lib/Collaboration/Collaborators/SearchTest.php
@@ -123,7 +123,7 @@ class SearchTest extends TestCase {
$this->search->registerPlugin(['shareType' => 'SHARE_TYPE_GROUP', 'class' => $groupPlugin]);
$this->search->registerPlugin(['shareType' => 'SHARE_TYPE_REMOTE', 'class' => $remotePlugin]);
- list($results, $moreResults) = $this->search->search($searchTerm, $shareTypes, false, $perPage, $perPage * ($page - 1));
+ [$results, $moreResults] = $this->search->search($searchTerm, $shareTypes, false, $perPage, $perPage * ($page - 1));
$this->assertEquals($expected, $results);
$this->assertSame($expectedMoreResults, $moreResults);
diff --git a/tests/lib/DB/MigratorTest.php b/tests/lib/DB/MigratorTest.php
index 6f3ebbb2b0c..6b758534897 100644
--- a/tests/lib/DB/MigratorTest.php
+++ b/tests/lib/DB/MigratorTest.php
@@ -138,7 +138,7 @@ class MigratorTest extends \Test\TestCase {
if ($this->isSQLite()) {
$this->markTestSkipped('sqlite does not throw errors when creating a new key on existing data');
}
- list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
+ [$startSchema, $endSchema] = $this->getDuplicateKeySchemas();
$migrator = $this->manager->getMigrator();
$migrator->migrate($startSchema);
@@ -157,7 +157,7 @@ class MigratorTest extends \Test\TestCase {
}
public function testChangeToString() {
- list($startSchema, $endSchema) = $this->getChangedTypeSchema('integer', 'string');
+ [$startSchema, $endSchema] = $this->getChangedTypeSchema('integer', 'string');
$migrator = $this->manager->getMigrator();
$migrator->migrate($startSchema);
$schema = new SchemaWrapper($this->connection);
@@ -184,7 +184,7 @@ class MigratorTest extends \Test\TestCase {
}
public function testUpgrade() {
- list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
+ [$startSchema, $endSchema] = $this->getDuplicateKeySchemas();
$migrator = $this->manager->getMigrator();
$migrator->migrate($startSchema);
@@ -202,7 +202,7 @@ class MigratorTest extends \Test\TestCase {
$this->config->setSystemValue('dbtableprefix', 'ownc_');
$this->tableName = strtolower($this->getUniqueID($this->config->getSystemValue('dbtableprefix') . 'test_'));
- list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
+ [$startSchema, $endSchema] = $this->getDuplicateKeySchemas();
$migrator = $this->manager->getMigrator();
$migrator->migrate($startSchema);
@@ -217,7 +217,7 @@ class MigratorTest extends \Test\TestCase {
}
public function testInsertAfterUpgrade() {
- list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
+ [$startSchema, $endSchema] = $this->getDuplicateKeySchemas();
$migrator = $this->manager->getMigrator();
$migrator->migrate($startSchema);
diff --git a/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php b/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php
index 2a3a4e9a9c1..76d9dab75ee 100644
--- a/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php
+++ b/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php
@@ -82,8 +82,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testComparison($comparison, $input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->comparison($doctrineInput1, $comparison, $doctrineInput2),
@@ -109,8 +109,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testEquals($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->eq($doctrineInput1, $doctrineInput2),
@@ -127,8 +127,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testNotEquals($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->neq($doctrineInput1, $doctrineInput2),
@@ -145,8 +145,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testLowerThan($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->lt($doctrineInput1, $doctrineInput2),
@@ -163,8 +163,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testLowerThanEquals($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->lte($doctrineInput1, $doctrineInput2),
@@ -181,8 +181,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testGreaterThan($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->gt($doctrineInput1, $doctrineInput2),
@@ -199,8 +199,8 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isInput2Literal
*/
public function testGreaterThanEquals($input1, $isInput1Literal, $input2, $isInput2Literal) {
- list($doctrineInput1, $ocInput1) = $this->helpWithLiteral($input1, $isInput1Literal);
- list($doctrineInput2, $ocInput2) = $this->helpWithLiteral($input2, $isInput2Literal);
+ [$doctrineInput1, $ocInput1] = $this->helpWithLiteral($input1, $isInput1Literal);
+ [$doctrineInput2, $ocInput2] = $this->helpWithLiteral($input2, $isInput2Literal);
$this->assertEquals(
$this->doctrineExpressionBuilder->gte($doctrineInput1, $doctrineInput2),
@@ -236,7 +236,7 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isLiteral
*/
public function testLike($input, $isLiteral) {
- list($doctrineInput, $ocInput) = $this->helpWithLiteral($input, $isLiteral);
+ [$doctrineInput, $ocInput] = $this->helpWithLiteral($input, $isLiteral);
$this->assertEquals(
$this->doctrineExpressionBuilder->like('`test`', $doctrineInput),
@@ -251,7 +251,7 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isLiteral
*/
public function testNotLike($input, $isLiteral) {
- list($doctrineInput, $ocInput) = $this->helpWithLiteral($input, $isLiteral);
+ [$doctrineInput, $ocInput] = $this->helpWithLiteral($input, $isLiteral);
$this->assertEquals(
$this->doctrineExpressionBuilder->notLike('`test`', $doctrineInput),
@@ -275,7 +275,7 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isLiteral
*/
public function testIn($input, $isLiteral) {
- list($doctrineInput, $ocInput) = $this->helpWithLiteral($input, $isLiteral);
+ [$doctrineInput, $ocInput] = $this->helpWithLiteral($input, $isLiteral);
$this->assertEquals(
$this->doctrineExpressionBuilder->in('`test`', $doctrineInput),
@@ -290,7 +290,7 @@ class ExpressionBuilderTest extends TestCase {
* @param bool $isLiteral
*/
public function testNotIn($input, $isLiteral) {
- list($doctrineInput, $ocInput) = $this->helpWithLiteral($input, $isLiteral);
+ [$doctrineInput, $ocInput] = $this->helpWithLiteral($input, $isLiteral);
$this->assertEquals(
$this->doctrineExpressionBuilder->notIn('`test`', $doctrineInput),
diff --git a/tests/lib/DB/QueryBuilder/QuoteHelperTest.php b/tests/lib/DB/QueryBuilder/QuoteHelperTest.php
index 3b5562b69c9..17e7ac14a02 100644
--- a/tests/lib/DB/QueryBuilder/QuoteHelperTest.php
+++ b/tests/lib/DB/QueryBuilder/QuoteHelperTest.php
@@ -136,7 +136,7 @@ class QuoteHelperTest extends \Test\TestCase {
}
if (substr_count($string, '.')) {
- list($alias, $columnName) = explode('.', $string);
+ [$alias, $columnName] = explode('.', $string);
return '`' . $alias . '`.`' . $columnName . '`';
}
diff --git a/tests/lib/Files/Cache/QuerySearchHelperTest.php b/tests/lib/Files/Cache/QuerySearchHelperTest.php
index 8344460f757..7ae0c2b38c2 100644
--- a/tests/lib/Files/Cache/QuerySearchHelperTest.php
+++ b/tests/lib/Files/Cache/QuerySearchHelperTest.php
@@ -111,7 +111,7 @@ class QuerySearchHelperTest extends TestCase {
$data['name'] = basename($data['path']);
$data['parent'] = -1;
if (isset($data['mimetype'])) {
- list($mimepart,) = explode('/', $data['mimetype']);
+ [$mimepart,] = explode('/', $data['mimetype']);
$data['mimepart'] = $this->mimetypeLoader->getId($mimepart);
$data['mimetype'] = $this->mimetypeLoader->getId($data['mimetype']);
} else {
diff --git a/tests/lib/Files/Config/UserMountCacheTest.php b/tests/lib/Files/Config/UserMountCacheTest.php
index d170049aab5..4ee0f730440 100644
--- a/tests/lib/Files/Config/UserMountCacheTest.php
+++ b/tests/lib/Files/Config/UserMountCacheTest.php
@@ -107,7 +107,7 @@ class UserMountCacheTest extends TestCase {
public function testNewMounts() {
$user = $this->userManager->get('u1');
- list($storage) = $this->getStorage(10);
+ [$storage] = $this->getStorage(10);
$mount = new MountPoint($storage, '/asd/');
$this->cache->registerMounts($user, [$mount]);
@@ -127,7 +127,7 @@ class UserMountCacheTest extends TestCase {
public function testSameMounts() {
$user = $this->userManager->get('u1');
- list($storage) = $this->getStorage(10);
+ [$storage] = $this->getStorage(10);
$mount = new MountPoint($storage, '/asd/');
$this->cache->registerMounts($user, [$mount]);
@@ -151,7 +151,7 @@ class UserMountCacheTest extends TestCase {
public function testRemoveMounts() {
$user = $this->userManager->get('u1');
- list($storage) = $this->getStorage(10);
+ [$storage] = $this->getStorage(10);
$mount = new MountPoint($storage, '/asd/');
$this->cache->registerMounts($user, [$mount]);
@@ -170,7 +170,7 @@ class UserMountCacheTest extends TestCase {
public function testChangeMounts() {
$user = $this->userManager->get('u1');
- list($storage) = $this->getStorage(10);
+ [$storage] = $this->getStorage(10);
$mount = new MountPoint($storage, '/bar/');
$this->cache->registerMounts($user, [$mount]);
@@ -193,7 +193,7 @@ class UserMountCacheTest extends TestCase {
public function testChangeMountId() {
$user = $this->userManager->get('u1');
- list($storage) = $this->getStorage(10);
+ [$storage] = $this->getStorage(10);
$mount = new MountPoint($storage, '/foo/', null, null, null, null);
$this->cache->registerMounts($user, [$mount]);
@@ -218,8 +218,8 @@ class UserMountCacheTest extends TestCase {
$user2 = $this->userManager->get('u2');
$user3 = $this->userManager->get('u3');
- list($storage1, $id1) = $this->getStorage(1);
- list($storage2, $id2) = $this->getStorage(2);
+ [$storage1, $id1] = $this->getStorage(1);
+ [$storage2, $id2] = $this->getStorage(2);
$mount1 = new MountPoint($storage1, '/foo/');
$mount2 = new MountPoint($storage2, '/bar/');
@@ -252,8 +252,8 @@ class UserMountCacheTest extends TestCase {
$user1 = $this->userManager->get('u1');
$user2 = $this->userManager->get('u2');
- list($storage1, $id1) = $this->getStorage(1);
- list($storage2, $id2) = $this->getStorage(2);
+ [$storage1, $id1] = $this->getStorage(1);
+ [$storage2, $id2] = $this->getStorage(2);
$mount1 = new MountPoint($storage1, '/foo/');
$mount2 = new MountPoint($storage2, '/bar/');
@@ -282,8 +282,8 @@ class UserMountCacheTest extends TestCase {
$user1 = $this->userManager->get('u1');
$user2 = $this->userManager->get('u2');
- list($storage1, $id1) = $this->getStorage(1);
- list($storage2, $id2) = $this->getStorage(2);
+ [$storage1, $id1] = $this->getStorage(1);
+ [$storage2, $id2] = $this->getStorage(2);
$mount1 = new MountPoint($storage1, '/foo/');
$mount2 = new MountPoint($storage2, '/bar/');
@@ -346,7 +346,7 @@ class UserMountCacheTest extends TestCase {
public function testGetMountsForFileIdRootId() {
$user1 = $this->userManager->get('u1');
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$mount1 = new MountPoint($storage1, '/foo/');
$this->cache->registerMounts($user1, [$mount1]);
@@ -368,7 +368,7 @@ class UserMountCacheTest extends TestCase {
$fileId = $this->createCacheEntry('/foo/bar', 2);
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$mount1 = new MountPoint($storage1, '/foo/');
$this->cache->registerMounts($user1, [$mount1]);
@@ -390,7 +390,7 @@ class UserMountCacheTest extends TestCase {
public function testGetMountsForFileIdSubFolderMount() {
$user1 = $this->userManager->get('u1');
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$folderId = $this->createCacheEntry('/foo', 2);
$fileId = $this->createCacheEntry('/foo/bar', 2);
@@ -424,7 +424,7 @@ class UserMountCacheTest extends TestCase {
public function testGetMountsForFileIdSubFolderMountOutside() {
$user1 = $this->userManager->get('u1');
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$folderId = $this->createCacheEntry('/foo', 2);
$fileId = $this->createCacheEntry('/bar/asd', 2);
@@ -452,7 +452,7 @@ class UserMountCacheTest extends TestCase {
public function testGetMountsForFileIdDeletedUser() {
$user1 = $this->userManager->get('u1');
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$rootId = $this->createCacheEntry('', 2);
$mount1 = new MountPoint($storage1, '/foo/');
$this->cache->registerMounts($user1, [$mount1]);
@@ -469,7 +469,7 @@ class UserMountCacheTest extends TestCase {
$user2 = $this->userManager->get('u2');
/** @var Storage $storage1 */
- list($storage1, $rootId) = $this->getStorage(2);
+ [$storage1, $rootId] = $this->getStorage(2);
$folderId = $this->createCacheEntry('files', 2, 100);
$fileId = $this->createCacheEntry('files/foo', 2, 7);
$storage1->getCache()->put($folderId, ['size' => 100]);
diff --git a/tests/lib/Files/FilesystemTest.php b/tests/lib/Files/FilesystemTest.php
index d9e6de898f0..5859bc2c772 100644
--- a/tests/lib/Files/FilesystemTest.php
+++ b/tests/lib/Files/FilesystemTest.php
@@ -99,9 +99,9 @@ class FilesystemTest extends \Test\TestCase {
\OC\Files\Filesystem::mount('\OC\Files\Storage\Local', self::getStorageData(), '/');
$this->assertEquals('/', \OC\Files\Filesystem::getMountPoint('/'));
$this->assertEquals('/', \OC\Files\Filesystem::getMountPoint('/some/folder'));
- list(, $internalPath) = \OC\Files\Filesystem::resolvePath('/');
+ [, $internalPath] = \OC\Files\Filesystem::resolvePath('/');
$this->assertEquals('', $internalPath);
- list(, $internalPath) = \OC\Files\Filesystem::resolvePath('/some/folder');
+ [, $internalPath] = \OC\Files\Filesystem::resolvePath('/some/folder');
$this->assertEquals('some/folder', $internalPath);
\OC\Files\Filesystem::mount('\OC\Files\Storage\Local', self::getStorageData(), '/some');
@@ -109,7 +109,7 @@ class FilesystemTest extends \Test\TestCase {
$this->assertEquals('/some/', \OC\Files\Filesystem::getMountPoint('/some/folder'));
$this->assertEquals('/some/', \OC\Files\Filesystem::getMountPoint('/some/'));
$this->assertEquals('/some/', \OC\Files\Filesystem::getMountPoint('/some'));
- list(, $internalPath) = \OC\Files\Filesystem::resolvePath('/some/folder');
+ [, $internalPath] = \OC\Files\Filesystem::resolvePath('/some/folder');
$this->assertEquals('folder', $internalPath);
}
@@ -421,7 +421,7 @@ class FilesystemTest extends \Test\TestCase {
'/' . $userId . '/',
\OC\Files\Filesystem::getMountPoint('/' . $userId . '/cache')
);
- list($storage, $internalPath) = \OC\Files\Filesystem::resolvePath('/' . $userId . '/cache');
+ [$storage, $internalPath] = \OC\Files\Filesystem::resolvePath('/' . $userId . '/cache');
$this->assertTrue($storage->instanceOfStorage('\OCP\Files\IHomeStorage'));
$this->assertEquals('cache', $internalPath);
$user = \OC::$server->getUserManager()->get($userId);
@@ -452,7 +452,7 @@ class FilesystemTest extends \Test\TestCase {
'/' . $userId . '/cache/',
\OC\Files\Filesystem::getMountPoint('/' . $userId . '/cache')
);
- list($storage, $internalPath) = \OC\Files\Filesystem::resolvePath('/' . $userId . '/cache');
+ [$storage, $internalPath] = \OC\Files\Filesystem::resolvePath('/' . $userId . '/cache');
$this->assertTrue($storage->instanceOfStorage('\OC\Files\Storage\Local'));
$this->assertEquals('', $internalPath);
$user = \OC::$server->getUserManager()->get($userId);
diff --git a/tests/lib/Files/Node/HookConnectorTest.php b/tests/lib/Files/Node/HookConnectorTest.php
index a5e73f18365..7990abad231 100644
--- a/tests/lib/Files/Node/HookConnectorTest.php
+++ b/tests/lib/Files/Node/HookConnectorTest.php
@@ -238,7 +238,7 @@ class HookConnectorTest extends TestCase {
$this->legacyDispatcher->addListener($expectedLegacyEvent, function ($event) use (&$dispatcherSourceNode, &$dispatcherTargetNode, &$dispatcherCalled) {
/** @var GenericEvent|APIGenericEvent $event */
$dispatcherCalled = true;
- list($dispatcherSourceNode, $dispatcherTargetNode) = $event->getSubject();
+ [$dispatcherSourceNode, $dispatcherTargetNode] = $event->getSubject();
});
$newDispatcherCalled = false;
diff --git a/tests/lib/Files/Storage/Wrapper/QuotaTest.php b/tests/lib/Files/Storage/Wrapper/QuotaTest.php
index 199ba5f2e26..82b3e3556cb 100644
--- a/tests/lib/Files/Storage/Wrapper/QuotaTest.php
+++ b/tests/lib/Files/Storage/Wrapper/QuotaTest.php
@@ -113,7 +113,7 @@ class QuotaTest extends \Test\Files\Storage\Storage {
$instance = $this->getLimitedStorage(16);
$inputStream = fopen('data://text/plain,foobarqwerty', 'r');
$outputStream = $instance->fopen('files/foo', 'w+');
- list($count, $result) = \OC_Helper::streamCopy($inputStream, $outputStream);
+ [$count, $result] = \OC_Helper::streamCopy($inputStream, $outputStream);
$this->assertEquals(12, $count);
$this->assertTrue($result);
fclose($inputStream);
@@ -124,7 +124,7 @@ class QuotaTest extends \Test\Files\Storage\Storage {
$instance = $this->getLimitedStorage(9);
$inputStream = fopen('data://text/plain,foobarqwerty', 'r');
$outputStream = $instance->fopen('files/foo', 'w+');
- list($count, $result) = \OC_Helper::streamCopy($inputStream, $outputStream);
+ [$count, $result] = \OC_Helper::streamCopy($inputStream, $outputStream);
$this->assertEquals(9, $count);
$this->assertFalse($result);
fclose($inputStream);
diff --git a/tests/lib/Group/MetaDataTest.php b/tests/lib/Group/MetaDataTest.php
index ebd9cd79f23..7ad6d631b4e 100644
--- a/tests/lib/Group/MetaDataTest.php
+++ b/tests/lib/Group/MetaDataTest.php
@@ -85,7 +85,7 @@ class MetaDataTest extends \Test\TestCase {
->with('')
->willReturn($groups);
- list($adminGroups, $ordinaryGroups) = $this->groupMetadata->get();
+ [$adminGroups, $ordinaryGroups] = $this->groupMetadata->get();
$this->assertSame(1, count($adminGroups));
$this->assertSame(2, count($ordinaryGroups));
@@ -105,7 +105,7 @@ class MetaDataTest extends \Test\TestCase {
->with('')
->willReturn($groups);
- list($adminGroups, $ordinaryGroups) = $this->groupMetadata->get();
+ [$adminGroups, $ordinaryGroups] = $this->groupMetadata->get();
$this->assertSame(1, count($adminGroups));
$this->assertSame(2, count($ordinaryGroups));
diff --git a/tests/lib/LegacyHelperTest.php b/tests/lib/LegacyHelperTest.php
index 5c640e91284..1dfc3dace6c 100644
--- a/tests/lib/LegacyHelperTest.php
+++ b/tests/lib/LegacyHelperTest.php
@@ -174,7 +174,7 @@ class LegacyHelperTest extends \Test\TestCase {
$target = fopen($target, 'w');
}
- list($count, $result) = \OC_Helper::streamCopy($source, $target);
+ [$count, $result] = \OC_Helper::streamCopy($source, $target);
if (is_resource($source)) {
fclose($source);
diff --git a/tests/lib/Security/IdentityProof/ManagerTest.php b/tests/lib/Security/IdentityProof/ManagerTest.php
index 760c4911873..16ceec248ae 100644
--- a/tests/lib/Security/IdentityProof/ManagerTest.php
+++ b/tests/lib/Security/IdentityProof/ManagerTest.php
@@ -193,7 +193,7 @@ class ManagerTest extends TestCase {
$manager = $this->getManager();
$data = 'MyTestData';
- list($resultPublicKey, $resultPrivateKey) = self::invokePrivate($manager, 'generateKeyPair');
+ [$resultPublicKey, $resultPrivateKey] = self::invokePrivate($manager, 'generateKeyPair');
openssl_sign($data, $signature, $resultPrivateKey);
$details = openssl_pkey_get_details(openssl_pkey_get_public($resultPublicKey));
diff --git a/tests/lib/TestCase.php b/tests/lib/TestCase.php
index 6cadf9693c3..08c59633e42 100644
--- a/tests/lib/TestCase.php
+++ b/tests/lib/TestCase.php
@@ -408,7 +408,7 @@ abstract class TestCase extends \PHPUnit\Framework\TestCase {
// get the user for which the fs is setup
$view = Filesystem::getView();
if ($view) {
- list(, $user) = explode('/', $view->getRoot());
+ [, $user] = explode('/', $view->getRoot());
} else {
$user = null;
}