nextcloud / server

apps/files_sharing/tests/ExpireSharesJobTest.php

Spacing +10 added lines, -10 removed lines

@@ -92,16 +92,16 @@
 
 	public static function dataExpireLinkShare() {
 		return [
-			[false,   '', false, false],
-			[false,   '',  true, false],
-			[true, 'P1D', false,  true],
-			[true, 'P1D',  true, false],
-			[true, 'P1W', false,  true],
-			[true, 'P1W',  true, false],
-			[true, 'P1M', false,  true],
-			[true, 'P1M',  true, false],
-			[true, 'P1Y', false,  true],
-			[true, 'P1Y',  true, false],
+			[false, '', false, false],
+			[false, '', true, false],
+			[true, 'P1D', false, true],
+			[true, 'P1D', true, false],
+			[true, 'P1W', false, true],
+			[true, 'P1W', true, false],
+			[true, 'P1M', false, true],
+			[true, 'P1M', true, false],
+			[true, 'P1Y', false, true],
+			[true, 'P1Y', true, false],
 		];
 	}
 

tests/lib/Encryption/ManagerTest.php

Spacing +7 added lines, -7 removed lines

@@ -66,7 +66,7 @@ 
 		$em = $this->createMock(IEncryptionModule::class);
 		$em->expects($this->any())->method('getId')->willReturn('id');
 		$em->expects($this->any())->method('getDisplayName')->willReturn('TestDummyModule0');
