ParameterTest - Code Metrics - intraface/intraface.dk - Measure and Improve Code Quality continuously with Scrutinizer

ParameterTest A
last analyzed 2017-03-20 12:20 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

Metric	Value
dl	0
loc	44
rs	10
c	0
b	0
f	0
wmc	6
lcom	1
cbo	3

6 Methods

Rating	Name	Size	Complexity
A	setUp()	5	1
A	tearDown()	4	1
A	createParameter()	4	1
A	testConstruction()	4	1
A	testSaveSavesValuesAndTheyCanBeGottenRightAway()	7	1
A	testSaveActuallyPersistsValuesForLaterUsage()	10	1

<?php
require_once 'CMSStubs.php';
require_once 'Intraface/Kernel.php';
require_once 'Intraface/modules/cms/Parameter.php';

class FakeObjectToPutIntoParameter
{
    public $kernel;

    function __construct()
    {
        $this->kernel = new Stub_Kernel;
    }

    function get($type)
    {
        switch ($type) {
            case 'identify_as':
                return 'cms_element';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            default:
                return 1;
                exit;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
        }
    }
}

class ParameterTest extends PHPUnit_Framework_TestCase
{
    protected $db;

    function setUp()
    {
        $this->db = MDB2::singleton(DB_DSN);
        $this->parameter = $this->createParameter();
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    function tearDown()
    {
        $this->db->exec('TRUNCATE cms_parameter');
    }

    function createParameter()
    {
        return new CMS_Parameter(new FakeObjectToPutIntoParameter);
    }

    function testConstruction()
    {
        $this->assertTrue(is_object($this->parameter));
    }

    function testSaveSavesValuesAndTheyCanBeGottenRightAway()
    {
        $parameter = 'parameter';
        $value = 'value';
        $this->assertTrue($this->parameter->save($parameter, $value));
        $this->assertEquals($value, $this->parameter->get($parameter));
    }

    function testSaveActuallyPersistsValuesForLaterUsage()
    {
        $parameter = 'parameter';
        $value = 'value';
        $this->assertTrue($this->parameter->save($parameter, $value));
        $this->assertEquals($value, $this->parameter->get($parameter));

        $p = $this->createParameter();
        $this->assertEquals($value, $p->get($parameter));
    }
}


1			<?php
2			require_once 'CMSStubs.php';
3			require_once 'Intraface/Kernel.php';
4			require_once 'Intraface/modules/cms/Parameter.php';
5
6			class FakeObjectToPutIntoParameter
7			{
8			public $kernel;
9
10			function __construct()
11			{
12			$this->kernel = new Stub_Kernel;
13			}
14
15			function get($type)
16			{
17			switch ($type) {
18			case 'identify_as':
19			return 'cms_element';
20			break;
			0 ignored issues – show Unused Code introduced 2016-04-20 09:50 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
21			default:
22			return 1;
23			exit;
			0 ignored issues – show Unused Code introduced 2016-04-20 09:50 UTC by Report Bug Copy Issue Report `die;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history... Coding Style Compatibility introduced 2016-04-20 09:50 UTC by Report Bug Copy Issue Report The method `get()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
24			}
25			}
26			}
27
28			class ParameterTest extends PHPUnit_Framework_TestCase
29			{
30			protected $db;
31
32			function setUp()
33			{
34			$this->db = MDB2::singleton(DB_DSN);
35			$this->parameter = $this->createParameter();
			0 ignored issues – show Bug introduced 2016-04-20 09:50 UTC by Report Bug Copy Issue Report The property `parameter` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
36			}
37
38			function tearDown()
39			{
40			$this->db->exec('TRUNCATE cms_parameter');
41			}
42
43			function createParameter()
44			{
45			return new CMS_Parameter(new FakeObjectToPutIntoParameter);
46			}
47
48			function testConstruction()
49			{
50			$this->assertTrue(is_object($this->parameter));
51			}
52
53			function testSaveSavesValuesAndTheyCanBeGottenRightAway()
54			{
55			$parameter = 'parameter';
56			$value = 'value';
57			$this->assertTrue($this->parameter->save($parameter, $value));
58			$this->assertEquals($value, $this->parameter->get($parameter));
59			}
60
61			function testSaveActuallyPersistsValuesForLaterUsage()
62			{
63			$parameter = 'parameter';
64			$value = 'value';
65			$this->assertTrue($this->parameter->save($parameter, $value));
66			$this->assertEquals($value, $this->parameter->get($parameter));
67
68			$p = $this->createParameter();
69			$this->assertEquals($value, $p->get($parameter));
70			}
71			}
72

intraface / intraface.dk

ParameterTest A last analyzed 2017-03-20 12:20 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

6 Methods

Duplication Side-by-Side

Filter issues like

ParameterTest A
last analyzed 2017-03-20 12:20 UTC