nextcloud / server

tests/lib/Traits/ClientServiceTrait.php

Indentation +83 added lines, -83 removed lines

@@ -11,95 +11,95 @@
 use OCP\Http\Client\IResponse;
 
 trait ClientServiceTrait {
-	/** @var IClientService|\PHPUnit\Framework\MockObject\MockObject */
-	private $clientService;
-	/** @var IClient|\PHPUnit\Framework\MockObject\MockObject */
-	private $client;
-	private $expectedGetRequests = [];
-	private $expectedPostRequests = [];
+    /** @var IClientService|\PHPUnit\Framework\MockObject\MockObject */
+    private $clientService;
+    /** @var IClient|\PHPUnit\Framework\MockObject\MockObject */
+    private $client;
+    private $expectedGetRequests = [];
+    private $expectedPostRequests = [];
 
-	/**
-	 * Wrapper to be forward compatible to phpunit 5.4+
-	 *
-	 * @param string $originalClassName
-	 * @return \PHPUnit\Framework\MockObject\MockObject
-	 */
-	abstract protected function createMock(string $originalClassName);
+    /**
+     * Wrapper to be forward compatible to phpunit 5.4+
+     *
+     * @param string $originalClassName
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
+    abstract protected function createMock(string $originalClassName);
 
-	/**
-	 * Returns a matcher that matches when the method is executed
-	 * zero or more times.
-	 *
-	 * @return \PHPUnit_Framework_MockObject_Matcher_AnyInvokedCount
-	 *
-	 * @since  Method available since Release 3.0.0
-	 */
-	abstract public static function any();
+    /**
+     * Returns a matcher that matches when the method is executed
+     * zero or more times.
+     *
+     * @return \PHPUnit_Framework_MockObject_Matcher_AnyInvokedCount
+     *
+     * @since  Method available since Release 3.0.0
+     */
+    abstract public static function any();
 
-	protected function setUpClientServiceTrait() {
-		$this->clientService = $this->createMock(IClientService::class);
-		$this->client = $this->createMock(IClient::class);
-		$this->clientService->expects($this->any())
-			->method('newClient')
-			->willReturn($this->client);
-		$this->client->expects($this->any())
-			->method('get')
-			->willReturnCallback(function ($url) {
-				if (!isset($this->expectedGetRequests[$url])) {
-					throw new \Exception('unexpected request: ' . $url);
-				}
-				$result = $this->expectedGetRequests[$url];
+    protected function setUpClientServiceTrait() {
+        $this->clientService = $this->createMock(IClientService::class);
+        $this->client = $this->createMock(IClient::class);
+        $this->clientService->expects($this->any())
+            ->method('newClient')
+            ->willReturn($this->client);
+        $this->client->expects($this->any())
+            ->method('get')
+            ->willReturnCallback(function ($url) {
+                if (!isset($this->expectedGetRequests[$url])) {
+                    throw new \Exception('unexpected request: ' . $url);
+                }
+                $result = $this->expectedGetRequests[$url];
 
-				if ($result instanceof \Exception) {
-					throw $result;
-				} else {
-					$response = $this->createMock(IResponse::class);
-					$response->expects($this->any())
-						->method('getBody')
-						->willReturn($result);
-					return $response;
-				}
-			});
-		$this->client->expects($this->any())
-			->method('post')
-			->willReturnCallback(function ($url) {
-				if (!isset($this->expectedPostRequests[$url])) {
-					throw new \Exception('unexpected request: ' . $url);
-				}
-				$result = $this->expectedPostRequests[$url];
+                if ($result instanceof \Exception) {
+                    throw $result;
+                } else {
+                    $response = $this->createMock(IResponse::class);
+                    $response->expects($this->any())
+                        ->method('getBody')
+                        ->willReturn($result);
+                    return $response;
+                }
+            });
+        $this->client->expects($this->any())
+            ->method('post')
+            ->willReturnCallback(function ($url) {
+                if (!isset($this->expectedPostRequests[$url])) {
+                    throw new \Exception('unexpected request: ' . $url);
+                }
+                $result = $this->expectedPostRequests[$url];
 
-				if ($result instanceof \Exception) {
-					throw $result;
-				} else {
-					$response = $this->createMock(IResponse::class);
-					$response->expects($this->any())
-						->method('getBody')
-						->willReturn($result);
-					return $response;
-				}
-			});
-	}
+                if ($result instanceof \Exception) {
+                    throw $result;
+                } else {
+                    $response = $this->createMock(IResponse::class);
+                    $response->expects($this->any())
+                        ->method('getBody')
+                        ->willReturn($result);
+                    return $response;
+                }
+            });
+    }
 
-	/**
-	 * @param string $url
-	 * @param string|\Exception $result
-	 */
-	protected function expectGetRequest($url, $result) {
-		$this->expectedGetRequests[$url] = $result;
-	}
+    /**
+     * @param string $url
+     * @param string|\Exception $result
+     */
+    protected function expectGetRequest($url, $result) {
+        $this->expectedGetRequests[$url] = $result;
+    }
 
-	/**
-	 * @param string $url
-	 * @param string|\Exception $result
-	 */
-	protected function expectPostRequest($url, $result) {
-		$this->expectedPostRequests[$url] = $result;
-	}
+    /**
+     * @param string $url
+     * @param string|\Exception $result
+     */
+    protected function expectPostRequest($url, $result) {
+        $this->expectedPostRequests[$url] = $result;
+    }
 
-	/**
-	 * @return IClientService|\PHPUnit\Framework\MockObject\MockObject
-	 */
-	protected function getClientService() {
-		return $this->clientService;
-	}
+    /**
+     * @return IClientService|\PHPUnit\Framework\MockObject\MockObject
+     */
+    protected function getClientService() {
+        return $this->clientService;
+    }
 }

