StringOptionsAwareTraitTest::testSetStringOptions() - Code Metrics - fwolf/fwlib - Measure and Improve Code Quality continuously with Scrutinizer

testSetStringOptions() A
last analyzed 2019-05-09 15:53 UTC

↳ Parent: StringOptionsAwareTraitTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	15
rs	9.7666
c	0
b	0
f	0

<?php
namespace FwlibTest\Config;

use Fwlib\Config\StringOptionsAwareTrait;
use Fwolf\Wrapper\PHPUnit\PHPUnitTestCase;
use PHPUnit_Framework_MockObject_MockObject as MockObject;

/**
 * @copyright   Copyright 2015 Fwolf
 * @license     http://www.gnu.org/licenses/lgpl.html LGPL-3.0+
 */
class StringOptionsAwareTraitTest extends PHPUnitTestCase
{
    /**
     * @var array
     */
    protected static $setConfigs = [];


    /**
     * @param   string[]    $methods
     * @return  MockObject|StringOptionsAwareTrait
     */
    protected function buildMock(array $methods = null)
    {
        $mock = $this->getMockBuilder(StringOptionsAwareTrait::class)
            ->setMethods($methods)
            ->getMockForTrait();

        return $mock;
    }


    public function testSetStringOptions()
    {
        $trait = $this->buildMock(['setConfigs']);
        $trait->expects($this->any())
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->method('setConfigs')
            ->willReturnCallback(function ($configs) {
                self::$setConfigs = $configs;
            });

        $trait->setStringOptions('foo, bar=42');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        $this->assertEquals(
            ['foo' => true, 'bar' => 42],
            self::$setConfigs
        );
    }
}


testSetStringOptions() A
last analyzed 2019-05-09 15:53 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

1			<?php
2			namespace FwlibTest\Config;
3
4			use Fwlib\Config\StringOptionsAwareTrait;
5			use Fwolf\Wrapper\PHPUnit\PHPUnitTestCase;
6			use PHPUnit_Framework_MockObject_MockObject as MockObject;
7
8			/**
9			* @copyright Copyright 2015 Fwolf
10			* @license http://www.gnu.org/licenses/lgpl.html LGPL-3.0+
11			*/
12			class StringOptionsAwareTraitTest extends PHPUnitTestCase
13			{
14			/**
15			* @var array
16			*/
17			protected static $setConfigs = [];
18
19
20			/**
21			* @param string[] $methods
22			* @return MockObject\|StringOptionsAwareTrait
23			*/
24			protected function buildMock(array $methods = null)
25			{
26			$mock = $this->getMockBuilder(StringOptionsAwareTrait::class)
27			->setMethods($methods)
28			->getMockForTrait();
29
30			return $mock;
31			}
32
33
34			public function testSetStringOptions()
35			{
36			$trait = $this->buildMock(['setConfigs']);
37			$trait->expects($this->any())
			0 ignored issues – show Bug introduced 2017-01-07 05:16 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Fwlib\Config\StringOptionsAwareTrait`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
38			->method('setConfigs')
39			->willReturnCallback(function ($configs) {
40			self::$setConfigs = $configs;
41			});
42
43			$trait->setStringOptions('foo, bar=42');
			0 ignored issues – show Bug introduced 2017-01-07 05:16 UTC by Report Bug Copy Issue Report The method `setStringOptions` does only exist in `Fwlib\Config\StringOptionsAwareTrait`, but not in `PHPUnit_Framework_MockObject_MockObject`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
44			$this->assertEquals(
45			['foo' => true, 'bar' => 42],
46			self::$setConfigs
47			);
48			}
49			}
50

fwolf / fwlib

testSetStringOptions() A last analyzed 2019-05-09 15:53 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

testSetStringOptions() A
last analyzed 2019-05-09 15:53 UTC