AbstractResourceTest::testInterface() - Code Metrics - Inspection of "First set up for types" - php-api-clients/resource-test-utilities - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a9d21e...16a33e )

by Cees-Jan

created 2016-08-05 21:05 UTC

AbstractResourceTest::testInterface() A

↳ Parent: AbstractResourceTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	0
cts	9
cp	0
rs	9.6666
c	0
b	0
f	0
nc	1
cc	1
eloc	5
nop	0
crap	2

<?php
declare(strict_types=1);

namespace ApiClients\Tools\ResourceTestUtilities;

use ApiClients\Foundation\Resource\ResourceInterface;

abstract class AbstractResourceTest extends \PHPUnit_Framework_TestCase
{
    abstract function getClass(): string;


    public function provideProperties()
    {
        foreach ((new \ReflectionClass($this->getClass()))->getProperties() as $property) {

            //
        }
    }

    /**
     * @dataProvider provideProperties
     */
    public function testProperties()
    {
        $this->assertTrue(true);
    }

    public function testInterface()
    {
        $this->assertTrue(
            is_subclass_of(

                $this->getClass(),
                ResourceInterface::class
            )
        );
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace ApiClients\Tools\ResourceTestUtilities;
5
6			use ApiClients\Foundation\Resource\ResourceInterface;
7
8			abstract class AbstractResourceTest extends \PHPUnit_Framework_TestCase
9			{
10			abstract function getClass(): string;
			0 ignored issues – show Best Practice introduced 2016-08-02 20:35 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
11
12			public function provideProperties()
13			{
14			foreach ((new \ReflectionClass($this->getClass()))->getProperties() as $property) {
			0 ignored issues – show Unused Code introduced 2016-08-02 20:35 UTC by Report Bug Copy Issue Report This `foreach` statement is empty and can be removed. This check looks for `foreach` loops that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. Consider removing the loop. Loading history...
15			//
16			}
17			}
18
19			/**
20			* @dataProvider provideProperties
21			*/
22			public function testProperties()
23			{
24			$this->assertTrue(true);
25			}
26
27			public function testInterface()
28			{
29			$this->assertTrue(
30			is_subclass_of(
			0 ignored issues – show Bug introduced 2016-08-02 20:35 UTC by Report Bug Copy Issue Report Due to PHP Bug #53727, `is_subclass_of` might return inconsistent results on some PHP versions if `\ApiClients\Foundation\R...esourceInterface::class` can be an interface. If so, you could instead use `ReflectionClass::implementsInterface`. Loading history...
31			$this->getClass(),
32			ResourceInterface::class
33			)
34			);
35			}
36			}
37

php-api-clients / resource-test-utilities

GitHub Access Token became invalid

Push — master ( a9d21e...16a33e )

AbstractResourceTest::testInterface() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like