NullStringControllerModelTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testCanBeCreated()	0	8	1
A	testReturnsResponseParameters()	0	14	1

<?php

/*
 * This file is part of the Symfony-Util package.
 *
 * (c) Jean-Bernard Addor
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

use PHPUnit\Framework\TestCase;
use Symfony\Component\HttpFoundation\Request;
use SymfonyUtil\Component\HttpFoundation\NullStringControllerModel;

/**
 * @covers \SymfonyUtil\Component\HttpFoundation\NullStringControllerModel
 */
final class NullStringControllerModelTest extends TestCase

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function testCanBeCreated()
    {
        $this->assertInstanceOf(
            // ::class, // 5.4 < php
            'SymfonyUtil\Component\HttpFoundation\NullStringControllerModel',
            new NullStringControllerModel('')

The call to NullStringControllerModel::__construct() has too many arguments starting with ''.

        );
    }

    public function testReturnsResponseParameters()
    {
        $this->assertInstanceOf(
            // ::class, // 5.4 < php
            'SymfonyUtil\Component\HttpFoundation\ResponseParameters',
            (new NullStringControllerModel())->__invoke('', new Request())
        );
        /*

70% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->assertInternalType('array', (new NullStringControllerModel())->__invoke('', new Request()));
        $this->assertSame([], (new NullStringControllerModel())->__invoke('', new Request()));
        $this->assertSame(0, count((new NullStringControllerModel())->__invoke('', new Request())));
        $this->assertEmpty((new NullStringControllerModel())->__invoke('', new Request()));
        */
    }
}