TranslatableRepositoryTest::testCreateQueryBuilderForCollection() - Code Metrics - Inspection of "Move more code from bundles to components" - php-lug/lug - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 74c3c7...591b88 )

by Eric
created 2016-02-27 11:48 UTC
testCreateQueryBuilderForCollection() B

↳ Parent: TranslatableRepositoryTest
Complexity

Conditions	1
Paths	1
Size

Total Lines	100
Code Lines	80
Duplication

Lines	0
Ratio	0 %
Importance

Changes	1
Bugs	0	Features	0
Metric	Value
c	1
b	0
f	0
dl	0
loc	100
rs	8.2857
cc	1
eloc	80
nc	1
nop	0
How to fix Long Method

<?php

/*
 * This file is part of the Lug package.
 *
 * (c) Eric GELOEN <[email protected]>
 *
 * For the full copyright and license information, please read the LICENSE
 * file that was distributed with this source code.
 */

namespace Lug\Component\Translation\Tests\Repository\Doctrine\ORM;

use Doctrine\ORM\EntityManager;
use Doctrine\ORM\Mapping\ClassMetadata;
use Doctrine\ORM\Query;
use Doctrine\ORM\Query\Expr;
use Doctrine\ORM\Query\Expr\Join;
use Doctrine\ORM\QueryBuilder;
use Lug\Component\Resource\Model\ResourceInterface;
use Lug\Component\Translation\Context\LocaleContextInterface;
use Lug\Component\Translation\Model\TranslatableInterface;
use Lug\Component\Translation\Model\TranslatableTrait;
use Lug\Component\Translation\Model\TranslationInterface;
use Lug\Component\Translation\Model\TranslationTrait;
use Lug\Component\Translation\Repository\Doctrine\ORM\TranslatableRepository;

/**
 * @author GeLo <[email protected]>
 */
class TranslatableRepositoryTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var TranslatableRepository
     */
    private $translatableRepository;

    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|EntityManager
     */
    private $entityManager;

    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|ClassMetadata
     */
    private $classMetadata;

    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|ResourceInterface
     */
    private $resource;

    /**
     * @var string
     */
    private $class;

    /**
     * @var \PHPUnit_Framework_MockObject_MockObject|LocaleContextInterface
     */
    private $localeContext;

    /**
     * {@inheritdoc}
     */
    protected function setUp()
    {
        $this->entityManager = $this->createEntityManagerMock();
        $this->classMetadata = $this->createClassMetadataMock();
        $this->resource = $this->createResourceMock();
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->localeContext = $this->createLocaleContextMock();
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->class = $this->classMetadata->name = TranslatableTest::class;

        $this->translatableRepository = new TranslatableRepository(
            $this->entityManager,
            $this->classMetadata,
            $this->resource,
            $this->localeContext
        );
    }

    public function testCreateQueryBuilder()
    {
        $this->classMetadata
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([]));

        $this->classMetadata
            ->expects($this->once())
            ->method('getAssociationMapping')
            ->with($this->identicalTo('translations'))
            ->will($this->returnValue(['targetEntity' => $translationClass = TranslationTest::class]));

        $translationClassMetadata = $this->createClassMetadataMock();
        $translationClassMetadata
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([]));

        $this->entityManager
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getClassMetadata')
            ->with($this->identicalTo($translationClass))
            ->will($this->returnValue($translationClassMetadata));

        $this->resource
            ->expects($this->once())
            ->method('getName')
            ->will($this->returnValue($name = 'resource'));

        $this->entityManager
            ->expects($this->once())
            ->method('createQueryBuilder')
            ->will($this->returnValue($queryBuilder = $this->createQueryBuilderMock()));

        $queryBuilder
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('select')
            ->with($this->identicalTo($name))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('from')
            ->with(
                $this->identicalTo($this->class),
                $this->identicalTo($name),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('addSelect')
            ->with($this->identicalTo('resource_translation'))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->exactly(2))
            ->method('getRootAliases')
            ->will($this->returnValue([$name]));

        $queryBuilder
            ->expects($this->once())
            ->method('leftJoin')
            ->with(
                $this->identicalTo($name.'.translations'),
                $this->identicalTo('resource_translation'),
                $this->isNull(),
                $this->isNull(),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $this->assertSame($queryBuilder, $this->translatableRepository->createQueryBuilder());
    }

    public function testCreateQueryBuilderForCollection()
    {
        $this->classMetadata
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([]));

        $this->classMetadata
            ->expects($this->once())
            ->method('getAssociationMapping')
            ->with($this->identicalTo('translations'))
            ->will($this->returnValue(['targetEntity' => $translationClass = TranslationTest::class]));

        $translationClassMetadata = $this->createClassMetadataMock();
        $translationClassMetadata
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([]));

        $this->entityManager
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getClassMetadata')
            ->with($this->identicalTo($translationClass))
            ->will($this->returnValue($translationClassMetadata));

        $this->resource
            ->expects($this->once())
            ->method('getName')
            ->will($this->returnValue($name = 'resource'));

        $this->entityManager
            ->expects($this->once())
            ->method('createQueryBuilder')
            ->will($this->returnValue($queryBuilder = $this->createQueryBuilderMock()));

        $queryBuilder
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('select')
            ->with($this->identicalTo($name))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('from')
            ->with(
                $this->identicalTo($this->class),
                $this->identicalTo($name),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('addSelect')
            ->with($this->identicalTo('resource_translation'))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->exactly(3))
            ->method('getRootAliases')
            ->will($this->returnValue([$name]));

        $this->localeContext
            ->expects($this->once())
            ->method('getLocales')
            ->will($this->returnValue($locales = ['fr']));

        $this->localeContext
            ->expects($this->once())
            ->method('getFallbackLocale')
            ->will($this->returnValue($fallbackLocale = 'en'));

        $queryBuilder
            ->expects($this->once())
            ->method('expr')
            ->will($this->returnValue($expr = $this->createExprMock()));

        $expr
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('in')
            ->with(
                $this->identicalTo('resource.locale'),
                $this->identicalTo(array_merge($locales, [$fallbackLocale]))
            )
            ->will($this->returnValue($expression = 'expression'));

        $queryBuilder
            ->expects($this->once())
            ->method('innerJoin')
            ->with(
                $this->identicalTo($name.'.translations'),
                $this->identicalTo('resource_translation'),
                $this->identicalTo(Join::WITH),
                $this->identicalTo($expression),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $this->assertSame($queryBuilder, $this->translatableRepository->createQueryBuilderForCollection());
    }

    public function testFindOneBy()
    {
        $this->classMetadata
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([
                $translatableField = 'translatable_field',
                $bothField = 'both_field',
            ]));

        $this->classMetadata
            ->expects($this->once())
            ->method('getAssociationMapping')
            ->with($this->identicalTo('translations'))
            ->will($this->returnValue(['targetEntity' => $translationClass = TranslationTest::class]));

        $translationClassMetadata = $this->createClassMetadataMock();
        $translationClassMetadata
            ->expects($this->once())
            ->method('getFieldNames')
            ->will($this->returnValue([
                $translationField = 'translation_field',
                $bothField,
            ]));

        $this->entityManager
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getClassMetadata')
            ->with($this->identicalTo($translationClass))
            ->will($this->returnValue($translationClassMetadata));

        $this->resource
            ->expects($this->once())
            ->method('getName')
            ->will($this->returnValue($name = 'resource'));

        $this->entityManager
            ->expects($this->once())
            ->method('createQueryBuilder')
            ->will($this->returnValue($queryBuilder = $this->createQueryBuilderMock()));

        $queryBuilder
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('select')
            ->with($this->identicalTo($name))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('from')
            ->with(
                $this->identicalTo($this->class),
                $this->identicalTo($name),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('addSelect')
            ->with($this->identicalTo('resource_translation'))
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->exactly(5))
            ->method('getRootAliases')
            ->will($this->returnValue([$name]));

        $queryBuilder
            ->expects($this->once())
            ->method('leftJoin')
            ->with(
                $this->identicalTo($name.'.translations'),
                $this->identicalTo($translation = 'resource_translation'),
                $this->isNull(),
                $this->isNull(),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->exactly(3))
            ->method('expr')
            ->will($this->returnValue($expr = $this->createExprMock()));

        $expr
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->at(0))
            ->method('eq')
            ->with(
                $this->identicalTo($name.'.'.$translatableField),
                $this->matchesRegularExpression('/:'.$name.'_'.$translatableField.'_[a-z0-9]{22}/')
            )
            ->will($this->returnValue($translatableWhere = 'translatable_where'));

        $expr
            ->expects($this->at(1))
            ->method('eq')
            ->with(
                $this->identicalTo($translation.'.'.$translationField),
                $this->matchesRegularExpression('/:'.$translation.'_'.$translationField.'_[a-z0-9]{22}/')
            )
            ->will($this->returnValue($translationWhere = 'translation_where'));

        $expr
            ->expects($this->at(2))
            ->method('eq')
            ->with(
                $this->identicalTo($name.'.'.$bothField),
                $this->matchesRegularExpression('/:'.$name.'_'.$bothField.'_[a-z0-9]{22}/')
            )
            ->will($this->returnValue($bothWhere = 'both_where'));

        $queryBuilder
            ->expects($this->exactly(3))
            ->method('andWhere')
            ->will($this->returnValueMap([
                [$translatableWhere, $queryBuilder],
                [$translationWhere, $queryBuilder],
                [$bothWhere, $queryBuilder],
            ]));

        $queryBuilder
            ->expects($this->at(9))
            ->method('setParameter')
            ->with(
                $this->matchesRegularExpression('/'.$name.'_'.$translatableField.'_[a-z0-9]{22}/'),
                $this->identicalTo($translatableValue = 'translatable_value'),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->at(13))
            ->method('setParameter')
            ->with(
                $this->matchesRegularExpression('/'.$translation.'_'.$translationField.'_[a-z0-9]{22}/'),
                $this->identicalTo($translationValue = 'translation_value'),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->at(17))
            ->method('setParameter')
            ->with(
                $this->matchesRegularExpression('/'.$name.'_'.$bothField.'_[a-z0-9]{22}/'),
                $this->identicalTo($bothValue = 'both_value'),
                $this->isNull()
            )
            ->will($this->returnSelf());

        $queryBuilder
            ->expects($this->once())
            ->method('getQuery')
            ->will($this->returnValue($query = $this->createQueryMock()));

        $query
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            ->expects($this->once())
            ->method('getOneOrNullResult')
            ->will($this->returnValue($result = 'result'));

        $this->assertSame($result, $this->translatableRepository->findOneBy([
            $translatableField => $translatableValue,
            $translationField  => $translationValue,
            $bothField         => $bothValue,
        ]));
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|EntityManager
     */
    private function createEntityManagerMock()
    {
        return $this->getMockBuilder(EntityManager::class)
            ->disableOriginalConstructor()
            ->getMock();
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|ClassMetadata
     */
    private function createClassMetadataMock()
    {
        return $this->getMockBuilder(ClassMetadata::class)
            ->disableOriginalConstructor()
            ->getMock();
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|ResourceInterface
     */
    private function createResourceMock()
    {
        return $this->getMock(ResourceInterface::class);
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|LocaleContextInterface
     */
    private function createLocaleContextMock()
    {
        return $this->getMock(LocaleContextInterface::class);
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|QueryBuilder
     */
    private function createQueryBuilderMock()
    {
        return $this->getMockBuilder(QueryBuilder::class)
            ->disableOriginalConstructor()
            ->getMock();
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|Expr
     */
    private function createExprMock()
    {
        return $this->getMock(Expr::class);
    }

    /**
     * @return \PHPUnit_Framework_MockObject_MockObject|Query
     */
    private function createQueryMock()
    {
        return $this->getMock(\stdClass::class, ['getOneOrNullResult']);
    }
}

/**
 * @author GeLo <[email protected]>
 */
class TranslatableTest implements TranslatableInterface

{
    use TranslatableTrait;
}

/**
 * @author GeLo <[email protected]>
 */
class TranslationTest implements TranslationInterface

{
    use TranslationTrait;
}

Push — master ( 74c3c7...591b88 )

testCreateQueryBuilderForCollection() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

php-lug / lug

Push — master ( 74c3c7...591b88 )

testCreateQueryBuilderForCollection() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like
