Fabrica\Bundle\CoreBundle\Tests\Command\UserSshKeyCreateCommandTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	41.86 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	5	1
A	tearDown()	0	4	1
A	testSimpleCase()	18	20	1
A	testNonExistingUser()	0	6	1

<?php

/**
 * This file is part of Fabrica.
 *
 * (c) Alexandre Salomé <alexandre.salome@gmail.com>
 * (c) Julien DIDIER <genzo.wm@gmail.com>
 *
 * This source file is subject to the GPL license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Fabrica\Bundle\CoreBundle\Tests\Command;

use Fabrica\Bundle\CoreBundle\Test\CommandTestCase;

class UserSshKeyCreateCommandTest extends CommandTestCase
{
    protected $client;

    protected function setUp(): void
    {
        $this->client = self::createClient();
        $this->client->startIsolation();
    }

    public function tearDown(): void
    {
        $this->client->stopIsolation();
    }

    public function testSimpleCase()

This method seems to be duplicated in your project.

    {
        list($statusCode ,$output) = $this->runCommand($this->client, 'fabrica:user-ssh-key-create "alice" foo bar');

The assignment to $statusCode is unused. Consider omitting it like so list($first,,$third).

        $this->assertEquals("The key named foo was successfully added to user alice!\n", $output);

        $em = $this->client->getKernel()->getContainer()->get('doctrine')->getManager();

        $userSshKey = $em->getRepository('FabricaCoreBundle:UserSshKey')->findOneBy(
            array(
            'title' => 'foo'
            )
        );

        $this->assertInstanceOf('Fabrica\Models\Code\UserSshKey', $userSshKey);

        $this->assertEquals('alice', $userSshKey->getUser()->getUsername());
        $this->assertEquals('foo', $userSshKey->getTitle());
        $this->assertEquals('bar', $userSshKey->getContent());
    }

    public function testNonExistingUser()
    {
        list($statusCode ,$output) = $this->runCommand($this->client, 'fabrica:user-ssh-key-create "foo" bar baz');

The assignment to $statusCode is unused. Consider omitting it like so list($first,,$third).

        $this->assertContains('User with username "foo" not found', $output);
    }
}