Adoptez le TDD sur vos projets Symfony2 existants

ADOPTEZ LE TDD SUR VOS PROJETS SYMFONY2 EXISTANTS

Créer, refactoriser, corriger du code…

Telle est la mission des développeurs

On a tous peur d’introduire des bugs

Si seulement il y avait eu de bons tests !

Adoptez le TDD

Benjamin Grandfond Technical team manager  benja-M-1  benjaM1 7

LE TDD 8 Test-driven Development

"Clean code that works" Ron Je!ries un des trois fondateurs
de l’ «Extreme Programming» (XP) 9

RED REFACTOR 1. écrire un test simple qui échoue 3.
éliminer la duplication GREEN 2. écrire le minimum de code pour faire passer le test Mantra 10

11 + moins de debug + code découplé + plus
de con!ance - prend du temps (au début) - plus de code (?) Avantages Inconvénients

12 • patience, courage, persévérance • intégration continue • exécution
des tests rapide • qualité des tests (lisibilité, conventions...) Pré requis pour le succès

Suite de Fibonacci class FibonacciTest extends \PHPUnit_Framework_TestCase { public function
testFibonacci() { $fibonacci = new Fibonacci(); $this->assertEquals(0, $fibonacci->fib(0)); } } RED 13

class Fibonacci { public function fib($value) { return 0; }
} Fake it! Suite de Fibonacci GREEN 14

class FibonacciTest extends \PHPUnit_Framework_TestCase { public function testFibonacci() { $fibonacci
= new Fibonacci(); $this->assertEquals(0, $fibonacci->fib(0)); $this->assertEquals(1, $fibonacci->fib(1)); } } Suite de Fibonacci RED 15 class FibonacciTest extends \PHPUnit_Framework_TestCase { public function testFibonacci() { $fibonacci = new Fibonacci(); $this->assertEquals(0, $fibonacci->fib(0)); $this->assertEquals(1, $fibonacci->fib(1)); } } duplication

class Fibonacci { public function fib($value) { if ($value ===
0) { return 0; } return 1; } } Suite de Fibonacci GREEN 16

class FibonacciTest extends \PHPUnit_Framework_TestCase { /** * @dataProvider getValues */
public function testFibonacci($value, $result) { $fibonacci = new Fibonacci(); $this->assertEquals($result, $fibonacci->fib($value)); } public function getValues() { return array( array(0, 0), array(1, 1), ); } } Suite de Fibonacci REFACTOR 17

public function getValues() { return array( array(0, 0), array(1, 1),
array(2, 1), ); } Suite de Fibonacci GREEN 18

public function getValues() { return array( array(0, 0), array(1, 1),
array(2, 1), array(3, 2), ); } RED Triangulation Suite de Fibonacci 19

0) { return 0; } if ($value <= 2) { return 1; } return 2; } } GREEN Suite de Fibonacci 20

0) { return 0; } if ($value <= 2) { return 1; } return 1 + 1; } } Suite de Fibonacci REFACTOR 21

0) { return 0; } if ($value <= 2) { return 1; } return $this->fib($value - 1) + $this->fib($value - 2); } } REFACTOR Suite de Fibonacci 22

METTRE EN PLACE LE TDD DANS VOS PROJETS SYMFONY2 23

Créer une nouvelle page 24

• récupérer les experts dans la base de données •
créer un template qui a!che la liste • créer un contrôleur • ajouter une nouvelle route Ce qu’on veut : Créer une nouvelle page 25

1. code retour de la page = 200 2. la
réponse contient une liste 3. le nombre de li = le nombre d’experts dans la base Ce qu’on va tester : test list Créer une nouvelle page 26

Créer une nouvelle page RED 27 <?php namespace Theodo\Bundle\ExpertBundle\Tests\Controller; use
Symfony\Bundle\FrameworkBundle\Test\WebTestCase; /** * ExpertControllerTest * * @author Benjamin Grandfond <[email protected]> * @group functional */ class ExpertControllerTest extends WebTestCase { public function testShouldDisplayTheExpertsList() { $client = self::createClient(); $crawler = $client->request('GET', '/experts'); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertGreaterThan(1, $crawler->filter('li')->count()); } }

