Skip to content

Commit 545af5e

Browse files
committed
minor #51500 Remove getUsername methods in tests (alamirault)
This PR was merged into the 7.0 branch. Discussion ---------- Remove `getUsername` methods in tests | Q | A | ------------- | --- | Branch? | 7.0 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A This is allowed thanks to bumps to `^6.4|^7.0` in composer.json on 7.0 branch Commits ------- 85cafd35ac Remove getUsername methods in tests
2 parents 9e72f0e + db20f0f commit 545af5e

File tree

3 files changed

+0
-14
lines changed

3 files changed

+0
-14
lines changed

Tests/EventListener/PasswordMigratingListenerTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,10 +175,6 @@ public function eraseCredentials(): void
175175
{
176176
}
177177

178-
public function getUsername(): string
179-
{
180-
}
181-
182178
public function getUserIdentifier(): string
183179
{
184180
}

Tests/Firewall/ContextListenerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -552,11 +552,6 @@ public function setUser($user): void
552552
$this->user = $user;
553553
}
554554

555-
public function getUsername(): string
556-
{
557-
return $this->user->getUserIdentifier();
558-
}
559-
560555
public function getUserIdentifier(): string
561556
{
562557
return $this->getUserIdentifier();

Tests/LoginLink/LoginLinkHandlerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -289,11 +289,6 @@ public function getSalt(): string
289289
return '';
290290
}
291291

292-
public function getUsername(): string
293-
{
294-
return $this->username;
295-
}
296-
297292
public function getUserIdentifier(): string
298293
{
299294
return $this->username;

0 commit comments

Comments
 (0)