DoctrineAdapterAbstractFactoryTest::createServiceConfigProvider() - Code Metrics - stefanotorresi/thorr-persistence-doctrine - Measure and Improve Code Quality continuously with Scrutinizer

createServiceConfigProvider() A
last analyzed 2016-01-20 16:34 UTC

↳ Parent: DoctrineAdapterAbstractFactoryTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	68
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	8
Bugs	3	Features	1

Metric	Value
c	8
b	3
f	1
dl	0
loc	68
rs	9.2447
cc	1
eloc	31
nc	1
nop	0

How to fix Long Method

<?php
/**
 * @author Stefano Torresi (http://stefanotorresi.it)
 * @license See the file LICENSE.txt for copying permission.
 * ************************************************
 */

namespace Thorr\Persistence\Doctrine\Test\DataMapper\Manager;

use Doctrine\Common\Persistence\ObjectManager;
use PHPUnit_Framework_MockObject_MockObject;
use PHPUnit_Framework_TestCase as TestCase;
use Thorr\Persistence\Doctrine\DataMapper\DoctrineAdapter;
use Thorr\Persistence\Doctrine\DataMapper\Manager\DoctrineAdapterAbstractFactory;
use Zend\ServiceManager\Exception as SMException;
use Zend\ServiceManager\ServiceLocatorInterface;

/**
 * @covers Thorr\Persistence\Doctrine\DataMapper\Manager\DoctrineAdapterAbstractFactory
 */
class DoctrineAdapterAbstractFactoryTest extends TestCase
{
    /**
     * @var DoctrineAdapterAbstractFactory
     */
    protected $abstractFactory;

    /**
     * @var ServiceLocatorInterface|PHPUnit_Framework_MockObject_MockObject
     */
    protected $serviceLocator;

    public function setUp()
    {
        $this->abstractFactory = new DoctrineAdapterAbstractFactory();
        $this->serviceLocator  = $this->getMock(ServiceLocatorInterface::class);
    }

    /**
     * @param $config
     * @param $expectedResult
     *
     * @dataProvider canCreateServiceConfigProvider()
     */
    public function testCanCreateService($config, $serviceName, $expectedResult)
    {
        $this->serviceLocator->expects($this->atLeastOnce())
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('get')
                             ->with('config')
                             ->willReturn($config);

        $this->assertSame(
            $expectedResult,
            $this->abstractFactory->canCreateServiceWithName($this->serviceLocator, 'unrelevant', $serviceName)
        );
    }