tests/lib/Security/FeaturePolicy/AddFeaturePolicyEventTest.php

Indentation +9 added lines, -9 removed lines

@@ -14,15 +14,15 @@
 use Test\TestCase;
 
 class AddFeaturePolicyEventTest extends TestCase {
-	public function testAddEvent(): void {
-		$manager = $this->createMock(FeaturePolicyManager::class);
-		$policy = $this->createMock(FeaturePolicy::class);
-		$event = new AddFeaturePolicyEvent($manager);
+    public function testAddEvent(): void {
+        $manager = $this->createMock(FeaturePolicyManager::class);
+        $policy = $this->createMock(FeaturePolicy::class);
+        $event = new AddFeaturePolicyEvent($manager);
 
-		$manager->expects($this->once())
-			->method('addDefaultPolicy')
-			->with($policy);
+        $manager->expects($this->once())
+            ->method('addDefaultPolicy')
+            ->with($policy);
 
-		$event->addPolicy($policy);
-	}
+        $event->addPolicy($policy);
+    }
 }

tests/lib/Security/IdentityProof/SignerTest.php

Indentation +130 added lines, -130 removed lines

@@ -18,8 +18,8 @@ 
 use Test\TestCase;
 
 class SignerTest extends TestCase {
-	/** @var string */
-	private $private = '-----BEGIN PRIVATE KEY-----
+    /** @var string */
+    private $private = '-----BEGIN PRIVATE KEY-----
 MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQDImc6QvHBjBIoo
 w9nnywiPNESleUuFJ2yQ3Gd/3w2BkblojlUAJAsUd/bQokjOH9d+7nqyzgSiXjRl
 iwKagY6NjcNEEq0X5KOMNwx6uEbtq3+7pA7H2JefNrnAuD+Fhp3Hyo3h1cse6hAq
@@ -48,8 +48,8 @@ 
 kC6HCb+CXsMRD7yp8KrrYnw=
 -----END PRIVATE KEY-----';
 
-	/** @var string */
-	private $public = '-----BEGIN PUBLIC KEY-----
+    /** @var string */
+    private $public = '-----BEGIN PUBLIC KEY-----
 MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAyJnOkLxwYwSKKMPZ58sI
 jzREpXlLhSdskNxnf98NgZG5aI5VACQLFHf20KJIzh/Xfu56ss4Eol40ZYsCmoGO
 jY3DRBKtF+SjjDcMerhG7at/u6QOx9iXnza5wLg/hYadx8qN4dXLHuoQKuma/IWg
@@ -59,130 +59,130 @@ 
 gQIDAQAB
 -----END PUBLIC KEY-----';
 
-	/** @var Key */
-	private $key;
-
-	/** @var Manager|\PHPUnit\Framework\MockObject\MockObject */
-	private $keyManager;
-
-	/** @var ITimeFactory|\PHPUnit\Framework\MockObject\MockObject */
-	private $timeFactory;
-
-	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $userManager;
-
-	/** @var Signer */
-	private $signer;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->key = new Key($this->public, $this->private);
-
-		$this->keyManager = $this->createMock(Manager::class);
-		$this->timeFactory = $this->createMock(ITimeFactory::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-		$this->signer = new Signer(
-			$this->keyManager,
-			$this->timeFactory,
-			$this->userManager
-		);
-	}
-
-	public function testSign(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getCloudId')
-			->willReturn('foo@example.com');
-
-		$this->timeFactory->method('getTime')
-			->willReturn(42);
-
-		$this->keyManager->method('getKey')
-			->with($this->equalTo($user))
-			->willReturn($this->key);
-
-		$data = [
-			'foo' => 'bar',
-			'abc' => 'def',
-			'xyz' => 123,
-		];
-
-		$expects = [
-			'message' => [
-				'data' => $data,
-				'type' => 'myType',
-				'signer' => 'foo@example.com',
-				'timestamp' => 42,
-			],
-			'signature' => 'E1fNdoWMX1QmSyKv+S3FtOgLe9niYGQFWOKGaMLxc2h7s3V++EIqJvw/NCLBfrUowzWkTzhkjfbHaf88Hz34WAn4sAwXYAO8cnboQs6SClKRzQ/nvbtLgd2wm9RQ8UTOM7wR6C7HpIn4qqJ4BTQ1bAwYAiJ2GoK+W8wC0o0Gpub2906j3JJ4cbc9lufd5ohWKCev8Ubem/EEKaRIZA7qHh+Q1MKXTaJQJlCjTMe5PyGy0fsmtVxsPls3/Fkd9sVeHEHSYHzOiF6ttlxWou4TdRbq3WSEVpt1DOOvkKI9w2+zBJ7IPH8CnVpXcdIzWDctUygZKzNMUQnweDOOziEdUw=='
-		];
-
-		$result = $this->signer->sign('myType', $data, $user);
-
-		$this->assertEquals($expects, $result);
-	}
-
-	public function testVerifyValid(): void {
-		$data = [
-			'message' => [
-				'data' => [
-					'foo' => 'bar',
-					'abc' => 'def',
-					'xyz' => 123,
-				],
-				'type' => 'myType',
-				'signer' => 'foo@example.com',
-				'timestamp' => 42,
-			],
-			'signature' => 'E1fNdoWMX1QmSyKv+S3FtOgLe9niYGQFWOKGaMLxc2h7s3V++EIqJvw/NCLBfrUowzWkTzhkjfbHaf88Hz34WAn4sAwXYAO8cnboQs6SClKRzQ/nvbtLgd2wm9RQ8UTOM7wR6C7HpIn4qqJ4BTQ1bAwYAiJ2GoK+W8wC0o0Gpub2906j3JJ4cbc9lufd5ohWKCev8Ubem/EEKaRIZA7qHh+Q1MKXTaJQJlCjTMe5PyGy0fsmtVxsPls3/Fkd9sVeHEHSYHzOiF6ttlxWou4TdRbq3WSEVpt1DOOvkKI9w2+zBJ7IPH8CnVpXcdIzWDctUygZKzNMUQnweDOOziEdUw=='
-		];
-
-		$user = $this->createMock(IUser::class);
-
-		$this->keyManager->method('getKey')
-			->with($this->equalTo($user))
-			->willReturn($this->key);
-
-		$this->userManager->method('get')
-			->with('foo')
-			->willReturn($user);
-
-		$this->assertTrue($this->signer->verify($data));
-	}
-
-	public function testVerifyInvalid(): void {
-		$data = [
-			'message' => [
-				'data' => [
-					'foo' => 'bar',
-					'abc' => 'def',
-					'xyz' => 123,
-				],
-				'type' => 'myType',
-				'signer' => 'foo@example.com',
-				'timestamp' => 42,
-			],
-			'signature' => 'Invalid sig'
-		];
-
-		$user = $this->createMock(IUser::class);
-
-		$this->keyManager->method('getKey')
-			->with($this->equalTo($user))
-			->willReturn($this->key);
-
-		$this->userManager->method('get')
-			->with('foo')
-			->willReturn($user);
-
-		$this->assertFalse($this->signer->verify($data));
-	}
-
-	public function testVerifyInvalidData(): void {
-		$data = [
-		];
-
-		$this->assertFalse($this->signer->verify($data));
-	}
+    /** @var Key */
+    private $key;
+
+    /** @var Manager|\PHPUnit\Framework\MockObject\MockObject */
+    private $keyManager;
+
+    /** @var ITimeFactory|\PHPUnit\Framework\MockObject\MockObject */
+    private $timeFactory;
+
+    /** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $userManager;
+
+    /** @var Signer */
+    private $signer;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->key = new Key($this->public, $this->private);
+
+        $this->keyManager = $this->createMock(Manager::class);
+        $this->timeFactory = $this->createMock(ITimeFactory::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+        $this->signer = new Signer(
+            $this->keyManager,
+            $this->timeFactory,
+            $this->userManager
+        );
+    }
+
+    public function testSign(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getCloudId')
+            ->willReturn('foo@example.com');
+
+        $this->timeFactory->method('getTime')
+            ->willReturn(42);
+
+        $this->keyManager->method('getKey')
+            ->with($this->equalTo($user))
+            ->willReturn($this->key);
+
+        $data = [
+            'foo' => 'bar',
+            'abc' => 'def',
+            'xyz' => 123,
+        ];
+
+        $expects = [
+            'message' => [
+                'data' => $data,
+                'type' => 'myType',
+                'signer' => 'foo@example.com',
+                'timestamp' => 42,
+            ],
+            'signature' => 'E1fNdoWMX1QmSyKv+S3FtOgLe9niYGQFWOKGaMLxc2h7s3V++EIqJvw/NCLBfrUowzWkTzhkjfbHaf88Hz34WAn4sAwXYAO8cnboQs6SClKRzQ/nvbtLgd2wm9RQ8UTOM7wR6C7HpIn4qqJ4BTQ1bAwYAiJ2GoK+W8wC0o0Gpub2906j3JJ4cbc9lufd5ohWKCev8Ubem/EEKaRIZA7qHh+Q1MKXTaJQJlCjTMe5PyGy0fsmtVxsPls3/Fkd9sVeHEHSYHzOiF6ttlxWou4TdRbq3WSEVpt1DOOvkKI9w2+zBJ7IPH8CnVpXcdIzWDctUygZKzNMUQnweDOOziEdUw=='
+        ];
+
+        $result = $this->signer->sign('myType', $data, $user);
+
+        $this->assertEquals($expects, $result);
+    }
+
+    public function testVerifyValid(): void {
+        $data = [
+            'message' => [
+                'data' => [
+                    'foo' => 'bar',
+                    'abc' => 'def',
+                    'xyz' => 123,
+                ],
+                'type' => 'myType',
+                'signer' => 'foo@example.com',
+                'timestamp' => 42,
+            ],
+            'signature' => 'E1fNdoWMX1QmSyKv+S3FtOgLe9niYGQFWOKGaMLxc2h7s3V++EIqJvw/NCLBfrUowzWkTzhkjfbHaf88Hz34WAn4sAwXYAO8cnboQs6SClKRzQ/nvbtLgd2wm9RQ8UTOM7wR6C7HpIn4qqJ4BTQ1bAwYAiJ2GoK+W8wC0o0Gpub2906j3JJ4cbc9lufd5ohWKCev8Ubem/EEKaRIZA7qHh+Q1MKXTaJQJlCjTMe5PyGy0fsmtVxsPls3/Fkd9sVeHEHSYHzOiF6ttlxWou4TdRbq3WSEVpt1DOOvkKI9w2+zBJ7IPH8CnVpXcdIzWDctUygZKzNMUQnweDOOziEdUw=='
+        ];
+
+        $user = $this->createMock(IUser::class);
+
+        $this->keyManager->method('getKey')
+            ->with($this->equalTo($user))
+            ->willReturn($this->key);
+
+        $this->userManager->method('get')
+            ->with('foo')
+            ->willReturn($user);
+
+        $this->assertTrue($this->signer->verify($data));
+    }
+
+    public function testVerifyInvalid(): void {
+        $data = [
+            'message' => [
+                'data' => [
+                    'foo' => 'bar',
+                    'abc' => 'def',
+                    'xyz' => 123,
+                ],
+                'type' => 'myType',
+                'signer' => 'foo@example.com',
+                'timestamp' => 42,
+            ],
+            'signature' => 'Invalid sig'
+        ];
+
+        $user = $this->createMock(IUser::class);
+
+        $this->keyManager->method('getKey')
+            ->with($this->equalTo($user))
+            ->willReturn($this->key);
+
+        $this->userManager->method('get')
+            ->with('foo')
+            ->willReturn($user);
+
+        $this->assertFalse($this->signer->verify($data));
+    }
+
+    public function testVerifyInvalidData(): void {
+        $data = [
+        ];
+
+        $this->assertFalse($this->signer->verify($data));
+    }
 }

