nextcloud / server

tests/lib/L10N/FactoryTest.php

Spacing +36 added lines, -36 removed lines

@@ -287,7 +287,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));
 	}
@@ -313,7 +313,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')
@@ -379,7 +379,7 @@ 
 			->willReturn($availableLanguages);
 
 		$factory->expects(self::any())
-			->method('respectDefaultLanguage')->willReturnCallback(function ($app, $lang) {
+			->method('respectDefaultLanguage')->willReturnCallback(function($app, $lang) {
 				return $lang;
 			});
 
@@ -398,13 +398,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']],
 		];
 	}
 
@@ -416,11 +416,11 @@ 
 	 */
 	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')
@@ -432,12 +432,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/'],
 		];
 	}
 
@@ -449,11 +449,11 @@ 
 	 */
 	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')
@@ -492,7 +492,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 {
@@ -528,16 +528,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;
 			});
 
@@ -563,8 +563,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')
@@ -581,8 +581,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())
@@ -603,8 +603,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())
@@ -633,8 +633,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())
@@ -716,7 +716,7 @@ 
 			->method('languageExists')->willReturn(true);
 		$factory->expects(self::any())
 			->method('findAvailableLanguages')
-			->willReturnCallback(function ($app) use ($availableLanguages) {
+			->willReturnCallback(function($app) use ($availableLanguages) {
 				return $availableLanguages;
 			});
 
@@ -729,7 +729,7 @@ 
 			]);
 
 		$result = $this->invokePrivate($factory, 'getLanguages');
-		$commonLanguagesCodes = array_map(function ($lang) {
+		$commonLanguagesCodes = array_map(function($lang) {
 			return $lang['code'];
 		}, $result['commonLanguages']);
 
@@ -738,9 +738,9 @@ 
 
 	public static function languageIteratorRequestProvider(): array {
 		return [
-			[ true, true],
-			[ false, true],
-			[ false, false],
+			[true, true],
+			[false, true],
+			[false, false],
 		];
 	}
 

tests/lib/Updater/ChangesCheckTest.php

Spacing +7 added lines, -7 removed lines

@@ -104,7 +104,7 @@ 
 
 	public static function changesXMLProvider(): array {
 		return [
-			[ # 0 - full example
+			[# 0 - full example
 				'<?xml version="1.0" encoding="utf-8" ?>
 <release xmlns:xsi= "http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://updates.nextcloud.com/changelog_server/schema.xsd"
@@ -161,7 +161,7 @@ 
 					],
 				]
 			],
-			[ # 1- admin part not translated
+			[# 1- admin part not translated
 				'<?xml version="1.0" encoding="utf-8" ?>
 <release xmlns:xsi= "http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://updates.nextcloud.com/changelog_server/schema.xsd"
@@ -212,7 +212,7 @@ 
 					],
 				]
 			],
-			[ # 2 - minimal set
+			[# 2 - minimal set
 				'<?xml version="1.0" encoding="utf-8" ?>
 <release xmlns:xsi= "http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://updates.nextcloud.com/changelog_server/schema.xsd"
@@ -240,7 +240,7 @@ 
 					],
 				]
 			],
-			[ # 3 - minimal set (procrastinator edition)
+			[# 3 - minimal set (procrastinator edition)
 				'<?xml version="1.0" encoding="utf-8" ?>
 <release xmlns:xsi= "http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://updates.nextcloud.com/changelog_server/schema.xsd"
@@ -264,7 +264,7 @@ 
 					],
 				]
 			],
