FixedResponseMiddlewareTest - 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

FixedResponseMiddlewareTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
wmc	1
lcom	1
cbo	2
dl	0
loc	18
rs	10
c	0
b	0
f	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testFixedResponseMiddleware()	0	13	1

<?php namespace Nimo\Tests;

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

use Nimo\Middlewares\FixedResponseMiddleware;

class FixedResponseMiddlewareTest extends NimoTestCase
{
    public function testFixedResponseMiddleware()
    {
        $answerRes = $this->getResponseMock();
        /** @noinspection PhpParamsInspection */
        $middleware = new FixedResponseMiddleware($answerRes);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $returnValue = $middleware->process(
            $this->getRequestMock(),
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->throwHandler()
        );

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


}


1			<?php namespace Nimo\Tests;
2
3			/**
4			* User: mcfog
5			* Date: 15/9/13
6			*/
7
8			use Nimo\Middlewares\FixedResponseMiddleware;
9
10			class FixedResponseMiddlewareTest extends NimoTestCase
11			{
12			public function testFixedResponseMiddleware()
13			{
14			$answerRes = $this->getResponseMock();
15			/** @noinspection PhpParamsInspection */
16			$middleware = new FixedResponseMiddleware($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 = $middleware->process(
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			$this->throwHandler()
21			);
22
23			$this->assertSame($answerRes, $returnValue);
24			}
25
26
27			}
28

LitPHP / nimo

Push — master ( d8d5ef...b9f7fc )

FixedResponseMiddlewareTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like