nextcloud / server

tests/lib/Accounts/AccountPropertyCollectionTest.php

Indentation +168 added lines, -168 removed lines

@@ -17,172 +17,172 @@
 use Test\TestCase;
 
 class AccountPropertyCollectionTest extends TestCase {
-	protected IAccountPropertyCollection $collection;
-
-	protected const COLLECTION_NAME = 'my_multivalue_property';
-
-	public function setUp(): void {
-		parent::setUp();
-
-		$this->collection = new AccountPropertyCollection(self::COLLECTION_NAME);
-	}
-
-	protected function makePropertyMock(string $propertyName): IAccountProperty&MockObject {
-		$mock = $this->createMock(IAccountProperty::class);
-		$mock->expects($this->any())
-			->method('getName')
-			->willReturn($propertyName);
-
-		return $mock;
-	}
-
-	public function testSetAndGetProperties(): void {
-		$propsBefore = $this->collection->getProperties();
-		$this->assertIsArray($propsBefore);
-		$this->assertEmpty($propsBefore);
-
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-
-		$this->collection->setProperties($props);
-		$propsAfter = $this->collection->getProperties();
-		$this->assertIsArray($propsAfter);
-		$this->assertCount(count($props), $propsAfter);
-	}
-
-	public function testSetPropertiesMixedInvalid(): void {
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock('sneaky_property'),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-
-		$this->expectException(InvalidArgumentException::class);
-		$this->collection->setProperties($props);
-	}
-
-	public function testAddProperty(): void {
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-		$this->collection->setProperties($props);
-
-		$additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
-		$this->collection->addProperty($additionalProperty);
-
-		$propsAfter = $this->collection->getProperties();
-		$this->assertCount(count($props) + 1, $propsAfter);
-		$this->assertNotFalse(array_search($additionalProperty, $propsAfter, true));
-	}
-
-	public function testAddPropertyInvalid(): void {
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-		$this->collection->setProperties($props);
-
-		$additionalProperty = $this->makePropertyMock('sneaky_property');
-		$exceptionThrown = false;
-		try {
-			$this->collection->addProperty($additionalProperty);
-		} catch (\InvalidArgumentException $e) {
-			$exceptionThrown = true;
-		} finally {
-			$propsAfter = $this->collection->getProperties();
-			$this->assertCount(count($props), $propsAfter);
-			$this->assertFalse(array_search($additionalProperty, $propsAfter, true));
-			$this->assertTrue($exceptionThrown);
-		}
-	}
-
-	public function testRemoveProperty(): void {
-		$additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$additionalProperty,
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-		$this->collection->setProperties($props);
-
-		$propsBefore = $this->collection->getProperties();
-		$this->collection->removeProperty($additionalProperty);
-		$propsAfter = $this->collection->getProperties();
-
-		$this->assertTrue(count($propsBefore) > count($propsAfter));
-		$this->assertCount(count($propsBefore) - 1, $propsAfter);
-		$this->assertFalse(array_search($additionalProperty, $propsAfter, true));
-	}
-
-	public function testRemovePropertyNotFound(): void {
-		$additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-		$this->collection->setProperties($props);
-
-		$propsBefore = $this->collection->getProperties();
-		$this->collection->removeProperty($additionalProperty);
-		$propsAfter = $this->collection->getProperties();
-
-		// no errors, gently
-		$this->assertCount(count($propsBefore), $propsAfter);
-	}
-
-	public function testRemovePropertyByValue(): void {
-		$additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
-		$additionalProperty->expects($this->any())
-			->method('getValue')
-			->willReturn('Lorem ipsum');
-
-		$additionalPropertyTwo = clone $additionalProperty;
-
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$additionalProperty,
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$additionalPropertyTwo
-		];
-		$this->collection->setProperties($props);
-
-		$propsBefore = $this->collection->getProperties();
-		$this->collection->removePropertyByValue('Lorem ipsum');
-		$propsAfter = $this->collection->getProperties();
-
-		$this->assertTrue(count($propsBefore) > count($propsAfter));
-		$this->assertCount(count($propsBefore) - 2, $propsAfter);
-		$this->assertFalse(array_search($additionalProperty, $propsAfter, true));
-		$this->assertFalse(array_search($additionalPropertyTwo, $propsAfter, true));
-	}
-
-	public function testRemovePropertyByValueNotFound(): void {
-		$additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
-		$additionalProperty->expects($this->any())
-			->method('getValue')
-			->willReturn('Lorem ipsum');
-
-		$props = [
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-			$this->makePropertyMock(self::COLLECTION_NAME),
-		];
-		$this->collection->setProperties($props);
-
-		$propsBefore = $this->collection->getProperties();
-		$this->collection->removePropertyByValue('Lorem ipsum');
-		$propsAfter = $this->collection->getProperties();
-
-		// no errors, gently
-		$this->assertCount(count($propsBefore), $propsAfter);
-	}
+    protected IAccountPropertyCollection $collection;
+
+    protected const COLLECTION_NAME = 'my_multivalue_property';
+
+    public function setUp(): void {
+        parent::setUp();
+
+        $this->collection = new AccountPropertyCollection(self::COLLECTION_NAME);
+    }
+
+    protected function makePropertyMock(string $propertyName): IAccountProperty&MockObject {
+        $mock = $this->createMock(IAccountProperty::class);
+        $mock->expects($this->any())
+            ->method('getName')
+            ->willReturn($propertyName);
+
+        return $mock;
+    }
+
+    public function testSetAndGetProperties(): void {
+        $propsBefore = $this->collection->getProperties();
+        $this->assertIsArray($propsBefore);
+        $this->assertEmpty($propsBefore);
+
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+
+        $this->collection->setProperties($props);
+        $propsAfter = $this->collection->getProperties();
+        $this->assertIsArray($propsAfter);
+        $this->assertCount(count($props), $propsAfter);
+    }
+
+    public function testSetPropertiesMixedInvalid(): void {
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock('sneaky_property'),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+
+        $this->expectException(InvalidArgumentException::class);
+        $this->collection->setProperties($props);
+    }
+
+    public function testAddProperty(): void {
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+        $this->collection->setProperties($props);
+
+        $additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
+        $this->collection->addProperty($additionalProperty);
+
+        $propsAfter = $this->collection->getProperties();
+        $this->assertCount(count($props) + 1, $propsAfter);
+        $this->assertNotFalse(array_search($additionalProperty, $propsAfter, true));
+    }
+
+    public function testAddPropertyInvalid(): void {
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+        $this->collection->setProperties($props);
+
+        $additionalProperty = $this->makePropertyMock('sneaky_property');
+        $exceptionThrown = false;
+        try {
+            $this->collection->addProperty($additionalProperty);
+        } catch (\InvalidArgumentException $e) {
+            $exceptionThrown = true;
+        } finally {
+            $propsAfter = $this->collection->getProperties();
+            $this->assertCount(count($props), $propsAfter);
+            $this->assertFalse(array_search($additionalProperty, $propsAfter, true));
+            $this->assertTrue($exceptionThrown);
+        }
+    }
+
+    public function testRemoveProperty(): void {
+        $additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $additionalProperty,
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+        $this->collection->setProperties($props);
+
+        $propsBefore = $this->collection->getProperties();
+        $this->collection->removeProperty($additionalProperty);
+        $propsAfter = $this->collection->getProperties();
+
+        $this->assertTrue(count($propsBefore) > count($propsAfter));
+        $this->assertCount(count($propsBefore) - 1, $propsAfter);
+        $this->assertFalse(array_search($additionalProperty, $propsAfter, true));
+    }
+
+    public function testRemovePropertyNotFound(): void {
+        $additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+        $this->collection->setProperties($props);
+
+        $propsBefore = $this->collection->getProperties();
+        $this->collection->removeProperty($additionalProperty);
+        $propsAfter = $this->collection->getProperties();
+
+        // no errors, gently
+        $this->assertCount(count($propsBefore), $propsAfter);
+    }
+
+    public function testRemovePropertyByValue(): void {
+        $additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
+        $additionalProperty->expects($this->any())
+            ->method('getValue')
+            ->willReturn('Lorem ipsum');
+
+        $additionalPropertyTwo = clone $additionalProperty;
+
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $additionalProperty,
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $additionalPropertyTwo
+        ];
+        $this->collection->setProperties($props);
+
+        $propsBefore = $this->collection->getProperties();
+        $this->collection->removePropertyByValue('Lorem ipsum');
+        $propsAfter = $this->collection->getProperties();
+
+        $this->assertTrue(count($propsBefore) > count($propsAfter));
+        $this->assertCount(count($propsBefore) - 2, $propsAfter);
+        $this->assertFalse(array_search($additionalProperty, $propsAfter, true));
+        $this->assertFalse(array_search($additionalPropertyTwo, $propsAfter, true));
+    }
+
+    public function testRemovePropertyByValueNotFound(): void {
+        $additionalProperty = $this->makePropertyMock(self::COLLECTION_NAME);
+        $additionalProperty->expects($this->any())
+            ->method('getValue')
+            ->willReturn('Lorem ipsum');
+
+        $props = [
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+            $this->makePropertyMock(self::COLLECTION_NAME),
+        ];
+        $this->collection->setProperties($props);
+
+        $propsBefore = $this->collection->getProperties();
+        $this->collection->removePropertyByValue('Lorem ipsum');
+        $propsAfter = $this->collection->getProperties();
+
+        // no errors, gently
+        $this->assertCount(count($propsBefore), $propsAfter);
+    }
 }

tests/lib/Accounts/HooksTest.php

Indentation +108 added lines, -108 removed lines

