nextcloud / server

tests/lib/RichObjectStrings/ValidatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 		}
 
 		$v = new Validator(new Definitions());
-		$v->validate('{' . $key . '}', [
+		$v->validate('{'.$key.'}', [
 			$key => [
 				'type' => 'highlight',
 				'id' => 'identifier',

tests/lib/RichObjectStrings/DefinitionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -58,18 +58,18 @@
 	}
 
 	public function validateParameter($parameter, $data) {
-		$this->assertTrue(is_array($data), 'Parameter ' . $parameter . ' is invalid');
+		$this->assertTrue(is_array($data), 'Parameter '.$parameter.' is invalid');
 		$this->assertArrayHasKey('since', $data);
-		$this->assertNotEmpty($data['since'], 'Since of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEmpty($data['since'], 'Since of parameter '.$parameter.' must not be empty');
 		$this->assertArrayHasKey('required', $data);
-		$this->assertTrue(is_bool($data['required']), 'Required of parameter ' . $parameter . ' must be a boolean');
+		$this->assertTrue(is_bool($data['required']), 'Required of parameter '.$parameter.' must be a boolean');
 		if ($parameter === 'id' || $parameter === 'name') {
-			$this->assertTrue($data['required'], 'Parameter ' . $parameter . ' must be required');
+			$this->assertTrue($data['required'], 'Parameter '.$parameter.' must be required');
 		}
 
 		$this->assertArrayHasKey('description', $data);
-		$this->assertNotEquals('', $data['description'], 'Description of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEquals('', $data['description'], 'Description of parameter '.$parameter.' must not be empty');
 		$this->assertArrayHasKey('example', $data);
-		$this->assertNotEquals('', $data['example'], 'Example of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEquals('', $data['example'], 'Example of parameter '.$parameter.' must not be empty');
 	}
 }

tests/lib/Collaboration/Collaborators/GroupPluginTest.php

Spacing +1 added lines, -1 removed lines

@@ -445,7 +445,7 @@
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->willReturnCallback(
-				function ($appName, $key, $default) use ($shareWithGroupOnly, $shareeEnumeration, $groupSharingDisabled) {
+				function($appName, $key, $default) use ($shareWithGroupOnly, $shareeEnumeration, $groupSharingDisabled) {
 					if ($appName !== 'core') {
 						return $default;
 					}

tests/lib/Collaboration/Collaborators/LookupPluginTest.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
 		$this->userSession->expects($this->any())->method('getUser')
 			->willReturn($user);
 		$this->cloudIdManager->expects($this->any())->method('resolveCloudId')
-			->willReturnCallback(function ($cloudId) {
+			->willReturnCallback(function($cloudId) {
 				if ($cloudId === 'user@myNextcloud.net') {
 					return new CloudId('user@myNextcloud.net', 'user', 'myNextcloud.net');
 				}
@@ -162,8 +162,8 @@ 
 		$client = $this->createMock(IClient::class);
 		$client->expects($this->once())
 			->method('get')
-			->willReturnCallback(function ($url) use ($searchParams, $response) {
-				$this->assertSame(strpos($url, $searchParams['server'] . '/users?search='), 0);
+			->willReturnCallback(function($url) use ($searchParams, $response) {
+				$this->assertSame(strpos($url, $searchParams['server'].'/users?search='), 0);
 				$this->assertNotFalse(strpos($url, urlencode($searchParams['search'])));
 				return $response;
 			});
@@ -226,8 +226,8 @@ 
 			$client = $this->createMock(IClient::class);
 			$client->expects($this->once())
 				->method('get')
-				->willReturnCallback(function ($url) use ($searchParams, $response) {
-					$this->assertSame(strpos($url, $searchParams['server'] . '/users?search='), 0);
+				->willReturnCallback(function($url) use ($searchParams, $response) {
+					$this->assertSame(strpos($url, $searchParams['server'].'/users?search='), 0);
 					$this->assertNotFalse(strpos($url, urlencode($searchParams['search'])));
 					return $response;
 				});
@@ -323,7 +323,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				true, true
 			],
 			[[
@@ -365,7 +365,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				true, false
 			],
 			[[
@@ -404,7 +404,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				false, true
 			],
 			[[
@@ -443,7 +443,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				false, false
 			],
 		];

tests/lib/Collaboration/Collaborators/UserPluginTest.php

Spacing +9 added lines, -9 removed lines

@@ -90,7 +90,7 @@ 
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->willReturnCallback(
-				function ($appName, $key, $default) use ($mockedSettings) {
+				function($appName, $key, $default) use ($mockedSettings) {
 					return $mockedSettings[$appName][$key] ?? $default;
 				}
 			);
@@ -445,7 +445,7 @@ 
 	): void {
 		$this->mockConfig(['core' => [
 			'shareapi_only_share_with_group_members' => $shareWithGroupOnly ? 'yes' : 'no',
-			'shareapi_allow_share_dialog_user_enumeration' => $shareeEnumeration? 'yes' : 'no',
+			'shareapi_allow_share_dialog_user_enumeration' => $shareeEnumeration ? 'yes' : 'no',
 			'shareapi_restrict_user_enumeration_to_group' => false ? 'yes' : 'no',
 			'shareapi_restrict_user_enumeration_to_phone' => $shareeEnumerationPhone ? 'yes' : 'no',
 		]]);
@@ -544,7 +544,7 @@ 
 
 		$this->session->expects($this->once())
 			->method('getUser')
-			->willReturnCallback(function () use ($currentUserId) {
+			->willReturnCallback(function() use ($currentUserId) {
 				if ($currentUserId !== null) {
 					return $this->getUserMock($currentUserId, $currentUserId);
 				}
@@ -732,7 +732,7 @@ 
 			$usersById[$user['uid']] = $user;
 		}
 
-		$mappedResultExact = array_map(function ($user) use ($usersById, $search) {
+		$mappedResultExact = array_map(function($user) use ($usersById, $search) {
 			return [
 				'label' => $search === $user ? $user : $usersById[$user]['displayName'],
 				'value' => ['shareType' => 0, 'shareWith' => $user],
@@ -742,7 +742,7 @@ 
 				'shareWithDisplayNameUnique' => $user,
 			];
 		}, $result['exact']);
-		$mappedResultWide = array_map(function ($user) {
+		$mappedResultWide = array_map(function($user) {
 			return [
 				'label' => $user,
 				'value' => ['shareType' => 0, 'shareWith' => $user],
@@ -755,7 +755,7 @@ 
 
 		$this->userManager
 			->method('get')
-			->willReturnCallback(function ($userId) use ($userResults) {
+			->willReturnCallback(function($userId) use ($userResults) {
 				if (isset($userResults[$userId])) {
 					return $this->getUserMock($userId, $userId);
 				}
@@ -763,7 +763,7 @@ 
 			});
 		$this->userManager
 			->method('searchDisplayName')
-			->willReturnCallback(function ($search) use ($matchingUsers) {
+			->willReturnCallback(function($search) use ($matchingUsers) {
 				$users = array_filter(
 					$matchingUsers,
 					fn ($user) => str_contains(strtolower($user['displayName']), strtolower($search))
@@ -782,7 +782,7 @@ 
 			->willReturn($this->getUserMock('test', 'foo'));
 		$this->groupManager->expects($this->any())
 			->method('getUserGroupIds')
-			->willReturnCallback(function ($user) use ($matchingUsers, $userGroups) {
+			->willReturnCallback(function($user) use ($matchingUsers, $userGroups) {
 				static $firstCall = true;
 				if ($firstCall) {
 					$firstCall = false;
@@ -791,7 +791,7 @@ 
 				}
 				$neededObject = array_filter(
 					$matchingUsers,
-					function ($e) use ($user) {
+					function($e) use ($user) {
 						return $user->getUID() === $e['uid'];
 					}
 				);

tests/lib/Collaboration/Collaborators/RemotePluginTest.php

Spacing +7 added lines, -7 removed lines

@@ -83,7 +83,7 @@ 
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->willReturnCallback(
-				function ($appName, $key, $default) use ($shareeEnumeration) {
+				function($appName, $key, $default) use ($shareeEnumeration) {
 					if ($appName === 'core' && $key === 'shareapi_allow_share_dialog_user_enumeration') {
 						return $shareeEnumeration ? 'yes' : 'no';
 					}
@@ -95,7 +95,7 @@ 
 
 		$this->contactsManager->expects($this->any())
 			->method('search')
-			->willReturnCallback(function ($search, $searchAttributes) use ($searchTerm, $contacts) {
+			->willReturnCallback(function($search, $searchAttributes) use ($searchTerm, $contacts) {
 				if ($search === $searchTerm) {
 					return $contacts;
 				}
@@ -393,17 +393,17 @@ 
 		foreach ($userPrefix as $user) {
 			foreach ($remotes as $remote) {
 				foreach ($protocols as $protocol) {
-					$baseUrl = $user . '@' . $protocol . $remote;
+					$baseUrl = $user.'@'.$protocol.$remote;
 
 					if ($protocol === 'https://') {
 						// https:// protocol is not expected in the final result
 						$protocol = '';
 					}
 
-					$testCases[] = [$baseUrl, $user, $protocol . $remote];
-					$testCases[] = [$baseUrl . '/', $user, $protocol . $remote];
-					$testCases[] = [$baseUrl . '/index.php', $user, $protocol . $remote];
-					$testCases[] = [$baseUrl . '/index.php/s/token', $user, $protocol . $remote];
+					$testCases[] = [$baseUrl, $user, $protocol.$remote];
+					$testCases[] = [$baseUrl.'/', $user, $protocol.$remote];
+					$testCases[] = [$baseUrl.'/index.php', $user, $protocol.$remote];
+					$testCases[] = [$baseUrl.'/index.php/s/token', $user, $protocol.$remote];
 				}
 			}
 		}

tests/lib/Collaboration/Collaborators/SearchTest.php

Spacing +11 added lines, -11 removed lines

@@ -49,7 +49,7 @@ 
 		$userPlugin = $this->createMock(ISearchPlugin::class);
 		$userPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedUserResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedUserResult, $expectedMoreResults) {
 				$type = new SearchResultType('users');
 				$searchResult->addResultSet($type, $mockedUserResult);
 				return $expectedMoreResults;
@@ -58,7 +58,7 @@ 
 		$groupPlugin = $this->createMock(ISearchPlugin::class);
 		$groupPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedGroupsResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedGroupsResult, $expectedMoreResults) {
 				$type = new SearchResultType('groups');
 				$searchResult->addResultSet($type, $mockedGroupsResult);
 				return $expectedMoreResults;
@@ -67,7 +67,7 @@ 
 		$remotePlugin = $this->createMock(ISearchPlugin::class);
 		$remotePlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedRemotesResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedRemotesResult, $expectedMoreResults) {
 				if ($mockedRemotesResult !== null) {
 					$type = new SearchResultType('remotes');
 					$searchResult->addResultSet($type, $mockedRemotesResult['results'], $mockedRemotesResult['exact']);
@@ -81,7 +81,7 @@ 
 		$mailPlugin = $this->createMock(ISearchPlugin::class);
 		$mailPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedMailResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedMailResult, $expectedMoreResults) {
 				$type = new SearchResultType('emails');
 				$searchResult->addResultSet($type, $mockedMailResult);
 				return $expectedMoreResults;
@@ -89,7 +89,7 @@ 
 
 		$this->container->expects($this->any())
 			->method('resolve')
-			->willReturnCallback(function ($class) use ($searchResult, $userPlugin, $groupPlugin, $remotePlugin, $mailPlugin) {
+			->willReturnCallback(function($class) use ($searchResult, $userPlugin, $groupPlugin, $remotePlugin, $mailPlugin) {
 				if ($class === SearchResult::class) {
 					return $searchResult;
 				} elseif ($class === $userPlugin) {
@@ -214,24 +214,24 @@ 
 				[IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_REMOTE, IShare::TYPE_EMAIL], // plugins
 				1, // page
 				10, // per page
-				[  // user result
+				[// user result
 					['label' => 'test One', 'value' => ['shareType' => IShare::TYPE_USER, 'shareWith' => 'test1']],
 				],
-				[  // group result
+				[// group result
 					['label' => 'testgroup1', 'value' => ['shareType' => IShare::TYPE_GROUP, 'shareWith' => 'testgroup1']],
 				],
-				[  // remote result
+				[// remote result
 					'results' => [
-						['label' => 'testz@remote.tld', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf','value' => ['shareType' => IShare::TYPE_REMOTE, 'shareWith' => 'testz@remote.tld']]
+						['label' => 'testz@remote.tld', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf', 'value' => ['shareType' => IShare::TYPE_REMOTE, 'shareWith' => 'testz@remote.tld']]
 					],
 					'exact' => [],
 					'exactIdMatch' => false,
 				],
-				[  //  mail result
+				[//  mail result
 					['label' => 'test Two', 'uuid' => 'b2321e9e-31af-43ac-a406-583fb26d1964', 'value' => ['shareType' => IShare::TYPE_EMAIL, 'shareWith' => 'test2@remote.tld']],
 					['label' => 'test One', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf', 'value' => ['shareType' => IShare::TYPE_EMAIL, 'shareWith' => 'testz@remote.tld']],
 				],
-				[  // expected result
+				[// expected result
 					'exact' => ['users' => [], 'groups' => [], 'remotes' => [], 'emails' => []],
 					'users' => [
 						['label' => 'test One', 'value' => ['shareType' => IShare::TYPE_USER, 'shareWith' => 'test1']],

tests/lib/Session/CryptoSessionDataTest.php

Spacing +3 added lines, -3 removed lines

@@ -24,12 +24,12 @@
 		$this->crypto = $this->createMock(ICrypto::class);
 		$this->crypto->expects($this->any())
 			->method('encrypt')
-			->willReturnCallback(function ($input) {
-				return '#' . $input . '#';
+			->willReturnCallback(function($input) {
+				return '#'.$input.'#';
 			});
 		$this->crypto->expects($this->any())
 			->method('decrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				if ($input === '') {
 					return '';
 				}

tests/lib/Session/CryptoWrappingTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,12 +32,12 @@ 
 			->getMock();
 		$this->crypto->expects($this->any())
 			->method('encrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				return $input;
 			});
 		$this->crypto->expects($this->any())
 			->method('decrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				if ($input === '') {
 					return '';
 				}
@@ -54,7 +54,7 @@ 
 		$this->wrappedSession->expects($this->once())
 			->method('get')
 			->with('encrypted_session_data')
-			->willReturnCallback(function () use ($encryptedValue) {
+			->willReturnCallback(function() use ($encryptedValue) {
 				return $encryptedValue;
 			});