Zenstruck\Porpaginas\Tests\Doctrine\Repository\ORMMatchableRepositoryTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	can_match_one_with_callable()	0	12	1
A	createRepository()	0	3	1
A	can_match_with_callable()	0	13	1

<?php

namespace Zenstruck\Porpaginas\Tests\Doctrine\Repository;

use Doctrine\ORM\QueryBuilder;
use Zenstruck\Porpaginas\Tests\Doctrine\Fixtures\ORMEntity;
use Zenstruck\Porpaginas\Tests\Doctrine\Fixtures\ORMEntityRepository;

/**
 * @author Kevin Bond <kevinbond@gmail.com>
 */
final class ORMMatchableRepositoryTest extends MatchableRepositoryTest
{
    /**
     * @test
     */
    public function can_match_one_with_callable(): void
    {
        $this->persistEntities(4);

        $object = $this->createRepository()->matchOne(function(QueryBuilder $qb, string $alias) {
            $qb->where("{$alias}.value = :value")
                ->setParameter('value', 'value 2')
            ;
        });

        $this->assertInstanceOf(ORMEntity::class, $object);
        $this->assertSame('value 2', $object->value);
    }

    /**
     * @test
     */
    public function can_match_with_callable(): void
    {
        $this->persistEntities(4);

        $objects = $this->createRepository()->match(function(QueryBuilder $qb, string $alias) {
            $qb->where("{$alias}.id > :value")
                ->setParameter('value', 2)
            ;
        });

        $this->assertCount(2, $objects);
        $this->assertSame('value 3', \iterator_to_array($objects)[0]->value);
        $this->assertSame('value 4', \iterator_to_array($objects)[1]->value);
    }

    protected function createRepository(): ORMEntityRepository
    {
        return new ORMEntityRepository($this->em);

It seems like $this->em can also be of type null; however, parameter $em of Zenstruck\Porpaginas\Tests\Doctrine\Fixtures\ORMEntityRepository::__construct() does only seem to accept Doctrine\ORM\EntityManagerInterface, maybe add an additional type check?

    }
}