ClosuresTest::filterAllowNullIsFalseAndNullValue() - Code Metrics - Inspection of "Merge pull request #102 from akoryak/closures" - traderinteractive/filter-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d5ea1d...4cb52c )

by Chad

created 2024-09-04 16:11 UTC

ClosuresTest::filterAllowNullIsFalseAndNullValue() A

↳ Parent: ClosuresTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	2
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	1
nc	1
nop	0

<?php

namespace TraderInteractive\Filter;

use PHPUnit\Framework\TestCase;
use TraderInteractive\Exceptions\FilterException;

/**
 * @coversDefaultClass \TraderInteractive\Filter\Closures
 * @covers ::<private>
 */
final class ClosuresTest extends TestCase
{
    /**
     * @test
     * @covers ::filter
     */
    public function filterAllowNullAndNullValue()
    {
        $result = Closures::filter(null, true);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $this->assertSame(null, $result);
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterAllowNullIsFalseAndNullValue()
    {
        $this->expectException(FilterException::class);
        Closures::filter(null, false);
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterClosure()
    {
        $closureFunction = function () {
            return 'do nothing';
        };
        $result = Closures::filter($closureFunction);
        $this->assertTrue($result instanceof \Closure);
    }

    public static function myCallableFunction()
    {
        return 'do nothing';
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterCallable()
    {
        $result = Closures::filter(['\\TraderInteractive\\Filter\\ClosuresTest', 'myCallableFunction']);
        $this->assertTrue($result instanceof \Closure);
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterNotCallableString()
    {
        $this->expectException(FilterException::class);
        Closures::filter('string');
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterNotCallableInt()
    {
        $this->expectException(FilterException::class);
        Closures::filter(123);
    }
}


1			<?php
2
3			namespace TraderInteractive\Filter;
4
5			use PHPUnit\Framework\TestCase;
6			use TraderInteractive\Exceptions\FilterException;
7
8			/**
9			* @coversDefaultClass \TraderInteractive\Filter\Closures
10			* @covers ::<private>
11			*/
12			final class ClosuresTest extends TestCase
13			{
14			/**
15			* @test
16			* @covers ::filter
17			*/
18			public function filterAllowNullAndNullValue()
19			{
20			$result = Closures::filter(null, true);
			0 ignored issues – show Bug introduced 2024-08-29 15:25 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$result` is correct as `TraderInteractive\Filter...res::filter(null, true)` targeting `TraderInteractive\Filter\Closures::filter()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
21			$this->assertSame(null, $result);
22			}
23
24			/**
25			* @test
26			* @covers ::filter
27			*/
28			public function filterAllowNullIsFalseAndNullValue()
29			{
30			$this->expectException(FilterException::class);
31			Closures::filter(null, false);
32			}
33
34			/**
35			* @test
36			* @covers ::filter
37			*/
38			public function filterClosure()
39			{
40			$closureFunction = function () {
41			return 'do nothing';
42			};
43			$result = Closures::filter($closureFunction);
44			$this->assertTrue($result instanceof \Closure);
45			}
46
47			public static function myCallableFunction()
48			{
49			return 'do nothing';
50			}
51
52			/**
53			* @test
54			* @covers ::filter
55			*/
56			public function filterCallable()
57			{
58			$result = Closures::filter(['\\TraderInteractive\\Filter\\ClosuresTest', 'myCallableFunction']);
59			$this->assertTrue($result instanceof \Closure);
60			}
61
62			/**
63			* @test
64			* @covers ::filter
65			*/
66			public function filterNotCallableString()
67			{
68			$this->expectException(FilterException::class);
69			Closures::filter('string');
70			}
71
72			/**
73			* @test
74			* @covers ::filter
75			*/
76			public function filterNotCallableInt()
77			{
78			$this->expectException(FilterException::class);
79			Closures::filter(123);
80			}
81			}
82

traderinteractive / filter-php

Push — master ( d5ea1d...4cb52c )

ClosuresTest::filterAllowNullIsFalseAndNullValue() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like