MiddlewareIncluedHandlerTest::testMiddlewareIncluedHandler() - 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

testMiddlewareIncluedHandler() A

↳ Parent: MiddlewareIncluedHandlerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	9
nc	1
nop	0

<?php

namespace Nimo\Tests;

use Nimo\Handlers\CallableHandler;
use Nimo\Handlers\MiddlewareIncluedHandler;
use PHPUnit\Framework\Assert;

class MiddlewareIncluedHandlerTest extends NimoTestCase
{

    public function testMiddlewareIncluedHandler()
    {
        $wrappedHandler = $this->throwHandler();
        $request = $this->getRequestMock();
        $response = $this->getResponseMock();
        $middleware = $this->assertedMiddleware($request, $wrappedHandler, $response);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        /**
         * @var CallableHandler $wrappedHandler
         */
        $handler = $wrappedHandler->includeMiddleware($middleware);
        Assert::assertInstanceOf(MiddlewareIncluedHandler::class, $handler);

        $actualResponse = $handler->handle($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        Assert::assertSame($response, $actualResponse);

    }
}


1			<?php
2
3			namespace Nimo\Tests;
4
5			use Nimo\Handlers\CallableHandler;
6			use Nimo\Handlers\MiddlewareIncluedHandler;
7			use PHPUnit\Framework\Assert;
8
9			class MiddlewareIncluedHandlerTest extends NimoTestCase
10			{
11
12			public function testMiddlewareIncluedHandler()
13			{
14			$wrappedHandler = $this->throwHandler();
15			$request = $this->getRequestMock();
16			$response = $this->getResponseMock();
17			$middleware = $this->assertedMiddleware($request, $wrappedHandler, $response);
			0 ignored issues – show Documentation introduced 2018-01-06 14:37 UTC by Report Bug Copy Issue Report `$request` 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... Documentation introduced 2018-01-06 14:37 UTC by Report Bug Copy Issue Report `$response` 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...
18
19			/**
20			* @var CallableHandler $wrappedHandler
21			*/
22			$handler = $wrappedHandler->includeMiddleware($middleware);
23			Assert::assertInstanceOf(MiddlewareIncluedHandler::class, $handler);
24
25			$actualResponse = $handler->handle($request);
			0 ignored issues – show Documentation introduced 2018-01-06 14:37 UTC by Report Bug Copy Issue Report `$request` 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...
26			Assert::assertSame($response, $actualResponse);
27
28			}
29			}
30

LitPHP / nimo

Push — master ( d8d5ef...b9f7fc )

testMiddlewareIncluedHandler() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like