TracePluginTest::provideFrames() - Code Metrics - Inspection of "Merge coverage improvements" - jnvsor/kint - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — test ( cac427...5d2fc6 )

by Jonathan

created 2018-01-15 21:50 UTC

TracePluginTest::provideFrames() B

↳ Parent: TracePluginTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	42
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	29
nc	1
nop	0
dl	0
loc	42
rs	8.8571
c	0
b	0
f	0

<?php

namespace Kint\Test\Parser;

use Kint\Object\BasicObject;
use Kint\Parser\Parser;
use Kint\Parser\TracePlugin;
use PHPUnit_Framework_TestCase;

class TracePluginTest extends PHPUnit_Framework_TestCase

{
    protected $blacklist_stash;


    /**
     * @covers \Kint\Parser\TracePlugin::parse
     */
    public function testParse()
    {
        $p = new Parser();

        $p->addPlugin(new TracePlugin());

        $bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);


        $o = BasicObject::blank();


        $o = $p->parse($bt, $o);

        $this->assertContains('trace', $o->hints);
        $this->assertInstanceOf('Kint\\Object\\TraceObject', $o);
        $this->assertInstanceOf('Kint\\Object\\TraceFrameObject', $o->value->contents[0]);
    }

    /**
     * @covers \Kint\Parser\TracePlugin::parse
     */
    public function testParseMismatch()
    {
        $bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);

        $b = BasicObject::blank();

        $parser = new Parser();
        $plugin = new TracePlugin();

        $incorrect = $parser->parse($bt, clone $b);
        $incorrect->value->contents[0]->name = 'newName';
        $parser->addPlugin($plugin);
        $plugin->parse($bt, $incorrect, Parser::TRIGGER_SUCCESS);

        array_shift($bt);
        $correct = $parser->parse($bt, clone $b);

        foreach ($correct->value->contents as $frame) {
            ++$frame->name;
        }

        $this->assertEquals($correct, $incorrect);
    }

    /**
     * @covers \Kint\Parser\TracePlugin::parse
     */
    public function testParseNoValue()
    {
        $p = new TracePlugin();


        $b = BasicObject::blank();

        $o = clone $b;

        $v = array();


        $p->parse($v, $o, Parser::TRIGGER_SUCCESS);

        $this->assertEquals($b, $o);
    }

    /**
     * @covers \Kint\Parser\TracePlugin::parse
     */
    public function testParseBlacklist()
    {
        $bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);

        $shortbt = $bt;
        array_shift($shortbt);

        $p = new Parser();

        $p->addPlugin(new TracePlugin());

        $b = BasicObject::blank();


        $o = $p->parse($shortbt, clone $b);


        foreach ($o->value->contents as $frame) {
            ++$frame->name;
        }

        TracePlugin::$blacklist[] = array(__CLASS__, __FUNCTION__);

        $this->assertEquals($o->value, $p->parse($bt, clone $b)->value);
    }

    public function provideTraces()
    {
        $bt = debug_backtrace(true);

        $bad_bt_1 = $bt;

        $bad_bt_1[0]['test'] = 'woot';

        $bad_bt_2 = $bt;

        $bad_bt_2[0]['function'] = 1234;


        return array(
            'empty' => array(
                'trace' => array(),
                'expect' => false,
            ),
            'backtrace' => array(
                'trace' => $bt,
                'expect' => true,
            ),
            'bad backtrace, extra key' => array(
                'trace' => $bad_bt_1,

                'expect' => false,
            ),
            'bad backtrace, wrong type' => array(
                'trace' => $bad_bt_2,

                'expect' => false,
            ),
            'mythical' => array(
                'trace' => array(
                    array(
                        'function' => 'mythical_internal_function_with_no_args_that_results_in_a_backtrace',
                        'file' => __FILE__,
                        'line' => 1,
                    ),
                ),
                'expect' => true,
            ),
            'normal array' => array(
                'trace' => array(1, 2, 3),
                'expect' => false,
            ),
        );
    }

    /**
     * @dataProvider provideTraces
     * @covers \Kint\Parser\TracePlugin::isTrace
     */
    public function testIsTrace(array $trace, $expected)
    {
        $this->assertEquals($expected, TracePlugin::isTrace($trace));
    }

    public function provideFrames()
    {
        return array(
            'function match' => array(
                'frame' => array(
                    'function' => 'testAWeirdFunctionName',
                ),
                'matches' => array('testaweirdfunctionname'),
                'expected' => true,
            ),
            'function no match denormalized' => array(
                'frame' => array(
                    'function' => 'testAWeirdFunctionName',
                ),
                'matches' => array('testAWeirdFunctionName'),
                'expected' => false,
            ),
            'function no match method' => array(
                'frame' => array(
                    'function' => 'testAWeirdFunctionName',
                ),
                'matches' => array(array('test', 'testaweirdfunctionname')),
                'expected' => false,
            ),
            'method no match function' => array(
                'frame' => array(
                    'function' => 'testAWeirdFunctionName',
                    'class' => 'test',
                ),
                'matches' => array('testAWeirdFunctionName'),
                'expected' => false,
            ),
            'method match' => array(
                'frame' => array(
                    'function' => 'testAWeirdFunctionName',
                    'class' => 'test',
                ),
                'matches' => array(array('test', 'testaweirdfunctionname')),
                'expected' => true,
            ),
        );
    }

    /**
     * @dataProvider provideFrames
     * @covers \Kint\Parser\TracePlugin::frameIsListed
     */
    public function testFrameIsListed(array $frame, array $matches, $expected)
    {
        $this->assertEquals($expected, TracePlugin::frameIsListed($frame, $matches));
    }

    /**
     * @covers \Kint\Parser\TracePlugin::normalizeAliases
     */
    public function testNormalizeAliases()
    {
        $input = array(
            'CamelCaseFunction',
            'snake_case_function',
            'One_of_the_FunctionsMyColleaguesMADE__',
            'stringThatCan\'tBeAfunction',
            'another string that can not be a function',
            array('clASs', 'meThod'),
            array($this, 'meThod'),
            array('a', 'b', 'c'),
        );

        $expected = array(
            'camelcasefunction',
            'snake_case_function',
            'one_of_the_functionsmycolleaguesmade__',
            array('class', 'method'),
        );

        TracePlugin::normalizeAliases($input);

        $this->assertEquals($expected, $input);
    }

    /**
     * @covers \Kint\Parser\TracePlugin::getTypes
     * @covers \Kint\Parser\TracePlugin::getTriggers
     */
    public function testHooks()
    {
        $p = new TracePlugin();


        $this->assertEquals(array('array'), $p->getTypes());
        $this->assertEquals(Parser::TRIGGER_SUCCESS, $p->getTriggers());
    }

    public function setUp()
    {
        $this->blacklist_stash = TracePlugin::$blacklist;
    }

    public function tearDown()
    {
        TracePlugin::$blacklist = $this->blacklist_stash;
    }
}