@@ -27,112 +27,112 @@
  */
 class HooksTest extends TestCase {
 
-	private LoggerInterface&MockObject $logger;
-	private AccountManager&MockObject $accountManager;
-	private Hooks $hooks;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->accountManager = $this->getMockBuilder(AccountManager::class)
-			->disableOriginalConstructor()->getMock();
-
-		$this->hooks = new Hooks($this->logger, $this->accountManager);
-	}
-
-	/**
-	 *
-	 * @param $params
-	 * @param $data
-	 * @param $setEmail
-	 * @param $setDisplayName
-	 * @param $error
-	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestChangeUserHook')]
-	public function testChangeUserHook($params, $data, $setEmail, $setDisplayName, $error): void {
-		if ($error) {
-			$this->accountManager->expects($this->never())->method('updateAccount');
-		} else {
-			$account = $this->createMock(IAccount::class);
-			$this->accountManager->expects($this->atLeastOnce())->method('getAccount')->willReturn($account);
-			if ($setEmail) {
-				$property = $this->createMock(IAccountProperty::class);
-				$property->expects($this->atLeastOnce())
-					->method('getValue')
-					->willReturn($data[IAccountManager::PROPERTY_EMAIL]['value']);
-				$property->expects($this->atLeastOnce())
-					->method('setValue')
-					->with($params['value']);
-
-				$account->expects($this->atLeastOnce())
-					->method('getProperty')
-					->with(IAccountManager::PROPERTY_EMAIL)
-					->willReturn($property);
-
-				$this->accountManager->expects($this->once())
-					->method('updateAccount')
-					->with($account);
-			} elseif ($setDisplayName) {
-				$property = $this->createMock(IAccountProperty::class);
-				$property->expects($this->atLeastOnce())
-					->method('getValue')
-					->willReturn($data[IAccountManager::PROPERTY_DISPLAYNAME]['value']);
-				$property->expects($this->atLeastOnce())
-					->method('setValue')
-					->with($params['value']);
-
-				$account->expects($this->atLeastOnce())
-					->method('getProperty')
-					->with(IAccountManager::PROPERTY_DISPLAYNAME)
-					->willReturn($property);
-
-				$this->accountManager->expects($this->once())
-					->method('updateAccount')
-					->with($account);
-			} else {
-				$this->accountManager->expects($this->never())->method('updateAccount');
-			}
-		}
-
-		$params['user'] = $this->createMock(IUser::class);
-		$this->hooks->changeUserHook($params['user'], $params['feature'], $params['value']);
-	}
-
-	public static function dataTestChangeUserHook(): array {
-		return [
-			[
-				['feature' => '', 'value' => ''],
-				[
-					IAccountManager::PROPERTY_EMAIL => ['value' => ''],
-					IAccountManager::PROPERTY_DISPLAYNAME => ['value' => '']
-				],
-				false, false, true
-			],
-			[
-				['feature' => 'foo', 'value' => 'bar'],
-				[
-					IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
-					IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
-				],
-				false, false, false
-			],
-			[
-				['feature' => 'eMailAddress', 'value' => 'newMail@example.com'],
-				[
-					IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
-					IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
-				],
-				true, false, false
-			],
-			[
-				['feature' => 'displayName', 'value' => 'newDisplayName'],
-				[
-					IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
-					IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
-				],
-				false, true, false
-			],
-		];
-	}
+    private LoggerInterface&MockObject $logger;
+    private AccountManager&MockObject $accountManager;
+    private Hooks $hooks;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->accountManager = $this->getMockBuilder(AccountManager::class)
+            ->disableOriginalConstructor()->getMock();
+
+        $this->hooks = new Hooks($this->logger, $this->accountManager);
+    }
+
+    /**
+     *
+     * @param $params
+     * @param $data
+     * @param $setEmail
+     * @param $setDisplayName
+     * @param $error
+     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataTestChangeUserHook')]
+    public function testChangeUserHook($params, $data, $setEmail, $setDisplayName, $error): void {
+        if ($error) {
+            $this->accountManager->expects($this->never())->method('updateAccount');
+        } else {
+            $account = $this->createMock(IAccount::class);
+            $this->accountManager->expects($this->atLeastOnce())->method('getAccount')->willReturn($account);
+            if ($setEmail) {
+                $property = $this->createMock(IAccountProperty::class);
+                $property->expects($this->atLeastOnce())
+                    ->method('getValue')
+                    ->willReturn($data[IAccountManager::PROPERTY_EMAIL]['value']);
+                $property->expects($this->atLeastOnce())
+                    ->method('setValue')
+                    ->with($params['value']);
+
+                $account->expects($this->atLeastOnce())
+                    ->method('getProperty')
+                    ->with(IAccountManager::PROPERTY_EMAIL)
+                    ->willReturn($property);
+
+                $this->accountManager->expects($this->once())
+                    ->method('updateAccount')
+                    ->with($account);
+            } elseif ($setDisplayName) {
+                $property = $this->createMock(IAccountProperty::class);
+                $property->expects($this->atLeastOnce())
+                    ->method('getValue')
+                    ->willReturn($data[IAccountManager::PROPERTY_DISPLAYNAME]['value']);
+                $property->expects($this->atLeastOnce())
+                    ->method('setValue')
+                    ->with($params['value']);
+
+                $account->expects($this->atLeastOnce())
+                    ->method('getProperty')
+                    ->with(IAccountManager::PROPERTY_DISPLAYNAME)
+                    ->willReturn($property);
+
+                $this->accountManager->expects($this->once())
+                    ->method('updateAccount')
+                    ->with($account);
+            } else {
+                $this->accountManager->expects($this->never())->method('updateAccount');
+            }
+        }
+
+        $params['user'] = $this->createMock(IUser::class);
+        $this->hooks->changeUserHook($params['user'], $params['feature'], $params['value']);
+    }
+
+    public static function dataTestChangeUserHook(): array {
+        return [
+            [
+                ['feature' => '', 'value' => ''],
+                [
+                    IAccountManager::PROPERTY_EMAIL => ['value' => ''],
+                    IAccountManager::PROPERTY_DISPLAYNAME => ['value' => '']
+                ],
+                false, false, true
+            ],
+            [
+                ['feature' => 'foo', 'value' => 'bar'],
+                [
+                    IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
+                    IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
+                ],
+                false, false, false
+            ],
+            [
+                ['feature' => 'eMailAddress', 'value' => 'newMail@example.com'],
+                [
+                    IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
+                    IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
+                ],
+                true, false, false
+            ],
+            [
+                ['feature' => 'displayName', 'value' => 'newDisplayName'],
+                [
+                    IAccountManager::PROPERTY_EMAIL => ['value' => 'oldMail@example.com'],
+                    IAccountManager::PROPERTY_DISPLAYNAME => ['value' => 'oldDisplayName']
+                ],
+                false, true, false
+            ],
+        ];
+    }
 }

tests/lib/Activity/ManagerTest.php

Indentation +267 added lines, -267 removed lines

@@ -26,274 +26,274 @@
 use Test\TestCase;
 
 class ManagerTest extends TestCase {
-	private Manager $activityManager;
-
-	protected IRequest&MockObject $request;
-	protected IUserSession&MockObject $session;
-	protected IConfig&MockObject $config;
-	protected IValidator&MockObject $validator;
-	protected IRichTextFormatter&MockObject $richTextFormatter;
-	private ITimeFactory&MockObject $time;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->request = $this->createMock(IRequest::class);
-		$this->session = $this->createMock(IUserSession::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->validator = $this->createMock(IValidator::class);
-		$this->richTextFormatter = $this->createMock(IRichTextFormatter::class);
-		$this->time = $this->createMock(ITimeFactory::class);
-
-		$this->activityManager = new Manager(
-			$this->request,
-			$this->session,
-			$this->config,
-			$this->validator,
-			$this->richTextFormatter,
-			$this->createMock(IL10N::class),
-			$this->time,
-		);
-
-		$this->assertSame([], self::invokePrivate($this->activityManager, 'getConsumers'));
-
-		$this->activityManager->registerConsumer(function () {
-			return new NoOpConsumer();
-		});
-
-		$this->assertNotEmpty(self::invokePrivate($this->activityManager, 'getConsumers'));
-		$this->assertNotEmpty(self::invokePrivate($this->activityManager, 'getConsumers'));
-	}
-
-	public function testGetConsumers(): void {
-		$consumers = self::invokePrivate($this->activityManager, 'getConsumers');
-
-		$this->assertNotEmpty($consumers);
-	}
-
-
-	public function testGetConsumersInvalidConsumer(): void {
-		$this->expectException(\InvalidArgumentException::class);
-
-		$this->activityManager->registerConsumer(function () {
-			return new \stdClass();
-		});
-
-		self::invokePrivate($this->activityManager, 'getConsumers');
-	}
-
-	public static function getUserFromTokenThrowInvalidTokenData(): array {
-		return [
-			[null, []],
-			['', []],
-			['12345678901234567890123456789', []],
-			['1234567890123456789012345678901', []],
-			['123456789012345678901234567890', []],
-			['123456789012345678901234567890', ['user1', 'user2']],
-		];
-	}
-
-	/**
-	 *
-	 * @param string $token
-	 * @param array $users
-	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('getUserFromTokenThrowInvalidTokenData')]
-	public function testGetUserFromTokenThrowInvalidToken($token, $users): void {
-		$this->expectException(\UnexpectedValueException::class);
-
-		$this->mockRSSToken($token, $token, $users);
-		self::invokePrivate($this->activityManager, 'getUserFromToken');
-	}
-
-	public static function getUserFromTokenData(): array {
-		return [
-			[null, '123456789012345678901234567890', 'user1'],
-			['user2', null, 'user2'],
-			['user2', '123456789012345678901234567890', 'user2'],
-		];
-	}
-
-	/**
-	 *
-	 * @param string $userLoggedIn
-	 * @param string $token
-	 * @param string $expected
-	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('getUserFromTokenData')]
-	public function testGetUserFromToken($userLoggedIn, $token, $expected): void {
-		if ($userLoggedIn !== null) {
-			$this->mockUserSession($userLoggedIn);
-		}
-		$this->mockRSSToken($token, '123456789012345678901234567890', ['user1']);
-
-		$this->assertEquals($expected, $this->activityManager->getCurrentUserId());
-	}
-
-	protected function mockRSSToken($requestToken, $userToken, $users) {
-		if ($requestToken !== null) {
-			$this->request->expects($this->any())
-				->method('getParam')
-				->with('token', '')
-				->willReturn($requestToken);
-		}
-
-		$this->config->expects($this->any())
-			->method('getUsersForUserValue')
-			->with('activity', 'rsstoken', $userToken)
-			->willReturn($users);
-	}
-
-	protected function mockUserSession($user) {
-		$mockUser = $this->getMockBuilder(IUser::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$mockUser->expects($this->any())
-			->method('getUID')
-			->willReturn($user);
-
-		$this->session->expects($this->any())
-			->method('isLoggedIn')
-			->willReturn(true);
-		$this->session->expects($this->any())
-			->method('getUser')
-			->willReturn($mockUser);
-	}
-
-
-	public function testPublishExceptionNoApp(): void {
-		$this->expectException(IncompleteActivityException::class);
-
-		$event = $this->activityManager->generateEvent();
-		$this->activityManager->publish($event);
-	}
-
-
-	public function testPublishExceptionNoType(): void {
-		$this->expectException(IncompleteActivityException::class);
-
-		$event = $this->activityManager->generateEvent();
-		$event->setApp('test');
-		$this->activityManager->publish($event);
-	}
-
-
-	public function testPublishExceptionNoAffectedUser(): void {
-		$this->expectException(IncompleteActivityException::class);
-
-		$event = $this->activityManager->generateEvent();
-		$event->setApp('test')
-			->setType('test_type');
-		$this->activityManager->publish($event);
-	}
-
-
-	public function testPublishExceptionNoSubject(): void {
-		$this->expectException(IncompleteActivityException::class);
-
-		$event = $this->activityManager->generateEvent();
-		$event->setApp('test')
-			->setType('test_type')
-			->setAffectedUser('test_affected');
-		$this->activityManager->publish($event);
-	}
-
-	public static function dataPublish(): array {
-		return [
-			[null, ''],
-			['test_author', 'test_author'],
-		];
-	}
-
-	/**
-	 * @param string|null $author
-	 * @param string $expected
-	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataPublish')]
-	public function testPublish($author, $expected): void {
-		if ($author !== null) {
-			$authorObject = $this->getMockBuilder(IUser::class)
-				->disableOriginalConstructor()
-				->getMock();
-			$authorObject->expects($this->once())
-				->method('getUID')
-				->willReturn($author);
-			$this->session->expects($this->atLeastOnce())
-				->method('getUser')
-				->willReturn($authorObject);
-		}
-
-		$time = time();
-		$this->time
-			->method('getTime')
-			->willReturn($time);
-
-		$event = $this->activityManager->generateEvent();
-		$event->setApp('test')
-			->setType('test_type')
-			->setSubject('test_subject', [])
-			->setAffectedUser('test_affected')
-			->setObject('file', 123);
-
-		$consumer = $this->getMockBuilder('OCP\Activity\IConsumer')
-			->disableOriginalConstructor()
-			->getMock();
-		$consumer->expects($this->once())
-			->method('receive')
-			->with($event)
-			->willReturnCallback(function (IEvent $event) use ($expected, $time): void {
-				$this->assertEquals($time, $event->getTimestamp(), 'Timestamp not set correctly');
-				$this->assertSame($expected, $event->getAuthor(), 'Author name not set correctly');
-			});
-		$this->activityManager->registerConsumer(function () use ($consumer) {
-			return $consumer;
-		});
-
-		$this->activityManager->publish($event);
-	}
-
-	public function testPublishAllManually(): void {
-		$event = $this->activityManager->generateEvent();
-		$event->setApp('test_app')
-			->setType('test_type')
-			->setAffectedUser('test_affected')
-			->setAuthor('test_author')
-			->setTimestamp(1337)
-			->setSubject('test_subject', ['test_subject_param'])
-			->setMessage('test_message', ['test_message_param'])
-			->setObject('test_object_type', 42, 'test_object_name')
-			->setLink('test_link')
-		;
-
-		$consumer = $this->getMockBuilder('OCP\Activity\IConsumer')
-			->disableOriginalConstructor()
-			->getMock();
-		$consumer->expects($this->once())
-			->method('receive')
-			->willReturnCallback(function (IEvent $event): void {
-				$this->assertSame('test_app', $event->getApp(), 'App not set correctly');
-				$this->assertSame('test_type', $event->getType(), 'Type not set correctly');
-				$this->assertSame('test_affected', $event->getAffectedUser(), 'Affected user not set correctly');
-				$this->assertSame('test_author', $event->getAuthor(), 'Author not set correctly');
-				$this->assertSame(1337, $event->getTimestamp(), 'Timestamp not set correctly');
-				$this->assertSame('test_subject', $event->getSubject(), 'Subject not set correctly');
-				$this->assertSame(['test_subject_param'], $event->getSubjectParameters(), 'Subject parameter not set correctly');
-				$this->assertSame('test_message', $event->getMessage(), 'Message not set correctly');
-				$this->assertSame(['test_message_param'], $event->getMessageParameters(), 'Message parameter not set correctly');
-				$this->assertSame('test_object_type', $event->getObjectType(), 'Object type not set correctly');
-				$this->assertSame(42, $event->getObjectId(), 'Object ID not set correctly');
-				$this->assertSame('test_object_name', $event->getObjectName(), 'Object name not set correctly');
-				$this->assertSame('test_link', $event->getLink(), 'Link not set correctly');
-			});
-		$this->activityManager->registerConsumer(function () use ($consumer) {
-			return $consumer;
-		});
-
-		$this->activityManager->publish($event);
-	}
+    private Manager $activityManager;
+
+    protected IRequest&MockObject $request;
+    protected IUserSession&MockObject $session;
+    protected IConfig&MockObject $config;
+    protected IValidator&MockObject $validator;
+    protected IRichTextFormatter&MockObject $richTextFormatter;
+    private ITimeFactory&MockObject $time;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->request = $this->createMock(IRequest::class);
+        $this->session = $this->createMock(IUserSession::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->validator = $this->createMock(IValidator::class);
+        $this->richTextFormatter = $this->createMock(IRichTextFormatter::class);
+        $this->time = $this->createMock(ITimeFactory::class);
+
+        $this->activityManager = new Manager(
+            $this->request,
+            $this->session,
+            $this->config,
+            $this->validator,
+            $this->richTextFormatter,
+            $this->createMock(IL10N::class),
+            $this->time,
+        );
+
+        $this->assertSame([], self::invokePrivate($this->activityManager, 'getConsumers'));
+
+        $this->activityManager->registerConsumer(function () {
+            return new NoOpConsumer();
+        });
+
+        $this->assertNotEmpty(self::invokePrivate($this->activityManager, 'getConsumers'));
+        $this->assertNotEmpty(self::invokePrivate($this->activityManager, 'getConsumers'));
+    }
+
+    public function testGetConsumers(): void {
+        $consumers = self::invokePrivate($this->activityManager, 'getConsumers');
+
+        $this->assertNotEmpty($consumers);
+    }
+
+
+    public function testGetConsumersInvalidConsumer(): void {
+        $this->expectException(\InvalidArgumentException::class);
+
+        $this->activityManager->registerConsumer(function () {
+            return new \stdClass();
+        });
+
+        self::invokePrivate($this->activityManager, 'getConsumers');
+    }
+
+    public static function getUserFromTokenThrowInvalidTokenData(): array {
+        return [
+            [null, []],
+            ['', []],
+            ['12345678901234567890123456789', []],
+            ['1234567890123456789012345678901', []],
+            ['123456789012345678901234567890', []],
+            ['123456789012345678901234567890', ['user1', 'user2']],
+        ];
+    }
+
+    /**
+     *
+     * @param string $token
+     * @param array $users
+     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('getUserFromTokenThrowInvalidTokenData')]
+    public function testGetUserFromTokenThrowInvalidToken($token, $users): void {
+        $this->expectException(\UnexpectedValueException::class);
+
+        $this->mockRSSToken($token, $token, $users);
+        self::invokePrivate($this->activityManager, 'getUserFromToken');
+    }
+
+    public static function getUserFromTokenData(): array {
+        return [
+            [null, '123456789012345678901234567890', 'user1'],
+            ['user2', null, 'user2'],
+            ['user2', '123456789012345678901234567890', 'user2'],
+        ];
+    }
+
+    /**
+     *
+     * @param string $userLoggedIn
+     * @param string $token
+     * @param string $expected
+     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('getUserFromTokenData')]
+    public function testGetUserFromToken($userLoggedIn, $token, $expected): void {
+        if ($userLoggedIn !== null) {
+            $this->mockUserSession($userLoggedIn);
+        }
+        $this->mockRSSToken($token, '123456789012345678901234567890', ['user1']);
+
+        $this->assertEquals($expected, $this->activityManager->getCurrentUserId());
+    }
+
+    protected function mockRSSToken($requestToken, $userToken, $users) {
+        if ($requestToken !== null) {
+            $this->request->expects($this->any())
+                ->method('getParam')
+                ->with('token', '')
+                ->willReturn($requestToken);
+        }
+
+        $this->config->expects($this->any())
+            ->method('getUsersForUserValue')
+            ->with('activity', 'rsstoken', $userToken)
+            ->willReturn($users);
+    }
+
+    protected function mockUserSession($user) {
+        $mockUser = $this->getMockBuilder(IUser::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $mockUser->expects($this->any())
+            ->method('getUID')
+            ->willReturn($user);
+
+        $this->session->expects($this->any())
+            ->method('isLoggedIn')
+            ->willReturn(true);
+        $this->session->expects($this->any())
+            ->method('getUser')
+            ->willReturn($mockUser);
+    }
+
+
+    public function testPublishExceptionNoApp(): void {
+        $this->expectException(IncompleteActivityException::class);
+
+        $event = $this->activityManager->generateEvent();
+        $this->activityManager->publish($event);
+    }
+
+
+    public function testPublishExceptionNoType(): void {
+        $this->expectException(IncompleteActivityException::class);
+
+        $event = $this->activityManager->generateEvent();
+        $event->setApp('test');
+        $this->activityManager->publish($event);
+    }
+
+
+    public function testPublishExceptionNoAffectedUser(): void {
+        $this->expectException(IncompleteActivityException::class);
+
+        $event = $this->activityManager->generateEvent();
+        $event->setApp('test')
+            ->setType('test_type');
+        $this->activityManager->publish($event);
+    }
+
+
+    public function testPublishExceptionNoSubject(): void {
+        $this->expectException(IncompleteActivityException::class);
+
+        $event = $this->activityManager->generateEvent();
+        $event->setApp('test')
+            ->setType('test_type')
+            ->setAffectedUser('test_affected');
+        $this->activityManager->publish($event);
+    }
+
+    public static function dataPublish(): array {
+        return [
+            [null, ''],
+            ['test_author', 'test_author'],
+        ];
+    }
+
+    /**
+     * @param string|null $author
+     * @param string $expected
+     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataPublish')]
+    public function testPublish($author, $expected): void {
+        if ($author !== null) {
+            $authorObject = $this->getMockBuilder(IUser::class)
+                ->disableOriginalConstructor()
+                ->getMock();
+            $authorObject->expects($this->once())
+                ->method('getUID')
+                ->willReturn($author);
+            $this->session->expects($this->atLeastOnce())
+                ->method('getUser')
+                ->willReturn($authorObject);
+        }
+
+        $time = time();
+        $this->time
+            ->method('getTime')
+            ->willReturn($time);
+
+        $event = $this->activityManager->generateEvent();
+        $event->setApp('test')
+            ->setType('test_type')
+            ->setSubject('test_subject', [])
+            ->setAffectedUser('test_affected')
+            ->setObject('file', 123);
+
+        $consumer = $this->getMockBuilder('OCP\Activity\IConsumer')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $consumer->expects($this->once())
+            ->method('receive')
+            ->with($event)
+            ->willReturnCallback(function (IEvent $event) use ($expected, $time): void {
+                $this->assertEquals($time, $event->getTimestamp(), 'Timestamp not set correctly');
+                $this->assertSame($expected, $event->getAuthor(), 'Author name not set correctly');
+            });
+        $this->activityManager->registerConsumer(function () use ($consumer) {
+            return $consumer;
+        });
+
+        $this->activityManager->publish($event);
+    }
+
+    public function testPublishAllManually(): void {
+        $event = $this->activityManager->generateEvent();
+        $event->setApp('test_app')
+            ->setType('test_type')
+            ->setAffectedUser('test_affected')
+            ->setAuthor('test_author')
+            ->setTimestamp(1337)
+            ->setSubject('test_subject', ['test_subject_param'])
+            ->setMessage('test_message', ['test_message_param'])
+            ->setObject('test_object_type', 42, 'test_object_name')
+            ->setLink('test_link')
+        ;
+
+        $consumer = $this->getMockBuilder('OCP\Activity\IConsumer')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $consumer->expects($this->once())
+            ->method('receive')
+            ->willReturnCallback(function (IEvent $event): void {
+                $this->assertSame('test_app', $event->getApp(), 'App not set correctly');
+                $this->assertSame('test_type', $event->getType(), 'Type not set correctly');
+                $this->assertSame('test_affected', $event->getAffectedUser(), 'Affected user not set correctly');
+                $this->assertSame('test_author', $event->getAuthor(), 'Author not set correctly');
+                $this->assertSame(1337, $event->getTimestamp(), 'Timestamp not set correctly');
+                $this->assertSame('test_subject', $event->getSubject(), 'Subject not set correctly');
+                $this->assertSame(['test_subject_param'], $event->getSubjectParameters(), 'Subject parameter not set correctly');
+                $this->assertSame('test_message', $event->getMessage(), 'Message not set correctly');
+                $this->assertSame(['test_message_param'], $event->getMessageParameters(), 'Message parameter not set correctly');
+                $this->assertSame('test_object_type', $event->getObjectType(), 'Object type not set correctly');
+                $this->assertSame(42, $event->getObjectId(), 'Object ID not set correctly');
+                $this->assertSame('test_object_name', $event->getObjectName(), 'Object name not set correctly');
+                $this->assertSame('test_link', $event->getLink(), 'Link not set correctly');
+            });
+        $this->activityManager->registerConsumer(function () use ($consumer) {
+            return $consumer;
+        });
+
+        $this->activityManager->publish($event);
+    }
 }
 
 class NoOpConsumer implements IConsumer {
-	public function receive(IEvent $event) {
-	}
+    public function receive(IEvent $event) {
+    }
 }

tests/lib/AppFramework/DependencyInjection/DIIntergrationTests.php

Indentation +90 added lines, -90 removed lines

@@ -22,97 +22,97 @@
 }
 
 class ClassB {
-	/**
-	 * ClassB constructor.
-	 *
-	 * @param Interface1 $interface1
-	 */
-	public function __construct(
-		public Interface1 $interface1,
-	) {
-	}
+    /**
+     * ClassB constructor.
+     *
+     * @param Interface1 $interface1
+     */
+    public function __construct(
+        public Interface1 $interface1,
+    ) {
+    }
 }
 
 class DIIntergrationTests extends TestCase {
-	private DIContainer $container;
-	private ServerContainer $server;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->server = new ServerContainer();
-		$this->container = new DIContainer('App1', [], $this->server);
-	}
-
-	public function testInjectFromServer(): void {
-		$this->server->registerService(Interface1::class, function () {
-			return new ClassA1();
-		});
-
-		$this->server->registerService(ClassB::class, function (SimpleContainer $c) {
-			return new ClassB(
-				$c->query(Interface1::class)
-			);
-		});
-
-		/** @var ClassB $res */
-		$res = $this->container->query(ClassB::class);
-		$this->assertSame(ClassA1::class, get_class($res->interface1));
-	}
-
-	public function testInjectDepFromServer(): void {
-		$this->server->registerService(Interface1::class, function () {
-			return new ClassA1();
-		});
-
-		$this->container->registerService(ClassB::class, function (SimpleContainer $c) {
-			return new ClassB(
-				$c->query(Interface1::class)
-			);
-		});
-
-		/** @var ClassB $res */
-		$res = $this->container->query(ClassB::class);
-		$this->assertSame(ClassA1::class, get_class($res->interface1));
-	}
-
-	public function testOverwriteDepFromServer(): void {
-		$this->server->registerService(Interface1::class, function () {
-			return new ClassA1();
-		});
-
-		$this->container->registerService(Interface1::class, function () {
-			return new ClassA2();
-		});
-
-		$this->container->registerService(ClassB::class, function (SimpleContainer $c) {
-			return new ClassB(
-				$c->query(Interface1::class)
-			);
-		});
-
-		/** @var ClassB $res */
-		$res = $this->container->query(ClassB::class);
-		$this->assertSame(ClassA2::class, get_class($res->interface1));
-	}
-
-	public function testIgnoreOverwriteInServerClass(): void {
-		$this->server->registerService(Interface1::class, function () {
-			return new ClassA1();
-		});
-
-		$this->container->registerService(Interface1::class, function () {
-			return new ClassA2();
-		});
-
-		$this->server->registerService(ClassB::class, function (SimpleContainer $c) {
-			return new ClassB(
-				$c->query(Interface1::class)
-			);
-		});
-
-		/** @var ClassB $res */
-		$res = $this->container->query(ClassB::class);
-		$this->assertSame(ClassA1::class, get_class($res->interface1));
-	}
+    private DIContainer $container;
+    private ServerContainer $server;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->server = new ServerContainer();
+        $this->container = new DIContainer('App1', [], $this->server);
+    }
+
+    public function testInjectFromServer(): void {
+        $this->server->registerService(Interface1::class, function () {
+            return new ClassA1();
+        });
+
+        $this->server->registerService(ClassB::class, function (SimpleContainer $c) {
+            return new ClassB(
+                $c->query(Interface1::class)
+            );
+        });
+
+        /** @var ClassB $res */
+        $res = $this->container->query(ClassB::class);
+        $this->assertSame(ClassA1::class, get_class($res->interface1));
+    }
+
+    public function testInjectDepFromServer(): void {
+        $this->server->registerService(Interface1::class, function () {
+            return new ClassA1();
+        });
+
+        $this->container->registerService(ClassB::class, function (SimpleContainer $c) {
+            return new ClassB(
+                $c->query(Interface1::class)
+            );
+        });
+
+        /** @var ClassB $res */
+        $res = $this->container->query(ClassB::class);
+        $this->assertSame(ClassA1::class, get_class($res->interface1));
+    }
+
+    public function testOverwriteDepFromServer(): void {
+        $this->server->registerService(Interface1::class, function () {
+            return new ClassA1();
+        });
+
+        $this->container->registerService(Interface1::class, function () {
+            return new ClassA2();
+        });
+
+        $this->container->registerService(ClassB::class, function (SimpleContainer $c) {
+            return new ClassB(
+                $c->query(Interface1::class)
+            );
+        });
+
+        /** @var ClassB $res */
+        $res = $this->container->query(ClassB::class);
+        $this->assertSame(ClassA2::class, get_class($res->interface1));
+    }
+
+    public function testIgnoreOverwriteInServerClass(): void {
+        $this->server->registerService(Interface1::class, function () {
+            return new ClassA1();
+        });
+
+        $this->container->registerService(Interface1::class, function () {
+            return new ClassA2();
+        });
+
+        $this->server->registerService(ClassB::class, function (SimpleContainer $c) {
+            return new ClassB(
+                $c->query(Interface1::class)
+            );
+        });
+
+        /** @var ClassB $res */
+        $res = $this->container->query(ClassB::class);
+        $this->assertSame(ClassA1::class, get_class($res->interface1));
+    }
 }

