Issues in NodeEventTest.php - All Issues - Inspection of "Merge branch '5.6'" - Kunstmaan/KunstmaanBundlesCMS - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ae5e03...0447ee )

by Jeroen

created 2020-08-07 09:00 UTC

NodeBundle/Tests/unit/Event/NodeEventTest.php (2 issues)

Labels

Documentation 2

Severity

Minor 2

Ruud Denivel 2

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace Kunstmaan\NodeBundle\Tests\Event;

use Kunstmaan\NodeBundle\Entity\HasNodeInterface;
use Kunstmaan\NodeBundle\Entity\Node;
use Kunstmaan\NodeBundle\Entity\NodeTranslation;
use Kunstmaan\NodeBundle\Entity\NodeVersion;
use Kunstmaan\NodeBundle\Event\NodeEvent;
use PHPUnit\Framework\TestCase;
use Symfony\Component\HttpFoundation\Response;

/**
 * Class NodeEventTest
 */
class NodeEventTest extends TestCase
{
    public function testGetSet()
    {
        /** @var Node $node */
        $node = $this->createMock(Node::class);
        /** @var NodeTranslation $nodeTranslation */
        $nodeTranslation = $this->createMock(NodeTranslation::class);
        /** @var NodeVersion $nodeVersion */
        $nodeVersion = $this->createMock(NodeVersion::class);
        $page = $this->createMock(HasNodeInterface::class);

        $event = new NodeEvent($node, $nodeTranslation, $nodeVersion, $page);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $this->assertInstanceOf(Node::class, $event->getNode());
        $this->assertInstanceOf(NodeTranslation::class, $event->getNodeTranslation());
        $this->assertInstanceOf(\get_class($page), $event->getPage());
        $this->assertInstanceOf(NodeVersion::class, $event->getNodeVersion());

        $event->setNode($node);
        $event->setNodeTranslation($nodeTranslation);
        $event->setNodeVersion($nodeVersion);
        $event->setPage($page);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $event->setResponse(new Response());

        $this->assertInstanceOf(Node::class, $event->getNode());
        $this->assertInstanceOf(NodeTranslation::class, $event->getNodeTranslation());
        $this->assertInstanceOf(\get_class($page), $event->getPage());
        $this->assertInstanceOf(NodeVersion::class, $event->getNodeVersion());
        $this->assertInstanceOf(Response::class, $event->getResponse());
    }
}


1			<?php
2
3			namespace Kunstmaan\NodeBundle\Tests\Event;
4
5			use Kunstmaan\NodeBundle\Entity\HasNodeInterface;
6			use Kunstmaan\NodeBundle\Entity\Node;
7			use Kunstmaan\NodeBundle\Entity\NodeTranslation;
8			use Kunstmaan\NodeBundle\Entity\NodeVersion;
9			use Kunstmaan\NodeBundle\Event\NodeEvent;
10			use PHPUnit\Framework\TestCase;
11			use Symfony\Component\HttpFoundation\Response;
12
13			/**
14			* Class NodeEventTest
15			*/
16			class NodeEventTest extends TestCase
17			{
18			public function testGetSet()
19			{
20			/** @var Node $node */
21			$node = $this->createMock(Node::class);
22			/** @var NodeTranslation $nodeTranslation */
23			$nodeTranslation = $this->createMock(NodeTranslation::class);
24			/** @var NodeVersion $nodeVersion */
25			$nodeVersion = $this->createMock(NodeVersion::class);
26			$page = $this->createMock(HasNodeInterface::class);
27
28			$event = new NodeEvent($node, $nodeTranslation, $nodeVersion, $page);
			0 ignored issues – show Documentation introduced 2019-03-03 18:46 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$page` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Kunstmaan\NodeBun...ntity\HasNodeInterface>`. 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...
29
30			$this->assertInstanceOf(Node::class, $event->getNode());
31			$this->assertInstanceOf(NodeTranslation::class, $event->getNodeTranslation());
32			$this->assertInstanceOf(\get_class($page), $event->getPage());
33			$this->assertInstanceOf(NodeVersion::class, $event->getNodeVersion());
34
35			$event->setNode($node);
36			$event->setNodeTranslation($nodeTranslation);
37			$event->setNodeVersion($nodeVersion);
38			$event->setPage($page);
			0 ignored issues – show Documentation introduced 2019-03-03 18:46 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$page` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Kunstmaan\NodeBun...ntity\HasNodeInterface>`. 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...
39			$event->setResponse(new Response());
40
41			$this->assertInstanceOf(Node::class, $event->getNode());
42			$this->assertInstanceOf(NodeTranslation::class, $event->getNodeTranslation());
43			$this->assertInstanceOf(\get_class($page), $event->getPage());
44			$this->assertInstanceOf(NodeVersion::class, $event->getNodeVersion());
45			$this->assertInstanceOf(Response::class, $event->getResponse());
46			}
47			}
48

Kunstmaan / KunstmaanBundlesCMS

Push — master ( ae5e03...0447ee )

NodeBundle/Tests/unit/Event/NodeEventTest.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like