tests/lib/Security/IdentityProof/KeyTest.php

Indentation +12 added lines, -12 removed lines

@@ -13,20 +13,20 @@
 use Test\TestCase;
 
 class KeyTest extends TestCase {
-	/** @var Key */
-	private $key;
+    /** @var Key */
+    private $key;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->key = new Key('public', 'private');
-	}
+        $this->key = new Key('public', 'private');
+    }
 
-	public function testGetPrivate(): void {
-		$this->assertSame('private', $this->key->getPrivate());
-	}
+    public function testGetPrivate(): void {
+        $this->assertSame('private', $this->key->getPrivate());
+    }
 
-	public function testGetPublic(): void {
-		$this->assertSame('public', $this->key->getPublic());
-	}
+    public function testGetPublic(): void {
+        $this->assertSame('public', $this->key->getPublic());
+    }
 }

tests/lib/Security/Bruteforce/CapabilitiesTest.php

Indentation +66 added lines, -66 removed lines

@@ -15,70 +15,70 @@
 use Test\TestCase;
 
 class CapabilitiesTest extends TestCase {
-	/** @var Capabilities */
-	private $capabilities;
-
-	/** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */
-	private $request;
-
-	/** @var IThrottler|\PHPUnit\Framework\MockObject\MockObject */
-	private $throttler;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->request = $this->createMock(IRequest::class);
-
-		$this->throttler = $this->createMock(IThrottler::class);
-
-		$this->capabilities = new Capabilities(
-			$this->request,
-			$this->throttler
-		);
-	}
-
-	public function testGetCapabilities(): void {
-		$this->throttler->expects($this->atLeastOnce())
-			->method('getDelay')
-			->with('10.10.10.10')
-			->willReturn(42);
-
-		$this->throttler->expects($this->atLeastOnce())
-			->method('isBypassListed')
-			->with('10.10.10.10')
-			->willReturn(true);
-
-		$this->request->method('getRemoteAddress')
-			->willReturn('10.10.10.10');
-
-		$expected = [
-			'bruteforce' => [
-				'delay' => 42,
-				'allow-listed' => true,
-			]
-		];
-		$result = $this->capabilities->getCapabilities();
-
-		$this->assertEquals($expected, $result);
-	}
-
-	public function testGetCapabilitiesOnCli(): void {
-		$this->throttler->expects($this->atLeastOnce())
-			->method('getDelay')
-			->with('')
-			->willReturn(0);
-
-		$this->request->method('getRemoteAddress')
-			->willReturn('');
-
-		$expected = [
-			'bruteforce' => [
-				'delay' => 0,
-				'allow-listed' => false,
-			]
-		];
-		$result = $this->capabilities->getCapabilities();
-
-		$this->assertEquals($expected, $result);
-	}
+    /** @var Capabilities */
+    private $capabilities;
+
+    /** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */
+    private $request;
+
+    /** @var IThrottler|\PHPUnit\Framework\MockObject\MockObject */
+    private $throttler;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->request = $this->createMock(IRequest::class);
+
+        $this->throttler = $this->createMock(IThrottler::class);
+
+        $this->capabilities = new Capabilities(
+            $this->request,
+            $this->throttler
+        );
+    }
+
+    public function testGetCapabilities(): void {
+        $this->throttler->expects($this->atLeastOnce())
+            ->method('getDelay')
+            ->with('10.10.10.10')
+            ->willReturn(42);
+
+        $this->throttler->expects($this->atLeastOnce())
+            ->method('isBypassListed')
+            ->with('10.10.10.10')
+            ->willReturn(true);
+
+        $this->request->method('getRemoteAddress')
+            ->willReturn('10.10.10.10');
+
+        $expected = [
+            'bruteforce' => [
+                'delay' => 42,
+                'allow-listed' => true,
+            ]
+        ];
+        $result = $this->capabilities->getCapabilities();
+
+        $this->assertEquals($expected, $result);
+    }
+
+    public function testGetCapabilitiesOnCli(): void {
+        $this->throttler->expects($this->atLeastOnce())
+            ->method('getDelay')
+            ->with('')
+            ->willReturn(0);
+
+        $this->request->method('getRemoteAddress')
+            ->willReturn('');
+
+        $expected = [
+            'bruteforce' => [
+                'delay' => 0,
+                'allow-listed' => false,
+            ]
+        ];
+        $result = $this->capabilities->getCapabilities();
+
+        $this->assertEquals($expected, $result);
+    }
 }