tests/lib/AppFramework/Db/EntityTest.php

Indentation +226 added lines, -226 removed lines

@@ -36,285 +36,285 @@
  * @method void setDatetime(\DateTimeImmutable $datetime)
  */
 class TestEntity extends Entity {
-	protected $email;
-	protected $testId;
-	protected $smallInt;
-	protected $bigInt;
-	protected $preName;
-	protected $trueOrFalse;
-	protected $anotherBool;
-	protected $text;
-	protected $longText;
-	protected $time;
-	protected $datetime;
-
-	public function __construct(
-		protected $name = null,
-	) {
-		$this->addType('testId', Types::INTEGER);
-		$this->addType('smallInt', Types::SMALLINT);
-		$this->addType('bigInt', Types::BIGINT);
-		$this->addType('anotherBool', Types::BOOLEAN);
-		$this->addType('text', Types::TEXT);
-		$this->addType('longText', Types::BLOB);
-		$this->addType('time', Types::TIME);
-		$this->addType('datetime', Types::DATETIME_IMMUTABLE);
-
-		// Legacy types
-		$this->addType('trueOrFalse', 'bool');
-		$this->addType('legacyInt', 'int');
-		$this->addType('doubleNowFloat', 'double');
-	}
-
-	public function setAnotherBool(bool $anotherBool): void {
-		parent::setAnotherBool($anotherBool);
-	}
+    protected $email;
+    protected $testId;
+    protected $smallInt;
+    protected $bigInt;
+    protected $preName;
+    protected $trueOrFalse;
+    protected $anotherBool;
+    protected $text;
+    protected $longText;
+    protected $time;
+    protected $datetime;
+
+    public function __construct(
+        protected $name = null,
+    ) {
+        $this->addType('testId', Types::INTEGER);
+        $this->addType('smallInt', Types::SMALLINT);
+        $this->addType('bigInt', Types::BIGINT);
+        $this->addType('anotherBool', Types::BOOLEAN);
+        $this->addType('text', Types::TEXT);
+        $this->addType('longText', Types::BLOB);
+        $this->addType('time', Types::TIME);
+        $this->addType('datetime', Types::DATETIME_IMMUTABLE);
+
+        // Legacy types
+        $this->addType('trueOrFalse', 'bool');
+        $this->addType('legacyInt', 'int');
+        $this->addType('doubleNowFloat', 'double');
+    }
+
+    public function setAnotherBool(bool $anotherBool): void {
+        parent::setAnotherBool($anotherBool);
+    }
 }
 
 
 class EntityTest extends \Test\TestCase {
-	private TestEntity $entity;
+    private TestEntity $entity;
 
-	protected function setUp(): void {
-		parent::setUp();
-		$this->entity = new TestEntity();
-	}
+    protected function setUp(): void {
+        parent::setUp();
+        $this->entity = new TestEntity();
+    }
 
 
-	public function testResetUpdatedFields(): void {
-		$entity = new TestEntity();
-		$entity->setId(3);
-		$entity->resetUpdatedFields();
+    public function testResetUpdatedFields(): void {
+        $entity = new TestEntity();
+        $entity->setId(3);
+        $entity->resetUpdatedFields();
 
-		$this->assertEquals([], $entity->getUpdatedFields());
-	}
+        $this->assertEquals([], $entity->getUpdatedFields());
+    }
 
 
-	public function testFromRow(): void {
-		$row = [
-			'pre_name' => 'john',
-			'email' => 'john@something.com',
-			'another_bool' => 1,
-		];
-		$this->entity = TestEntity::fromRow($row);
+    public function testFromRow(): void {
+        $row = [
+            'pre_name' => 'john',
+            'email' => 'john@something.com',
+            'another_bool' => 1,
+        ];
+        $this->entity = TestEntity::fromRow($row);
 
-		$this->assertEquals($row['pre_name'], $this->entity->getPreName());
-		$this->assertEquals($row['email'], $this->entity->getEmail());
-		$this->assertEquals($row['another_bool'], $this->entity->getAnotherBool());
-	}
+        $this->assertEquals($row['pre_name'], $this->entity->getPreName());
+        $this->assertEquals($row['email'], $this->entity->getEmail());
+        $this->assertEquals($row['another_bool'], $this->entity->getAnotherBool());
+    }
 
 
-	public function testGetSetId(): void {
-		$id = 3;
-		$this->entity->setId(3);
+    public function testGetSetId(): void {
+        $id = 3;
+        $this->entity->setId(3);
 
-		$this->assertEquals($id, $this->entity->getId());
-	}
+        $this->assertEquals($id, $this->entity->getId());
+    }
 
 
-	public function testColumnToPropertyNoReplacement(): void {
-		$column = 'my';
-		$this->assertEquals('my',
-			$this->entity->columnToProperty($column));
-	}
+    public function testColumnToPropertyNoReplacement(): void {
+        $column = 'my';
+        $this->assertEquals('my',
+            $this->entity->columnToProperty($column));
+    }
 
 
-	public function testColumnToProperty(): void {
-		$column = 'my_attribute';
-		$this->assertEquals('myAttribute',
-			$this->entity->columnToProperty($column));
-	}
+    public function testColumnToProperty(): void {
+        $column = 'my_attribute';
+        $this->assertEquals('myAttribute',
+            $this->entity->columnToProperty($column));
+    }
 
 
-	public function testPropertyToColumnNoReplacement(): void {
-		$property = 'my';
-		$this->assertEquals('my',
-			$this->entity->propertyToColumn($property));
-	}
+    public function testPropertyToColumnNoReplacement(): void {
+        $property = 'my';
+        $this->assertEquals('my',
+            $this->entity->propertyToColumn($property));
+    }
 
 
-	public function testSetterMarksFieldUpdated(): void {
-		$this->entity->setId(3);
+    public function testSetterMarksFieldUpdated(): void {
+        $this->entity->setId(3);
 
-		$this->assertContains('id', array_keys($this->entity->getUpdatedFields()));
-	}
+        $this->assertContains('id', array_keys($this->entity->getUpdatedFields()));
+    }
 
 
 
-	public function testCallShouldOnlyWorkForGetterSetter(): void {
-		$this->expectException(\BadFunctionCallException::class);
+    public function testCallShouldOnlyWorkForGetterSetter(): void {
+        $this->expectException(\BadFunctionCallException::class);
 
-		$this->entity->something();
-	}
+        $this->entity->something();
+    }
 
 
 
-	public function testGetterShouldFailIfAttributeNotDefined(): void {
-		$this->expectException(\BadFunctionCallException::class);
+    public function testGetterShouldFailIfAttributeNotDefined(): void {
+        $this->expectException(\BadFunctionCallException::class);
 
-		$this->entity->getTest();
-	}
+        $this->entity->getTest();
+    }
 
 
-	public function testSetterShouldFailIfAttributeNotDefined(): void {
-		$this->expectException(\BadFunctionCallException::class);
+    public function testSetterShouldFailIfAttributeNotDefined(): void {
+        $this->expectException(\BadFunctionCallException::class);
 
-		$this->entity->setTest();
-	}
+        $this->entity->setTest();
+    }
 
 
-	public function testFromRowShouldNotAssignEmptyArray(): void {
-		$row = [];
-		$entity2 = new TestEntity();
+    public function testFromRowShouldNotAssignEmptyArray(): void {
+        $row = [];
+        $entity2 = new TestEntity();
 
-		$this->entity = TestEntity::fromRow($row);
-		$this->assertEquals($entity2, $this->entity);
-	}
+        $this->entity = TestEntity::fromRow($row);
+        $this->assertEquals($entity2, $this->entity);
+    }
 
 
-	public function testIdGetsConvertedToInt(): void {
-		$row = ['id' => '4'];
+    public function testIdGetsConvertedToInt(): void {
+        $row = ['id' => '4'];
 
-		$this->entity = TestEntity::fromRow($row);
-		$this->assertSame(4, $this->entity->getId());
-	}
+        $this->entity = TestEntity::fromRow($row);
+        $this->assertSame(4, $this->entity->getId());
+    }
 
 
-	public function testSetType(): void {
-		$row = ['testId' => '4'];
+    public function testSetType(): void {
+        $row = ['testId' => '4'];
 
-		$this->entity = TestEntity::fromRow($row);
-		$this->assertSame(4, $this->entity->getTestId());
-	}
+        $this->entity = TestEntity::fromRow($row);
+        $this->assertSame(4, $this->entity->getTestId());
+    }
 
 
-	public function testFromParams(): void {
-		$params = [
-			'testId' => 4,
-			'email' => 'john@doe'
-		];
+    public function testFromParams(): void {
+        $params = [
+            'testId' => 4,
+            'email' => 'john@doe'
+        ];
 
-		$entity = TestEntity::fromParams($params);
+        $entity = TestEntity::fromParams($params);
 
-		$this->assertEquals($params['testId'], $entity->getTestId());
-		$this->assertEquals($params['email'], $entity->getEmail());
-		$this->assertTrue($entity instanceof TestEntity);
-	}
+        $this->assertEquals($params['testId'], $entity->getTestId());
+        $this->assertEquals($params['email'], $entity->getEmail());
+        $this->assertTrue($entity instanceof TestEntity);
+    }
 
-	public function testSlugify(): void {
-		$entity = new TestEntity();
-		$entity->setName('Slugify this!');
-		$this->assertEquals('slugify-this', $entity->slugify('name'));
-		$entity->setName('°!"§$%&/()=?`´ß\}][{³²#\'+~*-_.:,;<>|äöüÄÖÜSlugify this!');
-		$this->assertEquals('slugify-this', $entity->slugify('name'));
-	}
+    public function testSlugify(): void {
+        $entity = new TestEntity();
+        $entity->setName('Slugify this!');
+        $this->assertEquals('slugify-this', $entity->slugify('name'));
+        $entity->setName('°!"§$%&/()=?`´ß\}][{³²#\'+~*-_.:,;<>|äöüÄÖÜSlugify this!');
+        $this->assertEquals('slugify-this', $entity->slugify('name'));
+    }
 
 
-	public static function dataSetterCasts(): array {
-		return [
-			['Id', '3', 3],
-			['smallInt', '3', 3],
-			['bigInt', '' . PHP_INT_MAX, PHP_INT_MAX],
-			['trueOrFalse', 0, false],
-			['trueOrFalse', 1, true],
-			['anotherBool', 0, false],
-			['anotherBool', 1, true],
-			['text', 33, '33'],
-			['longText', PHP_INT_MAX, '' . PHP_INT_MAX],
-		];
-	}
+    public static function dataSetterCasts(): array {
+        return [
+            ['Id', '3', 3],
+            ['smallInt', '3', 3],
+            ['bigInt', '' . PHP_INT_MAX, PHP_INT_MAX],
+            ['trueOrFalse', 0, false],
+            ['trueOrFalse', 1, true],
+            ['anotherBool', 0, false],
+            ['anotherBool', 1, true],
+            ['text', 33, '33'],
+            ['longText', PHP_INT_MAX, '' . PHP_INT_MAX],
+        ];
+    }
 
 
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataSetterCasts')]
-	public function testSetterCasts(string $field, mixed $in, mixed $out): void {
-		$entity = new TestEntity();
-		$entity->{'set' . $field}($in);
-		$this->assertSame($out, $entity->{'get' . $field}());
-	}
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataSetterCasts')]
+    public function testSetterCasts(string $field, mixed $in, mixed $out): void {
+        $entity = new TestEntity();
+        $entity->{'set' . $field}($in);
+        $this->assertSame($out, $entity->{'get' . $field}());
+    }
 
 
-	public function testSetterDoesNotCastOnNull(): void {
-		$entity = new TestEntity();
-		$entity->setId(null);
-		$this->assertSame(null, $entity->getId());
-	}
+    public function testSetterDoesNotCastOnNull(): void {
+        $entity = new TestEntity();
+        $entity->setId(null);
+        $this->assertSame(null, $entity->getId());
+    }
 
-	public function testSetterConvertsResourcesToStringProperly(): void {
-		$string = 'Definitely a string';
-		$stream = fopen('php://memory', 'r+');
-		fwrite($stream, $string);
-		rewind($stream);
-
-		$entity = new TestEntity();
-		$entity->setLongText($stream);
-		fclose($stream);
-		$this->assertSame($string, $entity->getLongText());
-	}
-
-	public function testSetterConvertsDatetime() {
-		$entity = new TestEntity();
-		$entity->setDatetime('2024-08-19 15:26:00');
-		$this->assertEquals(new \DateTimeImmutable('2024-08-19 15:26:00'), $entity->getDatetime());
-	}
-
-	public function testSetterDoesNotConvertNullOnDatetime() {
-		$entity = new TestEntity();
-		$entity->setDatetime(null);
-		$this->assertNull($entity->getDatetime());
-	}
-
-	public function testSetterConvertsTime() {
-		$entity = new TestEntity();
-		$entity->setTime('15:26:00');
-		$this->assertEquals(new \DateTime('15:26:00'), $entity->getTime());
-	}
-
-	public function testGetFieldTypes(): void {
-		$entity = new TestEntity();
-		$this->assertEquals([
-			'id' => Types::INTEGER,
-			'testId' => Types::INTEGER,
-			'smallInt' => Types::SMALLINT,
-			'bigInt' => Types::BIGINT,
-			'anotherBool' => Types::BOOLEAN,
-			'text' => Types::TEXT,
-			'longText' => Types::BLOB,
-			'time' => Types::TIME,
-			'datetime' => Types::DATETIME_IMMUTABLE,
-			'trueOrFalse' => Types::BOOLEAN,
-			'legacyInt' => Types::INTEGER,
-			'doubleNowFloat' => Types::FLOAT,
-		], $entity->getFieldTypes());
-	}
-
-
-	public function testGetItInt(): void {
-		$entity = new TestEntity();
-		$entity->setId(3);
-		$this->assertEquals(Types::INTEGER, gettype($entity->getId()));
-	}
-
-
-	public function testFieldsNotMarkedUpdatedIfNothingChanges(): void {
-		$entity = new TestEntity('hey');
-		$entity->setName('hey');
-		$this->assertEquals(0, count($entity->getUpdatedFields()));
-	}
-
-	public function testIsGetter(): void {
-		$entity = new TestEntity();
-		$entity->setTrueOrFalse(false);
-		$entity->setAnotherBool(false);
-		$this->assertThat($entity->isTrueOrFalse(), new IsType(IsType::TYPE_BOOL));
-		$this->assertThat($entity->isAnotherBool(), new IsType(IsType::TYPE_BOOL));
-	}
-
-
-	public function testIsGetterShoudFailForOtherType(): void {
-		$this->expectException(\BadFunctionCallException::class);
-
-		$entity = new TestEntity();
-		$entity->setName('hello');
-		$this->assertThat($entity->isName(), new IsType(IsType::TYPE_BOOL));
-	}
+    public function testSetterConvertsResourcesToStringProperly(): void {
+        $string = 'Definitely a string';
+        $stream = fopen('php://memory', 'r+');
+        fwrite($stream, $string);
+        rewind($stream);
+
+        $entity = new TestEntity();
+        $entity->setLongText($stream);
+        fclose($stream);
+        $this->assertSame($string, $entity->getLongText());
+    }
+
+    public function testSetterConvertsDatetime() {
+        $entity = new TestEntity();
+        $entity->setDatetime('2024-08-19 15:26:00');
+        $this->assertEquals(new \DateTimeImmutable('2024-08-19 15:26:00'), $entity->getDatetime());
+    }
+
+    public function testSetterDoesNotConvertNullOnDatetime() {
+        $entity = new TestEntity();
+        $entity->setDatetime(null);
+        $this->assertNull($entity->getDatetime());
+    }
+
+    public function testSetterConvertsTime() {
+        $entity = new TestEntity();
+        $entity->setTime('15:26:00');
+        $this->assertEquals(new \DateTime('15:26:00'), $entity->getTime());
+    }
+
+    public function testGetFieldTypes(): void {
+        $entity = new TestEntity();
+        $this->assertEquals([
+            'id' => Types::INTEGER,
+            'testId' => Types::INTEGER,
+            'smallInt' => Types::SMALLINT,
+            'bigInt' => Types::BIGINT,
+            'anotherBool' => Types::BOOLEAN,
+            'text' => Types::TEXT,
+            'longText' => Types::BLOB,
+            'time' => Types::TIME,
+            'datetime' => Types::DATETIME_IMMUTABLE,
+            'trueOrFalse' => Types::BOOLEAN,
+            'legacyInt' => Types::INTEGER,
+            'doubleNowFloat' => Types::FLOAT,
+        ], $entity->getFieldTypes());
+    }
+
+
+    public function testGetItInt(): void {
+        $entity = new TestEntity();
+        $entity->setId(3);
+        $this->assertEquals(Types::INTEGER, gettype($entity->getId()));
+    }
+
+
+    public function testFieldsNotMarkedUpdatedIfNothingChanges(): void {
+        $entity = new TestEntity('hey');
+        $entity->setName('hey');
+        $this->assertEquals(0, count($entity->getUpdatedFields()));
+    }
+
+    public function testIsGetter(): void {
+        $entity = new TestEntity();
+        $entity->setTrueOrFalse(false);
+        $entity->setAnotherBool(false);
+        $this->assertThat($entity->isTrueOrFalse(), new IsType(IsType::TYPE_BOOL));
+        $this->assertThat($entity->isAnotherBool(), new IsType(IsType::TYPE_BOOL));
+    }
+
+
+    public function testIsGetterShoudFailForOtherType(): void {
+        $this->expectException(\BadFunctionCallException::class);
+
+        $entity = new TestEntity();
+        $entity->setName('hello');
+        $this->assertThat($entity->isName(), new IsType(IsType::TYPE_BOOL));
+    }
 }