-		$this->manager->registerEncryptionModule('id', 'TestDummyModule0', function () use ($em) {
+		$this->manager->registerEncryptionModule('id', 'TestDummyModule0', function() use ($em) {
 			return $em;
 		});
 		$this->assertFalse($this->manager->isEnabled());
@@ -124,7 +124,7 @@ 
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->with('core', 'default_encryption_module')
-			->willReturnCallback(function () {
+			->willReturnCallback(function() {
 				global $defaultId;
 				return $defaultId;
 			});
@@ -148,7 +148,7 @@ 
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->with('core', 'default_encryption_module')
-			->willReturnCallback(function () {
+			->willReturnCallback(function() {
 				global $defaultId;
 				return $defaultId;
 			});
@@ -173,7 +173,7 @@ 
 		$this->config->expects($this->any())
 			->method('getAppValue')
 			->with('core', 'default_encryption_module')
-			->willReturnCallback(function () {
+			->willReturnCallback(function() {
 				global $defaultId;
 				return $defaultId;
 			});
@@ -262,12 +262,12 @@ 
 		$encryptionModule = $this->createMock(IEncryptionModule::class);
 		$encryptionModule->expects($this->any())
 			->method('getId')
-			->willReturn('ID' . $id);
+			->willReturn('ID'.$id);
 		$encryptionModule->expects($this->any())
 			->method('getDisplayName')
-			->willReturn('TestDummyModule' . $id);
+			->willReturn('TestDummyModule'.$id);
 		/** @var IEncryptionModule $encryptionModule */
-		$manager->registerEncryptionModule('ID' . $id, 'TestDummyModule' . $id, function () use ($encryptionModule) {
+		$manager->registerEncryptionModule('ID'.$id, 'TestDummyModule'.$id, function() use ($encryptionModule) {
 			return $encryptionModule;
 		});
 	}

tests/lib/Encryption/UpdateTest.php

Spacing +4 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
 	public function testUpdate($path, $isDir, $allFiles, $numberOfFiles): void {
 		$updateMock = $this->getUpdateMock(['getOwnerPath']);
 		$updateMock->expects($this->once())->method('getOwnerPath')
-			->willReturnCallback(fn (OCPFile|Folder $node) => '/user/' . $node->getPath());
+			->willReturnCallback(fn (OCPFile | Folder $node) => '/user/'.$node->getPath());
 
 		$this->encryptionManager->expects($this->once())
 			->method('getEncryptionModule')
@@ -127,7 +127,7 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestPostRename')]
 	public function testPostRename($source, $target): void {
-		$updateMock = $this->getUpdateMock(['update','getOwnerPath']);
+		$updateMock = $this->getUpdateMock(['update', 'getOwnerPath']);
 
 		$sourceNode = $this->getFileMock($source, 'user');
 		$targetNode = $this->getFileMock($target, 'user');
@@ -137,7 +137,7 @@ 
 			$updateMock->expects($this->never())->method('update');
 		} else {
 			$updateMock->expects($this->once())->method('update')
-				->willReturnCallback(fn (OCPFile|Folder $node) => $this->assertSame(
+				->willReturnCallback(fn (OCPFile | Folder $node) => $this->assertSame(
 					$target,
 					$node->getPath(),
 					'update needs to be executed for the target destination'
@@ -159,7 +159,7 @@ 
 		$updateMock = $this->getUpdateMock(['update']);
 
 		$updateMock->expects($this->once())->method('update')
-			->willReturnCallback(fn (OCPFile|Folder $node) => $this->assertSame(
+			->willReturnCallback(fn (OCPFile | Folder $node) => $this->assertSame(
 				'/folder/test.txt',
 				$node->getPath(),
 				'update needs to be executed for the target destination'

tests/lib/MemoryInfoTest.php

Spacing +11 added lines, -11 removed lines

@@ -39,11 +39,11 @@ 
 
 	public static function getMemoryLimitTestData(): array {
 		return [
-			'unlimited' => ['-1', -1,],
-			'524288000 bytes' => ['524288000', 524288000,],
-			'500M' => ['500M', 524288000,],
-			'512000K' => ['512000K', 524288000,],
-			'2G' => ['2G', 2147483648,],
+			'unlimited' => ['-1', -1, ],
+			'524288000 bytes' => ['524288000', 524288000, ],
+			'500M' => ['500M', 524288000, ],
+			'512000K' => ['512000K', 524288000, ],
+			'2G' => ['2G', 2147483648, ],
 		];
 	}
 
@@ -54,7 +54,7 @@ 
 	 * @param int|float $expected The expected detected memory limit.
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('getMemoryLimitTestData')]
-	public function testMemoryLimit(string $iniValue, int|float $expected): void {
+	public function testMemoryLimit(string $iniValue, int | float $expected): void {
 		ini_set('memory_limit', $iniValue);
 		$memoryInfo = new MemoryInfo();
 		self::assertEquals($expected, $memoryInfo->getMemoryLimit());
@@ -62,10 +62,10 @@ 
 
 	public static function getSufficientMemoryTestData(): array {
 		return [
-			'unlimited' => [-1, true,],
-			'512M' => [512 * 1024 * 1024, true,],
-			'1G' => [1024 * 1024 * 1024, true,],
-			'256M' => [256 * 1024 * 1024, false,],
+			'unlimited' => [-1, true, ],
+			'512M' => [512 * 1024 * 1024, true, ],
+			'1G' => [1024 * 1024 * 1024, true, ],
+			'256M' => [256 * 1024 * 1024, false, ],
 		];
 	}
 
@@ -79,7 +79,7 @@ 
 	public function testIsMemoryLimitSufficient(int $memoryLimit, bool $expected): void {
 		/* @var MemoryInfo|MockObject $memoryInfo */
 		$memoryInfo = $this->getMockBuilder(MemoryInfo::class)
-			->onlyMethods(['getMemoryLimit',])
+			->onlyMethods(['getMemoryLimit', ])
 			->getMock();
 
 		$memoryInfo

tests/lib/Log/LogFactoryTest.php

Spacing +5 added lines, -5 removed lines

@@ -64,8 +64,8 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('fileTypeProvider')]
 	public function testFile(string $type): void {
-		$datadir = \OC::$SERVERROOT . '/data';
-		$defaultLog = $datadir . '/nextcloud.log';
+		$datadir = \OC::$SERVERROOT.'/data';
+		$defaultLog = $datadir.'/nextcloud.log';
 
 		$this->systemConfig->expects($this->exactly(3))
 			->method('getValue')
@@ -87,7 +87,7 @@ 
 			],
 			[
 				'/xdev/youshallfallback',
-				\OC::$SERVERROOT . '/data/nextcloud.log'
+				\OC::$SERVERROOT.'/data/nextcloud.log'
 			]
 		];
 	}
@@ -97,8 +97,8 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('logFilePathProvider')]
 	public function testFileCustomPath($path, $expected): void {
-		$datadir = \OC::$SERVERROOT . '/data';
-		$defaultLog = $datadir . '/nextcloud.log';
+		$datadir = \OC::$SERVERROOT.'/data';
+		$defaultLog = $datadir.'/nextcloud.log';
 
 		$this->systemConfig->expects($this->exactly(3))
 			->method('getValue')

tests/lib/Files/Stream/EncryptionTest.php

Spacing +5 added lines, -5 removed lines

@@ -128,7 +128,7 @@ 
 			$fileMock->expects($this->never())->method('getAccessList');
 		} else {
 			$fileMock->expects($this->once())->method('getAccessList')
-				->willReturnCallback(function ($sharePath) use ($expectedSharePath) {
+				->willReturnCallback(function($sharePath) use ($expectedSharePath) {
 					$this->assertSame($expectedSharePath, $sharePath);
 					return [];
 				});
@@ -277,7 +277,7 @@ 
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataFilesProvider')]
 	public function testWriteReadBigFile($testFile): void {
-		$expectedData = file_get_contents(\OC::$SERVERROOT . '/tests/data/' . $testFile);
+		$expectedData = file_get_contents(\OC::$SERVERROOT.'/tests/data/'.$testFile);
 		// write it
 		$fileName = tempnam('/tmp', 'FOO');
 		$stream = $this->getStream($fileName, 'w+', 0, self::DEFAULT_WRAPPER, strlen($expectedData));
@@ -320,7 +320,7 @@ 
 			->method('parentStreamSeek')
 			->willReturn(false);
 
-		$expectedData = file_get_contents(\OC::$SERVERROOT . '/tests/data/' . $testFile);
+		$expectedData = file_get_contents(\OC::$SERVERROOT.'/tests/data/'.$testFile);
 		// write it
 		$fileName = tempnam('/tmp', 'FOO');
 		$stream = $this->getStream($fileName, 'w+', 0, '\Test\Files\Stream\DummyEncryptionWrapper', strlen($expectedData));
@@ -363,7 +363,7 @@ 
 		$encryptionModule->expects($this->any())->method('end')->willReturn('');
 		$encryptionModule->expects($this->any())->method('isReadable')->willReturn(true);
 		$encryptionModule->expects($this->any())->method('needDetailedAccessList')->willReturn(false);
-		$encryptionModule->expects($this->any())->method('encrypt')->willReturnCallback(function ($data) {
+		$encryptionModule->expects($this->any())->method('encrypt')->willReturnCallback(function($data) {
 			// simulate different block size by adding some padding to the data
 			if (isset($data[6125])) {
 				return str_pad($data, 8192, 'X');
@@ -371,7 +371,7 @@ 
 			// last block
 			return $data;
 		});
-		$encryptionModule->expects($this->any())->method('decrypt')->willReturnCallback(function ($data) {
+		$encryptionModule->expects($this->any())->method('decrypt')->willReturnCallback(function($data) {
 			if (isset($data[8191])) {
 				return substr($data, 0, 6126);
 			}

tests/lib/Files/Type/DetectionTest.php

Spacing +9 added lines, -9 removed lines

@@ -22,8 +22,8 @@ 
 		$this->detection = new Detection(
 			Server::get(IURLGenerator::class),
 			Server::get(LoggerInterface::class),
-			\OC::$SERVERROOT . '/config/',
-			\OC::$SERVERROOT . '/resources/config/'
+			\OC::$SERVERROOT.'/config/',
+			\OC::$SERVERROOT.'/resources/config/'
 		);
 	}
 
@@ -73,7 +73,7 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataDetectContent')]
 	public function testDetectContent(string $path, string $expected): void {
-		$this->assertEquals($expected, $this->detection->detectContent(\OC::$SERVERROOT . '/tests/data' . $path));
+		$this->assertEquals($expected, $this->detection->detectContent(\OC::$SERVERROOT.'/tests/data'.$path));
 	}
 
 	public static function dataDetect(): array {
@@ -93,7 +93,7 @@ 
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataDetect')]
 	public function testDetect(string $path, string $expected): void {
-		$this->assertEquals($expected, $this->detection->detect(\OC::$SERVERROOT . '/tests/data' . $path));
+		$this->assertEquals($expected, $this->detection->detect(\OC::$SERVERROOT.'/tests/data'.$path));
 	}
 
 	public function testDetectString(): void {
@@ -118,7 +118,7 @@ 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataMimeTypeCustom')]
 	public function testDetectMimeTypeCustom(string $ext, string $mime): void {
 		$confDir = sys_get_temp_dir();
-		file_put_contents($confDir . '/mimetypemapping.dist.json', json_encode([]));
+		file_put_contents($confDir.'/mimetypemapping.dist.json', json_encode([]));
 
 		/** @var IURLGenerator $urlGenerator */
 		$urlGenerator = $this->getMockBuilder(IURLGenerator::class)
@@ -129,12 +129,12 @@ 
 		$logger = $this->createMock(LoggerInterface::class);
 
 		// Create new mapping file
-		file_put_contents($confDir . '/mimetypemapping.dist.json', json_encode([$ext => [$mime]]));
+		file_put_contents($confDir.'/mimetypemapping.dist.json', json_encode([$ext => [$mime]]));
 
 		$detection = new Detection($urlGenerator, $logger, $confDir, $confDir);
 		$mappings = $detection->getAllMappings();
 		$this->assertArrayHasKey($ext, $mappings);
-		$this->assertEquals($mime, $detection->detectPath('foo.' . $ext));
+		$this->assertEquals($mime, $detection->detectPath('foo.'.$ext));
 	}
 
 	public static function dataGetSecureMimeType(): array {
@@ -264,7 +264,7 @@ 
 		$urlGenerator->expects($this->exactly(2))
 			->method('imagePath')
 			->willReturnCallback(
-				function ($appName, $file) use (&$calls) {
+				function($appName, $file) use (&$calls) {
 					$expected = array_shift($calls);
 					$this->assertEquals($expected, [$appName, $file]);
 					if ($file === 'filetypes/my.png') {
@@ -297,7 +297,7 @@ 
 		$urlGenerator->expects($this->exactly(3))
 			->method('imagePath')
 			->willReturnCallback(
-				function ($appName, $file) use (&$calls) {
+				function($appName, $file) use (&$calls) {
 					$expected = array_shift($calls);
 					$this->assertEquals($expected, [$appName, $file]);
 					if ($file === 'filetypes/file.png') {

tests/lib/Files/Storage/Storage.php

Spacing +39 added lines, -39 removed lines

@@ -53,17 +53,17 @@ 
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('directoryProvider')]
 	public function testDirectories($directory): void {
-		$this->assertFalse($this->instance->file_exists('/' . $directory));
+		$this->assertFalse($this->instance->file_exists('/'.$directory));
 
-		$this->assertTrue($this->instance->mkdir('/' . $directory));
+		$this->assertTrue($this->instance->mkdir('/'.$directory));
 
-		$this->assertTrue($this->instance->file_exists('/' . $directory));
-		$this->assertTrue($this->instance->is_dir('/' . $directory));
-		$this->assertFalse($this->instance->is_file('/' . $directory));
-		$this->assertEquals('dir', $this->instance->filetype('/' . $directory));
-		$this->assertEquals(0, $this->instance->filesize('/' . $directory));
-		$this->assertTrue($this->instance->isReadable('/' . $directory));
-		$this->assertTrue($this->instance->isUpdatable('/' . $directory));
+		$this->assertTrue($this->instance->file_exists('/'.$directory));
+		$this->assertTrue($this->instance->is_dir('/'.$directory));
+		$this->assertFalse($this->instance->is_file('/'.$directory));
+		$this->assertEquals('dir', $this->instance->filetype('/'.$directory));
+		$this->assertEquals(0, $this->instance->filesize('/'.$directory));
+		$this->assertTrue($this->instance->isReadable('/'.$directory));
+		$this->assertTrue($this->instance->isUpdatable('/'.$directory));
 
 		$dh = $this->instance->opendir('/');
 		$content = [];
@@ -90,13 +90,13 @@ 
 			'permissions' => $this->instance->getPermissions($directory),
 		], $dirEntry);
 
-		$this->assertFalse($this->instance->mkdir('/' . $directory)); //can't create existing folders
-		$this->assertTrue($this->instance->rmdir('/' . $directory));
+		$this->assertFalse($this->instance->mkdir('/'.$directory)); //can't create existing folders
+		$this->assertTrue($this->instance->rmdir('/'.$directory));
 
 		$this->wait();
-		$this->assertFalse($this->instance->file_exists('/' . $directory));
+		$this->assertFalse($this->instance->file_exists('/'.$directory));
 
-		$this->assertFalse($this->instance->rmdir('/' . $directory)); //can't remove non existing folders
+		$this->assertFalse($this->instance->rmdir('/'.$directory)); //can't remove non existing folders
 
 		$dh = $this->instance->opendir('/');
 		$content = [];
@@ -131,12 +131,12 @@ 
 	}
 
 	public static function loremFileProvider(): array {
-		$root = \OC::$SERVERROOT . '/tests/data/';
+		$root = \OC::$SERVERROOT.'/tests/data/';
 		return [
 			// small file
-			[$root . 'lorem.txt'],
+			[$root.'lorem.txt'],
 			// bigger file (> 8 KB which is the standard PHP block size)
-			[$root . 'lorem-big.txt']
+			[$root.'lorem-big.txt']
 		];
 	}
 
@@ -164,15 +164,15 @@ 
 		$this->assertEquals('httpd/unix-directory', $this->instance->getMimeType('/'));
 		$this->assertEquals(false, $this->instance->getMimeType('/non/existing/file'));
 
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->instance->file_put_contents('/lorem.txt', file_get_contents($textFile, 'r'));
 		$this->assertEquals('text/plain', $this->instance->getMimeType('/lorem.txt'));
 
-		$pngFile = \OC::$SERVERROOT . '/tests/data/desktopapp.png';
+		$pngFile = \OC::$SERVERROOT.'/tests/data/desktopapp.png';
 		$this->instance->file_put_contents('/desktopapp.png', file_get_contents($pngFile, 'r'));
 		$this->assertEquals('image/png', $this->instance->getMimeType('/desktopapp.png'));
 
-		$svgFile = \OC::$SERVERROOT . '/tests/data/desktopapp.svg';
+		$svgFile = \OC::$SERVERROOT.'/tests/data/desktopapp.svg';
 		$this->instance->file_put_contents('/desktopapp.svg', file_get_contents($svgFile, 'r'));
 		$this->assertEquals('image/svg+xml', $this->instance->getMimeType('/desktopapp.svg'));
 	}
@@ -192,7 +192,7 @@ 
 	}
 
 	public function initSourceAndTarget($source, $target = null) {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->instance->file_put_contents($source, file_get_contents($textFile));
 		if ($target) {
 			$testContents = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ';
@@ -201,11 +201,11 @@ 
 	}
 
 	public function assertSameAsLorem($file) {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->assertEquals(
 			file_get_contents($textFile),
 			$this->instance->file_get_contents($file),
-			'Expected ' . $file . ' to be a copy of ' . $textFile
+			'Expected '.$file.' to be a copy of '.$textFile
 		);
 	}
 
@@ -215,9 +215,9 @@ 
 
 		$this->instance->copy($source, $target);
 
-		$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
+		$this->assertTrue($this->instance->file_exists($target), $target.' was not created');
 		$this->assertSameAsLorem($target);
-		$this->assertTrue($this->instance->file_exists($source), $source . ' was deleted');
+		$this->assertTrue($this->instance->file_exists($source), $source.' was deleted');
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('copyAndMoveProvider')]
@@ -227,8 +227,8 @@ 
 		$this->instance->rename($source, $target);
 
 		$this->wait();
-		$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
-		$this->assertFalse($this->instance->file_exists($source), $source . ' still exists');
+		$this->assertTrue($this->instance->file_exists($target), $target.' was not created');
+		$this->assertFalse($this->instance->file_exists($source), $source.' still exists');
 		$this->assertSameAsLorem($target);
 	}
 
@@ -238,8 +238,8 @@ 
 
 		$this->instance->copy($source, $target);
 
-		$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
-		$this->assertTrue($this->instance->file_exists($source), $source . ' was deleted');
+		$this->assertTrue($this->instance->file_exists($target), $target.' was not created');
+		$this->assertTrue($this->instance->file_exists($source), $source.' was deleted');
 		$this->assertSameAsLorem($target);
 		$this->assertSameAsLorem($source);
 	}
@@ -250,13 +250,13 @@ 
 
 		$this->instance->rename($source, $target);
 
-		$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
-		$this->assertFalse($this->instance->file_exists($source), $source . ' still exists');
+		$this->assertTrue($this->instance->file_exists($target), $target.' was not created');
+		$this->assertFalse($this->instance->file_exists($source), $source.' still exists');
 		$this->assertSameAsLorem($target);
 	}
 
 	public function testLocal(): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->instance->file_put_contents('/lorem.txt', file_get_contents($textFile));
 		$localFile = $this->instance->getLocalFile('/lorem.txt');
 		$this->assertTrue(file_exists($localFile));
@@ -283,7 +283,7 @@ 
 	}
 
 	public function testStat(): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$ctimeStart = time();
 		$this->instance->file_put_contents('/lorem.txt', file_get_contents($textFile));
 		$this->assertTrue($this->instance->isReadable('/lorem.txt'));
@@ -321,7 +321,7 @@ 
 		if ($this->instance instanceof Wrapper) {
 			$this->markTestSkipped('Cannot test update check on wrappers');
 		}
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$watcher = $this->instance->getWatcher();
 		$watcher->setPolicy(Watcher::CHECK_ALWAYS);
 		$this->instance->file_put_contents('/lorem.txt', file_get_contents($textFile));
@@ -330,7 +330,7 @@ 
 	}
 
 	public function testUnlink(): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->instance->file_put_contents('/lorem.txt', file_get_contents($textFile));
 
 		$this->assertTrue($this->instance->file_exists('/lorem.txt'));
@@ -343,7 +343,7 @@ 
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('fileNameProvider')]
 	public function testFOpen($fileName): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 
 		$fh = @$this->instance->fopen($fileName, 'r');
 		if ($fh) {
@@ -571,9 +571,9 @@ 
 
 		$this->instance->copyFromStorage($this->instance, $source, $target);
 
-		$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
+		$this->assertTrue($this->instance->file_exists($target), $target.' was not created');
 		$this->assertSameAsLorem($target);
-		$this->assertTrue($this->instance->file_exists($source), $source . ' was deleted');
+		$this->assertTrue($this->instance->file_exists($source), $source.' was deleted');
 	}
 
 	public function testIsCreatable(): void {
@@ -621,7 +621,7 @@ 
 	}
 
 	public function testWriteStream(): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 
 		if (!$this->instance->instanceOfStorage(IWriteStreamStorage::class)) {
 			$this->markTestSkipped('Not a WriteSteamStorage');
@@ -638,7 +638,7 @@ 
 	}
 
 	public function testFseekSize(): void {
-		$textFile = \OC::$SERVERROOT . '/tests/data/lorem.txt';
+		$textFile = \OC::$SERVERROOT.'/tests/data/lorem.txt';
 		$this->instance->file_put_contents('bar.txt', file_get_contents($textFile));
 
 		$size = $this->instance->filesize('bar.txt');

tests/lib/Files/Storage/Wrapper/EncodingTest.php

Spacing +48 added lines, -48 removed lines

@@ -41,13 +41,13 @@ 
 
 	public static function fileNameProvider(): array {
 		$a = parent::fileNameProvider();
-		$a[] = [self::NFD_NAME . '.txt'];
+		$a[] = [self::NFD_NAME.'.txt'];
 		return $a;
 	}
 
 	public static function copyAndMoveProvider(): array {
 		$a = parent::copyAndMoveProvider();
-		$a[] = [self::NFD_NAME . '.txt', self::NFC_NAME . '-renamed.txt'];
+		$a[] = [self::NFD_NAME.'.txt', self::NFC_NAME.'-renamed.txt'];
 		return $a;
 	}
 
@@ -107,27 +107,27 @@ 
 	public static function encodedDirectoriesProvider(): array {
 		return [
 			[self::NFD_NAME, self::NFC_NAME],
-			[self::NFD_NAME . '/' . self::NFD_NAME, self::NFC_NAME . '/' . self::NFC_NAME],
-			[self::NFD_NAME . '/' . self::NFC_NAME . '/' . self::NFD_NAME, self::NFC_NAME . '/' . self::NFC_NAME . '/' . self::NFC_NAME],
+			[self::NFD_NAME.'/'.self::NFD_NAME, self::NFC_NAME.'/'.self::NFC_NAME],
+			[self::NFD_NAME.'/'.self::NFC_NAME.'/'.self::NFD_NAME, self::NFC_NAME.'/'.self::NFC_NAME.'/'.self::NFC_NAME],
 		];
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('encodedDirectoriesProvider')]
 	public function testOperationInsideDirectory($sourceDir, $accessDir): void {
 		$this->sourceStorage->mkdir($sourceDir);
-		$this->instance->file_put_contents($accessDir . '/test.txt', 'bar');
-		$this->assertTrue($this->instance->file_exists($accessDir . '/test.txt'));
-		$this->assertEquals('bar', $this->instance->file_get_contents($accessDir . '/test.txt'));
+		$this->instance->file_put_contents($accessDir.'/test.txt', 'bar');
+		$this->assertTrue($this->instance->file_exists($accessDir.'/test.txt'));
+		$this->assertEquals('bar', $this->instance->file_get_contents($accessDir.'/test.txt'));
 
-		$this->sourceStorage->file_put_contents($sourceDir . '/' . self::NFD_NAME, 'foo');
-		$this->assertTrue($this->instance->file_exists($accessDir . '/' . self::NFC_NAME));
-		$this->assertEquals('foo', $this->instance->file_get_contents($accessDir . '/' . self::NFC_NAME));
+		$this->sourceStorage->file_put_contents($sourceDir.'/'.self::NFD_NAME, 'foo');
+		$this->assertTrue($this->instance->file_exists($accessDir.'/'.self::NFC_NAME));
+		$this->assertEquals('foo', $this->instance->file_get_contents($accessDir.'/'.self::NFC_NAME));
 
 		// try again to make it use cached path
-		$this->assertEquals('bar', $this->instance->file_get_contents($accessDir . '/test.txt'));
-		$this->assertTrue($this->instance->file_exists($accessDir . '/test.txt'));
-		$this->assertEquals('foo', $this->instance->file_get_contents($accessDir . '/' . self::NFC_NAME));
-		$this->assertTrue($this->instance->file_exists($accessDir . '/' . self::NFC_NAME));
+		$this->assertEquals('bar', $this->instance->file_get_contents($accessDir.'/test.txt'));
+		$this->assertTrue($this->instance->file_exists($accessDir.'/test.txt'));
+		$this->assertEquals('foo', $this->instance->file_get_contents($accessDir.'/'.self::NFC_NAME));
+		$this->assertTrue($this->instance->file_exists($accessDir.'/'.self::NFC_NAME));
 	}
 
 	public function testCacheExtraSlash(): void {
@@ -135,19 +135,19 @@ 
 		$this->assertEquals(3, $this->instance->file_put_contents(self::NFC_NAME, 'bar'));
 		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME));
 		clearstatcache();
-		$this->assertEquals(5, $this->instance->file_put_contents('/' . self::NFC_NAME, 'baric'));
+		$this->assertEquals(5, $this->instance->file_put_contents('/'.self::NFC_NAME, 'baric'));
 		$this->assertEquals('baric', $this->instance->file_get_contents(self::NFC_NAME));
 		clearstatcache();
-		$this->assertEquals(8, $this->instance->file_put_contents('/' . self::NFC_NAME, 'barbaric'));
-		$this->assertEquals('barbaric', $this->instance->file_get_contents('//' . self::NFC_NAME));
+		$this->assertEquals(8, $this->instance->file_put_contents('/'.self::NFC_NAME, 'barbaric'));
+		$this->assertEquals('barbaric', $this->instance->file_get_contents('//'.self::NFC_NAME));
 	}
 
 	public static function sourceAndTargetDirectoryProvider(): array {
 		return [
-			[self::NFC_NAME . '1', self::NFC_NAME . '2'],
-			[self::NFD_NAME . '1', self::NFC_NAME . '2'],
-			[self::NFC_NAME . '1', self::NFD_NAME . '2'],
-			[self::NFD_NAME . '1', self::NFD_NAME . '2'],
+			[self::NFC_NAME.'1', self::NFC_NAME.'2'],
+			[self::NFD_NAME.'1', self::NFC_NAME.'2'],
+			[self::NFC_NAME.'1', self::NFD_NAME.'2'],
+			[self::NFD_NAME.'1', self::NFD_NAME.'2'],
 		];
 	}
 
@@ -155,44 +155,44 @@ 
 	public function testCopyAndMoveEncodedFolder($sourceDir, $targetDir): void {
 		$this->sourceStorage->mkdir($sourceDir);
 		$this->sourceStorage->mkdir($targetDir);
-		$this->sourceStorage->file_put_contents($sourceDir . '/test.txt', 'bar');
-		$this->assertTrue($this->instance->copy(self::NFC_NAME . '1/test.txt', self::NFC_NAME . '2/test.txt'));
+		$this->sourceStorage->file_put_contents($sourceDir.'/test.txt', 'bar');
+		$this->assertTrue($this->instance->copy(self::NFC_NAME.'1/test.txt', self::NFC_NAME.'2/test.txt'));
 
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '1/test.txt'));
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '2/test.txt'));
-		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME . '2/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'1/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'2/test.txt'));
+		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME.'2/test.txt'));
 
-		$this->assertTrue($this->instance->rename(self::NFC_NAME . '1/test.txt', self::NFC_NAME . '2/test2.txt'));
-		$this->assertFalse($this->instance->file_exists(self::NFC_NAME . '1/test.txt'));
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '2/test2.txt'));
+		$this->assertTrue($this->instance->rename(self::NFC_NAME.'1/test.txt', self::NFC_NAME.'2/test2.txt'));
+		$this->assertFalse($this->instance->file_exists(self::NFC_NAME.'1/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'2/test2.txt'));
 
-		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME . '2/test2.txt'));
+		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME.'2/test2.txt'));
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('sourceAndTargetDirectoryProvider')]
 	public function testCopyAndMoveFromStorageEncodedFolder($sourceDir, $targetDir): void {
 		$this->sourceStorage->mkdir($sourceDir);
 		$this->sourceStorage->mkdir($targetDir);
-		$this->sourceStorage->file_put_contents($sourceDir . '/test.txt', 'bar');
-		$this->assertTrue($this->instance->copyFromStorage($this->instance, self::NFC_NAME . '1/test.txt', self::NFC_NAME . '2/test.txt'));
+		$this->sourceStorage->file_put_contents($sourceDir.'/test.txt', 'bar');
+		$this->assertTrue($this->instance->copyFromStorage($this->instance, self::NFC_NAME.'1/test.txt', self::NFC_NAME.'2/test.txt'));
 
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '1/test.txt'));
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '2/test.txt'));
-		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME . '2/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'1/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'2/test.txt'));
+		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME.'2/test.txt'));
 
-		$this->assertTrue($this->instance->moveFromStorage($this->instance, self::NFC_NAME . '1/test.txt', self::NFC_NAME . '2/test2.txt'));
-		$this->assertFalse($this->instance->file_exists(self::NFC_NAME . '1/test.txt'));
-		$this->assertTrue($this->instance->file_exists(self::NFC_NAME . '2/test2.txt'));
+		$this->assertTrue($this->instance->moveFromStorage($this->instance, self::NFC_NAME.'1/test.txt', self::NFC_NAME.'2/test2.txt'));
+		$this->assertFalse($this->instance->file_exists(self::NFC_NAME.'1/test.txt'));
+		$this->assertTrue($this->instance->file_exists(self::NFC_NAME.'2/test2.txt'));
 
-		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME . '2/test2.txt'));
+		$this->assertEquals('bar', $this->instance->file_get_contents(self::NFC_NAME.'2/test2.txt'));
 	}
 
 	public function testNormalizedDirectoryEntriesOpenDir(): void {
 		$this->sourceStorage->mkdir('/test');
-		$this->sourceStorage->mkdir('/test/' . self::NFD_NAME);
+		$this->sourceStorage->mkdir('/test/'.self::NFD_NAME);
 
-		$this->assertTrue($this->instance->file_exists('/test/' . self::NFC_NAME));
-		$this->assertTrue($this->instance->file_exists('/test/' . self::NFD_NAME));
+		$this->assertTrue($this->instance->file_exists('/test/'.self::NFC_NAME));
+		$this->assertTrue($this->instance->file_exists('/test/'.self::NFD_NAME));
 
 		$dh = $this->instance->opendir('/test');
 		$content = [];
@@ -208,10 +208,10 @@ 
 
 	public function testNormalizedDirectoryEntriesGetDirectoryContent(): void {
 		$this->sourceStorage->mkdir('/test');
-		$this->sourceStorage->mkdir('/test/' . self::NFD_NAME);
+		$this->sourceStorage->mkdir('/test/'.self::NFD_NAME);
 
-		$this->assertTrue($this->instance->file_exists('/test/' . self::NFC_NAME));
-		$this->assertTrue($this->instance->file_exists('/test/' . self::NFD_NAME));
+		$this->assertTrue($this->instance->file_exists('/test/'.self::NFC_NAME));
+		$this->assertTrue($this->instance->file_exists('/test/'.self::NFD_NAME));
 
 		$content = iterator_to_array($this->instance->getDirectoryContent('/test'));
 		$this->assertCount(1, $content);
@@ -220,12 +220,12 @@ 
 
 	public function testNormalizedGetMetaData(): void {
 		$this->sourceStorage->mkdir('/test');
-		$this->sourceStorage->mkdir('/test/' . self::NFD_NAME);
+		$this->sourceStorage->mkdir('/test/'.self::NFD_NAME);
 
-		$entry = $this->instance->getMetaData('/test/' . self::NFC_NAME);
+		$entry = $this->instance->getMetaData('/test/'.self::NFC_NAME);
 		$this->assertEquals(self::NFC_NAME, $entry['name']);
 
-		$entry = $this->instance->getMetaData('/test/' . self::NFD_NAME);
+		$entry = $this->instance->getMetaData('/test/'.self::NFD_NAME);
 		$this->assertEquals(self::NFC_NAME, $entry['name']);
 	}