nextcloud / server

tests/lib/Authentication/Token/ManagerTest.php

Spacing +7 added lines, -7 removed lines

@@ -100,7 +100,7 @@ 
 	public function testGenerateTokenTooLongName(): void {
 		$token = $this->createMock(IToken::class);
 		$token->method('getName')
-			->willReturn(str_repeat('a', 120) . '…');
+			->willReturn(str_repeat('a', 120).'…');
 
 
 		$this->publicKeyTokenProvider->expects($this->once())
@@ -110,7 +110,7 @@ 
 				'uid',
 				'loginName',
 				'password',
-				str_repeat('a', 120) . '…',
+				str_repeat('a', 120).'…',
 				IToken::TEMPORARY_TOKEN,
 				IToken::REMEMBER
 			)->willReturn($token);
@@ -158,7 +158,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('tokenData')]
-	public function testUpdateToken(IToken|string $token): void {
+	public function testUpdateToken(IToken | string $token): void {
 		if (is_string($token)) {
 			$token = $this->createMock($token);
 		}
@@ -171,7 +171,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('tokenData')]
-	public function testUpdateTokenActivity(IToken|string $token): void {
+	public function testUpdateTokenActivity(IToken | string $token): void {
 		if (is_string($token)) {
 			$token = $this->createMock($token);
 		}
@@ -184,7 +184,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('tokenData')]
-	public function testGetPassword(IToken|string $token): void {
+	public function testGetPassword(IToken | string $token): void {
 		if (is_string($token)) {
 			$token = $this->createMock($token);
 		}
@@ -199,7 +199,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('tokenData')]
-	public function testSetPassword(IToken|string $token): void {
+	public function testSetPassword(IToken | string $token): void {
 		if (is_string($token)) {
 			$token = $this->createMock($token);
 		}
@@ -374,7 +374,7 @@ 
 		$this->publicKeyTokenProvider
 			->expects($this->exactly(2))
 			->method('invalidateTokenById')
-			->willReturnCallback(function () use (&$calls): void {
+			->willReturnCallback(function() use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 			});

tests/lib/ErrorHandlerTest.php

Spacing +3 added lines, -3 removed lines

@@ -60,17 +60,17 @@
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('passwordProvider')]
 	public function testRemovePasswordFromError($username, $password): void {
-		$url = 'http://' . $username . ':' . $password . '@owncloud.org';
+		$url = 'http://'.$username.':'.$password.'@owncloud.org';
 		$expectedResult = 'http://xxx:xxx@owncloud.org';
 		$this->logger->expects(self::once())
 			->method('log')
 			->with(
 				ILogger::ERROR,
-				'Could not reach ' . $expectedResult . ' at file#4',
+				'Could not reach '.$expectedResult.' at file#4',
 				['app' => 'PHP'],
 			);
 
-		$result = $this->errorHandler->onError(E_USER_ERROR, 'Could not reach ' . $url, 'file', 4);
+		$result = $this->errorHandler->onError(E_USER_ERROR, 'Could not reach '.$url, 'file', 4);
 
 		self::assertTrue($result);
 	}

tests/lib/L10N/FactoryTest.php

Spacing +36 added lines, -36 removed lines

@@ -286,7 +286,7 @@ 
 		$factory->expects(self::once())
 			->method('findL10nDir')
 			->with($app)
-			->willReturn(\OC::$SERVERROOT . '/tests/data/l10n/');
+			->willReturn(\OC::$SERVERROOT.'/tests/data/l10n/');
 
 		self::assertEqualsCanonicalizing(['cs', 'de', 'en', 'ru'], $factory->findAvailableLanguages($app));
 	}
@@ -312,7 +312,7 @@ 
 		$factory->expects(self::once())
 			->method('findL10nDir')
 			->with($app)
-			->willReturn($this->serverRoot . '/apps/files/l10n/');
+			->willReturn($this->serverRoot.'/apps/files/l10n/');
 		$this->config
 			->expects(self::once())
 			->method('getSystemValueString')
@@ -378,7 +378,7 @@ 
 			->willReturn($availableLanguages);
 
 		$factory->expects(self::any())
-			->method('respectDefaultLanguage')->willReturnCallback(function ($app, $lang) {
+			->method('respectDefaultLanguage')->willReturnCallback(function($app, $lang) {
 				return $lang;
 			});
 
@@ -397,13 +397,13 @@ 
 
 	public static function dataGetL10nFilesForApp(): array {
 		return [
-			['', 'de', [\OC::$SERVERROOT . '/core/l10n/de.json']],
-			['core', 'ru', [\OC::$SERVERROOT . '/core/l10n/ru.json']],
-			['lib', 'ru', [\OC::$SERVERROOT . '/lib/l10n/ru.json']],
-			['settings', 'de', [\OC::$SERVERROOT . '/apps/settings/l10n/de.json']],
-			['files', 'de', [\OC::$SERVERROOT . '/apps/files/l10n/de.json']],
+			['', 'de', [\OC::$SERVERROOT.'/core/l10n/de.json']],
+			['core', 'ru', [\OC::$SERVERROOT.'/core/l10n/ru.json']],
+			['lib', 'ru', [\OC::$SERVERROOT.'/lib/l10n/ru.json']],
+			['settings', 'de', [\OC::$SERVERROOT.'/apps/settings/l10n/de.json']],
+			['files', 'de', [\OC::$SERVERROOT.'/apps/files/l10n/de.json']],
 			['files', '_lang_never_exists_', []],
-			['_app_never_exists_', 'de', [\OC::$SERVERROOT . '/core/l10n/de.json']],
+			['_app_never_exists_', 'de', [\OC::$SERVERROOT.'/core/l10n/de.json']],
 		];
 	}
 
@@ -415,11 +415,11 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataGetL10nFilesForApp')]
 	public function testGetL10nFilesForApp($app, $lang, $expected): void {
 		$factory = $this->getFactory();
-		if (in_array($app, ['settings','files'])) {
+		if (in_array($app, ['settings', 'files'])) {
 			$this->appManager
 				->method('getAppPath')
 				->with($app)
-				->willReturn(\OC::$SERVERROOT . '/apps/' . $app);
+				->willReturn(\OC::$SERVERROOT.'/apps/'.$app);
 		} else {
 			$this->appManager
 				->method('getAppPath')
@@ -431,12 +431,12 @@ 
 
 	public static function dataFindL10NDir(): array {
 		return [
-			['', \OC::$SERVERROOT . '/core/l10n/'],
-			['core', \OC::$SERVERROOT . '/core/l10n/'],
-			['lib', \OC::$SERVERROOT . '/lib/l10n/'],
-			['settings', \OC::$SERVERROOT . '/apps/settings/l10n/'],
-			['files', \OC::$SERVERROOT . '/apps/files/l10n/'],
-			['_app_never_exists_', \OC::$SERVERROOT . '/core/l10n/'],
+			['', \OC::$SERVERROOT.'/core/l10n/'],
+			['core', \OC::$SERVERROOT.'/core/l10n/'],
+			['lib', \OC::$SERVERROOT.'/lib/l10n/'],
+			['settings', \OC::$SERVERROOT.'/apps/settings/l10n/'],
+			['files', \OC::$SERVERROOT.'/apps/files/l10n/'],
+			['_app_never_exists_', \OC::$SERVERROOT.'/core/l10n/'],
 		];
 	}
 
@@ -448,11 +448,11 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataFindL10NDir')]
 	public function testFindL10NDir($app, $expected): void {
 		$factory = $this->getFactory();
-		if (in_array($app, ['settings','files'])) {
+		if (in_array($app, ['settings', 'files'])) {
 			$this->appManager
 				->method('getAppPath')
 				->with($app)
-				->willReturn(\OC::$SERVERROOT . '/apps/' . $app);
+				->willReturn(\OC::$SERVERROOT.'/apps/'.$app);
 		} else {
 			$this->appManager
 				->method('getAppPath')
@@ -491,7 +491,7 @@ 
 
 		$this->config->expects(self::any())
 			->method('getSystemValue')
-			->willReturnCallback(function ($var, $default) use ($defaultLang) {
+			->willReturnCallback(function($var, $default) use ($defaultLang) {
 				if ($var === 'default_language') {
 					return $defaultLang;
 				} else {
@@ -527,16 +527,16 @@ 
 		$factory = $this->getFactory(['languageExists', 'findAvailableLanguages', 'respectDefaultLanguage']);
 		$factory->expects(self::any())
 			->method('languageExists')
-			->willReturnCallback(function ($app, $lang) use ($availableLang) {
+			->willReturnCallback(function($app, $lang) use ($availableLang) {
 				return in_array($lang, $availableLang);
 			});
 		$factory->expects(self::any())
 			->method('findAvailableLanguages')
-			->willReturnCallback(function ($app) use ($availableLang) {
+			->willReturnCallback(function($app) use ($availableLang) {
 				return $availableLang;
 			});
 		$factory->expects(self::any())
-			->method('respectDefaultLanguage')->willReturnCallback(function ($app, $lang) {
+			->method('respectDefaultLanguage')->willReturnCallback(function($app, $lang) {
 				return $lang;
 			});
 
@@ -562,8 +562,8 @@ 
 		$this->config->expects(self::exactly(2))
 			->method('getSystemValue')
 			->willReturnMap([
-				['force_language', false, false,],
-				['default_language', false, 'cz',],
+				['force_language', false, false, ],
+				['default_language', false, 'cz', ],
 			]);
 		$factory->expects(self::once())
 			->method('languageExists')
@@ -580,8 +580,8 @@ 
 		$this->config->expects(self::exactly(2))
 			->method('getSystemValue')
 			->willReturnMap([
-				['force_language', false, false,],
-				['default_language', false, false,],
+				['force_language', false, false, ],
+				['default_language', false, false, ],
 			]);
 		$user = $this->createMock(IUser::class);
 		$this->userSession->expects(self::once())
@@ -602,8 +602,8 @@ 
 		$factory = $this->getFactory(['findAvailableLanguages', 'languageExists']);
 		$this->config->method('getSystemValue')
 			->willReturnMap([
-				['force_language', false, false,],
-				['default_language', false, false,],
+				['force_language', false, false, ],
+				['default_language', false, false, ],
 			]);
 		$user = $this->createMock(IUser::class);
 		$this->userSession->expects(self::once())
@@ -632,8 +632,8 @@ 
 		$factory = $this->getFactory(['findAvailableLanguages', 'languageExists']);
 		$this->config->method('getSystemValue')
 			->willReturnMap([
-				['force_language', false, false,],
-				['default_language', false, false,],
+				['force_language', false, false, ],
+				['default_language', false, false, ],
 			]);
 		$user = $this->createMock(IUser::class);
 		$this->userSession->expects(self::once())
@@ -715,7 +715,7 @@ 
 			->method('languageExists')->willReturn(true);
 		$factory->expects(self::any())
 			->method('findAvailableLanguages')
-			->willReturnCallback(function ($app) use ($availableLanguages) {
+			->willReturnCallback(function($app) use ($availableLanguages) {
 				return $availableLanguages;
 			});
 
@@ -728,7 +728,7 @@ 
 			]);
 
 		$result = $this->invokePrivate($factory, 'getLanguages');
-		$commonLanguagesCodes = array_map(function ($lang) {
+		$commonLanguagesCodes = array_map(function($lang) {
 			return $lang['code'];
 		}, $result['commonLanguages']);
 
@@ -737,9 +737,9 @@ 
 
 	public static function languageIteratorRequestProvider(): array {
 		return [
-			[ true, true],
-			[ false, true],
-			[ false, false],
+			[true, true],
+			[false, true],
+			[false, false],
 		];
 	}
 

tests/lib/L10N/L10nTest.php

Spacing +20 added lines, -20 removed lines

@@ -43,30 +43,30 @@ 
 	}
 
 	public function testSimpleTranslationWithTrailingColon(): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/de.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/de.json';
 		$l = new L10N($this->getFactory(), 'test', 'de', 'de_AT', [$transFile]);
 
 		$this->assertEquals('Files:', $l->t('Files:'));
 	}
 
 	public function testGermanPluralTranslations(): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/de.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/de.json';
 		$l = new L10N($this->getFactory(), 'test', 'de', 'de_AT', [$transFile]);
 
-		$this->assertEquals('1 Datei', (string)$l->n('%n file', '%n files', 1));
-		$this->assertEquals('2 Dateien', (string)$l->n('%n file', '%n files', 2));
+		$this->assertEquals('1 Datei', (string) $l->n('%n file', '%n files', 1));
+		$this->assertEquals('2 Dateien', (string) $l->n('%n file', '%n files', 2));
 	}
 
 	public function testRussianPluralTranslations(): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/ru.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/ru.json';
 		$l = new L10N($this->getFactory(), 'test', 'ru', 'ru_UA', [$transFile]);
 
-		$this->assertEquals('1 файл', (string)$l->n('%n file', '%n files', 1));
-		$this->assertEquals('2 файла', (string)$l->n('%n file', '%n files', 2));
-		$this->assertEquals('6 файлов', (string)$l->n('%n file', '%n files', 6));
-		$this->assertEquals('21 файл', (string)$l->n('%n file', '%n files', 21));
-		$this->assertEquals('22 файла', (string)$l->n('%n file', '%n files', 22));
-		$this->assertEquals('26 файлов', (string)$l->n('%n file', '%n files', 26));
+		$this->assertEquals('1 файл', (string) $l->n('%n file', '%n files', 1));
+		$this->assertEquals('2 файла', (string) $l->n('%n file', '%n files', 2));
+		$this->assertEquals('6 файлов', (string) $l->n('%n file', '%n files', 6));
+		$this->assertEquals('21 файл', (string) $l->n('%n file', '%n files', 21));
+		$this->assertEquals('22 файла', (string) $l->n('%n file', '%n files', 22));
+		$this->assertEquals('26 файлов', (string) $l->n('%n file', '%n files', 26));
 
 		/*
 		  1 file	1 файл	1 папка
@@ -82,21 +82,21 @@ 
 	}
 
 	public function testCzechPluralTranslations(): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/cs.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/cs.json';
 		$l = new L10N($this->getFactory(), 'test', 'cs', 'cs_CZ', [$transFile]);
 
-		$this->assertEquals('1 okno', (string)$l->n('%n window', '%n windows', 1));
-		$this->assertEquals('2 okna', (string)$l->n('%n window', '%n windows', 2));
-		$this->assertEquals('5 oken', (string)$l->n('%n window', '%n windows', 5));
+		$this->assertEquals('1 okno', (string) $l->n('%n window', '%n windows', 1));
+		$this->assertEquals('2 okna', (string) $l->n('%n window', '%n windows', 2));
+		$this->assertEquals('5 oken', (string) $l->n('%n window', '%n windows', 5));
 	}
 
 	public function testGermanPluralWithCzechLocaleTranslations(): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/de.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/de.json';
 		$l = new L10N($this->getFactory(), 'test', 'de', 'cs_CZ', [$transFile]);
 
-		$this->assertEquals('1 Datei', (string)$l->n('%n file', '%n files', 1));
-		$this->assertEquals('2 Dateien', (string)$l->n('%n file', '%n files', 2));
-		$this->assertEquals('5 Dateien', (string)$l->n('%n file', '%n files', 5));
+		$this->assertEquals('1 Datei', (string) $l->n('%n file', '%n files', 1));
+		$this->assertEquals('2 Dateien', (string) $l->n('%n file', '%n files', 2));
+		$this->assertEquals('5 Dateien', (string) $l->n('%n file', '%n files', 5));
 	}
 
 	public static function dataPlaceholders(): array {
@@ -114,7 +114,7 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataPlaceholders')]
 	public function testPlaceholders($string, $expected): void {
-		$transFile = \OC::$SERVERROOT . '/tests/data/l10n/de.json';
+		$transFile = \OC::$SERVERROOT.'/tests/data/l10n/de.json';
 		$l = new L10N($this->getFactory(), 'test', 'de', 'de_AT', [$transFile]);
 
 		$this->assertEquals($expected, $l->t($string, ['1', '2']));

tests/lib/Comments/ManagerTest.php

Spacing +46 added lines, -46 removed lines

@@ -136,7 +136,7 @@ 
 			])
 			->executeStatement();
 
-		$id = (string)$qb->getLastInsertId();
+		$id = (string) $qb->getLastInsertId();
 
 		$comment = $manager->get($id);
 		$this->assertInstanceOf(IComment::class, $comment);
@@ -185,14 +185,14 @@ 
 		// Verifying the root comment
 		$this->assertArrayHasKey('comment', $tree);
 		$this->assertInstanceOf(IComment::class, $tree['comment']);
-		$this->assertSame((string)$headId, $tree['comment']->getId());
+		$this->assertSame((string) $headId, $tree['comment']->getId());
 		$this->assertArrayHasKey('replies', $tree);
 		$this->assertCount(3, $tree['replies']);
 
 		// one level deep
 		foreach ($tree['replies'] as $reply) {
 			$this->assertInstanceOf(IComment::class, $reply['comment']);
-			$this->assertSame((string)$id, $reply['comment']->getId());
+			$this->assertSame((string) $id, $reply['comment']->getId());
 			$this->assertCount(0, $reply['replies']);
 			$id--;
 		}
@@ -207,7 +207,7 @@ 
 		// Verifying the root comment
 		$this->assertArrayHasKey('comment', $tree);
 		$this->assertInstanceOf(IComment::class, $tree['comment']);
-		$this->assertSame((string)$id, $tree['comment']->getId());
+		$this->assertSame((string) $id, $tree['comment']->getId());
 		$this->assertArrayHasKey('replies', $tree);
 		$this->assertCount(0, $tree['replies']);
 	}
@@ -223,19 +223,19 @@ 
 		$manager = $this->getManager();
 
 		for ($offset = 0; $offset < 3; $offset += 2) {
-			$tree = $manager->getTree((string)$headId, 2, $offset);
+			$tree = $manager->getTree((string) $headId, 2, $offset);
 
 			// Verifying the root comment
 			$this->assertArrayHasKey('comment', $tree);
 			$this->assertInstanceOf(IComment::class, $tree['comment']);
-			$this->assertSame((string)$headId, $tree['comment']->getId());
+			$this->assertSame((string) $headId, $tree['comment']->getId());
 			$this->assertArrayHasKey('replies', $tree);
 			$this->assertCount(2, $tree['replies']);
 
 			// one level deep
 			foreach ($tree['replies'] as $reply) {
 				$this->assertInstanceOf(IComment::class, $reply['comment']);
-				$this->assertSame((string)$idToVerify, $reply['comment']->getId());
+				$this->assertSame((string) $idToVerify, $reply['comment']->getId());
 				$this->assertCount(0, $reply['replies']);
 				$idToVerify--;
 			}
@@ -272,7 +272,7 @@ 
 			foreach ($comments as $key => $comment) {
 				$this->assertInstanceOf(IComment::class, $comment);
 				$this->assertSame('nice one', $comment->getMessage());
-				$this->assertSame((string)$idToVerify, $comment->getId(), 'ID wrong for comment ' . $key . ' on offset: ' . $offset);
+				$this->assertSame((string) $idToVerify, $comment->getId(), 'ID wrong for comment '.$key.' on offset: '.$offset);
 				$idToVerify--;
 			}
 			$offset += 3;
@@ -289,8 +289,8 @@ 
 		$comments = $manager->getForObject('files', 'file64', 0, 0, new \DateTime('-4 hours'));
 
 		$this->assertCount(2, $comments);
-		$this->assertSame((string)$id2, $comments[0]->getId());
-		$this->assertSame((string)$id1, $comments[1]->getId());
+		$this->assertSame((string) $id2, $comments[0]->getId());
+		$this->assertSame((string) $id1, $comments[1]->getId());
 	}
 
 	public function testGetForObjectWithLimitAndOffsetAndDateTimeConstraint(): void {
@@ -311,7 +311,7 @@ 
 			foreach ($comments as $comment) {
 				$this->assertInstanceOf(IComment::class, $comment);
 				$this->assertSame('nice one', $comment->getMessage());
-				$this->assertSame((string)$idToVerify, $comment->getId());
+				$this->assertSame((string) $idToVerify, $comment->getId());
 				$this->assertGreaterThanOrEqual(4, $comment->getId());
 				$idToVerify--;
 			}
@@ -336,7 +336,7 @@ 
 	public function testGetNumberOfUnreadCommentsForFolder(): void {
 		$folder = $this->createMock(Folder::class);
 		$fileIds = range(1111, 1114);
-		$children = array_map(function (int $id) {
+		$children = array_map(function(int $id) {
 			$file = $this->createMock(Folder::class);
 			$file->method('getId')
 				->willReturn($id);
@@ -365,8 +365,8 @@ 
 
 		$manager = $this->getManager();
 
-		$manager->setReadMark('files', (string)$fileIds[0], (new \DateTime())->modify('-1 days'), $user);
-		$manager->setReadMark('files', (string)$fileIds[2], (new \DateTime()), $user);
+		$manager->setReadMark('files', (string) $fileIds[0], (new \DateTime())->modify('-1 days'), $user);
+		$manager->setReadMark('files', (string) $fileIds[2], (new \DateTime()), $user);
 
 		$amount = $manager->getNumberOfUnreadCommentsForFolder($folder->getId(), $user);
 		$this->assertEquals([
@@ -393,7 +393,7 @@ 
 			$expected = array_reverse($expected);
 		}
 
-		$this->assertSame($expected, array_map(static fn (IComment $c): int => (int)$c->getId(), $comments));
+		$this->assertSame($expected, array_map(static fn (IComment $c): int => (int) $c->getId(), $comments));
 	}
 
 	public static function dataGetForObjectSince(): array {
@@ -423,7 +423,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('invalidCreateArgsProvider')]
-	public function testCreateCommentInvalidArguments(string|int $aType, string|int $aId, string|int $oType, string|int $oId): void {
+	public function testCreateCommentInvalidArguments(string | int $aType, string | int $aId, string | int $oType, string | int $oId): void {
 		$this->expectException(\InvalidArgumentException::class);
 
 		$manager = $this->getManager();
@@ -459,7 +459,7 @@ 
 		$done = $manager->delete('');
 		$this->assertFalse($done);
 
-		$id = (string)$this->addDatabaseEntry(0, 0);
+		$id = (string) $this->addDatabaseEntry(0, 0);
 		$comment = $manager->get($id);
 		$this->assertInstanceOf(IComment::class, $comment);
 		$done = $manager->delete($id);
@@ -574,7 +574,7 @@ 
 	}
 
 	public function testSaveAsChild(): void {
-		$id = (string)$this->addDatabaseEntry(0, 0);
+		$id = (string) $this->addDatabaseEntry(0, 0);
 
 		$manager = $this->getManager();
 
@@ -587,7 +587,7 @@ 
 				->setMessage('full ack')
 				->setVerb('comment')
 				// setting the creation time avoids using sleep() while making sure to test with different timestamps
-				->setCreationDateTime(new \DateTime('+' . $i . ' minutes'));
+				->setCreationDateTime(new \DateTime('+'.$i.' minutes'));
 
 			$manager->save($comment);
 
@@ -608,7 +608,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('invalidActorArgsProvider')]
-	public function testDeleteReferencesOfActorInvalidInput(string|int $type, string|int $id): void {
+	public function testDeleteReferencesOfActorInvalidInput(string | int $type, string | int $id): void {
 		$this->expectException(\InvalidArgumentException::class);
 
 		$manager = $this->getManager();
@@ -624,7 +624,7 @@ 
 		$manager = $this->getManager();
 
 		// just to make sure they are really set, with correct actor data
-		$comment = $manager->get((string)$ids[1]);
+		$comment = $manager->get((string) $ids[1]);
 		$this->assertSame('users', $comment->getActorType());
 		$this->assertSame('alice', $comment->getActorId());
 
@@ -632,7 +632,7 @@ 
 		$this->assertTrue($wasSuccessful);
 
 		foreach ($ids as $id) {
-			$comment = $manager->get((string)$id);
+			$comment = $manager->get((string) $id);
 			$this->assertSame(ICommentsManager::DELETED_USER, $comment->getActorType());
 			$this->assertSame(ICommentsManager::DELETED_USER, $comment->getActorId());
 		}
@@ -673,7 +673,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('invalidObjectArgsProvider')]
-	public function testDeleteCommentsAtObjectInvalidInput(string|int $type, string|int $id): void {
+	public function testDeleteCommentsAtObjectInvalidInput(string | int $type, string | int $id): void {
 		$this->expectException(\InvalidArgumentException::class);
 
 		$manager = $this->getManager();
@@ -689,7 +689,7 @@ 
 		$manager = $this->getManager();
 
 		// just to make sure they are really set, with correct actor data
-		$comment = $manager->get((string)$ids[1]);
+		$comment = $manager->get((string) $ids[1]);
 		$this->assertSame('files', $comment->getObjectType());
 		$this->assertSame('file64', $comment->getObjectId());
 
@@ -699,7 +699,7 @@ 
 		$verified = 0;
 		foreach ($ids as $id) {
 			try {
-				$manager->get((string)$id);
+				$manager->get((string) $id);
 			} catch (NotFoundException) {
 				$verified++;
 			}
@@ -733,7 +733,7 @@ 
 		);
 
 		// just to make sure they are really set, with correct actor data
-		$comment = $manager->get((string)$ids[1]);
+		$comment = $manager->get((string) $ids[1]);
 		$this->assertSame('files', $comment->getObjectType());
 		$this->assertSame('file64', $comment->getObjectId());
 
@@ -744,7 +744,7 @@ 
 		$exists = 0;
 		foreach ($ids as $id) {
 			try {
-				$manager->get((string)$id);
+				$manager->get((string) $id);
 				$exists++;
 			} catch (NotFoundException) {
 				$deleted++;
@@ -786,7 +786,7 @@ 
 		$exists = 0;
 		foreach ($ids as $id) {
 			try {
-				$manager->get((string)$id);
+				$manager->get((string) $id);
 				$exists++;
 			} catch (NotFoundException) {
 				$deleted++;
@@ -884,10 +884,10 @@ 
 			->method('handle');
 
 		$manager = $this->getManager();
-		$manager->registerEventHandler(function () use ($handler1) {
+		$manager->registerEventHandler(function() use ($handler1) {
 			return $handler1;
 		});
-		$manager->registerEventHandler(function () use ($handler2) {
+		$manager->registerEventHandler(function() use ($handler2) {
 			return $handler2;
 		});
 
@@ -912,11 +912,11 @@ 
 	public function testResolveDisplayName(): void {
 		$manager = $this->getManager();
 
-		$planetClosure = function ($name) {
+		$planetClosure = function($name) {
 			return ucfirst($name);
 		};
 
-		$galaxyClosure = function ($name) {
+		$galaxyClosure = function($name) {
 			return strtoupper($name);
 		};
 
@@ -933,7 +933,7 @@ 
 
 		$manager = $this->getManager();
 
-		$planetClosure = function ($name) {
+		$planetClosure = function($name) {
 			return ucfirst($name);
 		};
 		$manager->registerDisplayNameResolver('planet', $planetClosure);
@@ -946,7 +946,7 @@ 
 
 		$manager = $this->getManager();
 
-		$planetClosure = function ($name) {
+		$planetClosure = function($name) {
 			return ucfirst($name);
 		};
 		$manager->registerDisplayNameResolver(1337, $planetClosure);
@@ -958,7 +958,7 @@ 
 
 		$manager = $this->getManager();
 
-		$planetClosure = function ($name) {
+		$planetClosure = function($name) {
 			return ucfirst($name);
 		};
 
@@ -969,7 +969,7 @@ 
 	public function testResolveDisplayNameDirtyResolver(): void {
 		$manager = $this->getManager();
 
-		$planetClosure = function () {
+		$planetClosure = function() {
 			return null;
 		};
 
@@ -981,7 +981,7 @@ 
 
 		$manager = $this->getManager();
 
-		$planetClosure = function () {
+		$planetClosure = function() {
 			return null;
 		};
 
@@ -1068,14 +1068,14 @@ 
 			[$message, $actorId, $verb, $parentText] = $comment;
 			$parentId = null;
 			if ($parentText) {
-				$parentId = (string)$comments[$parentText]->getId();
+				$parentId = (string) $comments[$parentText]->getId();
 			}
 			$id = '';
 			if ($verb === 'reaction_deleted') {
-				$id = $comments[$message . '#' . $actorId]->getId();
+				$id = $comments[$message.'#'.$actorId]->getId();
 			}
 			$comment = $this->testSave($message, $actorId, $verb, $parentId, $id);
-			$comments[$comment->getMessage() . '#' . $comment->getActorId()] = $comment;
+			$comments[$comment->getMessage().'#'.$comment->getActorId()] = $comment;
 		}
 		$this->connection->commit();
 		return $comments;
@@ -1088,8 +1088,8 @@ 
 
 		$processedComments = $this->proccessComments($comments);
 		$comment = reset($processedComments);
-		$all = $manager->retrieveAllReactions((int)$comment->getId());
-		$actual = array_map(static function (IComment $row): array {
+		$all = $manager->retrieveAllReactions((int) $comment->getId());
+		$actual = array_map(static function(IComment $row): array {
 			return [
 				$row->getActorId(),
 				$row->getMessage(),
@@ -2350,8 +2350,8 @@ 
 
 		$processedComments = $this->proccessComments($comments);
 		$comment = reset($processedComments);
-		$all = $manager->retrieveAllReactionsWithSpecificReaction((int)$comment->getId(), $reaction);
-		$actual = array_map(static function (IComment $row): array {
+		$all = $manager->retrieveAllReactionsWithSpecificReaction((int) $comment->getId(), $reaction);
+		$actual = array_map(static function(IComment $row): array {
 			return [
 				$row->getActorId(),
 				$row->getMessage(),
@@ -2409,8 +2409,8 @@ 
 		if ($notFound) {
 			$this->expectException(NotFoundException::class);
 		}
-		$comment = $processedComments[$expected['message'] . '#' . $expected['actorId']];
-		$actual = $manager->getReactionComment((int)$comment->getParentId(), $comment->getActorType(), $comment->getActorId(), $comment->getMessage());
+		$comment = $processedComments[$expected['message'].'#'.$expected['actorId']];
+		$actual = $manager->getReactionComment((int) $comment->getParentId(), $comment->getActorType(), $comment->getActorId(), $comment->getMessage());
 		if (!$notFound) {
 			$this->assertEquals($expected['message'], $actual->getMessage());
 			$this->assertEquals($expected['actorId'], $actual->getActorId());

tests/lib/NavigationManagerTest.php

Spacing +15 added lines, -15 removed lines

@@ -39,7 +39,7 @@ 
 	/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
 	protected $config;
 
-	protected IEVentDispatcher|MockObject $dispatcher;
+	protected IEVentDispatcher | MockObject $dispatcher;
 
 	/** @var NavigationManager */
 	protected $navigationManager;
@@ -149,7 +149,7 @@ 
 		global $testAddClosureNumberOfCalls;
 		$testAddClosureNumberOfCalls = 0;
 
-		$this->navigationManager->add(function () use ($entry) {
+		$this->navigationManager->add(function() use ($entry) {
 			global $testAddClosureNumberOfCalls;
 			$testAddClosureNumberOfCalls++;
 
@@ -201,7 +201,7 @@ 
 		global $testAddClosureNumberOfCalls;
 		$testAddClosureNumberOfCalls = 0;
 
-		$this->navigationManager->add(function () use ($entry) {
+		$this->navigationManager->add(function() use ($entry) {
 			global $testAddClosureNumberOfCalls;
 			$testAddClosureNumberOfCalls++;
 
@@ -218,7 +218,7 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('providesNavigationConfig')]
 	public function testWithAppManager($expected, $navigation, $isAdmin = false): void {
 		$l = $this->createMock(IL10N::class);
-		$l->expects($this->any())->method('t')->willReturnCallback(function ($text, $parameters = []) {
+		$l->expects($this->any())->method('t')->willReturnCallback(function($text, $parameters = []) {
 			return vsprintf($text, $parameters);
 		});
 
@@ -236,14 +236,14 @@ 
 			->willReturn($navigation);
 		$this->urlGenerator->expects($this->any())
 			->method('imagePath')
-			->willReturnCallback(function ($appName, $file) {
+			->willReturnCallback(function($appName, $file) {
 				return "/apps/$appName/img/$file";
 			});
 		$this->appManager->expects($this->any())
 			->method('getAppIcon')
 			->willReturnCallback(fn (string $appName) => "/apps/$appName/img/app.svg");
 		$this->l10nFac->expects($this->any())->method('get')->willReturn($l);
-		$this->urlGenerator->expects($this->any())->method('linkToRoute')->willReturnCallback(function ($route) {
+		$this->urlGenerator->expects($this->any())->method('linkToRoute')->willReturnCallback(function($route) {
 			if ($route === 'core.login.logout') {
 				return 'https://example.com/logout';
 			}
@@ -265,7 +265,7 @@ 
 		$this->navigationManager->clear();
 		$this->dispatcher->expects($this->once())
 			->method('dispatchTyped')
-			->willReturnCallback(function ($event): void {
+			->willReturnCallback(function($event): void {
 				$this->assertInstanceOf(LoadAdditionalEntriesEvent::class, $event);
 			});
 		$entries = $this->navigationManager->getAll('all');
@@ -323,7 +323,7 @@ 
 			'logout' => [
 				'id' => 'logout',
 				'order' => 99999,
-				'href' => 'https://example.com/logout?requesttoken=' . urlencode(Util::callRegister()),
+				'href' => 'https://example.com/logout?requesttoken='.urlencode(Util::callRegister()),
 				'icon' => '/apps/core/img/actions/logout.svg',
 				'name' => 'Log out',
 				'active' => false,
@@ -503,7 +503,7 @@ 
 
 	public function testWithAppManagerAndApporder(): void {
 		$l = $this->createMock(IL10N::class);
-		$l->expects($this->any())->method('t')->willReturnCallback(function ($text, $parameters = []) {
+		$l->expects($this->any())->method('t')->willReturnCallback(function($text, $parameters = []) {
 			return vsprintf($text, $parameters);
 		});
 
@@ -531,7 +531,7 @@ 
 
 		$this->config->method('getUserValue')
 			->willReturnCallback(
-				function (string $userId, string $appName, string $key, mixed $default = '') use ($testOrder) {
+				function(string $userId, string $appName, string $key, mixed $default = '') use ($testOrder) {
 					$this->assertEquals('user001', $userId);
 					if ($key === 'apporder') {
 						return json_encode(['test' => ['app' => 'test', 'order' => $testOrder]]);
@@ -547,10 +547,10 @@ 
 		$this->appManager->expects($this->once())->method('getAppInfo')->with('test')->willReturn($navigation);
 		$this->appManager->expects($this->once())->method('getAppIcon')->with('test')->willReturn('/apps/test/img/app.svg');
 		$this->l10nFac->expects($this->any())->method('get')->willReturn($l);
-		$this->urlGenerator->expects($this->any())->method('imagePath')->willReturnCallback(function ($appName, $file) {
+		$this->urlGenerator->expects($this->any())->method('imagePath')->willReturnCallback(function($appName, $file) {
 			return "/apps/$appName/img/$file";
 		});
-		$this->urlGenerator->expects($this->any())->method('linkToRoute')->willReturnCallback(function ($route) {
+		$this->urlGenerator->expects($this->any())->method('linkToRoute')->willReturnCallback(function($route) {
 			if ($route === 'core.login.logout') {
 				return 'https://example.com/logout';
 			}
@@ -572,7 +572,7 @@ 
 		$this->navigationManager->clear();
 		$this->dispatcher->expects($this->once())
 			->method('dispatchTyped')
-			->willReturnCallback(function ($event): void {
+			->willReturnCallback(function($event): void {
 				$this->assertInstanceOf(LoadAdditionalEntriesEvent::class, $event);
 			});
 		$entries = $this->navigationManager->getAll();
@@ -737,13 +737,13 @@ 
 		$this->navigationManager->add([
 			'id' => 'settings',
 		]);
-		$this->navigationManager->add(static function (): array {
+		$this->navigationManager->add(static function(): array {
 			return [
 				'id' => 'closure1',
 				'href' => '/closure1',
 			];
 		});
-		$this->navigationManager->add(static function (): array {
+		$this->navigationManager->add(static function(): array {
 			return [
 				'id' => 'closure2',
 				'href' => '/closure2',

tests/lib/Security/SecureRandomTest.php

Spacing +2 added lines, -2 removed lines

@@ -62,9 +62,9 @@
 	#[\PHPUnit\Framework\Attributes\DataProvider('charCombinations')]
 	public function testScheme($charName, $chars): void {
 		$generator = $this->rng;
-		$scheme = constant('OCP\Security\ISecureRandom::' . $charName);
+		$scheme = constant('OCP\Security\ISecureRandom::'.$charName);
 		$randomString = $generator->generate(100, $scheme);
-		$matchesRegex = preg_match('/^' . $chars . '+$/', $randomString);
+		$matchesRegex = preg_match('/^'.$chars.'+$/', $randomString);
 		$this->assertSame(1, $matchesRegex);
 	}
 

tests/lib/Security/CertificateManagerTest.php

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
 		$this->createUser($this->username, '');
 
 		$storage = new Temporary();
-		$this->registerMount($this->username, $storage, '/' . $this->username . '/');
+		$this->registerMount($this->username, $storage, '/'.$this->username.'/');
 
 		\OC_Util::tearDownFS();
 		\OC_User::setUserId($this->username);
@@ -86,14 +86,14 @@ 
 		$this->assertSame([], $this->certificateManager->listCertificates());
 
 		// Add some certificates
-		$this->certificateManager->addCertificate(file_get_contents(__DIR__ . '/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
+		$this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
 		$certificateStore = [];
-		$certificateStore[] = new Certificate(file_get_contents(__DIR__ . '/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
+		$certificateStore[] = new Certificate(file_get_contents(__DIR__.'/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
 		$this->assertEqualsArrays($certificateStore, $this->certificateManager->listCertificates());
 
 		// Add another certificates
-		$this->certificateManager->addCertificate(file_get_contents(__DIR__ . '/../../data/certificates/expiredCertificate.crt'), 'ExpiredCertificate');
-		$certificateStore[] = new Certificate(file_get_contents(__DIR__ . '/../../data/certificates/expiredCertificate.crt'), 'ExpiredCertificate');
+		$this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), 'ExpiredCertificate');
+		$certificateStore[] = new Certificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), 'ExpiredCertificate');
 		$this->assertEqualsArrays($certificateStore, $this->certificateManager->listCertificates());
 	}
 
@@ -119,7 +119,7 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dangerousFileProvider')]
 	public function testAddDangerousFile($filename): void {
 		$this->expectException(InvalidPathException::class);
-		$this->certificateManager->addCertificate(file_get_contents(__DIR__ . '/../../data/certificates/expiredCertificate.crt'), $filename);
+		$this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/expiredCertificate.crt'), $filename);
 	}
 
 	public function testRemoveDangerousFile(): void {
@@ -127,7 +127,7 @@ 
 	}
 
 	public function testRemoveExistingFile(): void {
-		$this->certificateManager->addCertificate(file_get_contents(__DIR__ . '/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
+		$this->certificateManager->addCertificate(file_get_contents(__DIR__.'/../../data/certificates/goodCertificate.crt'), 'GoodCertificate');
 		$this->assertTrue($this->certificateManager->removeCertificate('GoodCertificate'));
 	}
 
@@ -170,7 +170,7 @@ 
 
 
 		$view->expects($this->any())->method('filemtime')
-			->willReturnCallback(function ($path) use ($targetBundleMtime) {
+			->willReturnCallback(function($path) use ($targetBundleMtime) {
 				if ($path === 'targetBundlePath') {
 					return $targetBundleMtime;
 				}

tests/lib/LoggerTest.php

Spacing +2 added lines, -2 removed lines

@@ -177,7 +177,7 @@ 
 		if (is_array($message)) {
 			$textMessage = $message['message'];
 		}
-		$this->logs[] = $level . ' ' . $textMessage;
+		$this->logs[] = $level.' '.$textMessage;
 	}
 
 	public static function userAndPasswordData(): array {
@@ -280,7 +280,7 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('userAndPasswordData')]
 	public function testDetectclosure(string $user, string $password): void {
 		$this->mockDefaultLogLevel();
-		$a = function ($user, $password): void {
+		$a = function($user, $password): void {
 			throw new \Exception('test');
 		};
 		$this->registry->expects($this->once())