Créer une nouvelle page RED 27 <?php namespace Theodo\Bundle\ExpertBundle\Tests\Controller; use
Symfony\Bundle\FrameworkBundle\Test\WebTestCase; /** * ExpertControllerTest * * @author Benjamin Grandfond <[email protected]> * @group functional */ class ExpertControllerTest extends WebTestCase { public function testShouldDisplayTheExpertsList() { $client = self::createClient(); $crawler = $client->request('GET', '/experts'); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertGreaterThan(1, $crawler->filter('li')->count()); } } TIP

Créer une nouvelle page 28 TIP

RED Créer une nouvelle page 29

1. créer la route 2. créer le contrôleur et l’action
3. créer un template avec une liste et deux éléments Il faut que le test passe avec un minimum de code Créer une nouvelle page 30

RED namespace Theodo\Bundle\ExpertBundle\Controller; use Symfony\Bundle\FrameworkBundle\Controller\Controller; use Sensio\Bundle\FrameworkExtraBundle\Configuration as Extra; class
ExpertController extends Controller { /** * @Extra\Route("/experts") * @Extra\Template() */ public function listAction() { return array(); } } {% extends '::base.html.twig' %} {% block body %} <ul> <li>Marek</li> <li>Pierre-Henri</li> </ul> {% endblock %} Créer une nouvelle page 31

GREEN Créer une nouvelle page 32

{% extends '::base.html.twig' %} {% block body %} <ul> {%
for expert in experts %} <li>{{ expert }}</li> {% endfor %} </ul> {% endblock %} On fait en sorte que les experts soient dynamiques public function listAction() { $experts = array( 'Marek', 'Pierre-Henri', ); return array( 'experts' => $experts ); } REFACTOR Créer une nouvelle page 33

REFACTOR Créer une nouvelle page 34

✔ code retour de la page = 200 ✔ la
réponse contient une liste ✘ le nombre de li = le nombre d’experts dans la base Ce qu’on a testé : Créer une nouvelle page 35

Utilisation de !xtures 36

• DoctrineFixturesBundle https://github.com/doctrine/DoctrineFixturesBundle • Faker https://github.com/fzaninotto/Faker • Alice https://github.com/nelmio/alice Les
outils à utiliser 37

REFACTOR # app/config/config_test.yml doctrine: dbal: driver: pdo_sqlite host: localhost port:
~ dbname: symfony user: root password: ~ path: %kernel.root_dir%/cache/database.db Créer une nouvelle page 38

/** * Generates the schema to use on the test
environment. * * @throws \Doctrine\DBAL\Schema\SchemaException */ protected static function generateSchema() { if (null === static::$kernel) { static::$kernel = static::createKernel(); } $em = static::$kernel->getContainer()->get('doctrine.orm.entity_manager'); // Get the metadata of the application to create the schema. $metadata = $em->getMetadataFactory()->getAllMetadata(); if (!empty($metadata)) { $tool = new \Doctrine\ORM\Tools\SchemaTool($em); $tool->dropDatabase(); $tool->createSchema($metadata); } else { throw new \Doctrine\DBAL\Schema\SchemaException( 'No Metadata Classes to process.' ); } } REFACTOR Créer une nouvelle page 39

public function testShouldDisplayTheExpertsList() { $client = self::createClient(); self::generateSchema(); $loader =
new \Nelmio\Alice\Loader\Base(); $objects = $loader->load($this->getExperts()); $persister = new \Nelmio\Alice\ORM\Doctrine( static::$kernel->getContainer()->get('doctrine.orm.entity_manager') ); $persister->persist($objects); $crawler = $client->request('GET', '/Experts'); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertGreaterThan(1, $crawler->filter('li')->count()); } private function getExperts() { return array( 'Theodo\Bundle\ExpertBundle\Entity\Expert' => array( 'expert{1..5}' => array( 'firstName' => '<firstName()>', 'lastName' => '<lastName()>', 'username' => '<userName()>' ), ) ); } Créer une nouvelle page REFACTOR 40

REFACTOR Créer une nouvelle page private function getExperts() { return
array( 'Theodo\Bundle\ExpertBundle\Entity\Expert' => array( 'expert{1..5}' => array( 'firstName' => '<firstName()>', 'lastName' => '<lastName()>', 'username' => '<userName()>' ), ) ); }

array( 'Theodo\Bundle\ExpertBundle\Entity\Expert' => array( 'expert{1..5}' => array( 'firstName' => '<firstName()>', 'lastName' => '<lastName()>', 'username' => '<userName()>' ), ) ); } créé 5 experts