    public function canCreateServiceConfigProvider()
    {
        return [
            [
                [],
                'anything',
                false,
            ],
            [
                [ 'thorr_persistence_dmm' => [] ],
                'anything',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [],
                    ],
                ],
                'anything',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [],
                        ],
                    ],
                ],
                'anything',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => 123,
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => DoctrineAdapter::class,
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                true,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => 'not-a-class',
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => [
                                    'class' => DoctrineAdapter::class,
                                ],
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                true,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => [],
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                false,
            ],
            [
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'adapters' => [
                                'SomeDataMapperServiceName' => [
                                    'class' => 'not-a-class',
                                ],
                            ],
                        ],
                    ],
                ],
                'SomeDataMapperServiceName',
                false,
            ],
        ];
    }

    /**
     * @param $config
     * @dataProvider createServiceConfigProvider
     */
    public function testCreateService($config, $requestedName, $expectedException)
    {
        $objectManagers = [
            'SomeObjectManagerService'    => $this->getMock(ObjectManager::class, [], [], 'SomeObjectManagerService'),
            'AnotherObjectManagerService' => $this->getMock(ObjectManager::class, [], [], 'AnotherObjectManagerService'),
        ];

        $this->serviceLocator->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('has')
            ->willReturnCallback(function ($name) use ($objectManagers) {
                return array_key_exists($name, $objectManagers);
            });

        $this->serviceLocator->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('get')
            ->willReturnCallback(function ($name) use ($config, $objectManagers) {
                switch ($name) {
                    case 'config' :
switch ($selector) {
    case "A": //right
        doSomething();
        break;
    case "B" : //wrong
        doSomethingElse();
        break;
}
                        return $config;
                    case 'SomeObjectManagerService' :
switch ($selector) {
    case "A": //right
        doSomething();
        break;
    case "B" : //wrong
        doSomethingElse();
        break;
}
                    case 'AnotherObjectManagerService' :
switch ($selector) {
    case "A": //right
        doSomething();
        break;
    case "B" : //wrong
        doSomethingElse();
        break;
}
                        return $objectManagers[$name];
                }
            });

        if ($expectedException) {
            $this->setExpectedException($expectedException[0], $expectedException[1]);
        }

        /** @var DoctrineAdapter $instance */
        $instance = $this->abstractFactory->createServiceWithName($this->serviceLocator, 'unrelevant', $requestedName);

        $this->assertInstanceOf(DoctrineAdapter::class, $instance);
        $this->assertSame(
            $requestedName,
            $config['thorr_persistence_dmm']['entity_data_mapper_map'][$instance->getEntityClass()]
        );

        if (isset($config['thorr_persistence_dmm']['doctrine']['adapters'][$requestedName]['object_manager'])) {
            $this->assertSame(
                $objectManagers[$config['thorr_persistence_dmm']['doctrine']['adapters'][$requestedName]['object_manager']],
                $instance->getObjectManager()
            );
        }
    }

    public function createServiceConfigProvider()
    {
        return [
            [
                // $config
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'object_manager' => 'SomeObjectManagerService',
                            'adapters'       => [
                                'SomeDataMapperServiceName' => DoctrineAdapter::class,
                            ],
                        ],
                        'entity_data_mapper_map' => [
                            'SomeEntityClass' => 'SomeDataMapperServiceName',
                        ],
                    ],
                ],
                // $requestedName
                'SomeDataMapperServiceName',
                // $expectedException
                null,
            ],
            [
                // $config
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'object_manager' => 'SomeObjectManagerService',
                            'adapters'       => [
                                'SomeDataMapperServiceName' => \stdClass::class,
                            ],
                        ],
                        'entity_data_mapper_map' => [
                            'SomeEntityClass' => 'SomeDataMapperServiceName',
                        ],
                    ],
                ],
                // $requestedName
                'SomeDataMapperServiceName',
                // $expectedException
                [ SMException\ServiceNotCreatedException::class, 'Invalid data mapper type' ],
            ],
            [
                // $config
                [
                    'thorr_persistence_dmm' => [
                        'doctrine' => [
                            'object_manager' => 'SomeObjectManagerService',
                            'adapters'       => [
                                'SomeDataMapperServiceName' => [
                                    'class'          => DoctrineAdapter::class,
                                    'object_manager' => 'AnotherObjectManagerService',
                                ],
                            ],
                        ],
                        'entity_data_mapper_map' => [
                            'SomeEntityClass' => 'SomeDataMapperServiceName',
                        ],
                    ],
                ],
                // $requestedName
                'SomeDataMapperServiceName',
                // $expectedException
                null,
            ],
        ];
    }

    public function testInvalidObjectManagerClass()
    {
        $config = [
            'thorr_persistence_dmm' => [
                'doctrine' => [
                    'object_manager' => 'SomeObjectManagerService',
                    'adapters'       => [
                        'SomeDataMapperServiceName' => DoctrineAdapter::class,
                    ],
                ],
                'entity_data_mapper_map' => [],
            ],
        ];

        $objectManager = $this->getMock(\stdClass::class);

        $this->serviceLocator->expects($this->atLeastOnce())
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('has')
            ->with($config['thorr_persistence_dmm']['doctrine']['object_manager'])
            ->willReturn(true);

        $this->serviceLocator->expects($this->atLeastOnce())
            ->method('get')
            ->willReturnCallback(function ($name) use ($config, $objectManager) {
                switch ($name) {
                    case 'config' :
switch ($selector) {
    case "A": //right
        doSomething();
        break;
    case "B" : //wrong
        doSomethingElse();
        break;
}
                        return $config;
                    case $config['thorr_persistence_dmm']['doctrine']['object_manager'] :
switch ($selector) {
    case "A": //right
        doSomething();
        break;
    case "B" : //wrong
        doSomethingElse();
        break;
}
                        return $objectManager;
                }
            });

        $this->setExpectedException(SMException\InvalidServiceNameException::class, 'Invalid object manager type');

        $this->abstractFactory->createServiceWithName($this->serviceLocator, 'unrelevant', 'SomeDataMapperServiceName');
    }
}


