TocifierTest::testTOC() - Code Metrics - Inspection of "Added standard Scrutinizer config" - ntd/silverstripe-autotoc - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#5)

by Helpful

created 2016-02-16 23:06 UTC

TocifierTest::testTOC() A

↳ Parent: TocifierTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	12
rs	9.4285
cc	1
eloc	9
nc	1
nop	0

<?php

class TocifierTest extends PHPUnit_Framework_TestCase
namespace YourVendor;

class YourClass { }
{
    public function testProcess()
    {
        $tocifier = new Tocifier(1234);
        $this->assertFalse($tocifier->process());

        $tocifier = new Tocifier('');
        $this->assertFalse($tocifier->process());

        $tocifier = new Tocifier(null);
        $this->assertFalse($tocifier->process());

        $tocifier = new Tocifier(array('1234'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertFalse($tocifier->process());

        $tocifier = new Tocifier('1234');
        $this->assertTrue($tocifier->process());
    }

    public function testHtml()
    {
        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
        $this->assertEquals($tocifier->getHtml(), '');
        $this->assertTrue($tocifier->process());
        $this->assertStringEqualsFile(__DIR__ . '/html1', $tocifier->getHtml());
    }

    public function testTOC()
    {
        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
        $this->assertEquals($tocifier->getTOC(), array());
        $this->assertTrue($tocifier->process());
        $this->assertNotNull($tocifier->getTOC());

        ob_start();
        $tocifier->dumpTOC();
        $returned = ob_get_clean();
        $this->assertStringEqualsFile(__DIR__ . '/toc1', $returned);
    }
}


1			<?php
2
3			class TocifierTest extends PHPUnit_Framework_TestCase
			0 ignored issues – show Coding Style Compatibility introduced 2016-02-16 23:08 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4			{
5			public function testProcess()
6			{
7			$tocifier = new Tocifier(1234);
8			$this->assertFalse($tocifier->process());
9
10			$tocifier = new Tocifier('');
11			$this->assertFalse($tocifier->process());
12
13			$tocifier = new Tocifier(null);
14			$this->assertFalse($tocifier->process());
15
16			$tocifier = new Tocifier(array('1234'));
			0 ignored issues – show Documentation introduced 2016-02-16 23:08 UTC by Report Bug Copy Issue Report `array('1234')` is of type `array<integer,string,{"0":"string"}>`, 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...
17			$this->assertFalse($tocifier->process());
18
19			$tocifier = new Tocifier('1234');
20			$this->assertTrue($tocifier->process());
21			}
22
23			public function testHtml()
24			{
25			$tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
26			$this->assertEquals($tocifier->getHtml(), '');
27			$this->assertTrue($tocifier->process());
28			$this->assertStringEqualsFile(__DIR__ . '/html1', $tocifier->getHtml());
29			}
30
31			public function testTOC()
32			{
33			$tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
34			$this->assertEquals($tocifier->getTOC(), array());
35			$this->assertTrue($tocifier->process());
36			$this->assertNotNull($tocifier->getTOC());
37
38			ob_start();
39			$tocifier->dumpTOC();
40			$returned = ob_get_clean();
41			$this->assertStringEqualsFile(__DIR__ . '/toc1', $returned);
42			}
43			}
44

ntd / silverstripe-autotoc

Pull Request — master (#5)

TocifierTest::testTOC() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like