nextcloud / server

tests/lib/BinaryFinderTest.php

Indentation +64 added lines, -64 removed lines

@@ -17,76 +17,76 @@
 use OCP\Server;
 
 class BinaryFinderTest extends TestCase {
-	private ICache $cache;
-	private ICacheFactory $cacheFactory;
-	private $oldEnv;
+    private ICache $cache;
+    private ICacheFactory $cacheFactory;
+    private $oldEnv;
 
-	protected function setUp(): void {
-		$this->oldEnv = getenv('PATH');
-		// BinaryFinder always includes the "PATH" environment variable into the search path,
-		// which we want to avoid in this test because they are not usually found in webserver
-		// deployments.
-		putenv('PATH=""');
-		$this->cacheFactory = $this->createMock(ICacheFactory::class);
-		$this->cache = new ArrayCache();
-		$this->cacheFactory->method('createLocal')->with('findBinaryPath')->willReturn($this->cache);
-	}
+    protected function setUp(): void {
+        $this->oldEnv = getenv('PATH');
+        // BinaryFinder always includes the "PATH" environment variable into the search path,
+        // which we want to avoid in this test because they are not usually found in webserver
+        // deployments.
+        putenv('PATH=""');
+        $this->cacheFactory = $this->createMock(ICacheFactory::class);
+        $this->cache = new ArrayCache();
+        $this->cacheFactory->method('createLocal')->with('findBinaryPath')->willReturn($this->cache);
+    }
 
-	protected function tearDown(): void {
-		putenv('PATH=' . $this->oldEnv);
-	}
+    protected function tearDown(): void {
+        putenv('PATH=' . $this->oldEnv);
+    }
 
-	public function testDefaultFindsCat() {
-		$config = $this->createMock(IConfig::class);
-		$config
-			->method('getSystemValue')
-			->with('binary_search_paths', $this->anything())
-			->willReturnCallback(function ($key, $default) {
-				return $default;
-			});
-		$finder = new BinaryFinder($this->cacheFactory, $config);
-		$this->assertStringEndsWith('/cat', $finder->findBinaryPath('cat'));
-		$this->assertStringEndsWith('/cat', $this->cache->get('cat'));
-	}
+    public function testDefaultFindsCat() {
+        $config = $this->createMock(IConfig::class);
+        $config
+            ->method('getSystemValue')
+            ->with('binary_search_paths', $this->anything())
+            ->willReturnCallback(function ($key, $default) {
+                return $default;
+            });
+        $finder = new BinaryFinder($this->cacheFactory, $config);
+        $this->assertStringEndsWith('/cat', $finder->findBinaryPath('cat'));
+        $this->assertStringEndsWith('/cat', $this->cache->get('cat'));
+    }
 
-	public function testDefaultDoesNotFindCata() {
-		$config = $this->createMock(IConfig::class);
-		$config
-			->method('getSystemValue')
-			->with('binary_search_paths', $this->anything())
-			->willReturnCallback(function ($key, $default) {
-				return $default;
-			});
-		$finder = new BinaryFinder($this->cacheFactory, $config);
-		$this->assertFalse($finder->findBinaryPath('cata'));
-		$this->assertFalse($this->cache->get('cata'));
-	}
+    public function testDefaultDoesNotFindCata() {
+        $config = $this->createMock(IConfig::class);
+        $config
+            ->method('getSystemValue')
+            ->with('binary_search_paths', $this->anything())
+            ->willReturnCallback(function ($key, $default) {
+                return $default;
+            });
+        $finder = new BinaryFinder($this->cacheFactory, $config);
+        $this->assertFalse($finder->findBinaryPath('cata'));
+        $this->assertFalse($this->cache->get('cata'));
+    }
 
-	public function testCustomPathFindsCat() {
-		$tmpdir = Server::get(ITempManager::class)->getTemporaryFolder();
-		touch($tmpdir . '/cat');
-		chmod($tmpdir . '/cat', 100);
+    public function testCustomPathFindsCat() {
+        $tmpdir = Server::get(ITempManager::class)->getTemporaryFolder();
+        touch($tmpdir . '/cat');
+        chmod($tmpdir . '/cat', 100);
 
-		$config = $this->createMock(IConfig::class);
-		$config
-			->method('getSystemValue')
-			->with('binary_search_paths', $this->anything())
-			->willReturn([$tmpdir]);
-		$finder = new BinaryFinder($this->cacheFactory, $config);
-		$this->assertEquals($tmpdir . '/cat', $finder->findBinaryPath('cat'));
-		$this->assertEquals($tmpdir . '/cat', $this->cache->get('cat'));
-	}
+        $config = $this->createMock(IConfig::class);
+        $config
+            ->method('getSystemValue')
+            ->with('binary_search_paths', $this->anything())
+            ->willReturn([$tmpdir]);
+        $finder = new BinaryFinder($this->cacheFactory, $config);
+        $this->assertEquals($tmpdir . '/cat', $finder->findBinaryPath('cat'));
+        $this->assertEquals($tmpdir . '/cat', $this->cache->get('cat'));
+    }
 
-	public function testWrongCustomPathDoesNotFindCat() {
-		$tmpdir = Server::get(ITempManager::class)->getTemporaryFolder();
+    public function testWrongCustomPathDoesNotFindCat() {
+        $tmpdir = Server::get(ITempManager::class)->getTemporaryFolder();
 
-		$config = $this->createMock(IConfig::class);
-		$config
-			->method('getSystemValue')
-			->with('binary_search_paths')
-			->willReturn([$tmpdir]);
-		$finder = new BinaryFinder($this->cacheFactory, $config);
-		$this->assertFalse($finder->findBinaryPath('cat'));
-		$this->assertFalse($this->cache->get('cat'));
-	}
+        $config = $this->createMock(IConfig::class);
+        $config
+            ->method('getSystemValue')
+            ->with('binary_search_paths')
+            ->willReturn([$tmpdir]);
+        $finder = new BinaryFinder($this->cacheFactory, $config);
+        $this->assertFalse($finder->findBinaryPath('cat'));
+        $this->assertFalse($this->cache->get('cat'));
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 /**
  * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
@@ -33,7 +33,7 @@ 
 	}
 
 	protected function tearDown(): void {
-		putenv('PATH=' . $this->oldEnv);
+		putenv('PATH='.$this->oldEnv);
 	}
 
 	public function testDefaultFindsCat() {
@@ -41,7 +41,7 @@ 
 		$config
 			->method('getSystemValue')
 			->with('binary_search_paths', $this->anything())
-			->willReturnCallback(function ($key, $default) {
+			->willReturnCallback(function($key, $default) {
 				return $default;
 			});
 		$finder = new BinaryFinder($this->cacheFactory, $config);
@@ -54,7 +54,7 @@ 
 		$config
 			->method('getSystemValue')
 			->with('binary_search_paths', $this->anything())
-			->willReturnCallback(function ($key, $default) {
+			->willReturnCallback(function($key, $default) {
 				return $default;
 			});
 		$finder = new BinaryFinder($this->cacheFactory, $config);
@@ -64,8 +64,8 @@ 
 
 	public function testCustomPathFindsCat() {
 		$tmpdir = Server::get(ITempManager::class)->getTemporaryFolder();
-		touch($tmpdir . '/cat');
-		chmod($tmpdir . '/cat', 100);
+		touch($tmpdir.'/cat');
+		chmod($tmpdir.'/cat', 100);
 
 		$config = $this->createMock(IConfig::class);
 		$config
@@ -73,8 +73,8 @@ 
 			->with('binary_search_paths', $this->anything())
 			->willReturn([$tmpdir]);
 		$finder = new BinaryFinder($this->cacheFactory, $config);
-		$this->assertEquals($tmpdir . '/cat', $finder->findBinaryPath('cat'));
-		$this->assertEquals($tmpdir . '/cat', $this->cache->get('cat'));
+		$this->assertEquals($tmpdir.'/cat', $finder->findBinaryPath('cat'));
+		$this->assertEquals($tmpdir.'/cat', $this->cache->get('cat'));
 	}
 
 	public function testWrongCustomPathDoesNotFindCat() {