[SF][SF2]Testowanie serwisu Opcje

[Matrix12]

Chce napisać dla testów rejestracje użytkownika. Ma być to osobny serwis, rejestracja wygląda w następujący sposób:

1. Kontroler przekazuje dane z formularza do serwisu register do metordy registerUser($data)
2. Metoda waliduje te dane, generuje token dla tego użytkownika, hashuje hasło, przesyła maila do użytkownika w celu potwierdzenia rejestracji.

Problem w tym, że nie wiem jak napisać speca do tej metody.. Nie wiem jak wyodrębnić poszczególne elementy do testowania..
Bo nie powinno być to przecież $this->registerUser($form->getData())->shouldReturn(true);





Będę wdzięczny za wszelkie wskazówki jak do tego podejść.

[destroyerr]

Napiszę, że nie powinieneś mieć dla tej metody jednego testu tylko wiele. Powinieneś mieć pewność, że są wykorzystywane zależności, które muszą być wywołane. Sprawdź czy metoda rzuca wyjątkami wtedy kiedy trzeba. Sprawdź wartość zwracaną (o ile coś zwracasz). Nie wiem co robi metoda registerUser więc nie mogę napisać precyzyjnie.

[Matrix12]

Masz rację tylko mam problem z testem integracyjnym.. że końcowy test przetestuje mi cały proces.. w niej ma być tylko $this->registerUser($dane)->shouldReturn(true)? Wiem że każdy element mam testować osobno ale potem tak integracyjnie jak to przetestować.

[kpt_lucek]

$this->registerUser($form->getData())->shouldReturn(true);

To mi raczej wygląda na test jednostkowy...

Mając mockowane $form->getData() musisz dodać expects, returnValue etc
Testując metodę swojego obiektu robisz self::assert{typAssertu} etc. poczytaj

[Matrix12]

[PHP] pobierz, plaintext 
 
Czyli test ma wyglądać tak :
 
$this->registerUser(Argument::type('Symfony\Component\Http\Form))->willReturn(true)
[PHP] pobierz, plaintext 

[destroyerr]

Skąd mamy wiedzieć jak ma wyglądać test, skoro nawet nie wiemy co Twój kod ma robić? Druga sprawa, ustalmy o jakim teście piszesz.
Jeśli chcesz pomocy to przedstaw publiczny interfejs obiektów które chcesz przetestować razem z zależnościami. Następnie pokaż spójne testy (jedna linijka mi nic nie mówi).

[Matrix12]

Mój serwis ma zapisywać użytkownika do bazy, w między czasie ma sprawdzić czy taki użytkownik już istnieje i ewentualnie go zapisać wysłać maila z potwierdzeniem rejestracji. Chce to przetestować.



Mój kod wygląda tak :

[PHP] pobierz, plaintext 
 
<?php
 
namespace Acme\UserBundle\Libs;
 
 
use Acme\UserBundle\Entity\User;
use Doctrine\Bundle\DoctrineBundle\Registry;
use Doctrine\ORM\EntityManager;
 
 
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('CustomUserBundle:User')->findOneBy(array(
            'username'=>$argument1
        ));
 
        if(!$user){
            return false;
        }else{
            return true;
        }
    }
 
 
 
 
    public function register($argument1)
    {
 
        $data =  $argument1->getData();
        $user = new User;
        try{
           $user->setName($data->getUsername());
                     $user->setEmail($data->getEmail());
           /* i inne settery */ 
            $this->doctrine->getManager()->persist($user);
            $this->doctrine->getManager()->flush();
        }catch(\Exception $e){
            echo $e->getMessage();
        }
 
 
 
 
 
 
    }
}
[PHP] pobierz, plaintext 

spec do niego :

[PHP] pobierz, plaintext 
<?php
 
namespace spec\Acme\UserBundle\Libs;
 
use Acme\UserBundle\Entity\User;
use Doctrine\Common\Persistence\ObjectRepository;
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\Form\Form;
use Symfony\Component\Form\FormConfigBuilder;
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_when_user_exist_in_system(Registry $doctrine, ObjectRepository $repository, User $user)
    {
 
        $doctrine->getManager()->willReturn($doctrine);
        $doctrine->getRepository('CustomUserBundle:User')->willReturn($repository);
 
        $repository->findOneBy(array('username'=>'user1'))->willReturn($user);
 
        $this->checkUser('user1')->shouldReturn(true);
    }
 
 
    function it_check_user_exist_in_system_when_user_not_exist_in_system(Registry $doctrine, ObjectRepository $repository, User $user)
    {
        $doctrine->getManager()->willReturn($doctrine);
        $doctrine->getRepository('CustomUserBundle:User')->willReturn($repository);
 
        $repository->findOneBy(array('username'=>'user123312312'))->willReturn(null);
 
        $this->checkUser('user123312312')->shouldReturn(false);
    }
 
    function it_register_user(Registry $doctrine, ObjectRepository $repository,EntityManager $entityManager , User $user, FormConfigBuilder $form)
    {
        $doctrine->getManager()->willReturn($doctrine);
//        $doctrine->getRepository('CustomUserBundle:User')->willReturn($repository);
 
 
        $siteClass =  Argument::exact('Acme\UserBundle\Entity\User')->getValue();
        $site = new $siteClass;
 
        $doctrine->getManager()->persist($site)->shouldBeCalled();
        $doctrine->getManager()->flush()->shouldBeCalled();
//        $entityManager->persist($site)->shouldBeCalled();
//        $entityManager->flush()->shouldBeCalled();
 
        $this->register($form)->shouldReturn(true);
    }
}
 
 
[PHP] pobierz, plaintext 

Problem miałem z sprawdzeniem użytkownika, ale poradziłem sobie teraz mam problem z perystencją. Chce spradzić czy użytkownik się zapiszę do bazy. Spec zwraca mi niby 100 % ale wywala też bład
Fatal error: Call to undefined method Prophecy\Prophecy\MethodProphecy::persist() in /var/www/example/spec/Acme/UserBundle/Libs/RegisterTestSpec.php on line 70 i nie wiem czy to do końca dobry test..
Czy powinienem używac setterów encji ? czy jest jakaś inna opcja obejścia tego ?