FixedResponseHandlerTest::testFixedResponseHandler() - Code Metrics - Inspection of "fix case" - LitPHP/nimo - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( d8d5ef...b9f7fc )

by mcfog

created 2018-01-06 14:35 UTC

testFixedResponseHandler() A

↳ Parent: FixedResponseHandlerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	6
nc	1
nop	0

<?php namespace Nimo\Tests;

/**
 * User: mcfog
 * Date: 15/9/13
 */

use Nimo\Handlers\FixedResponseHandler;

class FixedResponseHandlerTest extends NimoTestCase
{
    public function testFixedResponseHandler()
    {
        $answerRes = $this->getResponseMock();
        /** @noinspection PhpParamsInspection */
        $handler = FixedResponseHandler::wrap($answerRes);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $returnValue = $handler->handle(
            $this->getRequestMock()
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $this->assertSame($answerRes, $returnValue);
    }


}


1			<?php namespace Nimo\Tests;
2
3			/**
4			* User: mcfog
5			* Date: 15/9/13
6			*/
7
8			use Nimo\Handlers\FixedResponseHandler;
9
10			class FixedResponseHandlerTest extends NimoTestCase
11			{
12			public function testFixedResponseHandler()
13			{
14			$answerRes = $this->getResponseMock();
15			/** @noinspection PhpParamsInspection */
16			$handler = FixedResponseHandler::wrap($answerRes);
			0 ignored issues – show Documentation introduced 2018-01-06 14:37 UTC by Report Bug Copy Issue Report `$answerRes` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Psr\Http\Message\ResponseInterface>`. 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...
17
18			$returnValue = $handler->handle(
19			$this->getRequestMock()
			0 ignored issues – show Documentation introduced 2018-01-06 14:37 UTC by Report Bug Copy Issue Report `$this->getRequestMock()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Psr\Http\Message\ServerRequestInterface>`. 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...
20			);
21
22			$this->assertSame($answerRes, $returnValue);
23			}
24
25
26			}
27

LitPHP / nimo

Push — master ( d8d5ef...b9f7fc )

testFixedResponseHandler() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like