Passed
Push — master ( f80f70...56399a )
by Thomas Mauro
03:05 queued 10s
created

RpIdTokenSigNoneTest::getTestId()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 1
nc 1
nop 0
dl 0
loc 3
rs 10
c 1
b 0
f 0
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Facile\OpenIDClient\ConformanceTest\RpTest\IdToken;
6
7
use PHPUnit\Framework\Assert;
8
use Facile\OpenIDClient\ConformanceTest\RpTest\AbstractRpTest;
9
use Facile\OpenIDClient\ConformanceTest\TestInfo;
10
use Facile\OpenIDClient\Service\AuthorizationService;
11
use function Facile\OpenIDClient\base64url_encode;
12
13
/**
14
 * Use Code Flow and retrieve an unsigned ID Token.
15
 * This test is only applicable when response_type='code'.
16
 *
17
 * Accept the ID Token after doing ID Token validation.
18
 */
19
class RpIdTokenSigNoneTest extends AbstractRpTest
20
{
21
22
    public function getTestId(): string
23
    {
24
        return 'rp-id_token-sig-none';
25
    }
26
27
    public function execute(TestInfo $testInfo): void
28
    {
29
        $client = $this->registerClient($testInfo, ['id_token_signed_response_alg' => 'none']);
30
31
        Assert::assertSame('none', $client->getMetadata()->get('id_token_signed_response_alg'));
32
33
        // Get authorization redirect uri
34
        $authorizationService = new AuthorizationService();
0 ignored issues
show
Bug introduced by
The call to Facile\OpenIDClient\Serv...nService::__construct() has too few arguments starting with tokenSetFactory. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

34
        $authorizationService = /** @scrutinizer ignore-call */ new AuthorizationService();

This check compares calls to functions or methods with their respective definitions. If the call has less arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
35
        $uri = $authorizationService->getAuthorizationUri($client, [
36
            'response_type' => $testInfo->getResponseType(),
37
            'nonce' => base64url_encode(\random_bytes(32)),
38
        ]);
39
40
        // Simulate a redirect and create the server request
41
        $serverRequest = $this->simulateAuthRedirect($uri);
42
        $params = $authorizationService->getCallbackParams($serverRequest, $client);
43
44
        $tokenSet = $authorizationService->callback($client, $params);
45
46
        Assert::assertNotNull($tokenSet->getIdToken());
47
    }
48
}
49