Foo::getId() - Code Metrics - Inspection of "Merge pull request #1051 from FriendsOfSymfony/imp..." - FriendsOfSymfony/FOSElasticaBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ff8848...e1f7d6 )

by Karel

created 2016-08-04 10:17 UTC

Foo::getId() A

↳ Parent: Foo

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
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php

namespace FOS\ElasticaBundle\Tests\Doctrine;

use Elastica\Result;
use FOS\ElasticaBundle\Doctrine\ORM\ElasticaToModelTransformer;
use FOS\ElasticaBundle\Transformer\HighlightableModelInterface;
use Symfony\Component\PropertyAccess\PropertyAccess;

class AbstractElasticaToModelTransformerTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var \Doctrine\Common\Persistence\ManagerRegistry|\PHPUnit_Framework_MockObject_MockObject
     */
    protected $registry;

    /**
     * @var string
     */
    protected $objectClass = 'stdClass';

    protected function setUp()
    {
        $this->registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
            ->disableOriginalConstructor()
            ->getMock();
    }

    /**
     * Tests if ignore_missing option is properly handled in transformHybrid() method.
     */
    public function testIgnoreMissingOptionDuringTransformHybrid()
    {
        $transformer = $this->getMock(
            'FOS\ElasticaBundle\Doctrine\ORM\ElasticaToModelTransformer',
            array('findByIdentifiers'),
            array($this->registry, $this->objectClass, array('ignore_missing' => true))
        );

        $transformer->setPropertyAccessor(PropertyAccess::createPropertyAccessor());

        $firstOrmResult = new \stdClass();
        $firstOrmResult->id = 1;
        $secondOrmResult = new \stdClass();
        $secondOrmResult->id = 3;
        $transformer->expects($this->once())
            ->method('findByIdentifiers')
            ->with(array(1, 2, 3))
            ->willReturn(array($firstOrmResult, $secondOrmResult));

        $firstElasticaResult = new Result(array('_id' => 1));
        $secondElasticaResult = new Result(array('_id' => 2));
        $thirdElasticaResult = new Result(array('_id' => 3));

        $hybridResults = $transformer->hybridTransform(array($firstElasticaResult, $secondElasticaResult, $thirdElasticaResult));

        $this->assertCount(2, $hybridResults);
        $this->assertEquals($firstOrmResult, $hybridResults[0]->getTransformed());
        $this->assertEquals($firstElasticaResult, $hybridResults[0]->getResult());
        $this->assertEquals($secondOrmResult, $hybridResults[1]->getTransformed());
        $this->assertEquals($thirdElasticaResult, $hybridResults[1]->getResult());
    }

    public function testObjectClassCanBeSet()
    {
        $transformer = $this->createMockTransformer();
        $this->assertEquals('FOS\ElasticaBundle\Tests\Doctrine\Foo', $transformer->getObjectClass());
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.
}
    }

    public function resultsWithMatchingObjects()
    {
        $elasticaResults = $doctrineObjects = array();
        for ($i=1; $i<4; $i++) {
            $elasticaResults[] = new Result(array('_id' => $i, 'highlight' => array('foo')));
            $doctrineObjects[] = new Foo($i);
        }

        return array(
            array($elasticaResults, $doctrineObjects)
        );
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testObjectsAreTransformedByFindingThemByTheirIdentifiers($elasticaResults, $doctrineObjects)
    {
        $transformer = $this->createMockTransformer();

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue($doctrineObjects));

        $transformedObjects = $transformer->transform($elasticaResults);
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($doctrineObjects, $transformedObjects);
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testAnExceptionIsThrownWhenTheNumberOfFoundObjectsIsLessThanTheNumberOfResults(
        $elasticaResults,
        $doctrineObjects

    ) {
        $transformer = $this->createMockTransformer();

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue(array()));

        $this->setExpectedException(
            '\RuntimeException',
            'Cannot find corresponding Doctrine objects for all Elastica results.'
        );

        $transformer->transform($elasticaResults);
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.
}
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testAnExceptionIsNotThrownWhenTheNumberOfFoundObjectsIsLessThanTheNumberOfResultsIfOptionSet(
        $elasticaResults,
        $doctrineObjects

    ) {
        $transformer = $this->createMockTransformer(array('ignore_missing' => true));

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue(array()));

        $results = $transformer->transform($elasticaResults);
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(array(), $results);
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testHighlightsAreSetOnTransformedObjects($elasticaResults, $doctrineObjects)
    {
        $transformer = $this->createMockTransformer();

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue($doctrineObjects));

        $results = $transformer->transform($elasticaResults);
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.
}

        foreach($results as $result) {
            $this->assertInternalType('array', $result->highlights);
            $this->assertNotEmpty($result->highlights);
        }
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testResultsAreSortedByIdentifier($elasticaResults, $doctrineObjects)
    {
        rsort($doctrineObjects);

        $transformer = $this->createMockTransformer();

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue($doctrineObjects));

        $results = $transformer->transform($elasticaResults);
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->assertSame($doctrineObjects[2], $results[0]);
        $this->assertSame($doctrineObjects[1], $results[1]);
        $this->assertSame($doctrineObjects[0], $results[2]);
    }

    /**
     * @dataProvider resultsWithMatchingObjects
     */
    public function testHybridTransformReturnsDecoratedResults($elasticaResults, $doctrineObjects)
    {
        $transformer = $this->createMockTransformer();

        $transformer
            ->expects($this->once())
            ->method('findByIdentifiers')
            ->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
            ->will($this->returnValue($doctrineObjects));

        $results = $transformer->hybridTransform($elasticaResults);
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->assertNotEmpty($results);

        foreach ($results as $key => $result) {
            $this->assertInstanceOf('FOS\ElasticaBundle\HybridResult', $result);
            $this->assertSame($elasticaResults[$key], $result->getResult());
            $this->assertSame($doctrineObjects[$key], $result->getTransformed());
        }
    }

    public function testIdentifierFieldDefaultsToId()
    {
        $transformer = $this->createMockTransformer();
        $this->assertEquals('id', $transformer->getIdentifierField());
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.
}
    }

    private function createMockPropertyAccessor()
    {
        $callback = function ($object, $identifier) {
            return $object->$identifier;
        };

        $propertyAccessor = $this->getMock('Symfony\Component\PropertyAccess\PropertyAccessorInterface');
        $propertyAccessor
            ->expects($this->any())
            ->method('getValue')
            ->with($this->isType('object'), $this->isType('string'))
            ->will($this->returnCallback($callback));

        return $propertyAccessor;
    }

    /**
     * @param array $options
     *
     * @return \PHPUnit_Framework_MockObject_MockObject|\FOS\ElasticaBundle\Doctrine\AbstractElasticaToModelTransformer
     */
    private function createMockTransformer($options = array())
    {
        $objectClass = 'FOS\ElasticaBundle\Tests\Doctrine\Foo';
        $propertyAccessor = $this->createMockPropertyAccessor();

        $transformer = $this->getMockForAbstractClass(
            'FOS\ElasticaBundle\Doctrine\AbstractElasticaToModelTransformer',
            array($this->registry, $objectClass, $options)
        );

        $transformer->setPropertyAccessor($propertyAccessor);

        return $transformer;
    }
}