tests/lib/AppFramework/Db/QBMapperDBTest.php

Indentation +116 added lines, -116 removed lines

@@ -24,13 +24,13 @@ 
  * @method ?\DateTimeImmutable getDatetime()
  */
 class QBDBTestEntity extends Entity {
-	protected ?\DateTime $time = null;
-	protected ?\DateTimeImmutable $datetime = null;
+    protected ?\DateTime $time = null;
+    protected ?\DateTimeImmutable $datetime = null;
 
-	public function __construct() {
-		$this->addType('time', Types::TIME);
-		$this->addType('datetime', Types::DATETIME_IMMUTABLE);
-	}
+    public function __construct() {
+        $this->addType('time', Types::TIME);
+        $this->addType('datetime', Types::DATETIME_IMMUTABLE);
+    }
 }
 
 /**
@@ -39,24 +39,24 @@ 
  * @package Test\AppFramework\Db
  */
 class QBDBTestMapper extends QBMapper {
-	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'testing', QBDBTestEntity::class);
-	}
-
-	public function getParameterTypeForPropertyForTest(Entity $entity, string $property) {
-		return parent::getParameterTypeForProperty($entity, $property);
-	}
-
-	public function getById(int $id): QBDBTestEntity {
-		$qb = $this->db->getQueryBuilder();
-		$query = $qb
-			->select('*')
-			->from($this->tableName)
-			->where(
-				$qb->expr()->eq('id', $qb->createPositionalParameter($id, IQueryBuilder::PARAM_INT)),
-			);
-		return $this->findEntity($query);
-	}
+    public function __construct(IDBConnection $db) {
+        parent::__construct($db, 'testing', QBDBTestEntity::class);
+    }
+
+    public function getParameterTypeForPropertyForTest(Entity $entity, string $property) {
+        return parent::getParameterTypeForProperty($entity, $property);
+    }
+
+    public function getById(int $id): QBDBTestEntity {
+        $qb = $this->db->getQueryBuilder();
+        $query = $qb
+            ->select('*')
+            ->from($this->tableName)
+            ->where(
+                $qb->expr()->eq('id', $qb->createPositionalParameter($id, IQueryBuilder::PARAM_INT)),
+            );
+        return $this->findEntity($query);
+    }
 }
 
 /**
@@ -64,96 +64,96 @@ 
  * @group DB
  */
 class QBMapperDBTest extends TestCase {
-	protected IDBConnection $connection;
-	protected bool $schemaSetup = false;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->connection = Server::get(IDBConnection::class);
-		$this->prepareTestingTable();
-	}
-
-	public function testInsertDateTime(): void {
-		$mapper = new QBDBTestMapper($this->connection);
-		$entity = new QBDBTestEntity();
-		$entity->setTime(new \DateTime('2003-01-01 12:34:00'));
-		$entity->setDatetime(new \DateTimeImmutable('2000-01-01 23:45:00'));
-
-		$result = $mapper->insert($entity);
-		$this->assertNotNull($result->getId());
-	}
-
-	public function testRetrieveDateTime(): void {
-		$time = new \DateTime('2000-01-01 01:01:00');
-		$datetime = new \DateTimeImmutable('2000-01-01 02:02:00');
-
-		$mapper = new QBDBTestMapper($this->connection);
-		$entity = new QBDBTestEntity();
-		$entity->setTime($time);
-		$entity->setDatetime($datetime);
-
-		$result = $mapper->insert($entity);
-		$this->assertNotNull($result->getId());
-
-		$dbEntity = $mapper->getById($result->getId());
-		$this->assertEquals($time->format('H:i:s'), $dbEntity->getTime()->format('H:i:s'));
-		$this->assertEquals($datetime->format('Y-m-d H:i:s'), $dbEntity->getDatetime()->format('Y-m-d H:i:s'));
-		// The date is not saved for "time"
-		$this->assertNotEquals($time->format('Y'), $dbEntity->getTime()->format('Y'));
-	}
-
-	public function testUpdateDateTime(): void {
-		$time = new \DateTime('2000-01-01 01:01:00');
-		$datetime = new \DateTimeImmutable('2000-01-01 02:02:00');
-
-		$mapper = new QBDBTestMapper($this->connection);
-		$entity = new QBDBTestEntity();
-		$entity->setTime('now');
-		$entity->setDatetime('now');
-
-		/** @var QBDBTestEntity */
-		$entity = $mapper->insert($entity);
-		$this->assertNotNull($entity->getId());
-
-		// Update the values
-		$entity->setTime($time);
-		$entity->setDatetime($datetime);
-		$mapper->update($entity);
-
-		$dbEntity = $mapper->getById($entity->getId());
-		$this->assertEquals($time->format('H:i:s'), $dbEntity->getTime()->format('H:i:s'));
-		$this->assertEquals($datetime->format('Y-m-d H:i:s'), $dbEntity->getDatetime()->format('Y-m-d H:i:s'));
-	}
-
-	protected function prepareTestingTable(): void {
-		if ($this->schemaSetup) {
-			$this->connection->getQueryBuilder()->delete('testing')->executeStatement();
-		}
-
-		$prefix = Server::get(IConfig::class)->getSystemValueString('dbtableprefix', 'oc_');
-		$schema = $this->connection->createSchema();
-		try {
-			$schema->getTable($prefix . 'testing');
-			$this->connection->getQueryBuilder()->delete('testing')->executeStatement();
-		} catch (SchemaException $e) {
-			$this->schemaSetup = true;
-			$table = $schema->createTable($prefix . 'testing');
-			$table->addColumn('id', Types::BIGINT, [
-				'autoincrement' => true,
-				'notnull' => true,
-			]);
-
-			$table->addColumn('time', Types::TIME, [
-				'notnull' => false,
-			]);
-
-			$table->addColumn('datetime', Types::DATETIME_IMMUTABLE, [
-				'notnull' => false,
-			]);
-
-			$table->setPrimaryKey(['id']);
-			$this->connection->migrateToSchema($schema);
-		}
-	}
+    protected IDBConnection $connection;
+    protected bool $schemaSetup = false;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->connection = Server::get(IDBConnection::class);
+        $this->prepareTestingTable();
+    }
+
+    public function testInsertDateTime(): void {
+        $mapper = new QBDBTestMapper($this->connection);
+        $entity = new QBDBTestEntity();
+        $entity->setTime(new \DateTime('2003-01-01 12:34:00'));
+        $entity->setDatetime(new \DateTimeImmutable('2000-01-01 23:45:00'));
+
+        $result = $mapper->insert($entity);
+        $this->assertNotNull($result->getId());
+    }
+
+    public function testRetrieveDateTime(): void {
+        $time = new \DateTime('2000-01-01 01:01:00');
+        $datetime = new \DateTimeImmutable('2000-01-01 02:02:00');
+
+        $mapper = new QBDBTestMapper($this->connection);
+        $entity = new QBDBTestEntity();
+        $entity->setTime($time);
+        $entity->setDatetime($datetime);
+
+        $result = $mapper->insert($entity);
+        $this->assertNotNull($result->getId());
+
+        $dbEntity = $mapper->getById($result->getId());
+        $this->assertEquals($time->format('H:i:s'), $dbEntity->getTime()->format('H:i:s'));
+        $this->assertEquals($datetime->format('Y-m-d H:i:s'), $dbEntity->getDatetime()->format('Y-m-d H:i:s'));
+        // The date is not saved for "time"
+        $this->assertNotEquals($time->format('Y'), $dbEntity->getTime()->format('Y'));
+    }
+
+    public function testUpdateDateTime(): void {
+        $time = new \DateTime('2000-01-01 01:01:00');
+        $datetime = new \DateTimeImmutable('2000-01-01 02:02:00');
+
+        $mapper = new QBDBTestMapper($this->connection);
+        $entity = new QBDBTestEntity();
+        $entity->setTime('now');
+        $entity->setDatetime('now');
+
+        /** @var QBDBTestEntity */
+        $entity = $mapper->insert($entity);
+        $this->assertNotNull($entity->getId());
+
+        // Update the values
+        $entity->setTime($time);
+        $entity->setDatetime($datetime);
+        $mapper->update($entity);
+
+        $dbEntity = $mapper->getById($entity->getId());
+        $this->assertEquals($time->format('H:i:s'), $dbEntity->getTime()->format('H:i:s'));
+        $this->assertEquals($datetime->format('Y-m-d H:i:s'), $dbEntity->getDatetime()->format('Y-m-d H:i:s'));
+    }
+
+    protected function prepareTestingTable(): void {
+        if ($this->schemaSetup) {
+            $this->connection->getQueryBuilder()->delete('testing')->executeStatement();
+        }
+
+        $prefix = Server::get(IConfig::class)->getSystemValueString('dbtableprefix', 'oc_');
+        $schema = $this->connection->createSchema();
+        try {
+            $schema->getTable($prefix . 'testing');
+            $this->connection->getQueryBuilder()->delete('testing')->executeStatement();
+        } catch (SchemaException $e) {
+            $this->schemaSetup = true;
+            $table = $schema->createTable($prefix . 'testing');
+            $table->addColumn('id', Types::BIGINT, [
+                'autoincrement' => true,
+                'notnull' => true,
+            ]);
+
+            $table->addColumn('time', Types::TIME, [
+                'notnull' => false,
+            ]);
+
+            $table->addColumn('datetime', Types::DATETIME_IMMUTABLE, [
+                'notnull' => false,
+            ]);
+
+            $table->setPrimaryKey(['id']);
+            $this->connection->migrateToSchema($schema);
+        }
+    }
 }

