nextcloud / server

apps/settings/tests/SetupChecks/LoggingLevelTest.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->l10n->expects($this->any())
 			->method('t')
-			->willReturnCallback(function ($message, array $replace) {
+			->willReturnCallback(function($message, array $replace) {
 				return vsprintf($message, $replace);
 			});
 		$this->config = $this->createMock(IConfig::class);
@@ -60,7 +60,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataRun')]
-	public function testRun(string|int $value, string $expected): void {
+	public function testRun(string | int $value, string $expected): void {
 		$this->urlGenerator->method('linkToDocs')->willReturn('admin-logging');
 
 		$this->config->expects(self::once())

apps/settings/tests/SetupChecks/DataDirectoryProtectedTest.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->l10n->expects($this->any())
 			->method('t')
-			->willReturnCallback(function ($message, array $replace) {
+			->willReturnCallback(function($message, array $replace) {
 				return vsprintf($message, $replace);
 			});
 
@@ -56,7 +56,7 @@ 
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestStatusCode')]
 	public function testStatusCode(array $status, string $expected, bool $hasBody): void {
-		$responses = array_map(function ($state) use ($hasBody) {
+		$responses = array_map(function($state) use ($hasBody) {
 			$response = $this->createMock(IResponse::class);
 			$response->expects($this->any())->method('getStatusCode')->willReturn($state);
 			$response->expects(($this->atMost(1)))->method('getBody')->willReturn($hasBody ? '# Nextcloud data directory' : 'something else');
@@ -110,7 +110,7 @@ 
 	 * Helper function creates a nicer interface for mocking Generator behavior
 	 */
 	protected function generate(array $yield_values) {
-		return $this->returnCallback(function () use ($yield_values) {
+		return $this->returnCallback(function() use ($yield_values) {
 			yield from $yield_values;
 		});
 	}

apps/settings/tests/Controller/MailSettingsControllerTest.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
 		];
 		$this->config->expects($this->exactly(2))
 			->method('setSystemValues')
-			->willReturnCallback(function () use (&$calls): void {
+			->willReturnCallback(function() use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 			});
@@ -135,7 +135,7 @@ 
 
 		$this->l->expects($this->any())
 			->method('t')
-			->willReturnCallback(function ($text, $parameters = []) {
+			->willReturnCallback(function($text, $parameters = []) {
 				return vsprintf($text, $parameters);
 			});
 		$this->userSession

apps/settings/tests/Settings/Personal/Security/AuthtokensTest.php

Spacing +1 added lines, -1 removed lines

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

apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php

Spacing +2 added lines, -2 removed lines

@@ -247,7 +247,7 @@ 
 		$i = 0;
 		$job->expects($this->exactly(\count($notifications)))
 			->method('createNotifications')
-			->willReturnCallback(function () use ($notifications, &$i): void {
+			->willReturnCallback(function() use ($notifications, &$i): void {
 				$this->assertEquals($notifications[$i], func_get_args());
 				$i++;
 			});
@@ -265,7 +265,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataCreateNotifications')]
-	public function testCreateNotifications(string $app, string $version, string|false $lastNotification, string|false $callDelete, bool $createNotification, ?array $users, ?array $userNotifications): void {
+	public function testCreateNotifications(string $app, string $version, string | false $lastNotification, string | false $callDelete, bool $createNotification, ?array $users, ?array $userNotifications): void {
 		$job = $this->getJob([
 			'deleteOutdatedNotifications',
 			'getUsersToNotify',

apps/files_trashbin/tests/ExpirationTest.php

Spacing +45 added lines, -45 removed lines

@@ -30,55 +30,55 @@ 
 
 		return [
 			// Expiration is disabled - always should return false
-			[ 'disabled', $today, $back10Days, false, false],
-			[ 'disabled', $today, $back10Days, true, false],
-			[ 'disabled', $today, $ahead100Days, true, false],
+			['disabled', $today, $back10Days, false, false],
+			['disabled', $today, $back10Days, true, false],
+			['disabled', $today, $ahead100Days, true, false],
 
 			// Default: expire in 30 days or earlier when quota requirements are met
-			[ 'auto', $today, $back10Days, false, false],
-			[ 'auto', $today, $back35Days, false, false],
-			[ 'auto', $today, $back10Days, true, true],
-			[ 'auto', $today, $back35Days, true, true],
-			[ 'auto', $today, $ahead100Days, true, true],
+			['auto', $today, $back10Days, false, false],
+			['auto', $today, $back35Days, false, false],
+			['auto', $today, $back10Days, true, true],
+			['auto', $today, $back35Days, true, true],
+			['auto', $today, $ahead100Days, true, true],
 
 			// The same with 'auto'
-			[ 'auto, auto', $today, $back10Days, false, false],
-			[ 'auto, auto', $today, $back35Days, false, false],
-			[ 'auto, auto', $today, $back10Days, true, true],
-			[ 'auto, auto', $today, $back35Days, true, true],
+			['auto, auto', $today, $back10Days, false, false],
+			['auto, auto', $today, $back35Days, false, false],
+			['auto, auto', $today, $back10Days, true, true],
+			['auto, auto', $today, $back35Days, true, true],
 
 			// Keep for 15 days but expire anytime if space needed
-			[ '15, auto', $today, $back10Days, false, false],
-			[ '15, auto', $today, $back20Days, false, false],
-			[ '15, auto', $today, $back10Days, true, true],
-			[ '15, auto', $today, $back20Days, true, true],
-			[ '15, auto', $today, $ahead100Days, true, true],
+			['15, auto', $today, $back10Days, false, false],
+			['15, auto', $today, $back20Days, false, false],
+			['15, auto', $today, $back10Days, true, true],
+			['15, auto', $today, $back20Days, true, true],
+			['15, auto', $today, $ahead100Days, true, true],
 
 			// Expire anytime if space needed, Expire all older than max
-			[ 'auto, 15', $today, $back10Days, false, false],
-			[ 'auto, 15', $today, $back20Days, false, true],
-			[ 'auto, 15', $today, $back10Days, true, true],
-			[ 'auto, 15', $today, $back20Days, true, true],
-			[ 'auto, 15', $today, $ahead100Days, true, true],
+			['auto, 15', $today, $back10Days, false, false],
+			['auto, 15', $today, $back20Days, false, true],
+			['auto, 15', $today, $back10Days, true, true],
+			['auto, 15', $today, $back20Days, true, true],
+			['auto, 15', $today, $ahead100Days, true, true],
 
 			// Expire all older than max OR older than min if space needed
-			[ '15, 25', $today, $back10Days, false, false],
-			[ '15, 25', $today, $back20Days, false, false],
-			[ '15, 25', $today, $back30Days, false, true],
-			[ '15, 25', $today, $back10Days, false, false],
-			[ '15, 25', $today, $back20Days, true, true],
-			[ '15, 25', $today, $back30Days, true, true],
-			[ '15, 25', $today, $ahead100Days, true, false],
+			['15, 25', $today, $back10Days, false, false],
+			['15, 25', $today, $back20Days, false, false],
+			['15, 25', $today, $back30Days, false, true],
+			['15, 25', $today, $back10Days, false, false],
+			['15, 25', $today, $back20Days, true, true],
+			['15, 25', $today, $back30Days, true, true],
+			['15, 25', $today, $ahead100Days, true, false],
 
 			// Expire all older than max OR older than min if space needed
 			// Max<Min case
-			[ '25, 15', $today, $back10Days, false, false],
-			[ '25, 15', $today, $back20Days, false, false],
-			[ '25, 15', $today, $back30Days, false, true],
-			[ '25, 15', $today, $back10Days, false, false],
-			[ '25, 15', $today, $back20Days, true, false],
-			[ '25, 15', $today, $back30Days, true, true],
-			[ '25, 15', $today, $ahead100Days, true, false],
+			['25, 15', $today, $back10Days, false, false],
+			['25, 15', $today, $back20Days, false, false],
+			['25, 15', $today, $back30Days, false, true],
+			['25, 15', $today, $back10Days, false, false],
+			['25, 15', $today, $back20Days, true, false],
+			['25, 15', $today, $back30Days, true, true],
+			['25, 15', $today, $ahead100Days, true, false],
 		];
 	}
 
@@ -96,20 +96,20 @@ 
 
 	public static function timestampTestData(): array {
 		return [
-			[ 'disabled', false],
-			[ 'auto', false ],
-			[ 'auto,auto', false ],
-			[ 'auto, auto', false ],
-			[ 'auto, 3',  self::FAKE_TIME_NOW - (3 * self::SECONDS_PER_DAY) ],
-			[ '5, auto', false ],
-			[ '3, 5', self::FAKE_TIME_NOW - (5 * self::SECONDS_PER_DAY) ],
-			[ '10, 3', self::FAKE_TIME_NOW - (10 * self::SECONDS_PER_DAY) ],
+			['disabled', false],
+			['auto', false],
+			['auto,auto', false],
+			['auto, auto', false],
+			['auto, 3', self::FAKE_TIME_NOW - (3 * self::SECONDS_PER_DAY)],
+			['5, auto', false],
+			['3, 5', self::FAKE_TIME_NOW - (5 * self::SECONDS_PER_DAY)],
+			['10, 3', self::FAKE_TIME_NOW - (10 * self::SECONDS_PER_DAY)],
 		];
 	}
 
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('timestampTestData')]
-	public function testGetMaxAgeAsTimestamp(string $configValue, bool|int $expectedMaxAgeTimestamp): void {
+	public function testGetMaxAgeAsTimestamp(string $configValue, bool | int $expectedMaxAgeTimestamp): void {
 		$mockedConfig = $this->getMockedConfig($configValue);
 		$mockedTimeFactory = $this->getMockedTimeFactory(
 			self::FAKE_TIME_NOW

apps/federation/tests/DbHandlerTest.php

Spacing +9 added lines, -9 removed lines

@@ -68,9 +68,9 @@ 
 		$qResult->closeCursor();
 		$this->assertCount(1, $result);
 		$this->assertSame($expectedUrl, $result[0]['url']);
-		$this->assertSame($id, (int)$result[0]['id']);
+		$this->assertSame($id, (int) $result[0]['id']);
 		$this->assertSame($expectedHash, $result[0]['url_hash']);
-		$this->assertSame(TrustedServers::STATUS_PENDING, (int)$result[0]['status']);
+		$this->assertSame(TrustedServers::STATUS_PENDING, (int) $result[0]['status']);
 	}
 
 	public static function dataTestAddServer(): array {
@@ -93,8 +93,8 @@ 
 		$this->assertCount(2, $result);
 		$this->assertSame('server1', $result[0]['url']);
 		$this->assertSame('server2', $result[1]['url']);
-		$this->assertSame($id1, (int)$result[0]['id']);
-		$this->assertSame($id2, (int)$result[1]['id']);
+		$this->assertSame($id1, (int) $result[0]['id']);
+		$this->assertSame($id2, (int) $result[1]['id']);
 
 		$this->dbHandler->removeServer($id2);
 		$query = $this->connection->getQueryBuilder()->select('*')->from($this->dbTable);
@@ -104,7 +104,7 @@ 
 		$qResult->closeCursor();
 		$this->assertCount(1, $result);
 		$this->assertSame('server1', $result[0]['url']);
-		$this->assertSame($id1, (int)$result[0]['id']);
+		$this->assertSame($id1, (int) $result[0]['id']);
 	}
 
 
@@ -124,8 +124,8 @@ 
 		$this->assertSame(2, count($result));
 		$this->assertSame('server1', $result[0]['url']);
 		$this->assertSame('server2', $result[1]['url']);
-		$this->assertSame($id1, (int)$result[0]['id']);
-		$this->assertSame($id2, (int)$result[1]['id']);
+		$this->assertSame($id1, (int) $result[0]['id']);
+		$this->assertSame($id2, (int) $result[1]['id']);
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestServerExists')]
@@ -206,7 +206,7 @@ 
 		$result = $qResult->fetchAll();
 		$qResult->closeCursor();
 		$this->assertCount(1, $result);
-		$this->assertSame(TrustedServers::STATUS_PENDING, (int)$result[0]['status']);
+		$this->assertSame(TrustedServers::STATUS_PENDING, (int) $result[0]['status']);
 		$this->dbHandler->setServerStatus('http://server1', TrustedServers::STATUS_OK);
 		$query = $this->connection->getQueryBuilder()->select('*')->from($this->dbTable);
 
@@ -214,7 +214,7 @@ 
 		$result = $qResult->fetchAll();
 		$qResult->closeCursor();
 		$this->assertCount(1, $result);
-		$this->assertSame(TrustedServers::STATUS_OK, (int)$result[0]['status']);
+		$this->assertSame(TrustedServers::STATUS_OK, (int) $result[0]['status']);
 	}
 
 	public function testGetServerStatus(): void {

apps/files/tests/Command/DeleteOrphanedFilesTest.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
 
 		$this->loginAsUser($this->user1);
 
-		$view = new View('/' . $this->user1 . '/');
+		$view = new View('/'.$this->user1.'/');
 		$view->mkdir('files/test');
 
 		$fileInfo = $view->getFileInfo('files/test');
@@ -117,7 +117,7 @@ 
 		$output
 			->expects($this->exactly(3))
 			->method('writeln')
-			->willReturnCallback(function (string $message) use (&$calls): void {
+			->willReturnCallback(function(string $message) use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertSame($expected, $message);
 			});

apps/files/tests/Activity/ProviderTest.php

Spacing +3 added lines, -3 removed lines

@@ -91,7 +91,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataGetFile')]
-	public function testGetFile(array|string $parameter, ?int $eventId, string $id, string $name, string $path): void {
+	public function testGetFile(array | string $parameter, ?int $eventId, string $id, string $name, string $path): void {
 		$provider = $this->getProvider();
 
 		if ($eventId !== null) {
@@ -106,7 +106,7 @@ 
 		$this->url->expects($this->once())
 			->method('linkToRouteAbsolute')
 			->with('files.viewcontroller.showFile', ['fileid' => $id])
-			->willReturn('link-' . $id);
+			->willReturn('link-'.$id);
 
 		$result = self::invokePrivate($provider, 'getFile', [$parameter, $event]);
 
@@ -114,7 +114,7 @@ 
 		$this->assertSame($id, $result['id']);
 		$this->assertSame($name, $result['name']);
 		$this->assertSame($path, $result['path']);
-		$this->assertSame('link-' . $id, $result['link']);
+		$this->assertSame('link-'.$id, $result['link']);
 	}