From 04cba9702ce439e5cb1529f9c54ec649cafba098 Mon Sep 17 00:00:00 2001 From: James Griffin Date: Wed, 29 Jul 2026 22:40:51 -0300 Subject: [PATCH] Fix main: two signup fixtures use a now-rejected password MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit #154 and #155 each passed on their own branch and broke on landing together. #154 added two guardian-signup tests using 'password123' as their fixture; #155 then added PasswordPolicy, which rejects exactly that. Neither branch ever saw the other's change, because #155 was cut from main before #154 merged. Both tests now use the same policy-clearing fixture as the rest of the file. The deliberate 'password123' in the rejected-passwords provider stays — that one is the point. Co-Authored-By: Claude Opus 5 --- tests/Unit/Auth/RegistrationPageTest.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/Unit/Auth/RegistrationPageTest.php b/tests/Unit/Auth/RegistrationPageTest.php index 1beb1d4..7874c66 100644 --- a/tests/Unit/Auth/RegistrationPageTest.php +++ b/tests/Unit/Auth/RegistrationPageTest.php @@ -738,7 +738,7 @@ class RegistrationPageTest extends TestCase public function testGuardianSignupRejectsAHalfFilledChildRatherThanDroppingIt(): void { $_POST = [ - 'password' => 'password123', + 'password' => 'thistle-marrow-42', 'display_name' => 'Grace', 'us_is_guardian' => '1', 'children' => [ @@ -764,7 +764,7 @@ class RegistrationPageTest extends TestCase public function testGuardianSignupRejectsAChildWithoutAUsableBirthYear(string $submitted): void { $_POST = [ - 'password' => 'password123', + 'password' => 'thistle-marrow-42', 'display_name' => 'Grace', 'us_is_guardian' => '1', 'children' => [['name' => 'Ada', 'birth_year' => $submitted, 'answers' => []]], -- 2.54.0