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

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

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	312
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

Metric	Value
dl	0
loc	312
c	0
b	0
f	0
wmc	10
lcom	1
cbo	4
rs	10

10 Methods

Rating	Name	Size	Complexity
A	testConstructCorrectInterface()	6	1
A	testSetHashingAlgorithm()	6	1
A	testSetTokenSecret()	6	1
A	testGetSignatureBareUri()	28	1
A	testGetSignatureWithQueryString()	28	1
A	testGetSignatureWithAuthority()	28	1
A	testGetSignatureWithBarePathNonExplicitTrailingHostSlash()	31	1
A	testGetSignatureWithBarePathWithExplicitTrailingHostSlash()	31	1
A	testGetSignatureNoTokenSecretSet()	30	1
A	testGetSignatureThrowsExceptionOnUnsupportedAlgo()	32	1

<?php

namespace OAuthTest\Unit\OAuth1\Signature;

use OAuth\OAuth1\Signature\Signature;
use PHPUnit\Framework\TestCase;

class SignatureTest extends TestCase
{
    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     */
    public function testConstructCorrectInterface(): void
    {
        $signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertInstanceOf('\\OAuth\\OAuth1\\Signature\\SignatureInterface', $signature);
    }

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     */
    public function testSetHashingAlgorithm(): void
    {
        $signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertNull($signature->setHashingAlgorithm('foo'));
    }

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testSetTokenSecret(): void
    {
        $signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertNull($signature->setTokenSecret('foo'));
    }

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureBareUri(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');
        $signature->setTokenSecret('foo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/foo');

        self::assertSame('uoCpiII/Lg/cPiF0XrU2pj4eGFQ=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureWithQueryString(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');
        $signature->setTokenSecret('foo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/foo');

        self::assertSame('LxtD+WjJBRppIUvEI79iQ7I0hSo=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureWithAuthority(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');
        $signature->setTokenSecret('foo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('peehaa:pass');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/foo');

        self::assertSame('MHvkRndIntLrxiPkjkiCNsMEqv4=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureWithBarePathNonExplicitTrailingHostSlash(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');
        $signature->setTokenSecret('foo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('peehaa:pass');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/');
        $uri->expects(self::any())
            ->method('hasExplicitTrailingHostSlash')
            ->willReturn(false);

        self::assertSame('iFELDoiI5Oj9ixB3kHzoPvBpq0w=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureWithBarePathWithExplicitTrailingHostSlash(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');
        $signature->setTokenSecret('foo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('peehaa:pass');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/');
        $uri->expects(self::any())
            ->method('hasExplicitTrailingHostSlash')
            ->willReturn(true);

        self::assertSame('IEhUsArSTLvbQ3QYr0zzn+Rxpjg=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureNoTokenSecretSet(): void
    {
        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('HMAC-SHA1');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('peehaa:pass');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/');
        $uri->expects(self::any())
            ->method('hasExplicitTrailingHostSlash')
            ->willReturn(true);

        self::assertSame('YMHF7FYmLq7wzGnnHWYtd1VoBBE=', $signature->getSignature($uri, ['pee' => 'haa']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \OAuth\OAuth1\Signature\Signature::__construct
     * @covers \OAuth\OAuth1\Signature\Signature::buildSignatureDataString
     * @covers \OAuth\OAuth1\Signature\Signature::getSignature
     * @covers \OAuth\OAuth1\Signature\Signature::getSigningKey
     * @covers \OAuth\OAuth1\Signature\Signature::hash
     * @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
     * @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
     */
    public function testGetSignatureThrowsExceptionOnUnsupportedAlgo(): void
    {
        $this->expectException('\\OAuth\\OAuth1\\Signature\\Exception\\UnsupportedHashAlgorithmException');

        $credentials = $this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface');
        $credentials->expects(self::any())
            ->method('getConsumerSecret')
            ->willReturn('foo');

        $signature = new Signature($credentials);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $signature->setHashingAlgorithm('UnsupportedAlgo');

        $uri = $this->createMock('\\OAuth\\Common\\Http\\Uri\\UriInterface');
        $uri->expects(self::any())
            ->method('getQuery')
            ->willReturn('param1=value1');
        $uri->expects(self::any())
            ->method('getScheme')
            ->willReturn('http');
        $uri->expects(self::any())
            ->method('getRawAuthority')
            ->willReturn('peehaa:pass');
        $uri->expects(self::any())
            ->method('getPath')
            ->willReturn('/');
        $uri->expects(self::any())
            ->method('hasExplicitTrailingHostSlash')
            ->willReturn(true);

        $signature->getSignature($uri, ['pee' => 'haa']);
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\Signature;
4
5			use OAuth\OAuth1\Signature\Signature;
6			use PHPUnit\Framework\TestCase;
7
8			class SignatureTest extends TestCase
9			{
10			/**
11			* @covers \OAuth\OAuth1\Signature\Signature::__construct
12			*/
13			public function testConstructCorrectInterface(): void
14			{
15			$signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...\CredentialsInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Cons...r\CredentialsInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
16
17			self::assertInstanceOf('\\OAuth\\OAuth1\\Signature\\SignatureInterface', $signature);
18			}
19
20			/**
21			* @covers \OAuth\OAuth1\Signature\Signature::__construct
22			* @covers \OAuth\OAuth1\Signature\Signature::setHashingAlgorithm
23			*/
24			public function testSetHashingAlgorithm(): void
25			{
26			$signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...\CredentialsInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Cons...r\CredentialsInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
27
28			self::assertNull($signature->setHashingAlgorithm('foo'));
29			}
30
31			/**
32			* @covers \OAuth\OAuth1\Signature\Signature::__construct
33			* @covers \OAuth\OAuth1\Signature\Signature::setTokenSecret
34			*/
35			public function testSetTokenSecret(): void
36			{
37			$signature = new Signature($this->createMock('\\OAuth\\Common\\Consumer\\CredentialsInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...\CredentialsInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Cons...r\CredentialsInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...

Lusitanian / PHPoAuthLib

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

Complexity

Size/Duplication

Coupling/Cohesion

Importance

10 Methods

Duplication Side-by-Side

Filter issues like

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