Witam,



stworzyłem sobie serwis który chce aby rejestrował użytkownika. Chciałem przetestować metodę która weryfikuje czy użytkownik jest już w systemie, zwraca boola . Zrobiłem var_dump poprzez przeglądarkę i jak użytkownik istnieje zwraca mi jego obiekt a jeśli nie zwraca nulla.

Kod serwisu :

[PHP] pobierz, plaintext 
 
<?php
 
namespace Acme\UserBundle\Libs;
 
 
use Doctrine\Bundle\DoctrineBundle\Registry;
use Doctrine\ORM\EntityManager;
use Symfony\Component\DependencyInjection\Container;
 
class RegisterTest
{
 
    public  $doctrine;
    private $container;
 
    /**
     * @param $data
     */
    public function __construct(Registry $doctrine)
    {
        $this->doctrine = $doctrine;
    }
 
    public function checkUserExist($name)
    {
        $this->name = $name;
 
        $user = $this->doctrine->getManager()->getRepository('AcmeUserBundle:User')->findOneBy(array(
            'username'=>'m4ver'
        ));
 
 
        if(is_object($user) && !empty($user)){
            return  true;
        }else{
            return  false;
        }
 
    }
 
}
 
[PHP] pobierz, plaintext 

W przeglądarce var_dump zwraca mi true a spec false.. mimo że testuje tego samego użytkownika.. wiecie może dlaczego ?

Możesz się zdecydować co się tak naprawdę dzieje a co ma się dziać?
Jeżeli nie pokażesz testu to możemy sobie długo gdybać.

[PHP] pobierz, plaintext 
<?php
 
namespace spec\Acme\UserBundle\Libs;
 
use PhpSpec\ObjectBehavior;
use Prophecy\Argument;
use Symfony\Component\DependencyInjection\Container;
use Doctrine\Bundle\DoctrineBundle\Registry;
use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\EntityManager;
use Symfony\Component\HttpFoundation\Request;
 
use Symfony\Component\Routing\Router;
 
class RegisterTestSpec extends ObjectBehavior
{
 
    function let(Container $container, Registry $doctrine, EntityRepository $repository,EntityManager $entityManager, Request $request)
    {
        $container->get('doctrine')->willReturn($doctrine);
        $doctrine->getManager()->willReturn($entityManager);
        $entityManager->getRepository(Argument::any())->willReturn($repository);
 
        $this->beConstructedWith($doctrine);
    }
    function it_is_initializable()
    {
        $this->shouldHaveType('Acme\UserBundle\Libs\RegisterTest');
    }
 
    function it_check_user_exist_in_system()
    {
 
        $this->checkUser('user1')->shouldReturn(true);
    }
 
}
[PHP] pobierz, plaintext 

kod

[PHP] pobierz, plaintext 
<?php
 
namespace Acme\UserBundle\Libs;
 
 
use Doctrine\Bundle\DoctrineBundle\Registry;
 
 
class RegisterTest
{
 
    public  $doctrine;
 
    public function __construct(Registry $doctrine)
    {
        $this->doctrine = $doctrine;
    }
 
 
    public function checkUser($argument1)
    {
        try{
            $em =  $this->doctrine->getManager();
        }catch(\Exception $e){
            return $e->getMessage();
        }
 
 
        $user = $em->getRepository('AcmeUserBundle:User')->findOneBy(array(
            'username'=>$argument1
        ));
 
 
        if(!$user){
            return false;
        }else{
            return true;
        }
    }
 
 
 
}
 
[PHP] pobierz, plaintext 

jezeli skasuje tego ifa w kodzie i dam return $user to w przeglądarce daje obiekt, a jeżeli zostawie ifa daje mi boola true. Problem w tym że spec dalej false a jezeli w kodzie dam return $user daje null..

W RegisterTestSpec nie ustalasz co ma zwracać metoda findOneBy. Inna sprawa, że taki łańcuch zależności nie jest wskazany. Być może wystarczy dostarczać samo repozytorium, chyba, że w usłudze chcesz skorzystać z transakcji. Moim zdaniem powinieneś dostarczyć repozytorium i manedżera jako zależności.

Poradziłem sobie. Mam jednak takie pytanie z punktu widzenia "dobrych praktyk" Chce zapisać użytkownika do bazy.. i zastanawiam się czy w serwisie to zrobić czy w kontrolerze. A np. w serwisie nadać token do potwierdzenia takiego użytkownika hasować hasło wysłać potwierdzającego maila itd. a persist flush w controlerze.

Wszystko to robisz w usłudze. Włącznie z flushem.