Symplify\DefaultAutowire\Tests\CompleteTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	4
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	7	1
A	testSomeServiceAutowire()	0	12	1

<?php

namespace Symplify\DefaultAutowire\Tests;

use Doctrine\ORM\EntityManagerInterface;
use PHPUnit_Framework_TestCase;
use Symfony\Component\DependencyInjection\Container;
use Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher;
use Symfony\Component\EventDispatcher\EventDispatcher;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symplify\DefaultAutowire\Tests\Resources\Repository\SomeRepository;
use Symplify\DefaultAutowire\Tests\Source\SomeAutowiredService;
use Symplify\DefaultAutowire\Tests\Source\SomeService;

final class CompleteTest extends PHPUnit_Framework_TestCase
{
    /**
     * @var Container
     */
    private $container;

    protected function setUp()
    {
        $kernel = new AppKernel('test_env', true);
        $kernel->boot();

        $this->container = $kernel->getContainer();

It seems like $kernel->getContainer() can also be of type object<Symfony\Component\DependencyInjection\ContainerInterface>. However, the property $container is declared as type object<Symfony\Component\DependencyInjection\Container>. Maybe add an additional type check?

    }

    public function testSomeServiceAutowire()
    {
        /** @var SomeAutowiredService $someAutowiredService */
        $someAutowiredService = $this->container->get('some_autowired_service');

        $this->assertInstanceOf(SomeAutowiredService::class, $someAutowiredService);
        $this->assertInstanceOf(SomeService::class, $someAutowiredService->getSomeService());

        $this->assertInstanceOf(EventDispatcherInterface::class, $someAutowiredService->getEventDispatcher());
        $this->assertInstanceOf(EventDispatcher::class, $someAutowiredService->getEventDispatcher());
        $this->assertNotInstanceOf(TraceableEventDispatcher::class, $someAutowiredService->getEventDispatcher());
    }

//    public function testRepositoryAutowire()

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

//    {
//        /** @var SomeRepository $someRepository */
//        $someRepository = $this->container->get('some_repository');
//        $this->assertInstanceOf(SomeRepository::class, $someRepository);
//        $this->assertInstanceOf(EntityManagerInterface::class, $someRepository->getEntityManager());
//    }
}