upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-04-20 22:00:39 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 39

Fix unit tests

Browse source 
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
Joas Schilling 2023-02-09 16:15:47 +01:00
parent 7a85a1596e
commit a81d8ecef5
No known key found for this signature in database
GPG key ID: C400AAF20C1BB6FC
1 changed files with 42 additions and 47 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

89
tests/lib/Authentication/Token/PublicKeyTokenProviderTest.php
View file

@ -93,7 +93,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testGenerateToken() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -115,7 +115,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testGenerateTokenNoPassword(): void {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -138,7 +138,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testGenerateTokenLongPassword() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = '';
@ -157,7 +157,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testGenerateTokenInvalidName() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -222,7 +222,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testGetPassword() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -253,7 +253,7 @@ class PublicKeyTokenProviderTest extends TestCase {
public function testGetPasswordInvalidToken() {
$this->expectException(\OC\Authentication\Exceptions\InvalidTokenException::class);
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -270,7 +270,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testSetPassword() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -291,13 +291,13 @@ class PublicKeyTokenProviderTest extends TestCase {
$this->mapper->expects($this->once())
->method('update')
->with($this->callback(function ($token) use ($newpass) {
return $newpass === $this->tokenProvider->getPassword($token, 'token');
return $newpass === $this->tokenProvider->getPassword($token, 'tokentokentokentokentoken');
}));
$this->tokenProvider->setPassword($actual, $token, $newpass);
$this->assertSame($newpass, $this->tokenProvider->getPassword($actual, 'token'));
$this->assertSame($newpass, $this->tokenProvider->getPassword($actual, 'tokentokentokentokentoken'));
}
@ -312,12 +312,12 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testInvalidateToken() {
$this->mapper->expects($this->at(0))
$this->mapper->expects($this->exactly(2))
->method('invalidate')
->with(hash('sha512', 'token7'.'1f4h9s'));
$this->mapper->expects($this->at(1))
->method('invalidate')
->with(hash('sha512', 'token7'));
->withConsecutive(
[hash('sha512', 'token7'.'1f4h9s')],
[hash('sha512', 'token7')]
);
$this->tokenProvider->invalidateToken('token7');
}
@ -352,7 +352,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testRenewSessionTokenWithoutPassword() {
$token = 'oldId';
$token = 'oldIdtokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = null;
@ -364,7 +364,7 @@ class PublicKeyTokenProviderTest extends TestCase {
$this->mapper
->expects($this->once())
->method('getToken')
->with(hash('sha512', 'oldId' . '1f4h9s'))
->with(hash('sha512', 'oldIdtokentokentokentoken' . '1f4h9s'))
->willReturn($oldToken);
$this->mapper
->expects($this->once())
@ -384,11 +384,11 @@ class PublicKeyTokenProviderTest extends TestCase {
return $token === $oldToken;
}));
$this->tokenProvider->renewSessionToken('oldId', 'newId');
$this->tokenProvider->renewSessionToken('oldIdtokentokentokentoken', 'newIdtokentokentokentoken');
}
public function testRenewSessionTokenWithPassword(): void {
$token = 'oldId';
$token = 'oldIdtokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'password';
@ -404,7 +404,7 @@ class PublicKeyTokenProviderTest extends TestCase {
$this->mapper
->expects($this->once())
->method('getToken')
->with(hash('sha512', 'oldId' . '1f4h9s'))
->with(hash('sha512', 'oldIdtokentokentokentoken' . '1f4h9s'))
->willReturn($oldToken);
$this->mapper
->expects($this->once())
@ -416,7 +416,7 @@ class PublicKeyTokenProviderTest extends TestCase {
$token->getType() === IToken::DO_NOT_REMEMBER &&
$token->getLastActivity() === $this->time &&
$token->getPassword() !== null &&
$this->tokenProvider->getPassword($token, 'newId') === 'password';
$this->tokenProvider->getPassword($token, 'newIdtokentokentokentoken') === 'password';
}));
$this->mapper
->expects($this->once())
@ -425,7 +425,7 @@ class PublicKeyTokenProviderTest extends TestCase {
return $token === $oldToken;
}));
$this->tokenProvider->renewSessionToken('oldId', 'newId');
$this->tokenProvider->renewSessionToken('oldIdtokentokentokentoken', 'newIdtokentokentokentoken');
}
public function testGetToken(): void {
@ -438,37 +438,32 @@ class PublicKeyTokenProviderTest extends TestCase {
$this->mapper->method('getToken')
->with(
$this->callback(function (string $token) {
return hash('sha512', 'unhashedToken'.'1f4h9s') === $token;
return hash('sha512', 'unhashedTokentokentokentokentoken'.'1f4h9s') === $token;
})
)->willReturn($token);
$this->assertSame($token, $this->tokenProvider->getToken('unhashedToken'));
$this->assertSame($token, $this->tokenProvider->getToken('unhashedTokentokentokentokentoken'));
}
public function testGetInvalidToken() {
$this->expectException(InvalidTokenException::class);
$this->mapper->expects($this->at(0))
$this->mapper->expects($this->exactly(2))
->method('getToken')
->with(
$this->callback(function (string $token): bool {
return hash('sha512', 'unhashedToken'.'1f4h9s') === $token;
})
->withConsecutive(
[$this->callback(function (string $token): bool {
return hash('sha512', 'unhashedTokentokentokentokentoken'.'1f4h9s') === $token;
})],
[$this->callback(function (string $token): bool {
return hash('sha512', 'unhashedTokentokentokentokentoken') === $token;
})]
)->willThrowException(new DoesNotExistException('nope'));
$this->mapper->expects($this->at(1))
->method('getToken')
->with(
$this->callback(function (string $token): bool {
return hash('sha512', 'unhashedToken') === $token;
})
)->willThrowException(new DoesNotExistException('nope'));
$this->tokenProvider->getToken('unhashedToken');
$this->tokenProvider->getToken('unhashedTokentokentokentokentoken');
}
public function testGetExpiredToken() {
$token = 'token';
$token = 'tokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'passme';
@ -481,12 +476,12 @@ class PublicKeyTokenProviderTest extends TestCase {
$this->mapper->method('getToken')
->with(
$this->callback(function (string $token) {
return hash('sha512', 'token'.'1f4h9s') === $token;
return hash('sha512', 'tokentokentokentokentoken'.'1f4h9s') === $token;
})
)->willReturn($actual);
try {
$this->tokenProvider->getToken('token');
$this->tokenProvider->getToken('tokentokentokentokentoken');
$this->fail();
} catch (ExpiredTokenException $e) {
$this->assertSame($actual, $e->getToken());
@ -533,7 +528,7 @@ class PublicKeyTokenProviderTest extends TestCase {
}
public function testRotate() {
$token = 'oldtoken';
$token = 'oldtokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = 'password';
@ -546,13 +541,13 @@ class PublicKeyTokenProviderTest extends TestCase {
]);
$actual = $this->tokenProvider->generateToken($token, $uid, $user, $password, $name, $type, IToken::DO_NOT_REMEMBER);
$new = $this->tokenProvider->rotate($actual, 'oldtoken', 'newtoken');
$new = $this->tokenProvider->rotate($actual, 'oldtokentokentokentokentoken', 'newtokentokentokentokentoken');
$this->assertSame('password', $this->tokenProvider->getPassword($new, 'newtoken'));
$this->assertSame('password', $this->tokenProvider->getPassword($new, 'newtokentokentokentokentoken'));
}
public function testRotateNoPassword() {
$token = 'oldtoken';
$token = 'oldtokentokentokentokentoken';
$uid = 'user';
$user = 'User';
$password = null;
@ -563,7 +558,7 @@ class PublicKeyTokenProviderTest extends TestCase {
$oldPrivate = $actual->getPrivateKey();
$new = $this->tokenProvider->rotate($actual, 'oldtoken', 'newtoken');
$new = $this->tokenProvider->rotate($actual, 'oldtokentokentokentokentoken', 'newtokentokentokentokentoken');
$newPrivate = $new->getPrivateKey();
@ -595,7 +590,7 @@ class PublicKeyTokenProviderTest extends TestCase {
public function testUpdatePasswords() {
$uid = 'myUID';
$token1 = $this->tokenProvider->generateToken(
'foo',
'foobetokentokentokentoken',
$uid,
$uid,
'bar',
@ -603,7 +598,7 @@ class PublicKeyTokenProviderTest extends TestCase {
IToken::PERMANENT_TOKEN,
IToken::REMEMBER);
$token2 = $this->tokenProvider->generateToken(
'foobar',
'foobartokentokentokentoken',
$uid,
$uid,
'bar',