FileStorageTest::tearDown() - Code Metrics - Inspection of "Merge pull request #6 from djsmithme/psr-simple-ca..." - djsmithme/Cache - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 00fae0...ee941a )

by Dan

created 2017-03-21 15:12 UTC

FileStorageTest::tearDown() A

↳ Parent: FileStorageTest

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0

<?php

namespace Tests\Cache\Storage;

use Ds\Cache\Storage\FileStorage;

/**
 * Class FileStorageTest
 *
 * @package Tests\Cache\Storage
 */
class FileStorageTest extends \PHPUnit_Framework_TestCase
{

    /**
     * @var FileStorage
     */
    public $cacheStorage;

    /**
     * @var
     */
    public $testDir;

    /**
     *
     */
    public function setUp()
    {
        $this->testDir = __DIR__ . '/cacheTest';
        $this->cacheStorage = new FileStorage( $this->testDir, new \DateInterval('P1M'));
    }

    /**
     *
     */
    public function testUnwritableDirectory()
    {
        $this->expectException('\Exception');
        $this->cacheStorage = new FileStorage(false, new \DateInterval('P1M'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     *
     */
    public function testSet(){
        $actual = $this->cacheStorage->set('key','value',60*60);
        $this->assertEquals(true, $actual);
    }

    /**
     *
     */
    public function testHasNoValue(){
        $this->assertEquals($this->cacheStorage->has('someRandomKey'), false);
    }

    /**
     *
     */
    public function testHas(){
        $this->cacheStorage->set('foo','bar');
        $this->assertEquals($this->cacheStorage->has('foo'), true);
    }

    /**
     *
     */
    public function testHasExpired(){
        $this->cacheStorage->set('expired','bar', -1200);
        $this->assertEquals($this->cacheStorage->has('expired'), false);
    }

    /**
     *
     */
    public function testClear(){
       $this->assertEquals($this->cacheStorage->clear(), true);
    }

    /**
     *
     */
    public function testGet(){
        $expected = 'bat';
        $this->cacheStorage->set('baz',$expected);
        $actual = $this->cacheStorage->get('baz');
        $this->assertEquals($expected, $actual);
    }

    /**
     * Clean up any cache files left on system.
     */
    public function tearDown()
    {
        $dir = $this->testDir;

        if (is_dir( $dir)){
            array_map('unlink', glob("$dir/*.*"));
        }
        \rmdir($dir);
    }
    
}


1			<?php
2
3			namespace Tests\Cache\Storage;
4
5			use Ds\Cache\Storage\FileStorage;
6
7			/**
8			* Class FileStorageTest
9			*
10			* @package Tests\Cache\Storage
11			*/
12			class FileStorageTest extends \PHPUnit_Framework_TestCase
13			{
14
15			/**
16			* @var FileStorage
17			*/
18			public $cacheStorage;
19
20			/**
21			* @var
22			*/
23			public $testDir;
24
25			/**
26			*
27			*/
28			public function setUp()
29			{
30			$this->testDir = __DIR__ . '/cacheTest';
31			$this->cacheStorage = new FileStorage( $this->testDir, new \DateInterval('P1M'));
32			}
33
34			/**
35			*
36			*/
37			public function testUnwritableDirectory()
38			{
39			$this->expectException('\Exception');
40			$this->cacheStorage = new FileStorage(false, new \DateInterval('P1M'));
			0 ignored issues – show Documentation introduced 2017-03-21 15:27 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
41			}
42
43			/**
44			*
45			*/
46			public function testSet(){
47			$actual = $this->cacheStorage->set('key','value',60*60);
48			$this->assertEquals(true, $actual);
49			}
50
51			/**
52			*
53			*/
54			public function testHasNoValue(){
55			$this->assertEquals($this->cacheStorage->has('someRandomKey'), false);
56			}
57
58			/**
59			*
60			*/
61			public function testHas(){
62			$this->cacheStorage->set('foo','bar');
63			$this->assertEquals($this->cacheStorage->has('foo'), true);
64			}
65
66			/**
67			*
68			*/
69			public function testHasExpired(){
70			$this->cacheStorage->set('expired','bar', -1200);
71			$this->assertEquals($this->cacheStorage->has('expired'), false);
72			}
73
74			/**
75			*
76			*/
77			public function testClear(){
78			$this->assertEquals($this->cacheStorage->clear(), true);
79			}
80
81			/**
82			*
83			*/
84			public function testGet(){
85			$expected = 'bat';
86			$this->cacheStorage->set('baz',$expected);
87			$actual = $this->cacheStorage->get('baz');
88			$this->assertEquals($expected, $actual);
89			}
90
91			/**
92			* Clean up any cache files left on system.
93			*/
94			public function tearDown()
95			{
96			$dir = $this->testDir;
97
98			if (is_dir( $dir)){
99			array_map('unlink', glob("$dir/."));
100			}
101			\rmdir($dir);
102			}
103
104			}
105

djsmithme / Cache

Push — master ( 00fae0...ee941a )

FileStorageTest::tearDown() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like