nextcloud / server

apps/federatedfilesharing/tests/Settings/AdminTest.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 		];
 		$this->initialState->expects($this->exactly(10))
 			->method('provideInitialState')
-			->willReturnCallback(function () use (&$calls): void {
+			->willReturnCallback(function() use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertSame($expected, func_get_args());
 			});

apps/user_ldap/tests/Group_LDAPTest.php

Spacing +58 added lines, -58 removed lines

@@ -64,7 +64,7 @@ 
 			->willReturn($groupDN);
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn) use ($groupDN) {
+			->willReturnCallback(function($dn) use ($groupDN) {
 				if ($dn === $groupDN) {
 					return [
 						'uid=u11,ou=users,dc=foo,dc=bar',
@@ -112,7 +112,7 @@ 
 	private function enableGroups(): void {
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'ldapDynamicGroupMemberURL') {
 					return '';
 				} elseif ($name === 'ldapBaseGroups') {
@@ -133,7 +133,7 @@ 
 			->willReturn([]);
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				//the search operation will call readAttribute, thus we need
 				//to analyze the "dn". All other times we just need to return
 				//something that is neither null or false, but once an array
@@ -146,8 +146,8 @@ 
 			});
 		$this->access->expects($this->any())
 			->method('dn2username')
-			->willReturnCallback(function () {
-				return 'foobar' . Server::get(ISecureRandom::class)->generate(7);
+			->willReturnCallback(function() {
+				return 'foobar'.Server::get(ISecureRandom::class)->generate(7);
 			});
 		$this->access->expects($this->any())
 			->method('isDNPartOfBase')
@@ -287,7 +287,7 @@ 
 		$connection = $this->access->connection;
 		$connection->expects($this->once())
 			->method('getFromCache')
-			->with('primaryGroupIDtoName_' . $gid)
+			->with('primaryGroupIDtoName_'.$gid)
 			->willReturn('MyGroup');
 
 		$this->access->expects($this->never())
@@ -446,7 +446,7 @@ 
 
 		$uid = 'someUser';
 		$gid = 'someGroup';
-		$cacheKey = 'inGroup' . $uid . ':' . $gid;
+		$cacheKey = 'inGroup'.$uid.':'.$gid;
 
 		$this->access->connection->expects($this->once())
 			->method('getFromCache')
@@ -482,7 +482,7 @@ 
 
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				switch ($name) {
 					case 'ldapGroupMemberAssocAttr':
 						return 'member';
@@ -521,7 +521,7 @@ 
 
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				switch ($name) {
 					case 'ldapGroupMemberAssocAttr':
 						return 'member';
@@ -567,7 +567,7 @@ 
 
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				switch ($name) {
 					case 'ldapGroupMemberAssocAttr':
 						return 'memberUid';
@@ -636,7 +636,7 @@ 
 			->willReturn(null);
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				if ($attr === 'primaryGroupToken') {
 					return [1337];
 				} elseif ($attr === 'gidNumber') {
@@ -679,7 +679,7 @@ 
 			->willReturn(null);
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				if ($attr === 'primaryGroupToken') {
 					return [1337];
 				}
@@ -721,7 +721,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				if ($attr === 'primaryGroupToken') {
 					return [1337];
 				}
@@ -785,7 +785,7 @@ 
 	public function testGetUserGroupsMemberOfDisabled(): void {
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'useMemberOfToDetectMembership') {
 					return 0;
 				} elseif ($name === 'ldapDynamicGroupMemberURL') {
@@ -945,14 +945,14 @@ 
 		$groupFilter = '(&(objectclass=nextcloudGroup)(nextcloudEnabled=TRUE))';
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function (string $name) use ($nestedGroups, $groupFilter) {
+			->willReturnCallback(function(string $name) use ($nestedGroups, $groupFilter) {
 				switch ($name) {
 					case 'useMemberOfToDetectMembership':
 						return 0;
 					case 'ldapDynamicGroupMemberURL':
 						return '';
 					case 'ldapNestedGroups':
-						return (int)$nestedGroups;
+						return (int) $nestedGroups;
 					case 'ldapGroupMemberAssocAttr':
 						return 'member';
 					case 'ldapGroupFilter':
@@ -978,7 +978,7 @@ 
 			->willReturn([]);
 		$this->access->expects($this->any())
 			->method('combineFilterWithAnd')
-			->willReturnCallback(function (array $filterParts) {
+			->willReturnCallback(function(array $filterParts) {
 				// ⚠ returns a pseudo-filter only, not real LDAP Filter syntax
 				return implode('&', $filterParts);
 			});
@@ -1008,7 +1008,7 @@ 
 			->willReturn($expectedGroupsNames);
 		$this->access->expects($nestedGroups ? $this->atLeastOnce() : $this->once())
 			->method('fetchListOfGroups')
-			->willReturnCallback(function ($filter, $attr, $limit, $offset) use ($nestedGroups, $groupFilter, $group1, $group2, $group3, $dn) {
+			->willReturnCallback(function($filter, $attr, $limit, $offset) use ($nestedGroups, $groupFilter, $group1, $group2, $group3, $dn) {
 				static $firstRun = true;
 				if (!$nestedGroups) {
 					// When nested groups are enabled, groups cannot be filtered early as it would
@@ -1016,10 +1016,10 @@ 
 					$this->assertTrue(str_contains($filter, $groupFilter));
 				}
 				[$memberFilter] = explode('&', $filter);
-				if ($memberFilter === 'member=' . $dn) {
+				if ($memberFilter === 'member='.$dn) {
 					return [$group1, $group2];
 					return [];
-				} elseif ($memberFilter === 'member=' . $group2['dn'][0]) {
+				} elseif ($memberFilter === 'member='.$group2['dn'][0]) {
 					return [$group3];
 				} else {
 					return [];
@@ -1027,12 +1027,12 @@ 
 			});
 		$this->access->expects($this->any())
 			->method('dn2groupname')
-			->willReturnCallback(function (string $dn) {
+			->willReturnCallback(function(string $dn) {
 				return ldap_explode_dn($dn, 1)[0];
 			});
 		$this->access->expects($this->any())
 			->method('groupname2dn')
-			->willReturnCallback(function (string $gid) use ($group1, $group2, $group3) {
+			->willReturnCallback(function(string $gid) use ($group1, $group2, $group3) {
 				if ($gid === $group1['cn']) {
 					return $group1['dn'][0];
 				}
@@ -1248,11 +1248,11 @@ 
 		$base = 'dc=species,dc=earth';
 
 		$birdsDn = [
-			'uid=3723,' . $base,
-			'uid=8372,' . $base,
-			'uid=8427,' . $base,
-			'uid=2333,' . $base,
-			'uid=4754,' . $base,
+			'uid=3723,'.$base,
+			'uid=8372,'.$base,
+			'uid=8427,'.$base,
+			'uid=2333,'.$base,
+			'uid=4754,'.$base,
 		];
 		$birdsUid = [
 			'3723',
@@ -1262,51 +1262,51 @@ 
 			'4754',
 		];
 		$animalsDn = [
-			'uid=lion,' . $base,
-			'uid=tiger,' . $base,
+			'uid=lion,'.$base,
+			'uid=tiger,'.$base,
 		];
 		$plantsDn = [
-			'uid=flower,' . $base,
-			'uid=tree,' . $base,
+			'uid=flower,'.$base,
+			'uid=tree,'.$base,
 		];
 		$thingsDn = [
-			'uid=thing1,' . $base,
-			'uid=thing2,' . $base,
+			'uid=thing1,'.$base,
+			'uid=thing2,'.$base,
 		];
 
 		return [
-			[ #0 – test DNs
-				['cn=Birds,' . $base => $birdsDn],
-				['cn=Birds,' . $base => $birdsDn]
+			[#0 – test DNs
+				['cn=Birds,'.$base => $birdsDn],
+				['cn=Birds,'.$base => $birdsDn]
 			],
-			[ #1 – test uids
-				['cn=Birds,' . $base => $birdsUid],
-				['cn=Birds,' . $base => $birdsUid]
+			[#1 – test uids
+				['cn=Birds,'.$base => $birdsUid],
+				['cn=Birds,'.$base => $birdsUid]
 			],
-			[ #2 – test simple nested group
-				['cn=Animals,' . $base => array_merge($birdsDn, $animalsDn)],
+			[#2 – test simple nested group
+				['cn=Animals,'.$base => array_merge($birdsDn, $animalsDn)],
 				[
-					'cn=Animals,' . $base => array_merge(['cn=Birds,' . $base], $animalsDn),
-					'cn=Birds,' . $base => $birdsDn,
+					'cn=Animals,'.$base => array_merge(['cn=Birds,'.$base], $animalsDn),
+					'cn=Birds,'.$base => $birdsDn,
 				]
 			],
-			[ #3 – test recursive nested group
+			[#3 – test recursive nested group
 				[
-					'cn=Animals,' . $base => array_merge($birdsDn, $animalsDn),
-					'cn=Birds,' . $base => array_merge($birdsDn, $animalsDn),
+					'cn=Animals,'.$base => array_merge($birdsDn, $animalsDn),
+					'cn=Birds,'.$base => array_merge($birdsDn, $animalsDn),
 				],
 				[
-					'cn=Animals,' . $base => array_merge(['cn=Birds,' . $base,'cn=Birds,' . $base,'cn=Animals,' . $base], $animalsDn),
-					'cn=Birds,' . $base => array_merge(['cn=Animals,' . $base,'cn=Birds,' . $base], $birdsDn),
+					'cn=Animals,'.$base => array_merge(['cn=Birds,'.$base, 'cn=Birds,'.$base, 'cn=Animals,'.$base], $animalsDn),
+					'cn=Birds,'.$base => array_merge(['cn=Animals,'.$base, 'cn=Birds,'.$base], $birdsDn),
 				]
 			],
-			[ #4 – Complicated nested group
-				['cn=Things,' . $base => array_merge($birdsDn, $animalsDn, $thingsDn, $plantsDn)],
+			[#4 – Complicated nested group
+				['cn=Things,'.$base => array_merge($birdsDn, $animalsDn, $thingsDn, $plantsDn)],
 				[
-					'cn=Animals,' . $base => array_merge(['cn=Birds,' . $base], $animalsDn),
-					'cn=Birds,' . $base => $birdsDn,
-					'cn=Plants,' . $base => $plantsDn,
-					'cn=Things,' . $base => array_merge(['cn=Animals,' . $base,'cn=Plants,' . $base], $thingsDn),
+					'cn=Animals,'.$base => array_merge(['cn=Birds,'.$base], $animalsDn),
+					'cn=Birds,'.$base => $birdsDn,
+					'cn=Plants,'.$base => $plantsDn,
+					'cn=Things,'.$base => array_merge(['cn=Animals,'.$base, 'cn=Plants,'.$base], $thingsDn),
 				]
 			],
 		];
@@ -1316,7 +1316,7 @@ 
 	public function testGroupMembers(array $expectedResult, array $groupsInfo): void {
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($group) use ($groupsInfo) {
+			->willReturnCallback(function($group) use ($groupsInfo) {
 				if (isset($groupsInfo[$group])) {
 					return $groupsInfo[$group];
 				}
@@ -1325,7 +1325,7 @@ 
 
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function (string $name) {
+			->willReturnCallback(function(string $name) {
 				if ($name === 'ldapNestedGroups') {
 					return 1;
 				} elseif ($name === 'ldapGroupMemberAssocAttr') {
@@ -1352,7 +1352,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('displayNameProvider')]
-	public function testGetDisplayName(string $expected, bool|array $ldapResult): void {
+	public function testGetDisplayName(string $expected, bool | array $ldapResult): void {
 		$gid = 'graphic_novelists';
 
 		$this->access->expects($this->atLeastOnce())
@@ -1361,7 +1361,7 @@ 
 
 		$this->access->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'ldapGroupMemberAssocAttr') {
 					return 'member';
 				} elseif ($name === 'ldapGroupFilter') {

apps/user_ldap/tests/AccessTest.php

Spacing +20 added lines, -20 removed lines

@@ -220,7 +220,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dnInputDataProvider')]
-	public function testStringResemblesDN(string $input, array|bool $interResult, bool $expectedResult): void {
+	public function testStringResemblesDN(string $input, array | bool $interResult, bool $expectedResult): void {
 		[$lw, $con, $um, $helper] = $this->getConnectorAndLdapMock();
 		/** @var IConfig&MockObject $config */
 		$config = $this->createMock(IConfig::class);
@@ -228,7 +228,7 @@ 
 
 		$lw->expects($this->exactly(1))
 			->method('explodeDN')
-			->willReturnCallback(function ($dn) use ($input, $interResult) {
+			->willReturnCallback(function($dn) use ($input, $interResult) {
 				if ($dn === $input) {
 					return $interResult;
 				}
@@ -239,7 +239,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dnInputDataProvider')]
-	public function testStringResemblesDNLDAPmod(string $input, array|bool $interResult, bool $expectedResult): void {
+	public function testStringResemblesDNLDAPmod(string $input, array | bool $interResult, bool $expectedResult): void {
 		[, $con, $um, $helper] = $this->getConnectorAndLdapMock();
 		/** @var IConfig&MockObject $config */
 		$config = $this->createMock(IConfig::class);
@@ -517,7 +517,7 @@ 
 			->willReturn($fakeConnection);
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($key) use ($base) {
+			->willReturnCallback(function($key) use ($base) {
 				if (stripos($key, 'base') !== false) {
 					return [$base];
 				}
@@ -527,7 +527,7 @@ 
 		$this->ldap
 			->expects($this->any())
 			->method('isResource')
-			->willReturnCallback(function ($resource) {
+			->willReturnCallback(function($resource) {
 				return is_object($resource);
 			});
 		$this->ldap
@@ -558,10 +558,10 @@ 
 		$fakeLdapEntries = [
 			'count' => 2,
 			[
-				'dn' => 'uid=sgarth,' . $base,
+				'dn' => 'uid=sgarth,'.$base,
 			],
 			[
-				'dn' => 'uid=wwilson,' . $base,
+				'dn' => 'uid=wwilson,'.$base,
 			]
 		];
 
@@ -585,18 +585,18 @@ 
 		$fakeLdapEntries = [
 			'count' => 2,
 			[
-				'dn' => 'uid=sgarth,' . $base,
-				'uid' => [ 'sgarth' ],
+				'dn' => 'uid=sgarth,'.$base,
+				'uid' => ['sgarth'],
 			],
 			[
-				'dn' => 'uid=wwilson,' . $base,
-				'uid' => [ 'wwilson' ],
+				'dn' => 'uid=wwilson,'.$base,
+				'uid' => ['wwilson'],
 			]
 		];
 		$expected = $fakeLdapEntries;
 		unset($expected['count']);
-		array_walk($expected, function (&$v): void {
-			$v['dn'] = [$v['dn']];	// dn is translated into an array internally for consistency
+		array_walk($expected, function(&$v): void {
+			$v['dn'] = [$v['dn']]; // dn is translated into an array internally for consistency
 		});
 
 		$this->prepareMocksForSearchTests($base, $fakeConnection, $fakeSearchResultResource, $fakeLdapEntries);
@@ -608,7 +608,7 @@ 
 
 		$this->userMapper->expects($this->exactly($fakeLdapEntries['count']))
 			->method('getNameByDN')
-			->willReturnCallback(function ($fdn) {
+			->willReturnCallback(function($fdn) {
 				$parts = ldap_explode_dn($fdn, 0);
 				return $parts[0];
 			});
@@ -628,11 +628,11 @@ 
 		$fakeLdapEntries = [
 			'count' => 2,
 			[
-				'dn' => 'cn=Good Team,' . $base,
+				'dn' => 'cn=Good Team,'.$base,
 				'cn' => ['Good Team'],
 			],
 			[
-				'dn' => 'cn=Another Good Team,' . $base,
+				'dn' => 'cn=Another Good Team,'.$base,
 				'cn' => ['Another Good Team'],
 			]
 		];
@@ -642,8 +642,8 @@ 
 		$this->groupMapper->expects($this->any())
 			->method('getListOfIdsByDn')
 			->willReturn([
-				'cn=Good Team,' . $base => 'Good_Team',
-				'cn=Another Good Team,' . $base => 'Another_Good_Team',
+				'cn=Good Team,'.$base => 'Good_Team',
+				'cn=Another Good Team,'.$base => 'Another_Good_Team',
 			]);
 		$this->groupMapper->expects($this->never())
 			->method('getNameByDN');
@@ -711,8 +711,8 @@ 
 		$this->connection->expects($this->any())
 			->method('__get')
 			->willReturnMap([
-				[ 'ldapUserDisplayName', 'displayName' ],
-				[ 'ldapUserDisplayName2', null],
+				['ldapUserDisplayName', 'displayName'],
+				['ldapUserDisplayName2', null],
 			]);
 
 		$offlineUserMock = $this->createMock(OfflineUser::class);

apps/user_ldap/tests/LDAPTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('errorProvider')]
 	public function testSearchWithErrorHandler(string $errorMessage, bool $passThrough): void {
 		$wasErrorHandlerCalled = false;
-		$errorHandler = function ($number, $message, $file, $line) use (&$wasErrorHandlerCalled): void {
+		$errorHandler = function($number, $message, $file, $line) use (&$wasErrorHandlerCalled): void {
 			$wasErrorHandlerCalled = true;
 		};
 
@@ -46,7 +46,7 @@ 
 			->expects($this->once())
 			->method('invokeLDAPMethod')
 			->with('search', $this->anything(), $this->anything(), $this->anything(), $this->anything(), $this->anything())
-			->willReturnCallback(function () use ($errorMessage): void {
+			->willReturnCallback(function() use ($errorMessage): void {
 				trigger_error($errorMessage);
 			});
 

apps/user_ldap/tests/ConfigurationTest.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('configurationDataProvider')]
-	public function testSetValue(string $key, string|array $input, string|array $expected): void {
+	public function testSetValue(string $key, string | array $input, string | array $expected): void {
 		$this->configuration->setConfiguration([$key => $input]);
 		$this->assertSame($this->configuration->$key, $expected);
 	}

apps/user_ldap/tests/User_LDAPTest.php

Spacing +62 added lines, -62 removed lines

@@ -84,7 +84,7 @@ 
 	private function prepareMockForUserExists(): void {
 		$this->access->expects($this->any())
 			->method('username2dn')
-			->willReturnCallback(function ($uid) {
+			->willReturnCallback(function($uid) {
 				switch ($uid) {
 					case 'gunslinger':
 						return 'dnOfRoland,dc=test';
@@ -113,7 +113,7 @@ 
 	private function prepareAccessForCheckPassword(bool $noDisplayName = false): void {
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'ldapLoginFilter') {
 					return '%uid';
 				}
@@ -122,7 +122,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('fetchListOfUsers')
-			->willReturnCallback(function ($filter) {
+			->willReturnCallback(function($filter) {
 				if ($filter === 'roland') {
 					return [['dn' => ['dnOfRoland,dc=test']]];
 				}
@@ -130,7 +130,7 @@ 
 			});
 		$this->access->expects($this->any())
 			->method('fetchUsersByLoginName')
-			->willReturnCallback(function ($uid) {
+			->willReturnCallback(function($uid) {
 				if ($uid === 'roland') {
 					return [['dn' => ['dnOfRoland,dc=test']]];
 				}
@@ -155,7 +155,7 @@ 
 			->willReturn(true);
 		$this->access->expects($this->any())
 			->method('areCredentialsValid')
-			->willReturnCallback(function ($dn, $pwd) {
+			->willReturnCallback(function($dn, $pwd) {
 				if ($pwd === 'dt19') {
 					return true;
 				}
@@ -349,22 +349,22 @@ 
 	private function prepareAccessForGetUsers() {
 		$this->access->expects($this->once())
 			->method('escapeFilterPart')
-			->willReturnCallback(function ($search) {
+			->willReturnCallback(function($search) {
 				return $search;
 			});
 		$this->access->expects($this->any())
 			->method('getFilterPartForUserSearch')
-			->willReturnCallback(function ($search) {
+			->willReturnCallback(function($search) {
 				return $search;
 			});
 		$this->access->expects($this->any())
 			->method('combineFilterWithAnd')
-			->willReturnCallback(function ($param) {
+			->willReturnCallback(function($param) {
 				return $param[2];
 			});
 		$this->access->expects($this->any())
 			->method('fetchListOfUsers')
-			->willReturnCallback(function ($search, $a, $l, $o) {
+			->willReturnCallback(function($search, $a, $l, $o) {
 				$users = ['gunslinger', 'newyorker', 'ladyofshadows'];
 				if (empty($search)) {
 					$result = $users;
@@ -434,7 +434,7 @@ 
 
 	private function getUsers($search = '', $limit = null, $offset = null) {
 		$users = Server::get(IUserManager::class)->search($search, $limit, $offset);
-		$uids = array_map(function (IUser $user) {
+		$uids = array_map(function(IUser $user) {
 			return $user->getUID();
 		}, $users);
 		return $uids;
@@ -536,7 +536,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn) {
+			->willReturnCallback(function($dn) {
 				if ($dn === 'dnOfRoland,dc=test') {
 					return [];
 				}
@@ -561,7 +561,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn) {
+			->willReturnCallback(function($dn) {
 				if ($dn === 'dnOfRoland,dc=test') {
 					return [];
 				}
@@ -596,7 +596,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'homeFolderNamingRule') {
 					return 'attr:testAttribute';
 				}
@@ -605,7 +605,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				switch ($dn) {
 					case 'dnOfRoland,dc=test':
 						if ($attr === 'testAttribute') {
@@ -649,11 +649,11 @@ 
 		$this->prepareMockForUserExists();
 
 		$dataDir = Server::get(IConfig::class)->getSystemValue(
-			'datadirectory', \OC::$SERVERROOT . '/data');
+			'datadirectory', \OC::$SERVERROOT.'/data');
 
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'homeFolderNamingRule') {
 					return 'attr:testAttribute';
 				}
@@ -662,7 +662,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				switch ($dn) {
 					case 'dnOfLadyOfShadows,dc=test':
 						if ($attr === 'testAttribute') {
@@ -684,7 +684,7 @@ 
 			->willReturn('dnOfLadyOfShadows,dc=test');
 		$user->expects($this->any())
 			->method('getHomePath')
-			->willReturn($dataDir . '/susannah/');
+			->willReturn($dataDir.'/susannah/');
 
 		$this->userManager->expects($this->atLeastOnce())
 			->method('get')
@@ -696,7 +696,7 @@ 
 
 		/** @noinspection PhpUnhandledExceptionInspection */
 		$result = $backend->getHome('ladyofshadows');
-		$this->assertEquals($dataDir . '/susannah/', $result);
+		$this->assertEquals($dataDir.'/susannah/', $result);
 	}
 
 
@@ -708,7 +708,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'homeFolderNamingRule') {
 					return 'attr:testAttribute';
 				}
@@ -716,7 +716,7 @@ 
 			});
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				switch ($dn) {
 					default:
 						return false;
@@ -753,7 +753,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'homeFolderNamingRule') {
 					return 'attr:testAttribute';
 				}
@@ -800,7 +800,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('getFromCache')
-			->willReturnCallback(function ($uid) {
+			->willReturnCallback(function($uid) {
 				return true;
 			});
 
@@ -811,7 +811,7 @@ 
 	private function prepareAccessForGetDisplayName() {
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) {
+			->willReturnCallback(function($name) {
 				if ($name === 'ldapUserDisplayName') {
 					return 'displayname';
 				} elseif ($name === 'ldapUserDisplayName2') {
@@ -822,7 +822,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('readAttribute')
-			->willReturnCallback(function ($dn, $attr) {
+			->willReturnCallback(function($dn, $attr) {
 				switch ($dn) {
 					case 'dnOfRoland,dc=test':
 						if ($attr === 'displayname') {
@@ -846,7 +846,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('getConnectionResource')
-			->willReturnCallback(function () {
+			->willReturnCallback(function() {
 				return true;
 			});
 
@@ -868,13 +868,13 @@ 
 		$mapper = $this->createMock(UserMapping::class);
 		$mapper->expects($this->any())
 			->method('getUUIDByDN')
-			->willReturnCallback(function ($dn) {
+			->willReturnCallback(function($dn) {
 				return $dn;
 			});
 
 		$this->userManager->expects($this->any())
 			->method('get')
-			->willReturnCallback(function ($uid) use ($user1, $user2) {
+			->willReturnCallback(function($uid) use ($user1, $user2) {
 				if ($uid === 'gunslinger') {
 					return $user1;
 				} elseif ($uid === 'newyorker') {
@@ -884,7 +884,7 @@ 
 			});
 		$this->userManager->expects($this->any())
 			->method('exists')
-			->willReturnCallback(function ($uid) use ($user1, $user2) {
+			->willReturnCallback(function($uid) use ($user1, $user2) {
 				if ($uid === 'gunslinger') {
 					return true;
 				} elseif ($uid === 'newyorker') {
@@ -897,8 +897,8 @@ 
 			->willReturn($mapper);
 		$this->access->expects($this->any())
 			->method('getUserDnByUuid')
-			->willReturnCallback(function ($uuid) {
-				return $uuid . '1';
+			->willReturnCallback(function($uuid) {
+				return $uuid.'1';
 			});
 
 		//with displayName
@@ -913,7 +913,7 @@ 
 	public function testGetDisplayNamePublicAPI(): void {
 		$this->access->expects($this->any())
 			->method('username2dn')
-			->willReturnCallback(function ($uid) {
+			->willReturnCallback(function($uid) {
 				switch ($uid) {
 					case 'gunslinger':
 						return 'dnOfRoland,dc=test';
@@ -937,7 +937,7 @@ 
 
 		$this->connection->expects($this->any())
 			->method('getConnectionResource')
-			->willReturnCallback(function () {
+			->willReturnCallback(function() {
 				return true;
 			});
 
@@ -961,13 +961,13 @@ 
 		$mapper = $this->createMock(UserMapping::class);
 		$mapper->expects($this->any())
 			->method('getUUIDByDN')
-			->willReturnCallback(function ($dn) {
+			->willReturnCallback(function($dn) {
 				return $dn;
 			});
 
 		$this->userManager->expects($this->any())
 			->method('get')
-			->willReturnCallback(function ($uid) use ($user1, $user2) {
+			->willReturnCallback(function($uid) use ($user1, $user2) {
 				if ($uid === 'gunslinger') {
 					return $user1;
 				} elseif ($uid === 'newyorker') {
@@ -977,7 +977,7 @@ 
 			});
 		$this->userManager->expects($this->any())
 			->method('exists')
-			->willReturnCallback(function ($uid) use ($user1, $user2) {
+			->willReturnCallback(function($uid) use ($user1, $user2) {
 				if ($uid === 'gunslinger') {
 					return true;
 				} elseif ($uid === 'newyorker') {
@@ -990,8 +990,8 @@ 
 			->willReturn($mapper);
 		$this->access->expects($this->any())
 			->method('getUserDnByUuid')
-			->willReturnCallback(function ($uuid) {
-				return $uuid . '1';
+			->willReturnCallback(function($uuid) {
+				return $uuid.'1';
 			});
 
 		//with displayName
@@ -1073,11 +1073,11 @@ 
 
 		$this->connection->expects($this->exactly(2))
 			->method('getFromCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName))
+			->with($this->equalTo('loginName2UserName-'.$loginName))
 			->willReturnOnConsecutiveCalls(null, $username);
 		$this->connection->expects($this->once())
 			->method('writeToCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName), $this->equalTo($username));
+			->with($this->equalTo('loginName2UserName-'.$loginName), $this->equalTo($username));
 
 		$backend = new UserLDAP($this->access, $this->notificationManager, $this->pluginManager, $this->logger, $this->deletedUsersIndex);
 		$user = $this->createMock(User::class);
@@ -1116,11 +1116,11 @@ 
 
 		$this->connection->expects($this->exactly(2))
 			->method('getFromCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName))
+			->with($this->equalTo('loginName2UserName-'.$loginName))
 			->willReturnOnConsecutiveCalls(null, false);
 		$this->connection->expects($this->once())
 			->method('writeToCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName), false);
+			->with($this->equalTo('loginName2UserName-'.$loginName), false);
 
 		$this->userManager->expects($this->any())
 			->method('getAttributes')
@@ -1149,11 +1149,11 @@ 
 
 		$this->connection->expects($this->exactly(2))
 			->method('getFromCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName))
+			->with($this->equalTo('loginName2UserName-'.$loginName))
 			->willReturnOnConsecutiveCalls(null, false);
 		$this->connection->expects($this->once())
 			->method('writeToCache')
-			->with($this->equalTo('loginName2UserName-' . $loginName), $this->equalTo(false));
+			->with($this->equalTo('loginName2UserName-'.$loginName), $this->equalTo(false));
 
 		$this->userManager->expects($this->any())
 			->method('get')
@@ -1177,19 +1177,19 @@ 
 	private function prepareAccessForSetPassword($enablePasswordChange = true) {
 		$this->connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($name) use (&$enablePasswordChange) {
+			->willReturnCallback(function($name) use (&$enablePasswordChange) {
 				if ($name === 'ldapLoginFilter') {
 					return '%uid';
 				}
 				if ($name === 'turnOnPasswordChange') {
-					return $enablePasswordChange?1:0;
+					return $enablePasswordChange ? 1 : 0;
 				}
 				return null;
 			});
 		$this->connection->expects($this->any())
 			->method('getFromCache')
-			->willReturnCallback(function ($uid) {
-				if ($uid === 'userExists' . 'roland') {
+			->willReturnCallback(function($uid) {
+				if ($uid === 'userExists'.'roland') {
 					return true;
 				}
 				return null;
@@ -1197,7 +1197,7 @@ 
 
 		$this->access->expects($this->any())
 			->method('fetchListOfUsers')
-			->willReturnCallback(function ($filter) {
+			->willReturnCallback(function($filter) {
 				if ($filter === 'roland') {
 					return [['dn' => ['dnOfRoland,dc=test']]];
 				}
@@ -1205,7 +1205,7 @@ 
 			});
 		$this->access->expects($this->any())
 			->method('fetchUsersByLoginName')
-			->willReturnCallback(function ($uid) {
+			->willReturnCallback(function($uid) {
 				if ($uid === 'roland') {
 					return [['dn' => ['dnOfRoland,dc=test']]];
 				}
@@ -1221,7 +1221,7 @@ 
 			->willReturn(true);
 		$this->access->expects($this->any())
 			->method('setPassword')
-			->willReturnCallback(function ($uid, $password) {
+			->willReturnCallback(function($uid, $password) {
 				if (strlen($password) <= 5) {
 					throw new HintException('Password fails quality checking policy', '', 19);
 				}
@@ -1319,15 +1319,15 @@ 
 
 	public static function avatarDataProvider(): array {
 		return [
-			[ 'validImageData', false ],
-			[ 'corruptImageData', true ],
-			[ false, true]
+			['validImageData', false],
+			['corruptImageData', true],
+			[false, true]
 		];
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('avatarDataProvider')]
-	public function testCanChangeAvatar(string|bool $imageData, bool $expected): void {
-		$isValidImage = str_starts_with((string)$imageData, 'valid');
+	public function testCanChangeAvatar(string | bool $imageData, bool $expected): void {
+		$isValidImage = str_starts_with((string) $imageData, 'valid');
 
 		$user = $this->createMock(User::class);
 		$user->expects($this->once())
@@ -1442,16 +1442,16 @@ 
 
 	public static function actionProvider(): array {
 		return [
-			[ 'ldapUserAvatarRule', 'default', Backend::PROVIDE_AVATAR, true],
-			[ 'ldapUserAvatarRule', 'data:selfiePhoto', Backend::PROVIDE_AVATAR, true],
-			[ 'ldapUserAvatarRule', 'none', Backend::PROVIDE_AVATAR, false],
-			[ 'turnOnPasswordChange', 0, Backend::SET_PASSWORD, false],
-			[ 'turnOnPasswordChange', 1, Backend::SET_PASSWORD, true],
+			['ldapUserAvatarRule', 'default', Backend::PROVIDE_AVATAR, true],
+			['ldapUserAvatarRule', 'data:selfiePhoto', Backend::PROVIDE_AVATAR, true],
+			['ldapUserAvatarRule', 'none', Backend::PROVIDE_AVATAR, false],
+			['turnOnPasswordChange', 0, Backend::SET_PASSWORD, false],
+			['turnOnPasswordChange', 1, Backend::SET_PASSWORD, true],
 		];
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('actionProvider')]
-	public function testImplementsAction(string $configurable, string|int $value, int $actionCode, bool $expected): void {
+	public function testImplementsAction(string $configurable, string | int $value, int $actionCode, bool $expected): void {
 		$this->pluginManager->expects($this->once())
 			->method('getImplementedActions')
 			->willReturn(0);

apps/user_ldap/tests/Jobs/SyncTest.php

Spacing +21 added lines, -21 removed lines

@@ -88,7 +88,7 @@ 
 				0, 1000, 750
 			],
 			[
-				22,	0, 50
+				22, 0, 50
 			],
 			[
 				500, 500, 500
@@ -107,7 +107,7 @@ 
 		$this->config->expects($this->once())
 			->method('setAppValue')
 			->with('user_ldap', 'background_sync_interval', $this->anything())
-			->willReturnCallback(function ($a, $k, $interval) {
+			->willReturnCallback(function($a, $k, $interval) {
 				$this->assertTrue($interval >= SYNC::MIN_INTERVAL);
 				$this->assertTrue($interval <= SYNC::MAX_INTERVAL);
 				return true;
@@ -135,11 +135,11 @@ 
 
 	public static function moreResultsProvider(): array {
 		return [
-			[ 3, 3, true ],
-			[ 3, 5, true ],
-			[ 3, 2, false],
-			[ 0, 4, false],
-			[ null, 4, false]
+			[3, 3, true],
+			[3, 5, true],
+			[3, 2, false],
+			[0, 4, false],
+			[null, 4, false]
 		];
 	}
 
@@ -155,7 +155,7 @@ 
 			->willReturn($connection);
 		$connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($key) use ($pagingSize) {
+			->willReturnCallback(function($key) use ($pagingSize) {
 				if ($key === 'ldapPagingSize') {
 					return $pagingSize;
 				}
@@ -191,12 +191,12 @@ 
 		$lastCycle = ['prefix' => 's01', 'offset' => 1000];
 		$lastCycle2 = ['prefix' => '', 'offset' => 1000];
 		return [
-			[ null, ['s01'], ['prefix' => 's01', 'offset' => 0] ],
-			[ null, [''], ['prefix' => '', 'offset' => 0] ],
-			[ $lastCycle, ['s01', 's02'], ['prefix' => 's02', 'offset' => 0] ],
-			[ $lastCycle, [''], ['prefix' => '', 'offset' => 0] ],
-			[ $lastCycle2, ['', 's01'], ['prefix' => 's01', 'offset' => 0] ],
-			[ $lastCycle, [], null ],
+			[null, ['s01'], ['prefix' => 's01', 'offset' => 0]],
+			[null, [''], ['prefix' => '', 'offset' => 0]],
+			[$lastCycle, ['s01', 's02'], ['prefix' => 's02', 'offset' => 0]],
+			[$lastCycle, [''], ['prefix' => '', 'offset' => 0]],
+			[$lastCycle2, ['', 's01'], ['prefix' => 's01', 'offset' => 0]],
+			[$lastCycle, [], null],
 		];
 	}
 
@@ -214,7 +214,7 @@ 
 			];
 			$this->config->expects($this->exactly(2))
 				->method('setAppValue')
-				->willReturnCallback(function () use (&$calls): void {
+				->willReturnCallback(function() use (&$calls): void {
 					$expected = array_shift($calls);
 					$this->assertEquals($expected, func_get_args());
 				});
@@ -282,7 +282,7 @@ 
 	public function testRun(array $runData): void {
 		$this->config->expects($this->any())
 			->method('getAppValue')
-			->willReturnCallback(function ($app, $key, $default) use ($runData) {
+			->willReturnCallback(function($app, $key, $default) use ($runData) {
 				if ($app === 'core' && $key === 'backgroundjobs_mode') {
 					return 'cron';
 				}
@@ -295,11 +295,11 @@ 
 						return $runData['scheduledCycle']['offset'];
 					}
 					// for qualifiesToRun()
-					if ($key === $runData['scheduledCycle']['prefix'] . '_lastChange') {
+					if ($key === $runData['scheduledCycle']['prefix'].'_lastChange') {
 						return time() - 60 * 40;
 					}
 					// for getMinPagingSize
-					if ($key === $runData['scheduledCycle']['prefix'] . 'ldap_paging_size') {
+					if ($key === $runData['scheduledCycle']['prefix'].'ldap_paging_size') {
 						return $runData['pagingSize'];
 					}
 				}
@@ -314,14 +314,14 @@ 
 		];
 		$this->config->expects($this->exactly(3))
 			->method('setAppValue')
-			->willReturnCallback(function () use (&$calls): void {
+			->willReturnCallback(function() use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 			});
 		$this->config->expects($this->any())
 			->method('getAppKeys')
 			->with('user_ldap')
-			->willReturn([$runData['scheduledCycle']['prefix'] . 'ldap_paging_size']);
+			->willReturn([$runData['scheduledCycle']['prefix'].'ldap_paging_size']);
 
 		$this->helper->expects($this->any())
 			->method('getServerConfigurationPrefixes')
@@ -338,7 +338,7 @@ 
 			->willReturn($connection);
 		$connection->expects($this->any())
 			->method('__get')
-			->willReturnCallback(function ($key) use ($runData) {
+			->willReturnCallback(function($key) use ($runData) {
 				if ($key === 'ldapPagingSize') {
 					return $runData['pagingSize'];
 				}

apps/files_sharing/tests/MountProviderTest.php

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
 		$shareAttributes = $this->createMock(IShareAttributes::class);
 		$shareAttributes->method('toArray')->willReturn($attrs);
 		$shareAttributes->method('getAttribute')->willReturnCallback(
-			function ($scope, $key) use ($attrs) {
+			function($scope, $key) use ($attrs) {
 				$result = null;
 				foreach ($attrs as $attr) {
 					if ($attr['key'] === $key && $attr['scope'] === $scope) {
@@ -100,7 +100,7 @@ 
 			->method('getShareTime')
 			->willReturn(
 				// compute share time based on id, simulating share order
-				new \DateTime('@' . (1469193980 + 1000 * $id))
+				new \DateTime('@'.(1469193980 + 1000 * $id))
 			);
 		return $share;
 	}
@@ -155,7 +155,7 @@ 
 
 		$this->shareManager->expects($this->any())
 			->method('newShare')
-			->willReturnCallback(function () use ($rootFolder, $userManager) {
+			->willReturnCallback(function() use ($rootFolder, $userManager) {
 				return new Share($rootFolder, $userManager);
 			});
 
@@ -343,10 +343,10 @@ 
 		$rootFolder = $this->createMock(IRootFolder::class);
 		$userManager = $this->createMock(IUserManager::class);
 
-		$userShares = array_map(function ($shareSpec) {
+		$userShares = array_map(function($shareSpec) {
 			return $this->makeMockShare($shareSpec[0], $shareSpec[1], $shareSpec[2], $shareSpec[3], $shareSpec[4], $shareSpec[5]);
 		}, $userShares);
-		$groupShares = array_map(function ($shareSpec) {
+		$groupShares = array_map(function($shareSpec) {
 			return $this->makeMockShare($shareSpec[0], $shareSpec[1], $shareSpec[2], $shareSpec[3], $shareSpec[4], $shareSpec[5]);
 		}, $groupShares);
 
@@ -372,7 +372,7 @@ 
 
 		$this->shareManager->expects($this->any())
 			->method('newShare')
-			->willReturnCallback(function () use ($rootFolder, $userManager) {
+			->willReturnCallback(function() use ($rootFolder, $userManager) {
 				return new Share($rootFolder, $userManager);
 			});
 

apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php

Spacing +4 added lines, -4 removed lines

@@ -87,8 +87,8 @@ 
 			if (isset($storage['files_count'])) {
 				for ($i = 0; $i < $storage['files_count']; $i++) {
 					$filesQuery->setParameter('storage', $storage['numeric_id']);
-					$filesQuery->setParameter('path', 'file' . $i);
-					$filesQuery->setParameter('path_hash', md5('file' . $i));
+					$filesQuery->setParameter('path', 'file'.$i);
+					$filesQuery->setParameter('path_hash', md5('file'.$i));
 					$filesQuery->executeStatement();
 				}
 			}
@@ -169,14 +169,14 @@ 
 		$output
 			->expects($this->any())
 			->method('writeln')
-			->willReturnCallback(function (string $text) use (&$outputCalls): void {
+			->willReturnCallback(function(string $text) use (&$outputCalls): void {
 				$outputCalls[] = $text;
 			});
 
 		$this->cloudIdManager
 			->expects($this->any())
 			->method('getCloudId')
-			->willReturnCallback(function (string $user, string $remote) {
+			->willReturnCallback(function(string $user, string $remote) {
 				$cloudIdMock = $this->createMock(ICloudId::class);
 
 				// The remotes are already sanitized in the original data, so