ListTetantsCommandTest::testListWithTenantsAndOrganizationOption() - Code Metrics - Inspection of "Merge branch 'master' of github.com:superdesk/web-..." - SuperdeskWebPublisher/SWPMultiTenancyBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e32e27...6718d9 )

by Rafał

created 2017-02-20 20:24 UTC

testListWithTenantsAndOrganizationOption() A

↳ Parent: ListTetantsCommandTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	58
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	58
rs	9.639
c	0
b	0
f	0
cc	1
eloc	40
nc	1
nop	0

How to fix Long Method

<?php

/*
 * This file is part of the Superdesk Web Publisher MultiTenancy Bundle.
 *
 * Copyright 2016 Sourcefabric z.ú. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2016 Sourcefabric z.ú
 * @license http://www.superdesk.org/license
 */

namespace SWP\Bundle\MultiTenancyBundle\Tests\Command;

use SWP\Bundle\MultiTenancyBundle\Command\ListTenantsCommand;
use SWP\Component\MultiTenancy\Model\Organization;
use SWP\Component\MultiTenancy\Model\Tenant;
use SWP\Component\MultiTenancy\Repository\OrganizationRepositoryInterface;
use SWP\Component\MultiTenancy\Repository\TenantRepositoryInterface;
use Symfony\Component\Console\Application;
use Symfony\Component\Console\Tester\CommandTester;
use Symfony\Component\DependencyInjection\ContainerInterface;

class ListTetantsCommandTest extends \PHPUnit_Framework_TestCase
{
    const ORGANIZATION_CODE = '123456';

    private $commandTester;
    private $command;

    public function setUp()
    {
        $application = new Application();
        $application->add(new ListTenantsCommand());
        $this->command = $application->get('swp:tenant:list');
    }

    /**
     * @covers \SWP\Bundle\MultiTenancyBundle\Command\ListTenantsCommand
     */
    public function testExecuteWhenNoTenants()

    {
        $tenant = new Tenant();
        $tenant->setCode('123abc');
        $this->command->setContainer($this->getMockContainer(new Organization()));
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->commandTester = new CommandTester($this->command);
        $this->commandTester->execute(['command' => $this->command->getName()]);

        $this->assertContains(
            'There are no tenants defined.',
            trim($this->commandTester->getDisplay())
        );
    }

    /**
     * @covers \SWP\Bundle\MultiTenancyBundle\Command\ListTenantsCommand
     */
    public function testListWithTenants()
    {
        $tenant = new Tenant();
        $tenant->setCode('123abc');
        $tenant->setId('1');
        $tenant->setName('Test Tenant');
        $tenant->setCreatedAt(new \DateTime('2017-02-20 15:19:55'));
        $organization = new Organization();
        $organization->setCode('123456');
        $organization->setName('Test Organization');
        $organization->addTenant($tenant);
        $tenant->setOrganization($organization);
        $this->command->setContainer($this->getMockContainer($organization));
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->commandTester = new CommandTester($this->command);
        $this->commandTester->execute(['command' => $this->command->getName()]);

        $result = <<<'EOF'
List of all available tenants:
+----+--------+-------------+------------+---------------------+----------------------------------+
| Id | Code   | Name        | Is active? | Created at          | Organization                     |
+----+--------+-------------+------------+---------------------+----------------------------------+
| 1  | 123abc | Test Tenant | yes        | 2017-02-20 15:19:55 | Test Organization (code: 123456) |
+----+--------+-------------+------------+---------------------+----------------------------------+
EOF;

        $this->assertEquals($result, trim($this->commandTester->getDisplay()));
    }

