AbstractServiceTest - Code Metrics - Lusitanian/PHPoAuthLib - Measure and Improve Code Quality continuously with Scrutinizer

AbstractServiceTest A
last analyzed 2020-10-09 22:28 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	235
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

Metric	Value
dl	0
loc	235
c	0
b	0
f	0
wmc	9
lcom	1
cbo	6
rs	10

9 Methods

Rating	Name	Size	Complexity
A	testConstructCorrectInterface()	15	1
A	testConstructCorrectParent()	15	1
A	testRequestRequestTokenBuildAuthHeaderTokenRequestWithoutParams()	17	1
A	testGetAuthorizationUriWithoutParameters()	12	1
A	testGetAuthorizationUriWithParameters()	15	1
A	testRequestAccessTokenWithoutSecret()	23	1
A	testRequestAccessTokenWithSecret()	22	1
A	testRequest()	21	1
A	testRequestNonArrayBody()	20	1

<?php

namespace OAuthTest\Unit\OAuth1\Service;

use OAuthTest\Mocks\OAuth1\Service\Mock;
use PHPUnit\Framework\Assert;
use PHPUnit\Framework\TestCase;

class AbstractServiceTest extends TestCase
{
    /**
     * @covers \AbstractService::__construct
     */
    public function testConstructCorrectInterface(): void
    {
        $service = $this->getMockForAbstractClass(
            '\\OAuth\\OAuth1\\Service\\AbstractService',
            [
                $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
                $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
                $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
                $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
                $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface'),
            ]
        );

        self::assertInstanceOf('\\OAuth\\OAuth1\\Service\\ServiceInterface', $service);
    }

    /**
     * @covers \AbstractService::__construct
     */
    public function testConstructCorrectParent(): void
    {
        $service = $this->getMockForAbstractClass(
            '\\OAuth\\OAuth1\\Service\\AbstractService',
            [
                $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
                $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
                $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
                $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
                $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface'),
            ]
        );

        self::assertInstanceOf('\\OAuth\\Common\\Service\\AbstractService', $service);
    }

    /**
     * @covers \AbstractService::buildAuthorizationHeaderForTokenRequest
     * @covers \AbstractService::generateNonce
     * @covers \AbstractService::getBasicAuthorizationHeaderInfo
     * @covers \AbstractService::getExtraOAuthHeaders
     * @covers \AbstractService::getSignatureMethod
     * @covers \AbstractService::getVersion
     * @covers \AbstractService::parseRequestTokenResponse
     * @covers \AbstractService::requestRequestToken
     */
    public function testRequestRequestTokenBuildAuthHeaderTokenRequestWithoutParams(): void
    {
        $client = $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface');
        $client->expects(self::once())->method('retrieveResponse')->willReturnCallback(function ($endpoint, $array, $headers): void {

            Assert::assertSame('http://pieterhordijk.com/token', $endpoint->getAbsoluteUri());
        });

        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $client,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertInstanceOf('\\OAuth\\OAuth1\\Token\\StdOAuth1Token', $service->requestRequestToken());
    }

    /**
     * @covers \AbstractService::getAuthorizationEndpoint
     * @covers \AbstractService::getAuthorizationUri
     */
    public function testGetAuthorizationUriWithoutParameters(): void
    {
        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertSame('http://pieterhordijk.com/auth', $service->getAuthorizationUri()->getAbsoluteUri());
    }

    /**
     * @covers \AbstractService::getAuthorizationEndpoint
     * @covers \AbstractService::getAuthorizationUri
     */
    public function testGetAuthorizationUriWithParameters(): void
    {
        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertSame('http://pieterhordijk.com/auth?foo=bar&baz=beer', $service->getAuthorizationUri([
            'foo' => 'bar',
            'baz' => 'beer',
        ])->getAbsoluteUri());
    }

    /**
     * @covers \AbstractService::buildAuthorizationHeaderForAPIRequest
     * @covers \AbstractService::generateNonce
     * @covers \AbstractService::getAccessTokenEndpoint
     * @covers \AbstractService::getBasicAuthorizationHeaderInfo
     * @covers \AbstractService::getExtraOAuthHeaders
     * @covers \AbstractService::getSignatureMethod
     * @covers \AbstractService::getVersion
     * @covers \AbstractService::parseAccessTokenResponse
     * @covers \AbstractService::requestAccessToken
     * @covers \AbstractService::service
     */
    public function testRequestAccessTokenWithoutSecret(): void
    {
        $client = $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface');
        $client->expects(self::once())->method('retrieveResponse')->willReturnCallback(function ($endpoint, $array, $headers): void {

            Assert::assertSame('http://pieterhordijk.com/access', $endpoint->getAbsoluteUri());
        });

        $token = $this->createMock('\\OAuth\\OAuth1\\Token\\TokenInterface');
        $token->expects(self::once())->method('getRequestTokenSecret')->willReturn('baz');

        $storage = $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface');
        $storage->expects(self::any())->method('retrieveAccessToken')->willReturn($token);

        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $client,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $storage,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertInstanceOf('\\OAuth\\OAuth1\\Token\\StdOAuth1Token', $service->requestAccessToken('foo', 'bar'));
    }