class Foo implements HighlightableModelInterface
{
    public $id;
    public $highlights;

    public function __construct($id)
    {
        $this->id = $id;
    }

    public function getId()
    {
        return $this->id;
    }

    public function setElasticHighlights(array $highlights)
    {
        $this->highlights = $highlights;
    }
}


Push — master ( ff8848...e1f7d6 )

Foo::getId() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace FOS\ElasticaBundle\Tests\Doctrine;
4
5			use Elastica\Result;
6			use FOS\ElasticaBundle\Doctrine\ORM\ElasticaToModelTransformer;
7			use FOS\ElasticaBundle\Transformer\HighlightableModelInterface;
8			use Symfony\Component\PropertyAccess\PropertyAccess;
9
10			class AbstractElasticaToModelTransformerTest extends \PHPUnit_Framework_TestCase
11			{
12			/**
13			* @var \Doctrine\Common\Persistence\ManagerRegistry\|\PHPUnit_Framework_MockObject_MockObject
14			*/
15			protected $registry;
16
17			/**
18			* @var string
19			*/
20			protected $objectClass = 'stdClass';
21
22			protected function setUp()
23			{
24			$this->registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
25			->disableOriginalConstructor()
26			->getMock();
27			}
28
29			/**
30			* Tests if ignore_missing option is properly handled in transformHybrid() method.
31			*/
32			public function testIgnoreMissingOptionDuringTransformHybrid()
33			{
34			$transformer = $this->getMock(
35			'FOS\ElasticaBundle\Doctrine\ORM\ElasticaToModelTransformer',
36			array('findByIdentifiers'),
37			array($this->registry, $this->objectClass, array('ignore_missing' => true))
38			);
39
40			$transformer->setPropertyAccessor(PropertyAccess::createPropertyAccessor());
41
42			$firstOrmResult = new \stdClass();
43			$firstOrmResult->id = 1;
44			$secondOrmResult = new \stdClass();
45			$secondOrmResult->id = 3;
46			$transformer->expects($this->once())
47			->method('findByIdentifiers')
48			->with(array(1, 2, 3))
49			->willReturn(array($firstOrmResult, $secondOrmResult));
50
51			$firstElasticaResult = new Result(array('_id' => 1));
52			$secondElasticaResult = new Result(array('_id' => 2));
53			$thirdElasticaResult = new Result(array('_id' => 3));
54
55			$hybridResults = $transformer->hybridTransform(array($firstElasticaResult, $secondElasticaResult, $thirdElasticaResult));
56
57			$this->assertCount(2, $hybridResults);
58			$this->assertEquals($firstOrmResult, $hybridResults[0]->getTransformed());
59			$this->assertEquals($firstElasticaResult, $hybridResults[0]->getResult());
60			$this->assertEquals($secondOrmResult, $hybridResults[1]->getTransformed());
61			$this->assertEquals($thirdElasticaResult, $hybridResults[1]->getResult());
62			}
63
64			public function testObjectClassCanBeSet()
65			{
66			$transformer = $this->createMockTransformer();
67			$this->assertEquals('FOS\ElasticaBundle\Tests\Doctrine\Foo', $transformer->getObjectClass());
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `getObjectClass` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
68			}
69
70			public function resultsWithMatchingObjects()
71			{
72			$elasticaResults = $doctrineObjects = array();
73			for ($i=1; $i<4; $i++) {
74			$elasticaResults[] = new Result(array('_id' => $i, 'highlight' => array('foo')));
75			$doctrineObjects[] = new Foo($i);
76			}
77
78			return array(
79			array($elasticaResults, $doctrineObjects)
80			);
81			}
82
83			/**
84			* @dataProvider resultsWithMatchingObjects
85			*/
86			public function testObjectsAreTransformedByFindingThemByTheirIdentifiers($elasticaResults, $doctrineObjects)
87			{
88			$transformer = $this->createMockTransformer();
89
90			$transformer
91			->expects($this->once())
92			->method('findByIdentifiers')
93			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
94			->will($this->returnValue($doctrineObjects));
95
96			$transformedObjects = $transformer->transform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `transform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
97
98			$this->assertEquals($doctrineObjects, $transformedObjects);
99			}
100
101			/**
102			* @dataProvider resultsWithMatchingObjects
103			*/
104			public function testAnExceptionIsThrownWhenTheNumberOfFoundObjectsIsLessThanTheNumberOfResults(
105			$elasticaResults,
106			$doctrineObjects
			0 ignored issues – show Unused Code introduced 2016-03-16 11:45 UTC by Report Bug Copy Issue Report The parameter `$doctrineObjects` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
107			) {
108			$transformer = $this->createMockTransformer();
109
110			$transformer
111			->expects($this->once())
112			->method('findByIdentifiers')
113			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
114			->will($this->returnValue(array()));
115
116			$this->setExpectedException(
117			'\RuntimeException',
118			'Cannot find corresponding Doctrine objects for all Elastica results.'
119			);
120
121			$transformer->transform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `transform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
122			}
123
124			/**
125			* @dataProvider resultsWithMatchingObjects
126			*/
127			public function testAnExceptionIsNotThrownWhenTheNumberOfFoundObjectsIsLessThanTheNumberOfResultsIfOptionSet(
128			$elasticaResults,
129			$doctrineObjects
			0 ignored issues – show Unused Code introduced 2016-03-16 11:45 UTC by Report Bug Copy Issue Report The parameter `$doctrineObjects` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
130			) {
131			$transformer = $this->createMockTransformer(array('ignore_missing' => true));
132
133			$transformer
134			->expects($this->once())
135			->method('findByIdentifiers')
136			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
137			->will($this->returnValue(array()));
138
139			$results = $transformer->transform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `transform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
140
141			$this->assertEquals(array(), $results);
142			}
143
144			/**
145			* @dataProvider resultsWithMatchingObjects
146			*/
147			public function testHighlightsAreSetOnTransformedObjects($elasticaResults, $doctrineObjects)
148			{
149			$transformer = $this->createMockTransformer();
150
151			$transformer
152			->expects($this->once())
153			->method('findByIdentifiers')
154			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
155			->will($this->returnValue($doctrineObjects));
156
157			$results = $transformer->transform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `transform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
158
159			foreach($results as $result) {
160			$this->assertInternalType('array', $result->highlights);
161			$this->assertNotEmpty($result->highlights);
162			}
163			}
164
165			/**
166			* @dataProvider resultsWithMatchingObjects
167			*/
168			public function testResultsAreSortedByIdentifier($elasticaResults, $doctrineObjects)
169			{
170			rsort($doctrineObjects);
171
172			$transformer = $this->createMockTransformer();
173
174			$transformer
175			->expects($this->once())
176			->method('findByIdentifiers')
177			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
178			->will($this->returnValue($doctrineObjects));
179
180			$results = $transformer->transform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `transform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
181
182			$this->assertSame($doctrineObjects[2], $results[0]);
183			$this->assertSame($doctrineObjects[1], $results[1]);
184			$this->assertSame($doctrineObjects[0], $results[2]);
185			}
186
187			/**
188			* @dataProvider resultsWithMatchingObjects
189			*/
190			public function testHybridTransformReturnsDecoratedResults($elasticaResults, $doctrineObjects)
191			{
192			$transformer = $this->createMockTransformer();
193
194			$transformer
195			->expects($this->once())
196			->method('findByIdentifiers')
197			->with($this->equalTo(array(1, 2, 3)), $this->isType('boolean'))
198			->will($this->returnValue($doctrineObjects));
199
200			$results = $transformer->hybridTransform($elasticaResults);
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `hybridTransform` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
201
202			$this->assertNotEmpty($results);
203
204			foreach ($results as $key => $result) {
205			$this->assertInstanceOf('FOS\ElasticaBundle\HybridResult', $result);
206			$this->assertSame($elasticaResults[$key], $result->getResult());
207			$this->assertSame($doctrineObjects[$key], $result->getTransformed());
208			}
209			}
210
211			public function testIdentifierFieldDefaultsToId()
212			{
213			$transformer = $this->createMockTransformer();
214			$this->assertEquals('id', $transformer->getIdentifierField());
			0 ignored issues – show Bug introduced 2016-08-04 10:29 UTC by Report Bug Copy Issue Report The method `getIdentifierField` does only exist in `FOS\ElasticaBundle\Doctr...sticaToModelTransformer`, 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...
215			}
216
217			private function createMockPropertyAccessor()
218			{
219			$callback = function ($object, $identifier) {
220			return $object->$identifier;
221			};
222
223			$propertyAccessor = $this->getMock('Symfony\Component\PropertyAccess\PropertyAccessorInterface');
224			$propertyAccessor
225			->expects($this->any())
226			->method('getValue')
227			->with($this->isType('object'), $this->isType('string'))
228			->will($this->returnCallback($callback));
229
230			return $propertyAccessor;
231			}
232
233			/**
234			* @param array $options
235			*
236			* @return \PHPUnit_Framework_MockObject_MockObject\|\FOS\ElasticaBundle\Doctrine\AbstractElasticaToModelTransformer
237			*/
238			private function createMockTransformer($options = array())
239			{
240			$objectClass = 'FOS\ElasticaBundle\Tests\Doctrine\Foo';
241			$propertyAccessor = $this->createMockPropertyAccessor();
242
243			$transformer = $this->getMockForAbstractClass(
244			'FOS\ElasticaBundle\Doctrine\AbstractElasticaToModelTransformer',
245			array($this->registry, $objectClass, $options)
246			);
247
248			$transformer->setPropertyAccessor($propertyAccessor);
249
250			return $transformer;
251			}
252			}
253
254			class Foo implements HighlightableModelInterface
255			{
256			public $id;
257			public $highlights;
258
259			public function __construct($id)
260			{
261			$this->id = $id;
262			}
263
264			public function getId()
265			{
266			return $this->id;
267			}
268
269			public function setElasticHighlights(array $highlights)
270			{
271			$this->highlights = $highlights;
272			}
273			}
274

FriendsOfSymfony / FOSElasticaBundle

Push — master ( ff8848...e1f7d6 )

Foo::getId() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like