HandlerTest - Code Metrics - gbprod/elastica-specification - Measure and Improve Code Quality continuously with Scrutinizer

HandlerTest A
last analyzed 2018-02-22 19:05 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
wmc	4
c	2
b	0
f	0
lcom	1
cbo	8
dl	0
loc	79
rs	10

4 Methods

Rating	Name	Size	Complexity
A	setUp()	6	1
A	testConstructWillRegisterBaseFactoriess()	20	1
A	testRegisterFactoryAddFactoryInRegistry()	12	1
A	testHandle()	20	1

<?php

namespace Tests\GBProd\ElasticaSpecification;

use Elastica\QueryBuilder;
use Elastica\Query\AbstractQuery;
use GBProd\ElasticaSpecification\Handler;
use GBProd\ElasticaSpecification\QueryFactory\AndXFactory;
use GBProd\ElasticaSpecification\QueryFactory\Factory;
use GBProd\ElasticaSpecification\QueryFactory\NotFactory;
use GBProd\ElasticaSpecification\QueryFactory\OrXFactory;
use GBProd\ElasticaSpecification\Registry;
use GBProd\Specification\AndX;
use GBProd\Specification\Not;
use GBProd\Specification\OrX;
use GBProd\Specification\Specification;
use PHPUnit\Framework\TestCase;

class HandlerTest extends TestCase
{
    /**
     * @var Registry
     */
    private $registry;

    /**
     * @var QueryBuilder
     */
    private $qb;

    /**
     * @var Handler
     */
    private $handler;

    protected function setUp()
    {
        $this->registry = new Registry();
        $this->qb = new QueryBuilder();
        $this->handler = new Handler($this->registry, $this->qb);
    }

    public function testConstructWillRegisterBaseFactoriess()
    {
        $spec1 = $this->createMock(Specification::class);
        $spec2 = $this->createMock(Specification::class);

        $this->assertInstanceOf(
            AndXFactory::class,
            $this->registry->getFactory(new AndX($spec1, $spec2))
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        );

        $this->assertInstanceOf(
            OrXFactory::class,
            $this->registry->getFactory(new OrX($spec1, $spec2))
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        );

        $this->assertInstanceOf(
            NotFactory::class,
            $this->registry->getFactory(new Not($spec1))
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        );
    }

    public function testRegisterFactoryAddFactoryInRegistry()
    {
        $factory = $this->createMock(Factory::class);
        $spec = $this->createMock(Specification::class);

        $this->handler->registerFactory(get_class($spec), $factory);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $this->assertEquals(
            $factory,
            $this->registry->getFactory($spec)
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        );
    }

    public function testHandle()
    {
        $factory = $this->prophesize(Factory::class);

        $spec = $this->createMock(Specification::class);
        $this->handler->registerFactory(get_class($spec), $factory->reveal());

        $builtQuery = $this->getMockForAbstractClass(AbstractQuery::class);

        $factory
            ->create($spec, $this->qb)
            ->willReturn($builtQuery)
            ->shouldBeCalled()
        ;

        $this->assertEquals(
            $builtQuery,
            $this->handler->handle($spec)
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        );
    }
}


1			<?php
2
3			namespace Tests\GBProd\ElasticaSpecification;
4
5			use Elastica\QueryBuilder;
6			use Elastica\Query\AbstractQuery;
7			use GBProd\ElasticaSpecification\Handler;
8			use GBProd\ElasticaSpecification\QueryFactory\AndXFactory;
9			use GBProd\ElasticaSpecification\QueryFactory\Factory;
10			use GBProd\ElasticaSpecification\QueryFactory\NotFactory;
11			use GBProd\ElasticaSpecification\QueryFactory\OrXFactory;
12			use GBProd\ElasticaSpecification\Registry;
13			use GBProd\Specification\AndX;
14			use GBProd\Specification\Not;
15			use GBProd\Specification\OrX;
16			use GBProd\Specification\Specification;
17			use PHPUnit\Framework\TestCase;
18
19			class HandlerTest extends TestCase
20			{
21			/**
22			* @var Registry
23			*/
24			private $registry;
25
26			/**
27			* @var QueryBuilder
28			*/
29			private $qb;
30
31			/**
32			* @var Handler
33			*/
34			private $handler;
35
36			protected function setUp()
37			{
38			$this->registry = new Registry();
39			$this->qb = new QueryBuilder();
40			$this->handler = new Handler($this->registry, $this->qb);
41			}
42
43			public function testConstructWillRegisterBaseFactoriess()
44			{
45			$spec1 = $this->createMock(Specification::class);
46			$spec2 = $this->createMock(Specification::class);
47
48			$this->assertInstanceOf(
49			AndXFactory::class,
50			$this->registry->getFactory(new AndX($spec1, $spec2))
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec1` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec2` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
51			);
52
53			$this->assertInstanceOf(
54			OrXFactory::class,
55			$this->registry->getFactory(new OrX($spec1, $spec2))
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec1` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec2` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
56			);
57
58			$this->assertInstanceOf(
59			NotFactory::class,
60			$this->registry->getFactory(new Not($spec1))
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec1` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
61			);
62			}
63
64			public function testRegisterFactoryAddFactoryInRegistry()
65			{
66			$factory = $this->createMock(Factory::class);
67			$spec = $this->createMock(Specification::class);
68
69			$this->handler->registerFactory(get_class($spec), $factory);
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$factory` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\ElasticaSp...n\QueryFactory\Factory>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
70
71			$this->assertEquals(
72			$factory,
73			$this->registry->getFactory($spec)
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
74			);
75			}
76
77			public function testHandle()
78			{
79			$factory = $this->prophesize(Factory::class);
80
81			$spec = $this->createMock(Specification::class);
82			$this->handler->registerFactory(get_class($spec), $factory->reveal());
83
84			$builtQuery = $this->getMockForAbstractClass(AbstractQuery::class);
85
86			$factory
87			->create($spec, $this->qb)
88			->willReturn($builtQuery)
89			->shouldBeCalled()
90			;
91
92			$this->assertEquals(
93			$builtQuery,
94			$this->handler->handle($spec)
			0 ignored issues – show Documentation introduced 2018-02-01 07:54 UTC by Report Bug Copy Issue Report `$spec` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GBProd\Specification\Specification>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
95			);
96			}
97			}
98

gbprod / elastica-specification

HandlerTest A last analyzed 2018-02-22 19:05 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

4 Methods

Duplication Side-by-Side

Filter issues like

HandlerTest A
last analyzed 2018-02-22 19:05 UTC