1			<?php
2
3			namespace Kint\Test\Parser;
4
5			use Kint\Object\BasicObject;
6			use Kint\Parser\Parser;
7			use Kint\Parser\TracePlugin;
8			use PHPUnit_Framework_TestCase;
9
10			class TracePluginTest extends PHPUnit_Framework_TestCase
			0 ignored issues – show Coding Style introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report The property $blacklist_stash is not named in camelCase. This check marks property names that have not been written in camelCase. In camelCase names are written without any punctuation, the start of each new word being marked by a capital letter. Thus the name database connection string becomes `databaseConnectionString`. Loading history...
11			{
12			protected $blacklist_stash;
			0 ignored issues – show Coding Style introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report `$blacklist_stash` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
13
14			/**
15			* @covers \Kint\Parser\TracePlugin::parse
16			*/
17			public function testParse()
18			{
19			$p = new Parser();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $p. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
20			$p->addPlugin(new TracePlugin());
21
22			$bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $bt. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
23
24			$o = BasicObject::blank();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $o. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
25
26			$o = $p->parse($bt, $o);
27
28			$this->assertContains('trace', $o->hints);
29			$this->assertInstanceOf('Kint\\Object\\TraceObject', $o);
30			$this->assertInstanceOf('Kint\\Object\\TraceFrameObject', $o->value->contents[0]);
31			}
32
33			/**
34			* @covers \Kint\Parser\TracePlugin::parse
35			*/
36			public function testParseMismatch()
37			{
38			$bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $bt. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
39			$b = BasicObject::blank();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $b. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
40			$parser = new Parser();
41			$plugin = new TracePlugin();
42
43			$incorrect = $parser->parse($bt, clone $b);
44			$incorrect->value->contents[0]->name = 'newName';
45			$parser->addPlugin($plugin);
46			$plugin->parse($bt, $incorrect, Parser::TRIGGER_SUCCESS);
47
48			array_shift($bt);
49			$correct = $parser->parse($bt, clone $b);
50
51			foreach ($correct->value->contents as $frame) {
52			++$frame->name;
53			}
54
55			$this->assertEquals($correct, $incorrect);
56			}
57
58			/**
59			* @covers \Kint\Parser\TracePlugin::parse
60			*/
61			public function testParseNoValue()
62			{
63			$p = new TracePlugin();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $p. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
64
65			$b = BasicObject::blank();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $b. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
66			$o = clone $b;
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $o. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
67			$v = array();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $v. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
68
69			$p->parse($v, $o, Parser::TRIGGER_SUCCESS);
70
71			$this->assertEquals($b, $o);
72			}
73
74			/**
75			* @covers \Kint\Parser\TracePlugin::parse
76			*/
77			public function testParseBlacklist()
78			{
79			$bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $bt. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
80			$shortbt = $bt;
81			array_shift($shortbt);
82
83			$p = new Parser();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $p. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
84			$p->addPlugin(new TracePlugin());
85
86			$b = BasicObject::blank();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $b. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
87
88			$o = $p->parse($shortbt, clone $b);
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $o. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
89
90			foreach ($o->value->contents as $frame) {
91			++$frame->name;
92			}
93
94			TracePlugin::$blacklist[] = array(__CLASS__, __FUNCTION__);
95
96			$this->assertEquals($o->value, $p->parse($bt, clone $b)->value);
97			}
98
99			public function provideTraces()
100			{
101			$bt = debug_backtrace(true);
			0 ignored issues – show Comprehensibility introduced 2018-01-09 18:27 UTC by Report Bug Copy Issue Report Avoid variables with short names like $bt. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
102			$bad_bt_1 = $bt;
			0 ignored issues – show Coding Style introduced 2018-01-09 18:27 UTC by Report Bug Copy Issue Report `$bad_bt_1` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
103			$bad_bt_1[0]['test'] = 'woot';
			0 ignored issues – show Coding Style introduced 2018-01-09 18:27 UTC by Report Bug Copy Issue Report `$bad_bt_1` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
104			$bad_bt_2 = $bt;
			0 ignored issues – show Coding Style introduced 2018-01-09 18:27 UTC by Report Bug Copy Issue Report `$bad_bt_2` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
105			$bad_bt_2[0]['function'] = 1234;
			0 ignored issues – show Coding Style introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report `$bad_bt_2` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
106
107			return array(
108			'empty' => array(
109			'trace' => array(),
110			'expect' => false,
111			),
112			'backtrace' => array(
113			'trace' => $bt,
114			'expect' => true,
115			),
116			'bad backtrace, extra key' => array(
117			'trace' => $bad_bt_1,
			0 ignored issues – show Coding Style introduced 2018-01-09 18:27 UTC by Report Bug Copy Issue Report `$bad_bt_1` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
118			'expect' => false,
119			),
120			'bad backtrace, wrong type' => array(
121			'trace' => $bad_bt_2,
			0 ignored issues – show Coding Style introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report `$bad_bt_2` does not seem to conform to the naming convention (`^[a-z][a-zA-Z0-9]*$`). This check examines a number of code elements and verifies that they conform to the given naming conventions. You can set conventions for local variables, abstract classes, utility classes, constant, properties, methods, parameters, interfaces, classes, exceptions and special methods. Loading history...
122			'expect' => false,
123			),
124			'mythical' => array(
125			'trace' => array(
126			array(
127			'function' => 'mythical_internal_function_with_no_args_that_results_in_a_backtrace',
128			'file' => __FILE__,
129			'line' => 1,
130			),
131			),
132			'expect' => true,
133			),
134			'normal array' => array(
135			'trace' => array(1, 2, 3),
136			'expect' => false,
137			),
138			);
139			}
140
141			/**
142			* @dataProvider provideTraces
143			* @covers \Kint\Parser\TracePlugin::isTrace
144			*/
145			public function testIsTrace(array $trace, $expected)
146			{
147			$this->assertEquals($expected, TracePlugin::isTrace($trace));
148			}
149
150			public function provideFrames()
151			{
152			return array(
153			'function match' => array(
154			'frame' => array(
155			'function' => 'testAWeirdFunctionName',
156			),
157			'matches' => array('testaweirdfunctionname'),
158			'expected' => true,
159			),
160			'function no match denormalized' => array(
161			'frame' => array(
162			'function' => 'testAWeirdFunctionName',
163			),
164			'matches' => array('testAWeirdFunctionName'),
165			'expected' => false,
166			),
167			'function no match method' => array(
168			'frame' => array(
169			'function' => 'testAWeirdFunctionName',
170			),
171			'matches' => array(array('test', 'testaweirdfunctionname')),
172			'expected' => false,
173			),
174			'method no match function' => array(
175			'frame' => array(
176			'function' => 'testAWeirdFunctionName',
177			'class' => 'test',
178			),
179			'matches' => array('testAWeirdFunctionName'),
180			'expected' => false,
181			),
182			'method match' => array(
183			'frame' => array(
184			'function' => 'testAWeirdFunctionName',
185			'class' => 'test',
186			),
187			'matches' => array(array('test', 'testaweirdfunctionname')),
188			'expected' => true,
189			),
190			);
191			}
192
193			/**
194			* @dataProvider provideFrames
195			* @covers \Kint\Parser\TracePlugin::frameIsListed
196			*/
197			public function testFrameIsListed(array $frame, array $matches, $expected)
198			{
199			$this->assertEquals($expected, TracePlugin::frameIsListed($frame, $matches));
200			}
201
202			/**
203			* @covers \Kint\Parser\TracePlugin::normalizeAliases
204			*/
205			public function testNormalizeAliases()
206			{
207			$input = array(
208			'CamelCaseFunction',
209			'snake_case_function',
210			'One_of_the_FunctionsMyColleaguesMADE__',
211			'stringThatCan\'tBeAfunction',
212			'another string that can not be a function',
213			array('clASs', 'meThod'),
214			array($this, 'meThod'),
215			array('a', 'b', 'c'),
216			);
217
218			$expected = array(
219			'camelcasefunction',
220			'snake_case_function',
221			'one_of_the_functionsmycolleaguesmade__',
222			array('class', 'method'),
223			);
224
225			TracePlugin::normalizeAliases($input);
226
227			$this->assertEquals($expected, $input);
228			}
229
230			/**
231			* @covers \Kint\Parser\TracePlugin::getTypes
232			* @covers \Kint\Parser\TracePlugin::getTriggers
233			*/
234			public function testHooks()
235			{
236			$p = new TracePlugin();
			0 ignored issues – show Comprehensibility introduced 2018-01-13 16:08 UTC by Report Bug Copy Issue Report Avoid variables with short names like $p. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
237
238			$this->assertEquals(array('array'), $p->getTypes());
239			$this->assertEquals(Parser::TRIGGER_SUCCESS, $p->getTriggers());
240			}
241
242			public function setUp()
243			{
244			$this->blacklist_stash = TracePlugin::$blacklist;
245			}
246
247			public function tearDown()
248			{
249			TracePlugin::$blacklist = $this->blacklist_stash;
250			}
251			}
252

jnvsor / kint

Push — test ( cac427...5d2fc6 )

TracePluginTest::provideFrames() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like