tests/lib/AppFramework/Db/TransactionalTest.php

Indentation +49 added lines, -49 removed lines

@@ -15,64 +15,64 @@
 use Test\TestCase;
 
 class TransactionalTest extends TestCase {
-	private IDBConnection&MockObject $db;
+    private IDBConnection&MockObject $db;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->db = $this->createMock(IDBConnection::class);
-	}
+        $this->db = $this->createMock(IDBConnection::class);
+    }
 
-	public function testAtomicRollback(): void {
-		$test = new class($this->db) {
-			use TTransactional;
+    public function testAtomicRollback(): void {
+        $test = new class($this->db) {
+            use TTransactional;
 
-			public function __construct(
-				private IDBConnection $db,
-			) {
-			}
+            public function __construct(
+                private IDBConnection $db,
+            ) {
+            }
 
-			public function fail(): void {
-				$this->atomic(function (): void {
-					throw new RuntimeException('nope');
-				}, $this->db);
-			}
-		};
-		$this->db->expects(self::once())
-			->method('beginTransaction');
-		$this->db->expects(self::once())
-			->method('rollback');
-		$this->db->expects(self::never())
-			->method('commit');
-		$this->expectException(RuntimeException::class);
+            public function fail(): void {
+                $this->atomic(function (): void {
+                    throw new RuntimeException('nope');
+                }, $this->db);
+            }
+        };
+        $this->db->expects(self::once())
+            ->method('beginTransaction');
+        $this->db->expects(self::once())
+            ->method('rollback');
+        $this->db->expects(self::never())
+            ->method('commit');
+        $this->expectException(RuntimeException::class);
 
-		$test->fail();
-	}
+        $test->fail();
+    }
 