    /**
     * @covers \AbstractService::buildAuthorizationHeaderForAPIRequest
     * @covers \AbstractService::generateNonce
     * @covers \AbstractService::getAccessTokenEndpoint
     * @covers \AbstractService::getBasicAuthorizationHeaderInfo
     * @covers \AbstractService::getExtraOAuthHeaders
     * @covers \AbstractService::getSignatureMethod
     * @covers \AbstractService::getVersion
     * @covers \AbstractService::parseAccessTokenResponse
     * @covers \AbstractService::requestAccessToken
     * @covers \AbstractService::service
     */
    public function testRequestAccessTokenWithSecret(): void
    {
        $client = $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface');
        $client->expects(self::once())->method('retrieveResponse')->willReturnCallback(function ($endpoint, $array, $headers): void {

            Assert::assertSame('http://pieterhordijk.com/access', $endpoint->getAbsoluteUri());
        });

        $token = $this->createMock('\\OAuth\\OAuth1\\Token\\TokenInterface');

        $storage = $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface');
        $storage->expects(self::any())->method('retrieveAccessToken')->willReturn($token);

        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $client,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $storage,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertInstanceOf('\\OAuth\\OAuth1\\Token\\StdOAuth1Token', $service->requestAccessToken('foo', 'bar', $token));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @covers \AbstractService::buildAuthorizationHeaderForAPIRequest
     * @covers \AbstractService::determineRequestUriFromPath
     * @covers \AbstractService::generateNonce
     * @covers \AbstractService::getBasicAuthorizationHeaderInfo
     * @covers \AbstractService::getExtraApiHeaders
     * @covers \AbstractService::getSignatureMethod
     * @covers \AbstractService::getVersion
     * @covers \AbstractService::request
     * @covers \AbstractService::service
     */
    public function testRequest(): void
    {
        $client = $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface');
        $client->expects(self::once())->method('retrieveResponse')->willReturn('response!');

        $token = $this->createMock('\\OAuth\\OAuth1\\Token\\TokenInterface');
        //$token->expects($this->once())->method('getRequestTokenSecret')->will($this->returnValue('baz'));

        $storage = $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface');
        $storage->expects(self::any())->method('retrieveAccessToken')->willReturn($token);

        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $client,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $storage,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertSame('response!', $service->request('/my/awesome/path'));
    }

    /**
     * This test only captures a regression in php 5.3.
     *
     * @covers \AbstractService::request
     */
    public function testRequestNonArrayBody(): void
    {
        $client = $this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface');
        $client->expects(self::once())->method('retrieveResponse')->willReturn('response!');

        $token = $this->createMock('\\OAuth\\OAuth1\\Token\\TokenInterface');

        $storage = $this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface');
        $storage->expects(self::any())->method('retrieveAccessToken')->willReturn($token);

        $service = new Mock(
            $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $client,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $storage,
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface')
        );

        self::assertSame('response!', $service->request('/my/awesome/path', 'GET', 'A text body'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1			<?php
2
3			namespace OAuthTest\Unit\OAuth1\Service;
4
5			use OAuthTest\Mocks\OAuth1\Service\Mock;
6			use PHPUnit\Framework\Assert;
7			use PHPUnit\Framework\TestCase;
8
9			class AbstractServiceTest extends TestCase
10			{
11			/**
12			* @covers \AbstractService::__construct
13			*/
14			public function testConstructCorrectInterface(): void
15			{
16			$service = $this->getMockForAbstractClass(
17			'\\OAuth\\OAuth1\\Service\\AbstractService',
18			[
19			$this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
20			$this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
21			$this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),
22			$this->createMock('\\OAuth\\OAuth1\\Signature\\SignatureInterface'),
23			$this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface'),
24			]
25			);
26
27			self::assertInstanceOf('\\OAuth\\OAuth1\\Service\\ServiceInterface', $service);
28			}
29
30			/**
31			* @covers \AbstractService::__construct
32			*/
33			public function testConstructCorrectParent(): void
34			{
35			$service = $this->getMockForAbstractClass(
36			'\\OAuth\\OAuth1\\Service\\AbstractService',
37			[
38			$this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'),
39			$this->createMock('\\OAuth\\Common\\Http\\Client\\ClientInterface'),
40			$this->createMock('\\OAuth\\Common\\Storage\\TokenStorageInterface'),

Lusitanian / PHPoAuthLib

AbstractServiceTest A last analyzed 2020-10-09 22:28 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

9 Methods

Duplication Side-by-Side

Filter issues like

AbstractServiceTest A
last analyzed 2020-10-09 22:28 UTC