Merge pull request #131 from mano-lis/patch-1

coopTilleuls · Dec 5, 2023 · 45a6979 · 45a6979
2 parents abb88cc + 27e9f9e
commit 45a6979
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/src/Manager/PasswordTokenManager.php b/src/Manager/PasswordTokenManager.php
@@ -44,8 +44,8 @@ public function createPasswordToken($user, \DateTime $expiresAt = null, Provider
         }
 
         if (!$expiresAt) {
-            $expiredAt = new \DateTime($provider->getPasswordTokenExpiredIn());
-            $expiredAt->setTime((int) $expiredAt->format('H'), (int) $expiredAt->format('i'), (int) $expiredAt->format('s'), 0);
+            $expiresAt = new \DateTime($provider->getPasswordTokenExpiredIn());
+            $expiresAt->setTime((int) $expiresAt->format('H'), (int) $expiresAt->format('i'), (int) $expiresAt->format('s'), 0);
         }
 
         $tokenClass = $provider->getPasswordTokenClass();

diff --git a/tests/Manager/PasswordTokenManagerTest.php b/tests/Manager/PasswordTokenManagerTest.php
@@ -63,6 +63,21 @@ public function testCreatePasswordToken(): void
         $this->manager->createPasswordToken($this->userMock, new \DateTime('2016-10-11 10:00:00'));
     }
 
+    public function testCreatePasswordTokenWithoutExpirationDate(): void
+    {
+        $this->managerMock->expects($this->once())->method('persist')->with($this->callback(function ($object) {
+            return $object instanceof AbstractPasswordToken
+                && preg_match('/^[A-z\d]{50}$/', $object->getToken())
+                && $this->userMock === $object->getUser();
+        }));
+
+        $this->providerChainMock->expects($this->once())->method('get')->willReturn($this->providerMock);
+        $this->providerMock->expects($this->once())->method('getPasswordTokenClass')->willReturn(PasswordToken::class);
+        $this->providerMock->expects($this->once())->method('getManager')->willReturn($this->managerMock);
+
+        $this->manager->createPasswordToken($this->userMock);
+    }
+
     public function testFindOneByToken(): void
     {
         $this->managerMock->expects($this->once())->method('findOneBy')->with(PasswordToken::class, ['token' => 'foo'])->willReturn('bar');