-	public function testAtomicCommit(): void {
-		$test = new class($this->db) {
-			use TTransactional;
+    public function testAtomicCommit(): void {
+        $test = new class($this->db) {
+            use TTransactional;
 
-			public function __construct(
-				private IDBConnection $db,
-			) {
-			}
+            public function __construct(
+                private IDBConnection $db,
+            ) {
+            }
 
-			public function succeed(): int {
-				return $this->atomic(function () {
-					return 3;
-				}, $this->db);
-			}
-		};
-		$this->db->expects(self::once())
-			->method('beginTransaction');
-		$this->db->expects(self::never())
-			->method('rollback');
-		$this->db->expects(self::once())
-			->method('commit');
+            public function succeed(): int {
+                return $this->atomic(function () {
+                    return 3;
+                }, $this->db);
+            }
+        };
+        $this->db->expects(self::once())
+            ->method('beginTransaction');
+        $this->db->expects(self::never())
+            ->method('rollback');
+        $this->db->expects(self::once())
+            ->method('commit');
 
-		$result = $test->succeed();
+        $result = $test->succeed();
 
-		self::assertEquals(3, $result);
-	}
+        self::assertEquals(3, $result);
+    }
 }

tests/lib/AppFramework/Http/DataResponseTest.php

Indentation +41 added lines, -41 removed lines