-			[ # 4 - empty
+			[# 4 - empty
 				'',
 				[]
 			],
@@ -333,10 +333,10 @@ 
 
 	public static function changeDataProvider():array {
 		$testDataFound = $testDataNotFound = self::versionProvider();
-		array_walk($testDataFound, static function (&$params) {
+		array_walk($testDataFound, static function(&$params) {
 			$params[] = true;
 		});
-		array_walk($testDataNotFound, static function (&$params) {
+		array_walk($testDataNotFound, static function(&$params) {
 			$params[] = false;
 		});
 		return array_merge($testDataFound, $testDataNotFound);

tests/lib/DB/MigrationsTest.php

Spacing +9 added lines, -9 removed lines

@@ -41,8 +41,8 @@ 
  * @package Test\DB
  */
 class MigrationsTest extends \Test\TestCase {
-	private MigrationService|MockObject $migrationService;
-	private MockObject|IDBConnection $db;
+	private MigrationService | MockObject $migrationService;
+	private MockObject | IDBConnection $db;
 	private IAppManager $appManager;
 
 	protected function setUp(): void {
@@ -57,7 +57,7 @@ 
 
 	public function testGetters(): void {
 		$this->assertEquals('testing', $this->migrationService->getApp());
-		$this->assertEquals(\OC::$SERVERROOT . '/apps/testing/lib/Migration', $this->migrationService->getMigrationsDirectory());
+		$this->assertEquals(\OC::$SERVERROOT.'/apps/testing/lib/Migration', $this->migrationService->getMigrationsDirectory());
 		$this->assertEquals('OCA\Testing\Migration', $this->migrationService->getMigrationsNamespace());
 		$this->assertEquals('test_oc_migrations', $this->migrationService->getMigrationsTableName());
 	}
@@ -66,7 +66,7 @@ 
 		$this->migrationService = new MigrationService('core', $this->db);
 
 		$this->assertEquals('core', $this->migrationService->getApp());
-		$this->assertEquals(\OC::$SERVERROOT . '/core/Migrations', $this->migrationService->getMigrationsDirectory());
+		$this->assertEquals(\OC::$SERVERROOT.'/core/Migrations', $this->migrationService->getMigrationsDirectory());
 		$this->assertEquals('OC\Core\Migrations', $this->migrationService->getMigrationsNamespace());
 		$this->assertEquals('test_oc_migrations', $this->migrationService->getMigrationsTableName());
 	}
@@ -234,7 +234,7 @@ 
 		];
 		$this->migrationService->expects($this->exactly(2))
 			->method('executeStep')
-			->willReturnCallback(function () use (&$calls) {
+			->willReturnCallback(function() use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 			});
@@ -350,9 +350,9 @@ 
 	public function testEnsureOracleConstraintsValidWithPrimaryKeyDefault(): void {
 		$defaultName = 'PRIMARY';
 		if ($this->db->getDatabaseProvider() === IDBConnection::PLATFORM_POSTGRES) {
-			$defaultName = \str_repeat('a', 26) . '_' . \str_repeat('b', 30) . '_seq';
+			$defaultName = \str_repeat('a', 26).'_'.\str_repeat('b', 30).'_seq';
 		} elseif ($this->db->getDatabaseProvider() === IDBConnection::PLATFORM_ORACLE) {
-			$defaultName = \str_repeat('a', 26) . '_seq';
+			$defaultName = \str_repeat('a', 26).'_seq';
 		}
 
 		$index = $this->createMock(Index::class);
@@ -431,9 +431,9 @@ 
 
 		$defaultName = 'PRIMARY';
 		if ($this->db->getDatabaseProvider() === IDBConnection::PLATFORM_POSTGRES) {
-			$defaultName = \str_repeat('a', 27) . '_' . \str_repeat('b', 30) . '_seq';
+			$defaultName = \str_repeat('a', 27).'_'.\str_repeat('b', 30).'_seq';
 		} elseif ($this->db->getDatabaseProvider() === IDBConnection::PLATFORM_ORACLE) {
-			$defaultName = \str_repeat('a', 27) . '_seq';
+			$defaultName = \str_repeat('a', 27).'_seq';
 		}
 
 		$index = $this->createMock(Index::class);

tests/lib/User/AvailabilityCoordinatorTest.php

Spacing +4 added lines, -4 removed lines

@@ -26,10 +26,10 @@ 
 	private AvailabilityCoordinator $availabilityCoordinator;
 	private ICacheFactory $cacheFactory;
 	private ICache $cache;
-	private IConfig|MockObject $config;
+	private IConfig | MockObject $config;
 	private AbsenceService $absenceService;
 	private LoggerInterface $logger;
-	private MockObject|TimezoneService $timezoneService;
+	private MockObject | TimezoneService $timezoneService;
 
 	protected function setUp(): void {
 		parent::setUp();
@@ -90,12 +90,12 @@ 
 			->willReturn($absence);
 
 		$calls = [
-			[$user->getUID() . '_timezone', 'Europe/Berlin', 3600],
+			[$user->getUID().'_timezone', 'Europe/Berlin', 3600],
 			[$user->getUID(), '{"id":"420","startDate":1696111200,"endDate":1696802340,"shortMessage":"Vacation","message":"On vacation","replacementUserId":"batman","replacementUserDisplayName":"Bruce Wayne"}', 300],
 		];
 		$this->cache->expects(self::exactly(2))
 			->method('set')
-			->willReturnCallback(static function () use (&$calls): void {
+			->willReturnCallback(static function() use (&$calls): void {
 				$expected = array_shift($calls);
 				self::assertEquals($expected, func_get_args());
 			});

tests/lib/Collaboration/Resources/ProviderManagerTest.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	public function testGetResourceProvidersValidAndInvalidProvider(): void {
 		$this->serverContainer->expects($this->exactly(2))
 			->method('query')
-			->willReturnCallback(function (string $service) {
+			->willReturnCallback(function(string $service) {
 				if ($service === 'InvalidResourceProvider') {
 					throw new QueryException('A meaningful error message');
 				}

tests/Core/Command/Log/FileTest.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 		$this->config->method('getSystemValue')
 			->willReturnMap([
 				['log_type', 'file', 'log_type_value'],
-				['datadirectory', \OC::$SERVERROOT . '/data', '/data/directory/'],
+				['datadirectory', \OC::$SERVERROOT.'/data', '/data/directory/'],
 				['logfile', '/data/directory/nextcloud.log', '/var/log/nextcloud.log'],
 				['log_rotate_size', 100 * 1024 * 1024, 5 * 1024 * 1024],
 			]);
@@ -103,7 +103,7 @@ 
 		];
 		$this->consoleOutput->expects($this->exactly(3))
 			->method('writeln')
-			->willReturnCallback(function (string $message) use (&$calls) {
+			->willReturnCallback(function(string $message) use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected[0], $message);
 			});