Fix tests

kreait · Mar 4, 2025 · 75d6620 · 75d6620
1 parent f07ec1c
commit 75d6620
Showing 1 changed file with 28 additions and 11 deletions.
diff --git a/tests/Integration/Request/UpdateUserTest.php b/tests/Integration/Request/UpdateUserTest.php
@@ -5,10 +5,12 @@
 namespace Kreait\Firebase\Tests\Integration\Request;
 
 use DateTimeImmutable;
+use Kreait\Firebase\Auth\MfaInfo;
 use Kreait\Firebase\Contract\Auth;
 use Kreait\Firebase\Request\CreateUser;
 use Kreait\Firebase\Request\UpdateUser;
 use Kreait\Firebase\Tests\IntegrationTestCase;
+use Kreait\Firebase\Util\DT;
 use PHPUnit\Framework\Attributes\Test;
 
 use function bin2hex;
@@ -207,7 +209,9 @@ public function timeOfLastPasswordUpdateIsIncluded(): void
     #[Test]
     public function setMultiFactor(): void
     {
-        $user = $this->auth->createUser(CreateUser::new());
+        $user = $this->auth->createUser(
+            CreateUser::new()->withVerifiedEmail(self::randomEmail(__FUNCTION__)),
+        );
 
         $factor = [
             'mfaEnrollmentId' => '85dc3f7b-7bef-45b9-b9e6-0a1c2c656fed',
@@ -216,18 +220,27 @@ public function setMultiFactor(): void
             'enrolledAt' => '2025-02-28T15:30:00Z',
         ];
 
-        $check = $this->auth->updateUser($user->uid, ['enrolledfactors' => [$factor]]);
+        $enrolledAt = DT::toUTCDateTimeImmutable($factor['enrolledAt']);
+
+        try {
+            $check = $this->auth->updateUser($user->uid, ['multifactors' => [$factor]]);
 
-        $this->assertEquals($factor['mfaEnrollmentId'], $check->mfaInfo?->mfaEnrollmentId);
-        $this->assertEquals($factor['phoneInfo'], $check->mfaInfo?->phoneInfo);
-        $this->assertEquals($factor['displayName'], $check->mfaInfo?->displayName);
-        $this->assertEquals($factor['enrolledAt'], $check->mfaInfo?->enrolledAt);
+            $this->assertInstanceOf(MfaInfo::class, $check->mfaInfo);
+            $this->assertSame($factor['mfaEnrollmentId'], $check->mfaInfo->mfaEnrollmentId);
+            $this->assertSame($factor['phoneInfo'], $check->mfaInfo->phoneInfo);
+            $this->assertSame($factor['displayName'], $check->mfaInfo->displayName);
+            $this->assertEquals($enrolledAt, $check->mfaInfo->enrolledAt);
+        } finally {
+            $this->auth->deleteUser($user->uid);
+        }
     }
 
     #[Test]
     public function resetMultiFactor(): void
     {
-        $user = $this->auth->createUser(CreateUser::new());
+        $user = $this->auth->createUser(
+            CreateUser::new()->withVerifiedEmail(self::randomEmail(__FUNCTION__)),
+        );
 
         $factor = [
             'mfaEnrollmentId' => '85dc3f7b-7bef-45b9-b9e6-0a1c2c656fed',
@@ -236,12 +249,16 @@ public function resetMultiFactor(): void
             'enrolledAt' => '2025-02-28T15:30:00Z',
         ];
 
-        $updatedUser = $this->auth->updateUser($user->uid, ['enrolledfactors' => [$factor]]);
+        try {
+            $updatedUser = $this->auth->updateUser($user->uid, ['multifactors' => [$factor]]);
 
-        $this->assertNotNull($updatedUser->mfaInfo);
+            $this->assertInstanceOf(MfaInfo::class, $updatedUser->mfaInfo);
 
-        $check = $this->auth->updateUser($user->uid, ['resetmultifactor' => true]);
+            $check = $this->auth->updateUser($user->uid, ['resetmultifactor' => true]);
 
-        $this->assertNull($check->mfaInfo);
+            $this->assertNotInstanceOf(MfaInfo::class, $check->mfaInfo);
+        } finally {
+            $this->auth->deleteUser($user->uid);
+        }
     }
 }