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

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

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	239
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

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

15 Methods

Rating	Name	Size	Complexity
A	testConstructCorrectInterface()	6	1
A	testConstructWithoutStartingSession()	8	1
A	testConstructTryingToStartWhileSessionAlreadyExists()	8	1
A	testConstructWithExistingSessionKey()	10	1
A	testStoreAccessTokenIsAlreadyArray()	9	1
A	testStoreAccessTokenIsNotArray()	11	1
A	testRetrieveAccessTokenValid()	8	1
A	testRetrieveAccessTokenThrowsExceptionWhenTokenIsNotFound()	8	1
A	testHasAccessTokenTrue()	8	1
A	testHasAccessTokenFalse()	6	1
A	testClearTokenIsNotSet()	6	1
A	testClearTokenSet()	10	1
A	testClearAllTokens()	13	1
A	testDestruct()	6	1
A	testSerializeUnserialize()	13	1

<?php

namespace OAuthTest\Unit\Common\Storage;

use OAuth\Common\Storage\Session;
use PHPUnit\Framework\TestCase;

class SessionTest extends TestCase
{
    /**
     * @covers \Session::__construct
     *
     * @runInSeparateProcess
     */
    public function testConstructCorrectInterface(): void
    {
        $storage = new Session();

        self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
    }

    /**
     * @covers \Session::__construct
     *
     * @runInSeparateProcess
     */
    public function testConstructWithoutStartingSession(): void
    {
        session_start();

        $storage = new Session(false);

        self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
    }

    /**
     * @covers \Session::__construct
     *
     * @runInSeparateProcess
     */
    public function testConstructTryingToStartWhileSessionAlreadyExists(): void
    {
        session_start();

        $storage = new Session();

        self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
    }

