Doctrine\Bundle\DoctrineBundle\Tests\Command\DropDatabaseDoctrineTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	120
Duplicated Lines	50.83 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	testExecute()	0	23	1
A	testExecuteWithoutOptionForceWillFailWithAttentionMessage()	23	23	1
B	getMockContainer()	37	37	1

<?php


namespace Doctrine\Bundle\DoctrineBundle\Tests\Command;

use Doctrine\Bundle\DoctrineBundle\Command\DropDatabaseDoctrineCommand;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Console\Application;
use Symfony\Component\Console\Tester\CommandTester;

class DropDatabaseDoctrineTest extends TestCase
{

    public function testExecute()
    {
        $connectionName = 'default';
        $dbName = 'test';
        $params = array(
            'url' => "sqlite:///". sys_get_temp_dir() ."/test.db",
            'path' => sys_get_temp_dir() . "/" . $dbName,
            'driver' => 'pdo_sqlite',
        );

        $application = new Application();
        $application->add(new DropDatabaseDoctrineCommand());

        $command = $application->find('doctrine:database:drop');
        $command->setContainer($this->getMockContainer($connectionName, $params));

        $commandTester = new CommandTester($command);
        $commandTester->execute(
            array_merge(array('command' => $command->getName(), '--force' => true))
        );

        $this->assertContains("Dropped database for connection named " . sys_get_temp_dir() . "/" . $dbName . "" , $commandTester->getDisplay());
    }

    /*

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

    public function testExecuteWithConnectionUrlParam()
    {
        $connectionName = 'default';
        $dbName = 'test';
        $params = array(
            'url' => "mysql://root@127.0.0.1:3306/" . $dbName,
            'user' => 'root',
            'dbname' => $dbName,
            'driver' => "pdo_mysql"
        );

        $application = new Application();
        $application->add(new DropDatabaseDoctrineCommand());

        $command = $application->find('doctrine:database:drop');
        $command->setContainer($this->getMockContainer($connectionName, $params));

        $commandTester = new CommandTester($command);
        $commandTester->execute(
            array_merge(array('command' => $command->getName(), '--force' => true, '--if-exists' => true))
        );

        $this->assertContains("Database for connection named `". $dbName ."` doesn't exist. Skipped.", $commandTester->getDisplay());
    }*/

    public function testExecuteWithoutOptionForceWillFailWithAttentionMessage()

This method seems to be duplicated in your project.

    {
        $connectionName = 'default';
        $dbName = 'test';
        $params = array(
            'path' => sys_get_temp_dir() . "/" . $dbName,
            'driver' => 'pdo_sqlite',
        );

        $application = new Application();
        $application->add(new DropDatabaseDoctrineCommand());

        $command = $application->find('doctrine:database:drop');
        $command->setContainer($this->getMockContainer($connectionName, $params));

        $commandTester = new CommandTester($command);
        $commandTester->execute(
            array_merge(array('command' => $command->getName()))
        );

        $this->assertContains("Would drop the database named " . sys_get_temp_dir() . "/" . $dbName . ".", $commandTester->getDisplay());
        $this->assertContains("Please run the operation with --force to execute", $commandTester->getDisplay());
    }

    /**
     * @param string     $connectionName Connection name
     * @param array|null $params         Connection parameters
     * @return \PHPUnit_Framework_MockObject_MockObject
     */
    private function getMockContainer($connectionName, $params = null)

This method seems to be duplicated in your project.

    {
        // Mock the container and everything you'll need here
        $mockDoctrine = $this->getMockBuilder('Doctrine\Common\Persistence\ConnectionRegistry')
            ->getMock();

        $mockDoctrine->expects($this->any())
            ->method('getDefaultConnectionName')
            ->withAnyParameters()
            ->willReturn($connectionName);

        $mockConnection = $this->getMockBuilder('Doctrine\DBAL\Connection')
            ->disableOriginalConstructor()
            ->setMethods(array('getParams'))
            ->getMockForAbstractClass();

        $mockConnection->expects($this->any())
            ->method('getParams')
            ->withAnyParameters()
            ->willReturn($params);

        $mockDoctrine->expects($this->any())
            ->method('getConnection')
            ->withAnyParameters()
            ->willReturn($mockConnection);

        $mockContainer = $this->getMockBuilder('Symfony\Component\DependencyInjection\Container')
            ->setMethods(array('get'))
            ->getMock();

        $mockContainer->expects($this->any())
            ->method('get')
            ->with('doctrine')
            ->willReturn($mockDoctrine);

        return $mockContainer;
    }
}