Skip to content

Commit baac836

Browse files
committed
minor #51534 [DoctrineBridge][Security] Rename loadUserByUsername tests to loadUserByIdentifier (alamirault)
This PR was merged into the 6.3 branch. Discussion ---------- [DoctrineBridge][Security] Rename `loadUserByUsername` tests to `loadUserByIdentifier` | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead --> | License | MIT | Doc PR | symfony/symfony-docs#... <!-- required for new features --> Tests use `loadUserByIdentifier` method but name were `loadUserByUsername` (legacy) Commits ------- e32c833e04 [DoctrineBridge][Security] Rename loadUserByUsername tests to loadUserByIdentifier
2 parents e399e6a + 1dde6ee commit baac836

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

Tests/User/ChainUserProviderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323

2424
class ChainUserProviderTest extends TestCase
2525
{
26-
public function testLoadUserByUsername()
26+
public function testLoadUserByIdentifier()
2727
{
2828
$provider1 = $this->createMock(InMemoryUserProvider::class);
2929
$provider1
@@ -45,7 +45,7 @@ public function testLoadUserByUsername()
4545
$this->assertSame($account, $provider->loadUserByIdentifier('foo'));
4646
}
4747

48-
public function testLoadUserByUsernameThrowsUserNotFoundException()
48+
public function testLoadUserByIdentifierThrowsUserNotFoundException()
4949
{
5050
$this->expectException(UserNotFoundException::class);
5151
$provider1 = $this->createMock(InMemoryUserProvider::class);

Tests/User/InMemoryUserProviderTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public function testCreateUserAlreadyExist()
6868
$provider->createUser(new InMemoryUser('fabien', 'foo'));
6969
}
7070

71-
public function testLoadUserByUsernameDoesNotExist()
71+
public function testLoadUserByIdentifierDoesNotExist()
7272
{
7373
$this->expectException(UserNotFoundException::class);
7474
$provider = new InMemoryUserProvider();

0 commit comments

Comments
 (0)