ci: Fix unit tests

Signed-off-by: Joas Schilling <[email protected]>

tests/lib/Validator/CommonPasswordsValidatorTest.php

@@ -37,7 +37,7 @@ protected function setUp(): void {
 	/**
 	 * @dataProvider dataValidate
 	 */
-	public function testValidate(string $password, bool $enforced, bool $valid) {
+	public function testValidate(string $password, bool $enforced, bool $valid): void {
 		$this->config->method('getEnforceNonCommonPassword')
 			->willReturn($enforced);
 
@@ -49,15 +49,15 @@ public function testValidate(string $password, bool $enforced, bool $valid) {
 		$this->validator->validate($password);
 	}
 
-	public static function dataValidate() {
+	public static function dataValidate(): array {
 		$attempts = [
 			['banana', false,  true],
 			['bananabananabananabanana', false,  true],
 			['banana',  true, false],
 			['bananabananabananabanana',  true,  true],
 		];
 		for ($i = 1; $i <= 39; $i++) {
-			$attempts[] = [str_repeat('$', $i), true, true];
+			$attempts[] = [str_repeat('$', $i), true, $i !== 6];
 		}
 		return $attempts;
 	}