Skip to content

Commit 906c13d

Browse files
committed
wip - refactor helper unit test
1 parent 20857c9 commit 906c13d

File tree

1 file changed

+14
-32
lines changed

1 file changed

+14
-32
lines changed

tests/Unit/ResetPasswordHelperTest.php

+14-32
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
use SymfonyCasts\Bundle\ResetPassword\Exception\ExpiredResetPasswordTokenException;
1515
use SymfonyCasts\Bundle\ResetPassword\Exception\InvalidResetPasswordTokenException;
1616
use SymfonyCasts\Bundle\ResetPassword\Exception\TooManyPasswordRequestsException;
17+
use SymfonyCasts\Bundle\ResetPassword\Generator\ResetPasswordRandomGenerator;
1718
use SymfonyCasts\Bundle\ResetPassword\Generator\ResetPasswordTokenGenerator;
1819
use SymfonyCasts\Bundle\ResetPassword\Model\ResetPasswordRequestInterface;
1920
use SymfonyCasts\Bundle\ResetPassword\Persistence\ResetPasswordRequestRepositoryInterface;
@@ -27,35 +28,16 @@
2728
*/
2829
class ResetPasswordHelperTest extends TestCase
2930
{
30-
/**
31-
* @var MockObject|ResetPasswordRequestRepositoryInterface
32-
*/
33-
private $mockRepo;
34-
35-
/**
36-
* @var MockObject|ResetPasswordTokenGenerator
37-
*/
38-
private $mockTokenGenerator;
39-
40-
/**
41-
* @var MockObject|ResetPasswordRequestInterface
42-
*/
43-
private $mockResetRequest;
44-
45-
/**
46-
* @var MockObject|ResetPasswordCleaner
47-
*/
48-
private $mockCleaner;
49-
50-
/**
51-
* @var string
52-
*/
53-
private $randomToken;
31+
private MockObject&ResetPasswordRequestRepositoryInterface $mockRepo;
32+
private ResetPasswordTokenGenerator $tokenGenerator;
33+
private MockObject&ResetPasswordRequestInterface $mockResetRequest;
34+
private MockObject|ResetPasswordCleaner $mockCleaner;
35+
private string $randomToken;
5436

5537
protected function setUp(): void
5638
{
5739
$this->mockRepo = $this->createMock(ResetPasswordRequestRepositoryInterface::class);
58-
$this->mockTokenGenerator = $this->createMock(ResetPasswordTokenGenerator::class);
40+
$this->tokenGenerator = new ResetPasswordTokenGenerator('secret-key', new ResetPasswordRandomGenerator());
5941
$this->mockCleaner = $this->createMock(ResetPasswordCleaner::class);
6042
$this->mockResetRequest = $this->createMock(ResetPasswordRequestInterface::class);
6143
$this->randomToken = bin2hex(random_bytes(20));
@@ -112,7 +94,7 @@ public function testHasUserThrottlingReturnsNullIfNotBeforeThrottleTime(): void
11294
;
11395

11496
$helper = new ResetPasswordHelper(
115-
$this->mockTokenGenerator,
97+
$this->tokenGenerator,
11698
$this->mockCleaner,
11799
$this->mockRepo,
118100
99999999,
@@ -131,7 +113,7 @@ public function testExceptionThrownIfRequestBeforeThrottleLimit(): void
131113
;
132114

133115
$helper = new ResetPasswordHelper(
134-
$this->mockTokenGenerator,
116+
$this->tokenGenerator,
135117
$this->mockCleaner,
136118
$this->mockRepo,
137119
99999999,
@@ -332,7 +314,7 @@ public function testExpiresAtUsesCurrentTimeZone(): void
332314
public function testExpiresAtUsingDefault(): void
333315
{
334316
$helper = new ResetPasswordHelper(
335-
$this->mockTokenGenerator,
317+
$this->tokenGenerator,
336318
$this->mockCleaner,
337319
$this->mockRepo,
338320
60,
@@ -349,7 +331,7 @@ public function testExpiresAtUsingDefault(): void
349331
public function testExpiresAtUsingOverride(): void
350332
{
351333
$helper = new ResetPasswordHelper(
352-
$this->mockTokenGenerator,
334+
$this->tokenGenerator,
353335
$this->mockCleaner,
354336
$this->mockRepo,
355337
60,
@@ -366,7 +348,7 @@ public function testExpiresAtUsingOverride(): void
366348
public function testFakeTokenExpiresAtUsingDefault(): void
367349
{
368350
$helper = new ResetPasswordHelper(
369-
$this->mockTokenGenerator,
351+
$this->tokenGenerator,
370352
$this->mockCleaner,
371353
$this->mockRepo,
372354
60,
@@ -383,7 +365,7 @@ public function testFakeTokenExpiresAtUsingDefault(): void
383365
public function testFakeTokenExpiresAtUsingOverride(): void
384366
{
385367
$helper = new ResetPasswordHelper(
386-
$this->mockTokenGenerator,
368+
$this->tokenGenerator,
387369
$this->mockCleaner,
388370
$this->mockRepo,
389371
60,
@@ -400,7 +382,7 @@ public function testFakeTokenExpiresAtUsingOverride(): void
400382
private function getPasswordResetHelper(): ResetPasswordHelper
401383
{
402384
return new ResetPasswordHelper(
403-
$this->mockTokenGenerator,
385+
$this->tokenGenerator,
404386
$this->mockCleaner,
405387
$this->mockRepo,
406388
99999999,

0 commit comments

Comments
 (0)