tests/lib/Security/CSP/ContentSecurityPolicyNonceManagerTest.php

Indentation +44 added lines, -44 removed lines

@@ -17,55 +17,55 @@
 use Test\TestCase;
 
 class ContentSecurityPolicyNonceManagerTest extends TestCase {
-	/** @var CsrfTokenManager&MockObject */
-	private $CSRFTokenManager;
-	/** @var Request&MockObject */
-	private $request;
-	/** @var ContentSecurityPolicyNonceManager */
-	private $nonceManager;
+    /** @var CsrfTokenManager&MockObject */
+    private $CSRFTokenManager;
+    /** @var Request&MockObject */
+    private $request;
+    /** @var ContentSecurityPolicyNonceManager */
+    private $nonceManager;
 
-	protected function setUp(): void {
-		$this->CSRFTokenManager = $this->createMock(CsrfTokenManager::class);
-		$this->request = $this->createMock(Request::class);
-		$this->nonceManager = new ContentSecurityPolicyNonceManager(
-			$this->CSRFTokenManager,
-			$this->request
-		);
-	}
+    protected function setUp(): void {
+        $this->CSRFTokenManager = $this->createMock(CsrfTokenManager::class);
+        $this->request = $this->createMock(Request::class);
+        $this->nonceManager = new ContentSecurityPolicyNonceManager(
+            $this->CSRFTokenManager,
+            $this->request
+        );
+    }
 
-	public function testGetNonce(): void {
-		$secret = base64_encode('secret');
-		$tokenValue = base64_encode('secret' ^ 'value_') . ':' . $secret;
-		$token = $this->createMock(CsrfToken::class);
-		$token
-			->expects($this->once())
-			->method('getEncryptedValue')
-			->willReturn($tokenValue);
+    public function testGetNonce(): void {
+        $secret = base64_encode('secret');
+        $tokenValue = base64_encode('secret' ^ 'value_') . ':' . $secret;
+        $token = $this->createMock(CsrfToken::class);
+        $token
+            ->expects($this->once())
+            ->method('getEncryptedValue')
+            ->willReturn($tokenValue);
 
-		$this->CSRFTokenManager
-			->expects($this->once())
-			->method('getToken')
-			->willReturn($token);
+        $this->CSRFTokenManager
+            ->expects($this->once())
+            ->method('getToken')
+            ->willReturn($token);
 
-		$this->assertSame($secret, $this->nonceManager->getNonce());
-		// call it twice but `getEncryptedValue` is expected to be called only once
-		$this->assertSame($secret, $this->nonceManager->getNonce());
-	}
+        $this->assertSame($secret, $this->nonceManager->getNonce());
+        // call it twice but `getEncryptedValue` is expected to be called only once
+        $this->assertSame($secret, $this->nonceManager->getNonce());
+    }
 
-	public function testGetNonceServerVar(): void {
-		$token = 'SERVERNONCE';
-		$this->request
-			->method('__isset')
-			->with('server')
-			->willReturn(true);
+    public function testGetNonceServerVar(): void {
+        $token = 'SERVERNONCE';
+        $this->request
+            ->method('__isset')
+            ->with('server')
+            ->willReturn(true);
 
-		$this->request
-			->method('__get')
-			->with('server')
-			->willReturn(['CSP_NONCE' => $token]);
+        $this->request
+            ->method('__get')
+            ->with('server')
+            ->willReturn(['CSP_NONCE' => $token]);
 
-		$this->assertSame($token, $this->nonceManager->getNonce());
-		// call it twice but `CSP_NONCE` variable is expected to be loaded only once
-		$this->assertSame($token, $this->nonceManager->getNonce());
-	}
+        $this->assertSame($token, $this->nonceManager->getNonce());
+        // call it twice but `CSP_NONCE` variable is expected to be loaded only once
+        $this->assertSame($token, $this->nonceManager->getNonce());
+    }
 }

