NullCacheTest - Code Metrics - Inspection of "Merge pull request #12 from chadicus/master" - subjective-php/psr-cache-helper - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 807d05...dfc103 )

by Chad

created 2018-07-15 15:23 UTC

NullCacheTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	128
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
wmc	9
lcom	1
cbo	2
dl	0
loc	128
rs	10
c	0
b	0
f	0

9 Methods

Rating	Name	Size	Complexity
A	setUp()	4	1
A	get()	5	1
A	set()	4	1
A	delete()	4	1
A	clear()	4	1
A	getMultiple()	8	1
A	setMultiple()	4	1
A	deleteMultiple()	4	1
A	has()	4	1

<?php

namespace SubjectivePHPTest\Psr\SimpleCache;

use SubjectivePHP\Psr\SimpleCache\NullCache;

/**
 * @coversDefaultClass SubjectivePHP\Psr\SimpleCache\NullCache
 * @covers ::<private>
 * @covers ::<protected>
 */
final class NullCacheTest extends \PHPUnit\Framework\TestCase
{
    /**
     * NullCache instance to use in tests.
     *
     * @var NullCache
     */
    private $cache;

    /**
     * Prepare each test
     *
     * @return void
     */
    public function setUp()
    {
        $this->cache = new NullCache();
    }

    /**
     * Verify basic behavior of get().
     *
     * @test
     * @covers ::get
     *
     * @return void
     */
    public function get()
    {
        $default = new \StdClass();
        $this->assertSame($default, $this->cache->get('a key', $default));
    }

    /**
     * Verify basic behavior of set().
     *
     * @test
     * @covers ::set
     *
     * @return void
     */
    public function set()
    {
        $this->assertTrue($this->cache->set('a key', 'some data'));
    }

    /**
     * Verify basic behavior of delete().
     *
     * @test
     * @covers ::delete
     *
     * @return void
     */
    public function delete()
    {
        $this->assertTrue($this->cache->delete('a key'));
    }

    /**
     * Verify basic behavior of clear().
     *
     * @test
     * @covers ::clear
     *
     * @return void
     */
    public function clear()
    {
        $this->assertTrue($this->cache->clear());
    }

    /**
     * Verify basic behavior of getMultple().
     *
     * @test
     * @covers ::getMultiple
     *
     * @return void
     */
    public function getMultiple()
    {
        $default = new \StdClass();
        $this->assertSame(
            ['key1' => $default, 'key2' => $default],
            $this->cache->getMultiple(['key1', 'key2'], $default)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Verify basic behavior of setMultiple().
     *
     * @test
     * @covers ::setMultiple
     *
     * @return void
     */
    public function setMultiple()
    {
        $this->assertTrue($this->cache->setMultiple(['key' => 'some data', 'key2' => 'some more data']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Verify basic behavior of deleteMultiple().
     *
     * @test
     * @covers ::deleteMultiple
     *
     * @return void
     */
    public function deleteMultiple()
    {
        $this->assertTrue($this->cache->deleteMultiple(['key1', 'key2']));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Verify basic behavior of has().
     *
     * @test
     * @covers ::has
     *
     * @return void
     */
    public function has()
    {
        $this->assertFalse($this->cache->has('key1'));
    }
}


1			<?php
2
3			namespace SubjectivePHPTest\Psr\SimpleCache;
4
5			use SubjectivePHP\Psr\SimpleCache\NullCache;
6
7			/**
8			* @coversDefaultClass SubjectivePHP\Psr\SimpleCache\NullCache
9			* @covers ::<private>
10			* @covers ::<protected>
11			*/
12			final class NullCacheTest extends \PHPUnit\Framework\TestCase
13			{
14			/**
15			* NullCache instance to use in tests.
16			*
17			* @var NullCache
18			*/
19			private $cache;
20
21			/**
22			* Prepare each test
23			*
24			* @return void
25			*/
26			public function setUp()
27			{
28			$this->cache = new NullCache();
29			}
30
31			/**
32			* Verify basic behavior of get().
33			*
34			* @test
35			* @covers ::get
36			*
37			* @return void
38			*/
39			public function get()
40			{
41			$default = new \StdClass();
42			$this->assertSame($default, $this->cache->get('a key', $default));
43			}
44
45			/**
46			* Verify basic behavior of set().
47			*
48			* @test
49			* @covers ::set
50			*
51			* @return void
52			*/
53			public function set()
54			{
55			$this->assertTrue($this->cache->set('a key', 'some data'));
56			}
57
58			/**
59			* Verify basic behavior of delete().
60			*
61			* @test
62			* @covers ::delete
63			*
64			* @return void
65			*/
66			public function delete()
67			{
68			$this->assertTrue($this->cache->delete('a key'));
69			}
70
71			/**
72			* Verify basic behavior of clear().
73			*
74			* @test
75			* @covers ::clear
76			*
77			* @return void
78			*/
79			public function clear()
80			{
81			$this->assertTrue($this->cache->clear());
82			}
83
84			/**
85			* Verify basic behavior of getMultple().
86			*
87			* @test
88			* @covers ::getMultiple
89			*
90			* @return void
91			*/
92			public function getMultiple()
93			{
94			$default = new \StdClass();
95			$this->assertSame(
96			['key1' => $default, 'key2' => $default],
97			$this->cache->getMultiple(['key1', 'key2'], $default)
			0 ignored issues – show Documentation introduced 2018-07-15 15:09 UTC by Report Bug Copy Issue Report `array('key1', 'key2')` is of type `array<integer,string,{"0":"string","1":"string"}>`, but the function expects a `object<SubjectivePHP\Psr\SimpleCache\iterable>`. 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...
98			);
99			}
100
101			/**
102			* Verify basic behavior of setMultiple().
103			*
104			* @test
105			* @covers ::setMultiple
106			*
107			* @return void
108			*/
109			public function setMultiple()
110			{
111			$this->assertTrue($this->cache->setMultiple(['key' => 'some data', 'key2' => 'some more data']));
			0 ignored issues – show Documentation introduced 2018-07-15 15:09 UTC by Report Bug Copy Issue Report `array('key' => 'some dat...2' => 'some more data')` is of type `array<string,string,{"ke...ring","key2":"string"}>`, but the function expects a `object<SubjectivePHP\Psr\SimpleCache\iterable>`. 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...
112			}
113
114			/**
115			* Verify basic behavior of deleteMultiple().
116			*
117			* @test
118			* @covers ::deleteMultiple
119			*
120			* @return void
121			*/
122			public function deleteMultiple()
123			{
124			$this->assertTrue($this->cache->deleteMultiple(['key1', 'key2']));
			0 ignored issues – show Documentation introduced 2018-07-15 15:09 UTC by Report Bug Copy Issue Report `array('key1', 'key2')` is of type `array<integer,string,{"0":"string","1":"string"}>`, but the function expects a `object<SubjectivePHP\Psr\SimpleCache\iterable>`. 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...
125			}
126
127			/**
128			* Verify basic behavior of has().
129			*
130			* @test
131			* @covers ::has
132			*
133			* @return void
134			*/
135			public function has()
136			{
137			$this->assertFalse($this->cache->has('key1'));
138			}
139			}
140

subjective-php / psr-cache-helper

Push — master ( 807d05...dfc103 )

NullCacheTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

9 Methods

Duplication Side-by-Side

Filter issues like