    /**
     * @covers \Session::__construct
     *
     * @runInSeparateProcess
     */
    public function testConstructWithExistingSessionKey(): void
    {
        session_start();

        $_SESSION['lusitanian_oauth_token'] = [];

        $storage = new Session();

        self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testStoreAccessTokenIsAlreadyArray(): void
    {
        $storage = new Session();

        self::assertInstanceOf(
            '\\OAuth\\Common\\Storage\\Session',
            $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'))
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \Session::__construct
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testStoreAccessTokenIsNotArray(): void
    {
        $storage = new Session();

        $_SESSION['lusitanian_oauth_token'] = 'foo';

        self::assertInstanceOf(
            '\\OAuth\\Common\\Storage\\Session',
            $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'))
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @covers \Session::__construct
     * @covers \Session::hasAccessToken
     * @covers \Session::retrieveAccessToken
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testRetrieveAccessTokenValid(): void
    {
        $storage = new Session();

        $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertInstanceOf('\\OAuth\\Common\\Token\\TokenInterface', $storage->retrieveAccessToken('foo'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::hasAccessToken
     * @covers \Session::retrieveAccessToken
     *
     * @runInSeparateProcess
     */
    public function testRetrieveAccessTokenThrowsExceptionWhenTokenIsNotFound(): void
    {
        $this->expectException('\\OAuth\\Common\\Storage\\Exception\\TokenNotFoundException');

        $storage = new Session();

        $storage->retrieveAccessToken('foo');
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::hasAccessToken
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testHasAccessTokenTrue(): void
    {
        $storage = new Session();

        $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertTrue($storage->hasAccessToken('foo'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::hasAccessToken
     *
     * @runInSeparateProcess
     */
    public function testHasAccessTokenFalse(): void
    {
        $storage = new Session();

        self::assertFalse($storage->hasAccessToken('foo'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::clearToken
     *
     * @runInSeparateProcess
     */
    public function testClearTokenIsNotSet(): void
    {
        $storage = new Session();

        self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearToken('foo'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::clearToken
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testClearTokenSet(): void
    {
        $storage = new Session();

        $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertTrue($storage->hasAccessToken('foo'));
        self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearToken('foo'));
        self::assertFalse($storage->hasAccessToken('foo'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::clearAllTokens
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testClearAllTokens(): void
    {
        $storage = new Session();

        $storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $storage->storeAccessToken('bar', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        self::assertTrue($storage->hasAccessToken('foo'));
        self::assertTrue($storage->hasAccessToken('bar'));
        self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearAllTokens());
        self::assertFalse($storage->hasAccessToken('foo'));
        self::assertFalse($storage->hasAccessToken('bar'));
    }

    /**
     * @covers \Session::__construct
     * @covers \Session::__destruct
     *
     * @runInSeparateProcess
     */
    public function testDestruct(): void
    {
        $storage = new Session();

        unset($storage);
    }

    /**
     * @covers \Session::retrieveAccessToken
     * @covers \Session::storeAccessToken
     *
     * @runInSeparateProcess
     */
    public function testSerializeUnserialize(): void
    {
        $mock = $this->createMock('\\OAuth\\Common\\Token\\AbstractToken', ['__sleep']);

        $mock->expects(self::once())
            ->method('__sleep')
            ->willReturn(['accessToken']);

        $storage = new Session();
        $storage->storeAccessToken('foo', $mock);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $retrievedToken = $storage->retrieveAccessToken('foo');

        self::assertInstanceOf('\\OAuth\\Common\\Token\\AbstractToken', $retrievedToken);
    }
}


1			<?php
2
3			namespace OAuthTest\Unit\Common\Storage;
4
5			use OAuth\Common\Storage\Session;
6			use PHPUnit\Framework\TestCase;
7
8			class SessionTest extends TestCase
9			{
10			/**
11			* @covers \Session::__construct
12			*
13			* @runInSeparateProcess
14			*/
15			public function testConstructCorrectInterface(): void
16			{
17			$storage = new Session();
18
19			self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
20			}
21
22			/**
23			* @covers \Session::__construct
24			*
25			* @runInSeparateProcess
26			*/
27			public function testConstructWithoutStartingSession(): void
28			{
29			session_start();
30
31			$storage = new Session(false);
32
33			self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
34			}
35
36			/**
37			* @covers \Session::__construct
38			*
39			* @runInSeparateProcess
40			*/
41			public function testConstructTryingToStartWhileSessionAlreadyExists(): void
42			{
43			session_start();
44
45			$storage = new Session();
46
47			self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
48			}
49
50			/**
51			* @covers \Session::__construct
52			*
53			* @runInSeparateProcess
54			*/
55			public function testConstructWithExistingSessionKey(): void
56			{
57			session_start();
58
59			$_SESSION['lusitanian_oauth_token'] = [];
60
61			$storage = new Session();
62
63			self::assertInstanceOf('\\OAuth\\Common\\Storage\\TokenStorageInterface', $storage);
64			}
65
66			/**
67			* @covers \Session::__construct
68			* @covers \Session::storeAccessToken
69			*
70			* @runInSeparateProcess
71			*/
72			public function testStoreAccessTokenIsAlreadyArray(): void
73			{
74			$storage = new Session();
75
76			self::assertInstanceOf(
77			'\\OAuth\\Common\\Storage\\Session',
78			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'))
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
79			);
80			}
81
82			/**
83			* @covers \Session::__construct
84			* @covers \Session::storeAccessToken
85			*
86			* @runInSeparateProcess
87			*/
88			public function testStoreAccessTokenIsNotArray(): void
89			{
90			$storage = new Session();
91
92			$_SESSION['lusitanian_oauth_token'] = 'foo';
93
94			self::assertInstanceOf(
95			'\\OAuth\\Common\\Storage\\Session',
96			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'))
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
97			);
98			}
99
100			/**
101			* @covers \Session::__construct
102			* @covers \Session::hasAccessToken
103			* @covers \Session::retrieveAccessToken
104			* @covers \Session::storeAccessToken
105			*
106			* @runInSeparateProcess
107			*/
108			public function testRetrieveAccessTokenValid(): void
109			{
110			$storage = new Session();
111
112			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
113
114			self::assertInstanceOf('\\OAuth\\Common\\Token\\TokenInterface', $storage->retrieveAccessToken('foo'));
115			}
116
117			/**
118			* @covers \Session::__construct
119			* @covers \Session::hasAccessToken
120			* @covers \Session::retrieveAccessToken
121			*
122			* @runInSeparateProcess
123			*/
124			public function testRetrieveAccessTokenThrowsExceptionWhenTokenIsNotFound(): void
125			{
126			$this->expectException('\\OAuth\\Common\\Storage\\Exception\\TokenNotFoundException');
127
128			$storage = new Session();
129
130			$storage->retrieveAccessToken('foo');
131			}
132
133			/**
134			* @covers \Session::__construct
135			* @covers \Session::hasAccessToken
136			* @covers \Session::storeAccessToken
137			*
138			* @runInSeparateProcess
139			*/
140			public function testHasAccessTokenTrue(): void
141			{
142			$storage = new Session();
143
144			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
145
146			self::assertTrue($storage->hasAccessToken('foo'));
147			}
148
149			/**
150			* @covers \Session::__construct
151			* @covers \Session::hasAccessToken
152			*
153			* @runInSeparateProcess
154			*/
155			public function testHasAccessTokenFalse(): void
156			{
157			$storage = new Session();
158
159			self::assertFalse($storage->hasAccessToken('foo'));
160			}
161
162			/**
163			* @covers \Session::__construct
164			* @covers \Session::clearToken
165			*
166			* @runInSeparateProcess
167			*/
168			public function testClearTokenIsNotSet(): void
169			{
170			$storage = new Session();
171
172			self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearToken('foo'));
173			}
174
175			/**
176			* @covers \Session::__construct
177			* @covers \Session::clearToken
178			* @covers \Session::storeAccessToken
179			*
180			* @runInSeparateProcess
181			*/
182			public function testClearTokenSet(): void
183			{
184			$storage = new Session();
185
186			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
187
188			self::assertTrue($storage->hasAccessToken('foo'));
189			self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearToken('foo'));
190			self::assertFalse($storage->hasAccessToken('foo'));
191			}
192
193			/**
194			* @covers \Session::__construct
195			* @covers \Session::clearAllTokens
196			* @covers \Session::storeAccessToken
197			*
198			* @runInSeparateProcess
199			*/
200			public function testClearAllTokens(): void
201			{
202			$storage = new Session();
203
204			$storage->storeAccessToken('foo', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
205			$storage->storeAccessToken('bar', $this->createMock('\\OAuth\\Common\\Token\\TokenInterface'));
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$this->createMock('\\OAu...Token\\TokenInterface')` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
206
207			self::assertTrue($storage->hasAccessToken('foo'));
208			self::assertTrue($storage->hasAccessToken('bar'));
209			self::assertInstanceOf('\\OAuth\\Common\\Storage\\Session', $storage->clearAllTokens());
210			self::assertFalse($storage->hasAccessToken('foo'));
211			self::assertFalse($storage->hasAccessToken('bar'));
212			}
213
214			/**
215			* @covers \Session::__construct
216			* @covers \Session::__destruct
217			*
218			* @runInSeparateProcess
219			*/
220			public function testDestruct(): void
221			{
222			$storage = new Session();
223
224			unset($storage);
225			}
226
227			/**
228			* @covers \Session::retrieveAccessToken
229			* @covers \Session::storeAccessToken
230			*
231			* @runInSeparateProcess
232			*/
233			public function testSerializeUnserialize(): void
234			{
235			$mock = $this->createMock('\\OAuth\\Common\\Token\\AbstractToken', ['__sleep']);
			0 ignored issues – show Unused Code introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report The call to `SessionTest::createMock()` has too many arguments starting with `array('__sleep')`. This check compares calls to functions or methods with their respective definitions. If the call has more 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. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
236			$mock->expects(self::once())
237			->method('__sleep')
238			->willReturn(['accessToken']);
239
240			$storage = new Session();
241			$storage->storeAccessToken('foo', $mock);
			0 ignored issues – show Documentation introduced 2020-06-30 20:36 UTC by Report Bug Copy Issue Report `$mock` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<OAuth\Common\Token\TokenInterface>`. 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...
242			$retrievedToken = $storage->retrieveAccessToken('foo');
243
244			self::assertInstanceOf('\\OAuth\\Common\\Token\\AbstractToken', $retrievedToken);
245			}
246			}
247

Lusitanian / PHPoAuthLib

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

Complexity

Size/Duplication

Coupling/Cohesion

Importance

15 Methods

Duplication Side-by-Side

Filter issues like

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