array( 'Theodo\Bundle\ExpertBundle\Entity\Expert' => array( 'expert{1..5}' => array( 'firstName' => '<firstName()>', 'lastName' => '<lastName()>', 'username' => '<userName()>' ), ) ); } formateurs Faker créé 5 experts

REFACTOR public function testShouldDisplayTheExpertsList() { $client = self::createClient(); self::generateSchema(); $loader
= new \Nelmio\Alice\Loader\Base(); $objects = $loader->load($this->getExperts()); $persister = new \Nelmio\Alice\ORM\Doctrine( static::$kernel->getContainer()->get('doctrine.orm.entity_manager') ); $persister->persist($objects); // ... } Créer une nouvelle page

= new \Nelmio\Alice\Loader\Base(); $objects = $loader->load($this->getExperts()); $persister = new \Nelmio\Alice\ORM\Doctrine( static::$kernel->getContainer()->get('doctrine.orm.entity_manager') ); $persister->persist($objects); // ... } Créer une nouvelle page transforme les ﬁxtures en objets

= new \Nelmio\Alice\Loader\Base(); $objects = $loader->load($this->getExperts()); $persister = new \Nelmio\Alice\ORM\Doctrine( static::$kernel->getContainer()->get('doctrine.orm.entity_manager') ); $persister->persist($objects); // ... } Créer une nouvelle page transforme les ﬁxtures en objets persiste les objets dans la base

REFACTOR /** * @Extra\Route("/experts") * @Extra\Template() */ public function listAction()
{ $experts = $this->getDoctrine() ->getRepository('TheodoBundleExpertBundle:Expert') ->findAll() ; return array( 'experts' => $experts ); } Créer une nouvelle page 43

REFACTOR Créer une nouvelle page 44

✔ code retour de la page = 200 ✔ la
réponse contient une liste ✔ le nombre de li = le nombre d’experts dans la base Ce qu’on a testé : Créer une nouvelle page 45

Réutiliser l’existant ... qui n’a pas de tests 46

Réutiliser l’existant public function listAction() { $experts = $this->getDoctrine() ->getRepository('TheodoBundleExpertBundle:Expert')
->findAll() ; $expert = new Expert(); $builder = $this->createFormBuilder($Expert); $form = $builder ->add('firstName') ->add('lastName') ->add('username') ->getForm(); ; if ($this->getRequest()->isMethod('post')) { $form->bind($this->getRequest()); if ($form->isValid()) { $em = $this->getDoctrine()->getManager(); $em->persist($form->getData()); $em->flush(); // ici on envoie un email, au nouvel expert // on créé ses comptes sur tous les services utilisés par la boite etc... return $this->redirect($this->generateUrl('theodoexpertbundle_expert_list')); } } return array('experts' => $experts, 'form’ => $form->createView()); } 47 Contrôleur

->findAll() ; $expert = new Expert(); $builder = $this->createFormBuilder($Expert); $form = $builder ->add('firstName') ->add('lastName') ->add('username') ->getForm(); ; if ($this->getRequest()->isMethod('post')) { $form->bind($this->getRequest()); if ($form->isValid()) { $em = $this->getDoctrine()->getManager(); $em->persist($form->getData()); $em->flush(); // ici on envoie un email, au nouvel expert // on créé ses comptes sur tous les services utilisés par la boite etc... return $this->redirect($this->generateUrl('theodoexpertbundle_expert_list')); } } return array('experts' => $experts, 'form’ => $form->createView()); } 47 Contrôleur création du formulaire

->findAll() ; $expert = new Expert(); $builder = $this->createFormBuilder($Expert); $form = $builder ->add('firstName') ->add('lastName') ->add('username') ->getForm(); ; if ($this->getRequest()->isMethod('post')) { $form->bind($this->getRequest()); if ($form->isValid()) { $em = $this->getDoctrine()->getManager(); $em->persist($form->getData()); $em->flush(); // ici on envoie un email, au nouvel expert // on créé ses comptes sur tous les services utilisés par la boite etc... return $this->redirect($this->generateUrl('theodoexpertbundle_expert_list')); } } return array('experts' => $experts, 'form’ => $form->createView()); } 47 Contrôleur création du formulaire sauvegarde l’expert

for expert in experts %} <li>{{ expert }}</li> {% endfor %} </ul> <form action="{{ path('theodoexpertbundle_expert_list') }}" method="post"> {{ form_errors(form) }} {{ form_widget(form) }} <button type="submit">Save</button> </form> {% endblock %} Réutiliser l’existant 48 Template

Ce qu’on va tester : • l’a!chage de la liste
et du formulaire • la sauvegarde d’un nouvel expert test list Réutiliser l’existant 49

Ce qu’on va tester : • l’a!chage de la liste
et du formulaire • la sauvegarde d’un nouvel expert test list Réutiliser l’existant 49 • la page d’édition • la sauvegarde de l’édition d’un expert new new

public function testShouldDisplayTheExpertsList() { $client = self::createClient(); self::generateSchema(); // load
fixtures ... $crawler = $client->request('GET', '/experts'); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertGreaterThan(1, $crawler->filter('li')->count()); $this->assertCount(1, $crawler->filter('form')); } Note : ceci est un nouveau test ! Tester l’existant 50 ✘ l’a!chage de la liste et du formulaire

RED GREEN ou ? Tester l’existant 51

Tester l’existant 52 ✘ l’a!chage de la liste et du
formulaire

53 Ne faites pas con!ance aux tests écrits après le
code, testez les tests !

{% extends '::base.html.twig' %} {% block body %} <ul> </ul>
{% endblock %} Tester les tests 54 ✘ l’a!chage de la liste et du formulaire

55 Tester les tests ✘ l’a!chage de la liste et
du formulaire

for expert in experts %} <li>{{ expert }}</li> {% endfor %} </ul> {% endblock %} Tester les tests 56 ✘ l’a!chage de la liste et du formulaire

57 Tester les tests ✘ l’a!chage de la liste et
du formulaire

Tester les tests 58 {% extends '::base.html.twig' %} {% block
body %} <ul> {% for expert in experts %} <li>{{ expert }}</li> {% endfor %} </ul> <form action="{{ path('theodoexpertbundle_expert_list') }}" method="post"> {{ form_errors(form) }} {{ form_widget(form) }} <button type="submit">Save</button> </form> {% endblock %} ✘ l’a!chage de la liste et du formulaire

59 Tester les tests GREEN ✔ l’a!chage de la liste
et du formulaire

Véri!ons que la sauvegarde du formulaire fonctionne ! 60

public function testShouldSaveANewExpert() { $client = self::createClient(); $csrfToken = $client->getContainer()
->get('form.csrf_provider') ->generateCsrfToken('unknown'); $client->request('POST', '/experts', array( 'form' => array( 'firstName' => 'Benjamin', 'lastName' => 'Grandfond', 'username' => 'benjaming', '_token' => $csrfToken, ) )); $this->assertEquals(302, $client->getResponse()->getStatusCode()); $crawler = $client->followRedirect(); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertRegExp('~Benjamin~', $crawler->filter('li')->text()); } 61 Tester l’existant ✘ sauvegarde d’un nouvel expert

public function testShouldSaveANewExpert() { $client = self::createClient(); $csrfToken = $client->getContainer()
->get('form.csrf_provider') ->generateCsrfToken('unknown'); $client->request('POST', '/experts', array( 'form' => array( 'firstName' => 'Benjamin', 'lastName' => 'Grandfond', 'username' => 'benjaming', '_token' => $csrfToken, ) )); $this->assertEquals(302, $client->getResponse()->getStatusCode()); $crawler = $client->followRedirect(); $this->assertEquals(200, $client->getResponse()->getStatusCode()); $this->assertRegExp('~Benjamin~', $crawler->filter('li')->text()); } 61 Tester l’existant TIP ✘ sauvegarde d’un nouvel expert

# app/config/config_test.yml parameters: form.csrf_provider.class: 'Theodo\Bundle\ExpertBundle\Form\Extension\Csrf \CsrfProvider\MockCsrfProvider' Mockez les services en
changeant la classe utilisée par le container. TIP 62 Tester l’existant

namespace Theodo\Bundle\ExpertBundle\Form\Extension\Csrf\CsrfProvider; use Symfony\Component\Form\Extension\Csrf\CsrfProvider\CsrfProviderInterface; class MockCsrfProvider implements CsrfProviderInterface { public
function generateCsrfToken($intention) { return $intention; } public function isCsrfTokenValid($intention, $token) { return $intention == $token; } } TIP 63 Tester l’existant

64 Tester l’existant ✔ la sauvegarde d’un nouvel expert

Encore une fois : Ne faites pas con!ance aux tests
écrits après le code, testez les tests ! 65

✔ l’a!chage de la liste et du formulaire ✔ la
sauvegarde d’un nouvel expert ✘ la page d’édition ✘ la sauvegarde de l’édition d’un expert Ce qu’on a testé : test list 66 Réutiliser l’existant

• validité du formulaire • retourne une redirection • retourne
le formulaire erroné • bind le formulaire avec la request • sauvegarder l’entité du formulaire Refactorisation : Création d’un handler pour sauvegarder un expert. Ce qu’on va tester : test list Sauvegarde du formulaire 67

RED namespace Theodo\Bundle\ExpertBundle\Tests\Form\Handler; use Theodo\Bundle\ExpertBundle\Form\Handler\ExpertFormHandler; class ExpertFormHandlerTest extends \PHPUnit_Framework_TestCase {
public function testShouldHandleAnUnsuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form = $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $handler = new ExpertFormHandler(); $this->assertEquals($form, $handler->handle($form, $request)); } } Refactoriser l’existant 68 ✘ retourne le formulaire erroné

69 Mock ?!

70 Un mock est un objet qui simule un autre
objet et permet de contrôler les interactions avec l‘objet testé. • véri"er le nombre d’appels d’une méthode • spéci"er le retour de la méthode • véri"er les paramètres d’une méthode • ...

71 • PHPUnit Mock Object https://github.com/sebastianbergmann/phpunit-mock-objects • Mockery https://github.com/padraic/mockery •
Phake https://github.com/mlively/Phake Les frameworks

RED 72 Refactoriser l’existant ✘ retourne le formulaire erroné

RED namespace Theodo\Bundle\ExpertBundle\Form\Handler; use Symfony\Component\Form\FormInterface; use Symfony\Component\HttpFoundation\Request; class ExpertFormHandler {
public function handle(FormInterface $form, Request $request) { return $form; } } 73 Refactoriser l’existant ✘ retourne le formulaire erroné

RED namespace Theodo\Bundle\ExpertBundle\Form\Handler; use Symfony\Component\Form\FormInterface; use Symfony\Component\HttpFoundation\Request; class ExpertFormHandler {
public function handle(FormInterface $form, Request $request) { return $form; } } 73 Refactoriser l’existant ✘ retourne le formulaire erroné code minimum !

GREEN 74 Refactoriser l’existant ✔ retourne le formulaire erroné

RED public function testShouldHandleASuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form
= $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $form->expects($this->once()) ->method('isValid') ->will($this->returnValue(true)) ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request) $this->assertInstanceOf( 'Symfony\Component\HttpFoundation\RedirectResponse', $response ); $this->assertEquals('/experts', $response->getTargetUrl()); } 75 Refactoriser l’existant ✘ validité du formulaire ✘ retourne une redirection

= $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $form->expects($this->once()) ->method('isValid') ->will($this->returnValue(true)) ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request) $this->assertInstanceOf( 'Symfony\Component\HttpFoundation\RedirectResponse', $response ); $this->assertEquals('/experts', $response->getTargetUrl()); } 75 Refactoriser l’existant ✘ validité du formulaire ✘ retourne une redirection le formulaire est valide

= $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $form->expects($this->once()) ->method('isValid') ->will($this->returnValue(true)) ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request) $this->assertInstanceOf( 'Symfony\Component\HttpFoundation\RedirectResponse', $response ); $this->assertEquals('/experts', $response->getTargetUrl()); } 75 Refactoriser l’existant ✘ validité du formulaire ✘ retourne une redirection le formulaire est valide handle() retourne une redirection vers /experts

