Bludata\Lumen\Tests\Repositories\BaseRepositoryTest

Complexity

Total Complexity

8

Size/Duplication

Total Lines	88
Duplicated Lines	35.23 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
	getRepository()	0	1	?
	getMockObject()	0	1	?
A	getFlushedMockObject()	0	10	1
A	getMockArray()	0	4	1
A	getFlushedMockArray()	0	4	1
A	testFindBy()	0	11	1
A	testFindOneBy()	11	11	1
A	testFind()	11	11	1
A	testRemove()	0	11	1
A	testFindAll()	9	9	1

<?php

namespace Bludata\Lumen\Tests\Repositories;

use Bludata\Lumen\Tests\BaseTest;

abstract class BaseRepositoryTest extends BaseTest
{
    abstract public function getRepository();

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

    abstract public function getMockObject(array $except = []);

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

    public function getFlushedMockObject(array $except = [])

The return type could not be reliably inferred; please add a @return annotation.

    {
        $entity = $this->getMockObject($except);

        $this->getRepository()
             ->save($entity)
             ->flush($entity);

        return $entity;
    }

    public function getMockArray(array $except = [])

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->getMockObject($except)->toArray();
    }

    public function getFlushedMockArray(array $except = [])

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->getFlushedMockObject($except)->toArray();
    }

    public function testFindAll()

This method seems to be duplicated in your project.

    {
        $entity = $this->getFlushedMockObject();

$entity is not used, you could remove the assignment.

        $findAll = $this->getRepository()->findAll()->getResult();

        $this->assertGreaterThan(0, count($findAll));
        $this->assertInstanceOf($this->getRepository()->getEntityName(), $findAll[0]);

This line exceeds maximum limit of 80 characters; contains 86 characters

    }

    public function testFindBy()
    {
        $entity = $this->getFlushedMockObject();

        $repository = $this->getRepository();

        $findBy = $repository->findBy(['id' => $entity->getId()]);

        $this->assertGreaterThan(0, count($findBy));
        $this->assertInstanceOf($repository->getEntityName(), $findBy[0]);
    }

    public function testFindOneBy()

This method seems to be duplicated in your project.

    {
        $entity = $this->getFlushedMockObject();

        $repository = $this->getRepository();

        $findOneBy = $repository->findOneBy(['id' => $entity->getId()]);

        $this->assertInstanceOf($repository->getEntityName(), $findOneBy);
        $this->assertEquals($entity->getId(), $findOneBy->getId());
    }

    public function testFind()

This method seems to be duplicated in your project.

    {
        $entity = $this->getFlushedMockObject();

        $repository = $this->getRepository();

        $find = $repository->find($entity->getId());

        $this->assertInstanceOf($repository->getEntityName(), $find);
        $this->assertEquals($entity->getId(), $find->getId());
    }

    /**
     * @expectedException Symfony\Component\HttpKernel\Exception\NotFoundHttpException

This line exceeds maximum limit of 80 characters; contains 86 characters

     */
    public function testRemove()
    {
        $repository = $this->getRepository();

        $entity = $this->getFlushedMockObject();

        $repository->remove($entity)
                   ->flush($entity);

        $repository->find($entity->getId());
    }
}