Passed
Pull Request — master (#233)
by Dmitriy
02:41
created

JsonFormatterTest   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 12
Duplicated Lines 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 8
c 1
b 0
f 0
dl 0
loc 12
rs 10
wmc 1

1 Method

Rating   Name   Duplication   Size   Complexity  
A testFormatter() 0 10 1
1
<?php
2
3
namespace Yiisoft\Yii\Web\Tests;
4
5
use Nyholm\Psr7\Factory\Psr17Factory;
6
use PHPUnit\Framework\TestCase;
7
use Yiisoft\Yii\Web\Formatter\JsonResponseFormatter;
8
use Yiisoft\Yii\Web\WebResponse;
9
10
class JsonFormatterTest extends TestCase
11
{
12
    public function testFormatter(): void
13
    {
14
        $factory = new Psr17Factory();
15
        $webResponse = new WebResponse(['test' => 'test'], 200, $factory, $factory);
0 ignored issues
show
Unused Code introduced by
The call to Yiisoft\Yii\Web\WebResponse::__construct() has too many arguments starting with $factory. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

15
        $webResponse = /** @scrutinizer ignore-call */ new WebResponse(['test' => 'test'], 200, $factory, $factory);

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
16
        $formatter = new JsonResponseFormatter();
17
        $result = $formatter->format($webResponse);
18
        $result->getBody()->rewind();
19
20
        $this->assertSame('{"test":"test"}', $result->getBody()->getContents());
21
        $this->assertSame(['application/json'], $result->getHeader('Content-Type'));
22
    }
23
}
24