    /**
     * @covers \SWP\Bundle\MultiTenancyBundle\Command\CreateTenantCommand
     */
    public function testListWithTenantsAndOrganizationOption()
    {
        $tenant = new Tenant();
        $tenant->setCode('123abc');
        $tenant->setId('1');
        $tenant->setName('Test Tenant');
        $tenant->setCreatedAt(new \DateTime('2017-02-20 15:19:55'));
        $organization = new Organization();
        $organization->setCode('123456');
        $organization->setName('Test Organization');
        $organization->addTenant($tenant);

        $tenant->setOrganization($organization);
        $this->command->setContainer($this->getMockContainer($organization));
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->commandTester = new CommandTester($this->command);
        $this->commandTester->execute(['command' => $this->command->getName()]);

        $result = <<<'EOF'
List of all available tenants:
+----+--------+-------------+------------+---------------------+----------------------------------+
| Id | Code   | Name        | Is active? | Created at          | Organization                     |
+----+--------+-------------+------------+---------------------+----------------------------------+
| 1  | 123abc | Test Tenant | yes        | 2017-02-20 15:19:55 | Test Organization (code: 123456) |
+----+--------+-------------+------------+---------------------+----------------------------------+
EOF;

        $this->assertEquals($result, trim($this->commandTester->getDisplay()));

        $tenant2 = new Tenant();
        $tenant2->setCode('345def');
        $tenant2->setId('2');
        $tenant2->setName('Test Tenant 2');
        $tenant2->setCreatedAt(new \DateTime('2017-02-20 15:19:55'));
        $organization2 = new Organization();
        $organization2->setCode('789012');
        $organization2->setName('Test Organization2');
        $organization2->addTenant($tenant2);
        $tenant2->setOrganization($organization2);

        $this->command->setContainer($this->getMockContainer($organization2));
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->commandTester = new CommandTester($this->command);
        $this->commandTester->execute(['command' => $this->command->getName(), '-o' => '123456']);
        $result = <<<'EOF'
There are no tenants defined.
EOF;
        $this->assertEquals($result, trim($this->commandTester->getDisplay()));

        $this->commandTester->execute(['command' => $this->command->getName()]);
        $result = <<<'EOF'
List of all available tenants:
+----+--------+---------------+------------+---------------------+-----------------------------------+
| Id | Code   | Name          | Is active? | Created at          | Organization                      |
+----+--------+---------------+------------+---------------------+-----------------------------------+
| 2  | 345def | Test Tenant 2 | yes        | 2017-02-20 15:19:55 | Test Organization2 (code: 789012) |
+----+--------+---------------+------------+---------------------+-----------------------------------+
EOF;
        $this->assertEquals($result, trim($this->commandTester->getDisplay()));
    }

    private function getMockContainer($mockOrganization = null)
    {
        $mockRepoOrganization = $this->getMockBuilder(OrganizationRepositoryInterface::class)
            ->getMock();

        $mockRepoOrganization->expects($this->any())
            ->method('findOneByCode')
            ->with(self::ORGANIZATION_CODE)
            ->willReturn($mockOrganization);

        $mockRepo = $this->getMockBuilder(TenantRepositoryInterface::class)
            ->getMock();

        $mockRepo->expects($this->any())
            ->method('findAll')
            ->willReturn($mockOrganization->getTenants());

        $mockContainer = $this->getMockBuilder(ContainerInterface::class)
            ->getMock();

        $mockContainer->expects($this->any())
            ->method('get')
            ->will($this->returnValueMap([
                ['swp.repository.tenant', 1, $mockRepo],
                ['swp.repository.organization', 1, $mockRepoOrganization],
            ]));

        return $mockContainer;
    }

    /**
     * @param $input
     *
     * @return resource
     */
    protected function getInputStream($input)
    {
        $stream = fopen('php://memory', 'r+', false);
        fwrite($stream, $input);
        rewind($stream);

        return $stream;
    }
}


Push — master ( e32e27...6718d9 )

testListWithTenantsAndOrganizationOption() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

SuperdeskWebPublisher / SWPMultiTenancyBundle

Push — master ( e32e27...6718d9 )

testListWithTenantsAndOrganizationOption() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like