@@ -16,59 +16,59 @@
 use OCP\Server;
 
 class DataResponseTest extends \Test\TestCase {
-	private DataResponse $response;
+    private DataResponse $response;
 
-	protected function setUp(): void {
-		parent::setUp();
-		$this->response = new DataResponse();
-	}
+    protected function setUp(): void {
+        parent::setUp();
+        $this->response = new DataResponse();
+    }
 
 
-	public function testSetData(): void {
-		$params = ['hi', 'yo'];
-		$this->response->setData($params);
+    public function testSetData(): void {
+        $params = ['hi', 'yo'];
+        $this->response->setData($params);
 
-		$this->assertEquals(['hi', 'yo'], $this->response->getData());
-	}
+        $this->assertEquals(['hi', 'yo'], $this->response->getData());
+    }
 
 
-	public function testConstructorAllowsToSetData(): void {
-		$data = ['hi'];
-		$code = 300;
-		$response = new DataResponse($data, $code);
+    public function testConstructorAllowsToSetData(): void {
+        $data = ['hi'];
+        $code = 300;
+        $response = new DataResponse($data, $code);
 
-		$this->assertEquals($data, $response->getData());
-		$this->assertEquals($code, $response->getStatus());
-	}
+        $this->assertEquals($data, $response->getData());
+        $this->assertEquals($code, $response->getStatus());
+    }
 
 
-	public function testConstructorAllowsToSetHeaders(): void {
-		$data = ['hi'];
-		$code = 300;
-		$headers = ['test' => 'something'];
-		$response = new DataResponse($data, $code, $headers);
+    public function testConstructorAllowsToSetHeaders(): void {
+        $data = ['hi'];
+        $code = 300;
+        $headers = ['test' => 'something'];
+        $response = new DataResponse($data, $code, $headers);
 
-		$expectedHeaders = [
-			'Cache-Control' => 'no-cache, no-store, must-revalidate',
-			'Content-Security-Policy' => "default-src 'none';base-uri 'none';manifest-src 'self';frame-ancestors 'none'",
-			'Feature-Policy' => "autoplay 'none';camera 'none';fullscreen 'none';geolocation 'none';microphone 'none';payment 'none'",
-			'X-Robots-Tag' => 'noindex, nofollow',
-			'X-Request-Id' => Server::get(IRequest::class)->getId(),
-		];
-		$expectedHeaders = array_merge($expectedHeaders, $headers);
+        $expectedHeaders = [
+            'Cache-Control' => 'no-cache, no-store, must-revalidate',
+            'Content-Security-Policy' => "default-src 'none';base-uri 'none';manifest-src 'self';frame-ancestors 'none'",
+            'Feature-Policy' => "autoplay 'none';camera 'none';fullscreen 'none';geolocation 'none';microphone 'none';payment 'none'",
+            'X-Robots-Tag' => 'noindex, nofollow',
+            'X-Request-Id' => Server::get(IRequest::class)->getId(),
+        ];
+        $expectedHeaders = array_merge($expectedHeaders, $headers);
 
-		$this->assertEquals($data, $response->getData());
-		$this->assertEquals($code, $response->getStatus());
-		$this->assertEquals($expectedHeaders, $response->getHeaders());
-	}
+        $this->assertEquals($data, $response->getData());
+        $this->assertEquals($code, $response->getStatus());
+        $this->assertEquals($expectedHeaders, $response->getHeaders());
+    }
 
 
-	public function testChainability(): void {
-		$params = ['hi', 'yo'];
-		$this->response->setData($params)
-			->setStatus(Http::STATUS_NOT_FOUND);
+    public function testChainability(): void {
+        $params = ['hi', 'yo'];
+        $this->response->setData($params)
+            ->setStatus(Http::STATUS_NOT_FOUND);
 
-		$this->assertEquals(Http::STATUS_NOT_FOUND, $this->response->getStatus());
-		$this->assertEquals(['hi', 'yo'], $this->response->getData());
-	}
+        $this->assertEquals(Http::STATUS_NOT_FOUND, $this->response->getStatus());
+        $this->assertEquals(['hi', 'yo'], $this->response->getData());
+    }
 }