createServiceConfigProvider() A
last analyzed 2016-01-20 16:34 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2			/**
3			* @author Stefano Torresi (http://stefanotorresi.it)
4			* @license See the file LICENSE.txt for copying permission.
5			* ************************************************
6			*/
7
8			namespace Thorr\Persistence\Doctrine\Test\DataMapper\Manager;
9
10			use Doctrine\Common\Persistence\ObjectManager;
11			use PHPUnit_Framework_MockObject_MockObject;
12			use PHPUnit_Framework_TestCase as TestCase;
13			use Thorr\Persistence\Doctrine\DataMapper\DoctrineAdapter;
14			use Thorr\Persistence\Doctrine\DataMapper\Manager\DoctrineAdapterAbstractFactory;
15			use Zend\ServiceManager\Exception as SMException;
16			use Zend\ServiceManager\ServiceLocatorInterface;
17
18			/**
19			* @covers Thorr\Persistence\Doctrine\DataMapper\Manager\DoctrineAdapterAbstractFactory
20			*/
21			class DoctrineAdapterAbstractFactoryTest extends TestCase
22			{
23			/**
24			* @var DoctrineAdapterAbstractFactory
25			*/
26			protected $abstractFactory;
27
28			/**
29			* @var ServiceLocatorInterface\|PHPUnit_Framework_MockObject_MockObject
30			*/
31			protected $serviceLocator;
32
33			public function setUp()
34			{
35			$this->abstractFactory = new DoctrineAdapterAbstractFactory();
36			$this->serviceLocator = $this->getMock(ServiceLocatorInterface::class);
37			}
38
39			/**
40			* @param $config
41			* @param $expectedResult
42			*
43			* @dataProvider canCreateServiceConfigProvider()
44			*/
45			public function testCanCreateService($config, $serviceName, $expectedResult)
46			{
47			$this->serviceLocator->expects($this->atLeastOnce())
			0 ignored issues – show Bug introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Zend\ServiceManager\ServiceLocatorInterface`. 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...
48			->method('get')
49			->with('config')
50			->willReturn($config);
51
52			$this->assertSame(
53			$expectedResult,
54			$this->abstractFactory->canCreateServiceWithName($this->serviceLocator, 'unrelevant', $serviceName)
55			);
56			}
57
58			public function canCreateServiceConfigProvider()
59			{
60			return [
61			[
62			[],
63			'anything',
64			false,
65			],
66			[
67			[ 'thorr_persistence_dmm' => [] ],
68			'anything',
69			false,
70			],
71			[
72			[
73			'thorr_persistence_dmm' => [
74			'doctrine' => [],
75			],
76			],
77			'anything',
78			false,
79			],
80			[
81			[
82			'thorr_persistence_dmm' => [
83			'doctrine' => [
84			'adapters' => [],
85			],
86			],
87			],
88			'anything',
89			false,
90			],
91			[
92			[
93			'thorr_persistence_dmm' => [
94			'doctrine' => [
95			'adapters' => [
96			'SomeDataMapperServiceName' => 123,
97			],
98			],
99			],
100			],
101			'SomeDataMapperServiceName',
102			false,
103			],
104			[
105			[
106			'thorr_persistence_dmm' => [
107			'doctrine' => [
108			'adapters' => [
109			'SomeDataMapperServiceName' => DoctrineAdapter::class,
110			],
111			],
112			],
113			],
114			'SomeDataMapperServiceName',
115			true,
116			],
117			[
118			[
119			'thorr_persistence_dmm' => [
120			'doctrine' => [
121			'adapters' => [
122			'SomeDataMapperServiceName' => 'not-a-class',
123			],
124			],
125			],
126			],
127			'SomeDataMapperServiceName',
128			false,
129			],
130			[
131			[
132			'thorr_persistence_dmm' => [
133			'doctrine' => [
134			'adapters' => [
135			'SomeDataMapperServiceName' => [
136			'class' => DoctrineAdapter::class,
137			],
138			],
139			],
140			],
141			],
142			'SomeDataMapperServiceName',
143			true,
144			],
145			[
146			[
147			'thorr_persistence_dmm' => [
148			'doctrine' => [
149			'adapters' => [
150			'SomeDataMapperServiceName' => [],
151			],
152			],
153			],
154			],
155			'SomeDataMapperServiceName',
156			false,
157			],
158			[
159			[
160			'thorr_persistence_dmm' => [
161			'doctrine' => [
162			'adapters' => [
163			'SomeDataMapperServiceName' => [
164			'class' => 'not-a-class',
165			],
166			],
167			],
168			],
169			],
170			'SomeDataMapperServiceName',
171			false,
172			],
173			];
174			}
175
176			/**
177			* @param $config
178			* @dataProvider createServiceConfigProvider
179			*/
180			public function testCreateService($config, $requestedName, $expectedException)
181			{
182			$objectManagers = [
183			'SomeObjectManagerService' => $this->getMock(ObjectManager::class, [], [], 'SomeObjectManagerService'),
184			'AnotherObjectManagerService' => $this->getMock(ObjectManager::class, [], [], 'AnotherObjectManagerService'),
185			];
186
187			$this->serviceLocator->expects($this->any())
			0 ignored issues – show Bug introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Zend\ServiceManager\ServiceLocatorInterface`. 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...
188			->method('has')
189			->willReturnCallback(function ($name) use ($objectManagers) {
190			return array_key_exists($name, $objectManagers);
191			});
192
193			$this->serviceLocator->expects($this->any())
			0 ignored issues – show Bug introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Zend\ServiceManager\ServiceLocatorInterface`. 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...
194			->method('get')
195			->willReturnCallback(function ($name) use ($config, $objectManagers) {
196			switch ($name) {
197			case 'config' :
			0 ignored issues – show Coding Style introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report There must be no space before the colon in a CASE statement As per the PSR-2 coding standard, there must not be a space in front of the colon in case statements. switch ($selector) { case "A": //right doSomething(); break; case "B" : //wrong doSomethingElse(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
198			return $config;
199			case 'SomeObjectManagerService' :
			0 ignored issues – show Coding Style introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report There must be no space before the colon in a CASE statement As per the PSR-2 coding standard, there must not be a space in front of the colon in case statements. switch ($selector) { case "A": //right doSomething(); break; case "B" : //wrong doSomethingElse(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
200			case 'AnotherObjectManagerService' :
			0 ignored issues – show Coding Style introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report There must be no space before the colon in a CASE statement As per the PSR-2 coding standard, there must not be a space in front of the colon in case statements. switch ($selector) { case "A": //right doSomething(); break; case "B" : //wrong doSomethingElse(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
201			return $objectManagers[$name];
202			}
203			});
204
205			if ($expectedException) {
206			$this->setExpectedException($expectedException[0], $expectedException[1]);
207			}
208
209			/** @var DoctrineAdapter $instance */
210			$instance = $this->abstractFactory->createServiceWithName($this->serviceLocator, 'unrelevant', $requestedName);
211
212			$this->assertInstanceOf(DoctrineAdapter::class, $instance);
213			$this->assertSame(
214			$requestedName,
215			$config['thorr_persistence_dmm']['entity_data_mapper_map'][$instance->getEntityClass()]
216			);
217
218			if (isset($config['thorr_persistence_dmm']['doctrine']['adapters'][$requestedName]['object_manager'])) {
219			$this->assertSame(
220			$objectManagers[$config['thorr_persistence_dmm']['doctrine']['adapters'][$requestedName]['object_manager']],
221			$instance->getObjectManager()
222			);
223			}
224			}
225
226			public function createServiceConfigProvider()
227			{
228			return [
229			[
230			// $config
231			[
232			'thorr_persistence_dmm' => [
233			'doctrine' => [
234			'object_manager' => 'SomeObjectManagerService',
235			'adapters' => [
236			'SomeDataMapperServiceName' => DoctrineAdapter::class,
237			],
238			],
239			'entity_data_mapper_map' => [
240			'SomeEntityClass' => 'SomeDataMapperServiceName',
241			],
242			],
243			],
244			// $requestedName
245			'SomeDataMapperServiceName',
246			// $expectedException
247			null,
248			],
249			[
250			// $config
251			[
252			'thorr_persistence_dmm' => [
253			'doctrine' => [
254			'object_manager' => 'SomeObjectManagerService',
255			'adapters' => [
256			'SomeDataMapperServiceName' => \stdClass::class,
257			],
258			],
259			'entity_data_mapper_map' => [
260			'SomeEntityClass' => 'SomeDataMapperServiceName',
261			],
262			],
263			],
264			// $requestedName
265			'SomeDataMapperServiceName',
266			// $expectedException
267			[ SMException\ServiceNotCreatedException::class, 'Invalid data mapper type' ],
268			],
269			[
270			// $config
271			[
272			'thorr_persistence_dmm' => [
273			'doctrine' => [
274			'object_manager' => 'SomeObjectManagerService',
275			'adapters' => [
276			'SomeDataMapperServiceName' => [
277			'class' => DoctrineAdapter::class,
278			'object_manager' => 'AnotherObjectManagerService',
279			],
280			],
281			],
282			'entity_data_mapper_map' => [
283			'SomeEntityClass' => 'SomeDataMapperServiceName',
284			],
285			],
286			],
287			// $requestedName
288			'SomeDataMapperServiceName',
289			// $expectedException
290			null,
291			],
292			];
293			}
294
295			public function testInvalidObjectManagerClass()
296			{
297			$config = [
298			'thorr_persistence_dmm' => [
299			'doctrine' => [
300			'object_manager' => 'SomeObjectManagerService',
301			'adapters' => [
302			'SomeDataMapperServiceName' => DoctrineAdapter::class,
303			],
304			],
305			'entity_data_mapper_map' => [],
306			],
307			];
308
309			$objectManager = $this->getMock(\stdClass::class);
310
311			$this->serviceLocator->expects($this->atLeastOnce())
			0 ignored issues – show Bug introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report The method `expects` does only exist in `PHPUnit_Framework_MockObject_MockObject`, but not in `Zend\ServiceManager\ServiceLocatorInterface`. 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...
312			->method('has')
313			->with($config['thorr_persistence_dmm']['doctrine']['object_manager'])
314			->willReturn(true);
315
316			$this->serviceLocator->expects($this->atLeastOnce())
317			->method('get')
318			->willReturnCallback(function ($name) use ($config, $objectManager) {
319			switch ($name) {
320			case 'config' :
			0 ignored issues – show Coding Style introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report There must be no space before the colon in a CASE statement As per the PSR-2 coding standard, there must not be a space in front of the colon in case statements. switch ($selector) { case "A": //right doSomething(); break; case "B" : //wrong doSomethingElse(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
321			return $config;
322			case $config['thorr_persistence_dmm']['doctrine']['object_manager'] :
			0 ignored issues – show Coding Style introduced 2016-01-19 18:24 UTC by Report Bug Copy Issue Report There must be no space before the colon in a CASE statement As per the PSR-2 coding standard, there must not be a space in front of the colon in case statements. switch ($selector) { case "A": //right doSomething(); break; case "B" : //wrong doSomethingElse(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
323			return $objectManager;
324			}
325			});
326
327			$this->setExpectedException(SMException\InvalidServiceNameException::class, 'Invalid object manager type');
328
329			$this->abstractFactory->createServiceWithName($this->serviceLocator, 'unrelevant', 'SomeDataMapperServiceName');
330			}
331			}
332

stefanotorresi / thorr-persistence-doctrine

createServiceConfigProvider() A last analyzed 2016-01-20 16:34 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

createServiceConfigProvider() A
last analyzed 2016-01-20 16:34 UTC