Skip to content

Commit 58cf6c7

Browse files
committed
test: Add missing override
Also update stale comment that mentioned a function that no longer exists. Signed-off-by: Micke Nordin <kano@sunet.se>
1 parent 99ac009 commit 58cf6c7

3 files changed

Lines changed: 5 additions & 2 deletions

File tree

apps/cloud_federation_api/lib/Db/OcmTokenMapMapper.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,9 @@ public function getByAccessTokenId(int $accessTokenId): OcmTokenMap {
3434
}
3535

3636
/**
37-
* All mappings for a refresh token. Unlike findByRefreshToken this
38-
* tolerates the duplicate rows a concurrent exchange can create.
37+
* All mappings for a refresh token. Returns every match, since a refresh
38+
* token can back several access tokens and concurrent exchanges may add
39+
* duplicate rows.
3940
*
4041
* @return OcmTokenMap[]
4142
*/

apps/cloud_federation_api/tests/Listener/ShareDeletedListenerTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ class ShareDeletedListenerTest extends TestCase {
2222
private IProvider&MockObject $tokenProvider;
2323
private ShareDeletedListener $listener;
2424

25+
#[\Override]
2526
protected function setUp(): void {
2627
parent::setUp();
2728

apps/cloud_federation_api/tests/Service/OcmTokenServiceTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ class OcmTokenServiceTest extends TestCase {
2424
private IProvider&MockObject $tokenProvider;
2525
private OcmTokenService $service;
2626

27+
#[\Override]
2728
protected function setUp(): void {
2829
parent::setUp();
2930

0 commit comments

Comments
 (0)