RED 76 Refactoriser l’existant ✘ validité du formulaire ✘ retourne
une redirection

RED namespace Theodo\Bundle\ExpertBundle\Form\Handler; use Symfony\Component\Form\FormInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\RedirectResponse; class
ExpertFormHandler { public function handle(FormInterface $form, Request $request) { if ($form->isValid()) { return new RedirectResponse("/experts"); } return $form; } } 77 Refactoriser l’existant ✘ validité du formulaire ✘ retourne une redirection

GREEN 78 Refactoriser l’existant ✔ validité du formulaire ✔ retourne
une redirection

REFACTOR public function testShouldHandleAnUnsuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form
= $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request); // ... 79 Refactoriser l’existant public function testShouldHandleASuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form = $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request); // ...

= $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request); // ... 79 Refactoriser l’existant public function testShouldHandleASuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form = $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock() ; $handler = new ExpertFormHandler(); $response = $handler->handle($form, $request); // ... Duplication

REFACTOR class ExpertFormHandlerTest extends \PHPUnit_Framework_TestCase { protected $handler; public function
setUp() { $this->handler = new ExpertFormHandler(); } //... 80 Refactoriser l’existant

REFACTOR public function getFormMock() { return $this ->getMockBuilder('Symfony\Component\Form\Test\FormInterface') ->disableOriginalConstructor() ->getMock();
} 81 Refactoriser l’existant

= $this->getFromMock(); $response = $this->handler->handle($form, $request); // ... 82 Refactoriser l’existant public function testShouldHandleASuccessfulFormSubmission() { $request = new \Symfony\Component\HttpFoundation\Request(); $form = $this->getFromMock(); // ... $response = $this->handler->handle($form, $request); // ...

REFACTOR 83 Refactoriser l’existant

RED public function testShouldHandleASuccessfulFormSubmission() { // ... $form->expects($this->once()) ->method('bind') ->with($this->identicalTo($request))
; // ... } 84 Refactoriser l’existant ✘ bind le formulaire avec la request

RED 85 Refactoriser l’existant ✘ bind le formulaire avec la
request

RED namespace Theodo\Bundle\ExpertBundle\Form\Handler; use Symfony\Component\Form\FormInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\RedirectResponse; class
ExpertFormHandler { public function handle(FormInterface $form, Request $request) { $form->bind($request); if ($form->isValid()) { return new RedirectResponse("/experts"); } return $form; } } 86 Refactoriser l’existant ✘ bind le formulaire avec la request

✔ validité du formulaire ✔ retourne une redirection ✔ retourne
le formulaire erroné ✔ bind le formulaire avec la request ✘ sauvegarder l’entité du formulaire Sauvegarde du formulaire test list 87

le formulaire erroné ✔ bind le formulaire avec la request ✘ sauvegarder l’entité du formulaire Sauvegarde du formulaire test list 87 new ✘ génère la route avec le router

GREEN 88 Refactoriser l’existant ✔ bind le formulaire avec la
request

RED 89 Refactoriser l’existant ✘ génère la route avec le
router class ExpertFormHandlerTest extends \PHPUnit_Framework_TestCase { protected $handler; protected $router; public function setUp() { $this->router = $this->getMock('Symfony\Component\Routing\RouterInterface'); $this->handler = new ExpertFormHandler($this->router); } }

router class ExpertFormHandlerTest extends \PHPUnit_Framework_TestCase { protected $handler; protected $router; public function setUp() { $this->router = $this->getMock('Symfony\Component\Routing\RouterInterface'); $this->handler = new ExpertFormHandler($this->router); } } mock le router

router class ExpertFormHandlerTest extends \PHPUnit_Framework_TestCase { protected $handler; protected $router; public function setUp() { $this->router = $this->getMock('Symfony\Component\Routing\RouterInterface'); $this->handler = new ExpertFormHandler($this->router); } } mock le router injecte le router

RED public function testShouldHandleASuccessfulFormSubmission() { // ... $this->router->expects($this->once()) ->method('generate') ->with($this->equalTo('theodoexpertbundle_expert_list')
->will($this->returnValue('/experts')) ; $response = $this->handler->handle($form, $request); // ... } 90 Refactoriser l’existant ✘ génère la route avec le router

router

RED namespace Theodo\Bundle\ExpertBundle\Form\Handler; use ...; use Symfony\Component\Routing\RouterInterface; class ExpertFormHandler {
/** * @var \Symfony\Component\Routing\RouterInterface */ protected $router; /** * @param RouterInterface $router */ public function __construct(RouterInterface $router) { $this->router = $router; } // ... } 92 Refactoriser l’existant ✘ génère la route avec le router

RED public function handle(FormInterface $form, Request $request) { $form->bind($request); if
($form->isValid()) { $url = $this->router->generate( 'theodoexpertbundle_expert_list' ); return new RedirectResponse($url); } return $form; } 93 Refactoriser l’existant ✘ génère la route avec le router

GREEN 94 Refactoriser l’existant ✔ génère la route avec le
router

le formulaire erroné ✔ bind le formulaire avec la request ✔ génère la route avec le router ✘ sauvegarder l’entité du formulaire Sauvegarde du formulaire 95 test list

RED protected $em; public function setUp() { $this->router = $this->getMock('Symfony\Component\Routing\RouterInterface');
$this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager') ->disableOriginalConstructor() ->getMock() ; $this->handler = new ExpertFormHandler($this->router, $this->em); } 96 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire

$this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager') ->disableOriginalConstructor() ->getMock() ; $this->handler = new ExpertFormHandler($this->router, $this->em); } 96 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire mock l’entity manager

$this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager') ->disableOriginalConstructor() ->getMock() ; $this->handler = new ExpertFormHandler($this->router, $this->em); } 96 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire mock l’entity manager injecte l’entity manager

RED public function testShouldHandleASuccessfulFormSubmission() { // ... $expert = new
\Theodo\Bundle\ExpertBundle\Entity\Expert(); $form->expects($this->once()) ->method('getData') ->will($this->returnValue($expert)); // ... $this->em->expects($this->once()) ->method('persist') ->with($this->identicalTo($expert)) ; $this->em->expects($this->once()) ->method('flush'); // ... 97 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire

\Theodo\Bundle\ExpertBundle\Entity\Expert(); $form->expects($this->once()) ->method('getData') ->will($this->returnValue($expert)); // ... $this->em->expects($this->once()) ->method('persist') ->with($this->identicalTo($expert)) ; $this->em->expects($this->once()) ->method('flush'); // ... 97 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire getData retournera un expert

\Theodo\Bundle\ExpertBundle\Entity\Expert(); $form->expects($this->once()) ->method('getData') ->will($this->returnValue($expert)); // ... $this->em->expects($this->once()) ->method('persist') ->with($this->identicalTo($expert)) ; $this->em->expects($this->once()) ->method('flush'); // ... 97 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire getData retournera un expert cet expert sera persisté

\Theodo\Bundle\ExpertBundle\Entity\Expert(); $form->expects($this->once()) ->method('getData') ->will($this->returnValue($expert)); // ... $this->em->expects($this->once()) ->method('persist') ->with($this->identicalTo($expert)) ; $this->em->expects($this->once()) ->method('flush'); // ... 97 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire getData retournera un expert cet expert sera persisté l’em sera "ushé

RED 98 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire

RED class ExpertFormHandler { /** @var \Symfony\Component\Routing\RouterInterface */ protected $router;
/** @var \Doctrine\ORM\EntityManager */ protected $manager; /** * @param RouterInterface $router * @param EntityManager $manager */ public function __construct(RouterInterface $router, EntityManager $manager) { $this->router = $router; $this->manager = $manager; } 99 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire

($form->isValid()) { $this->manager->persist($form->getData()); $this->manager->flush(); $route = 'theodoexpertbundle_expert_list'; return new RedirectResponse($this->router->generate($route)); } return $form; } 100 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire

($form->isValid()) { $this->manager->persist($form->getData()); $this->manager->flush(); $route = 'theodoexpertbundle_expert_list'; return new RedirectResponse($this->router->generate($route)); } return $form; } 100 Refactoriser l’existant ✘ sauvegarder l’entité du formulaire persiste l’expert puis "ush

101 GREEN Refactoriser l’existant ✔ sauvegarder l’entité du formulaire

le formulaire erroné ✔ bind le formulaire avec la request ✔ génère la route avec le router ✔ sauvegarder l’entité du formulaire Sauvegarde du formulaire 102 test list

Maintenant on peut faire l’édition d’un expert en TDD !
103

Conclusion • Soyez patient • Persévérez • (Ré)introduisez progressivement des
tests • Testez ce qui est primordial • Refactorisez, corrigez, créez en TDD

Questions ?  benja-M-1  benjaM1  [email protected]:benja-M-1/s"ive-paris-2013.git https://speakerdeck.com/benjam1/adoptez-le-tdd-sﬂive-paris-2013 Venez
me retrouvez au stand Theodo pour en savoir plus ! 105

Adoptez le TDD sur vos projets Symfony2 existants

Adoptez le TDD sur vos projets Symfony2 existants

More Decks by Benjamin Grandfond

Other Decks in Programming

Featured

Transcript