tests/lib/Security/CSP/AddContentSecurityPolicyEventTest.php

Indentation +9 added lines, -9 removed lines

@@ -15,15 +15,15 @@
 use Test\TestCase;
 
 class AddContentSecurityPolicyEventTest extends TestCase {
-	public function testAddEvent(): void {
-		$cspManager = $this->createMock(ContentSecurityPolicyManager::class);
-		$policy = $this->createMock(ContentSecurityPolicy::class);
-		$event = new AddContentSecurityPolicyEvent($cspManager);
+    public function testAddEvent(): void {
+        $cspManager = $this->createMock(ContentSecurityPolicyManager::class);
+        $policy = $this->createMock(ContentSecurityPolicy::class);
+        $event = new AddContentSecurityPolicyEvent($cspManager);
 
-		$cspManager->expects($this->once())
-			->method('addDefaultPolicy')
-			->with($policy);
+        $cspManager->expects($this->once())
+            ->method('addDefaultPolicy')
+            ->with($policy);
 
-		$event->addPolicy($policy);
-	}
+        $event->addPolicy($policy);
+    }
 }

tests/lib/Security/Events/GenerateSecurePasswordEventTest.php

Indentation +16 added lines, -16 removed lines

@@ -14,20 +14,20 @@
 
 class GenerateSecurePasswordEventTest extends \Test\TestCase {
 
-	public function testDefaultProperties(): void {
-		$event = new GenerateSecurePasswordEvent();
-		$this->assertNull($event->getPassword());
-		$this->assertEquals(PasswordContext::ACCOUNT, $event->getContext());
-	}
-
-	public function testSettingPassword(): void {
-		$event = new GenerateSecurePasswordEvent();
-		$event->setPassword('example');
-		$this->assertEquals('example', $event->getPassword());
-	}
-
-	public function testSettingContext(): void {
-		$event = new GenerateSecurePasswordEvent(PasswordContext::SHARING);
-		$this->assertEquals(PasswordContext::SHARING, $event->getContext());
-	}
+    public function testDefaultProperties(): void {
+        $event = new GenerateSecurePasswordEvent();
+        $this->assertNull($event->getPassword());
+        $this->assertEquals(PasswordContext::ACCOUNT, $event->getContext());
+    }
+
+    public function testSettingPassword(): void {
+        $event = new GenerateSecurePasswordEvent();
+        $event->setPassword('example');
+        $this->assertEquals('example', $event->getPassword());
+    }
+
+    public function testSettingContext(): void {
+        $event = new GenerateSecurePasswordEvent(PasswordContext::SHARING);
+        $this->assertEquals(PasswordContext::SHARING, $event->getContext());
+    }
 }

tests/lib/Security/Events/ValidatePasswordPolicyEventTest.php

Indentation +10 added lines, -10 removed lines

@@ -14,15 +14,15 @@
 
 class ValidatePasswordPolicyEventTest extends \Test\TestCase {
 
-	public function testDefaultProperties(): void {
-		$password = 'example';
-		$event = new ValidatePasswordPolicyEvent($password);
-		$this->assertEquals($password, $event->getPassword());
-		$this->assertEquals(PasswordContext::ACCOUNT, $event->getContext());
-	}
+    public function testDefaultProperties(): void {
+        $password = 'example';
+        $event = new ValidatePasswordPolicyEvent($password);
+        $this->assertEquals($password, $event->getPassword());
+        $this->assertEquals(PasswordContext::ACCOUNT, $event->getContext());
+    }
 
-	public function testSettingContext(): void {
-		$event = new ValidatePasswordPolicyEvent('example', PasswordContext::SHARING);
-		$this->assertEquals(PasswordContext::SHARING, $event->getContext());
-	}
+    public function testSettingContext(): void {
+        $event = new ValidatePasswordPolicyEvent('example', PasswordContext::SHARING);
+        $this->assertEquals(PasswordContext::SHARING, $event->getContext());
+    }
 }