tests/lib/AppFramework/Http/ContentSecurityPolicyTest.php

Indentation +368 added lines, -368 removed lines

@@ -18,500 +18,500 @@
  * @package OC\AppFramework\Http
  */
 class ContentSecurityPolicyTest extends \Test\TestCase {
-	private ContentSecurityPolicy $contentSecurityPolicy;
+    private ContentSecurityPolicy $contentSecurityPolicy;
 
-	protected function setUp(): void {
-		parent::setUp();
-		$this->contentSecurityPolicy = new ContentSecurityPolicy();
-	}
+    protected function setUp(): void {
+        parent::setUp();
+        $this->contentSecurityPolicy = new ContentSecurityPolicy();
+    }
 
-	public function testGetPolicyDefault(): void {
-		$defaultPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
-		$this->assertSame($defaultPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+    public function testGetPolicyDefault(): void {
+        $defaultPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+        $this->assertSame($defaultPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyScriptDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyScriptDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyScriptDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com www.nextcloud.org;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyScriptDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com www.nextcloud.org;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowScriptDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowScriptDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowScriptDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowScriptDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' www.nextcloud.com;style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowScriptDomainMultipleStacked(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowScriptDomainMultipleStacked(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.org')->disallowScriptDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedScriptDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowScriptDomain('www.nextcloud.org')->disallowScriptDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyScriptDisallowEval(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyScriptDisallowEval(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->allowEvalScript(false);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->allowEvalScript(false);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStyleDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStyleDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStyleDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com www.nextcloud.org 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStyleDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com www.nextcloud.org 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowStyleDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowStyleDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowStyleDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowStyleDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowStyleDomainMultipleStacked(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowStyleDomainMultipleStacked(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.org')->disallowStyleDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowStyleDomain('www.nextcloud.org')->disallowStyleDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStyleAllowInline(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStyleAllowInline(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->allowInlineStyle(true);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->allowInlineStyle(true);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStyleAllowInlineWithDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStyleAllowInlineWithDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' www.nextcloud.com 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedStyleDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStyleDisallowInline(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStyleDisallowInline(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->allowInlineStyle(false);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->allowInlineStyle(false);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyImageDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyImageDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyImageDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com www.nextcloud.org;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyImageDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com www.nextcloud.org;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowImageDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowImageDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowImageDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowImageDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob: www.nextcloud.com;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowImageDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowImageDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.org')->disallowImageDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedImageDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowImageDomain('www.nextcloud.org')->disallowImageDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyFontDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyFontDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyFontDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com www.nextcloud.org;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyFontDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com www.nextcloud.org;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFontDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFontDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFontDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFontDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data: www.nextcloud.com;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFontDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFontDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.org')->disallowFontDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFontDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFontDomain('www.nextcloud.org')->disallowFontDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyConnectDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com;media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyConnectDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com;media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyConnectDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com www.nextcloud.org;media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyConnectDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com www.nextcloud.org;media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowConnectDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowConnectDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowConnectDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com;media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowConnectDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self' www.nextcloud.com;media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowConnectDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowConnectDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.org')->disallowConnectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedConnectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowConnectDomain('www.nextcloud.org')->disallowConnectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyMediaDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyMediaDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyMediaDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyMediaDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowMediaDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowMediaDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowMediaDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowMediaDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self' www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowMediaDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowMediaDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.org')->disallowMediaDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedMediaDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowMediaDomain('www.nextcloud.org')->disallowMediaDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyObjectDomainValid(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyObjectDomainValid(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyObjectDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyObjectDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowObjectDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowObjectDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowObjectDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowObjectDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';object-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowObjectDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowObjectDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.org')->disallowObjectDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedObjectDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowObjectDomain('www.nextcloud.org')->disallowObjectDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetAllowedFrameDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetAllowedFrameDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyFrameDomainValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyFrameDomainValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com www.nextcloud.org;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFrameDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFrameDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFrameDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.org')->disallowFrameDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFrameDomain('www.nextcloud.org')->disallowFrameDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetAllowedChildSrcDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src child.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetAllowedChildSrcDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src child.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyChildSrcValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src child.nextcloud.com child.nextcloud.org;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyChildSrcValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src child.nextcloud.com child.nextcloud.org;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('child.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowChildSrcDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowChildSrcDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowChildSrcDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowChildSrcDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';child-src www.nextcloud.com;frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowChildSrcDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowChildSrcDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org')->disallowChildSrcDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org')->disallowChildSrcDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
 
 
-	public function testGetAllowedFrameAncestorDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' sub.nextcloud.com;form-action 'self'";
+    public function testGetAllowedFrameAncestorDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' sub.nextcloud.com;form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameAncestorDomain('sub.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameAncestorDomain('sub.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyFrameAncestorValidMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' sub.nextcloud.com foo.nextcloud.com;form-action 'self'";
+    public function testGetPolicyFrameAncestorValidMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' sub.nextcloud.com foo.nextcloud.com;form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameAncestorDomain('sub.nextcloud.com');
-		$this->contentSecurityPolicy->addAllowedFrameAncestorDomain('foo.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameAncestorDomain('sub.nextcloud.com');
+        $this->contentSecurityPolicy->addAllowedFrameAncestorDomain('foo.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameAncestorDomain(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFrameAncestorDomain(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameAncestorDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFrameAncestorDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameAncestorDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFrameAncestorDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameAncestorDomainMultiple(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' www.nextcloud.com;form-action 'self'";
+    public function testGetPolicyDisallowFrameAncestorDomainMultiple(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self' www.nextcloud.com;form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedFrameAncestorDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowFrameAncestorDomain('www.nextcloud.org');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedFrameAncestorDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowFrameAncestorDomain('www.nextcloud.org');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyDisallowFrameAncestorDomainMultipleStakes(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyDisallowFrameAncestorDomainMultipleStakes(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
-		$this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org')->disallowChildSrcDomain('www.nextcloud.com');
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->addAllowedChildSrcDomain('www.nextcloud.com');
+        $this->contentSecurityPolicy->disallowChildSrcDomain('www.nextcloud.org')->disallowChildSrcDomain('www.nextcloud.com');
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyUnsafeEval(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' 'unsafe-eval';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyUnsafeEval(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' 'unsafe-eval';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->allowEvalScript(true);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->allowEvalScript(true);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyUnsafeWasmEval(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' 'wasm-unsafe-eval';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyUnsafeWasmEval(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self' 'wasm-unsafe-eval';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->allowEvalWasm(true);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->allowEvalWasm(true);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyNonce(): void {
-		$nonce = base64_encode('my-nonce');
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyNonce(): void {
+        $nonce = base64_encode('my-nonce');
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useJsNonce($nonce);
-		$this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useJsNonce($nonce);
+        $this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyNonceDefault(): void {
-		$nonce = base64_encode('my-nonce');
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'nonce-$nonce';script-src-elem 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyNonceDefault(): void {
+        $nonce = base64_encode('my-nonce');
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'nonce-$nonce';script-src-elem 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useJsNonce($nonce);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useJsNonce($nonce);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyNonceStrictDynamic(): void {
-		$nonce = base64_encode('my-nonce');
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyNonceStrictDynamic(): void {
+        $nonce = base64_encode('my-nonce');
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useJsNonce($nonce);
-		$this->contentSecurityPolicy->useStrictDynamic(true);
-		$this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useJsNonce($nonce);
+        $this->contentSecurityPolicy->useStrictDynamic(true);
+        $this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyNonceStrictDynamicDefault(): void {
-		$nonce = base64_encode('my-nonce');
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyNonceStrictDynamicDefault(): void {
+        $nonce = base64_encode('my-nonce');
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'strict-dynamic' 'nonce-$nonce';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useJsNonce($nonce);
-		$this->contentSecurityPolicy->useStrictDynamic(true);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useJsNonce($nonce);
+        $this->contentSecurityPolicy->useStrictDynamic(true);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStrictDynamicOnScriptsOff(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStrictDynamicOnScriptsOff(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useStrictDynamicOnScripts(false);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 
-	public function testGetPolicyStrictDynamicAndStrictDynamicOnScripts(): void {
-		$expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
+    public function testGetPolicyStrictDynamicAndStrictDynamicOnScripts(): void {
+        $expectedPolicy = "default-src 'none';base-uri 'none';manifest-src 'self';script-src 'self';style-src 'self' 'unsafe-inline';img-src 'self' data: blob:;font-src 'self' data:;connect-src 'self';media-src 'self';frame-ancestors 'self';form-action 'self'";
 
-		$this->contentSecurityPolicy->useStrictDynamic(true);
-		$this->contentSecurityPolicy->useStrictDynamicOnScripts(true);
-		$this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
-	}
+        $this->contentSecurityPolicy->useStrictDynamic(true);
+        $this->contentSecurityPolicy->useStrictDynamicOnScripts(true);
+        $this->assertSame($expectedPolicy, $this->contentSecurityPolicy->buildPolicy());
+    }
 }