Pour cela nous utiliseront le plugin Hudson Emma pour l’affichage dans Hudson, ainsi que le plugin symfony agEmmaCoverageReport pour générer le fichier xml au format emma.
Le format « emma » est à la basée utilisé en java, plus d’infos sur le site du projet.

Utilisation de la tache de génération du rapport xml

L’extension xdebug doit être installée (mais pas forcément activée).

Les fichiers testés sont trouvés en utilisant l’autoload, les fichiers de tests doivent être nommés de la façon suivante : nomDeLaClasseTesteeTest.php, par exemple le classe « qualifactCellFormatterEuro » aura pour fichier de test « qualifactCellFormatterEuroTest.php ».

Voyons tout d’abord comment fonctionne la génération du fichier xml de rapport :

Si xdebug est activé par défaut :

./symfony ecr:report --xml=cheminOuSauverLeXml.xml

Mais sur le serveur d’intégration cela n’est probablement pas le cas, et il serait
gênant d’avoir à l’activer pour les autres tests alors que qu’il est seulement utilisé pour le coverage.
Pour cela on peut indiquer à la tache le chemin ou se situe l’extension xdebug (par exemple sur une Mandriva il se situe ici : « /usr/lib/php/extensions/xdebug.so »). La tâche sera alors appelée de cette façon.

./symfony ecr:report --xml=cheminOuSauverLeXml.xml --xdebug-extension-path=/usr/lib/php/extensions/xdebug.so

Intégration dans Hudson CI

Création et configuration du job

• Dans le menu de gauche : « Nouvelle Tache »
• Cliquez sur « Construire un projet free-style »
• Donnez un nom au job
• Entrez alors l’url du dépôt
• Après checkout nous voulons lancer les tests unitaires ainsi que la tache créant

le rapport de coverage, nous allons donc ajouter une étape au build. Celle ci sera de
type « Executer un script shell »

• Dans la liste des commandes nous allons donc lancer les tests et le coverage de cette façon :

$WORKSPACE/symfony test:unit --xml=log/unit.xml
$WORKSPACE/symfony ecr:report --xml=log/coverage.xml



• Pour les résultats des tests unitaires, cochons la case « Publier le rapport des résultats de tests JUnit » puis dans « XML des rapports de test » nous indiquons ou chercher le fichier XML en saisissant : « log/unit.xml »
• Pour le code coverage, cochons la case « Record Emma coverage report » puis dans « Folders or files containing Emma XML reports » nous indiquons où chercher le fichier XML en saisissant : « log/coverage.xml »
• Il faudra ensuite exécuter le job

Affichage du rapport de code coverage

Après lancement du job les résultats du code coverage apparaissent dans la section « Coverage Trend ».

Pour un détail du coverage il faut cliquer sur le graphique.

La couverture du code par les tests unitaire est alors présentée par package ainsi que par classe, méthode, bloc et ligne.

Les blocs sont tous à 0 car c’est une notion seulement utilisée en java. Une méthode ou une classe est couverte si toutes les lignes qui la contiennent sont couvertes. Ainsi certaines classes, ne contenant aucune ligne de code, ayant juste leur déclaration, seront indiquées comme couvertes à 100%.

La rue de Sully va retrouver son calme, les locaux occupent dorénavant 10 fenêtres de large

Un dernier coup de peinture à la charte de notre suite logicielle

exposer les équipes de développement en plein soleil ? En voilà une idée étrange.

Les chaises ergonomiques ont été déballées

notre équipe support cable les baies

Tout va être fin prêt pour mettre le cap sur de nouveaux défis pour la team PMSIPilot

Plus de photos sur Flickr

Bon, mais d’abord, pour les nouveaux, qu’est-ce donc qu’un Mock ? Je ne saurais mieux l’expliquer que Padraic dans son superbe blogpost, mais pour les pressés, on va faire une version courte. Un mock est un remplacement d’objet pour isoler un test. Par exemple, on peut créer des mocks de base de données pour éviter d’instancier tout Propel a chaque test. Voilà, ça c’est pour la théorie.

Toujours pas clair ? Un petit exemple alors. Tout à l’heure j’ai écris des tests (disons pour une classe qui s’appellerait Foobar). C’est une super classe, qui fait son job, pas de problème, mais qui demande en constructeur 2 objets: un Foo et un Bar. Disons que Foo et Bar coûtent un peu cher à instancier. J’ai donc 2 choix: instancier des vrais objets (c’est mal, mais c’est qu’on fait en général), ou utiliser des mocks (c’est bien, mais on le fait pas parcequ’on a la flemme d’écrire les mocks). Qu’à cela ne tienne, avec Mockery, c’est fait en 2 coups de cuiler a pot:
$foo = Mockery::mock('Foo', array('getCreatedAt' => date('Y-m-d H:i:s'));
$bar = Mockery::mock('Bar', array(
'getIsSaved' => true,
'getTitle' => 'recherche titre',
'getDescription' => 'recherche description',
'getIsPublic'=> true ));
$object = new Foobar($foo, bar);
Donc ça c’est la forme un peu « rapide » de création de mock. Pour chaque méthode que je spécifie dans l’array d’options, mockery assigne en valeur de retour la valeur en face. Donc $recherche->getTite() renverra « recherche titre » par exemple. C’est cool non ? Bien sur on peut aller super loin avec Mockery, comme par exemple dire « la méthode tartenpion de l’objet pmsipilotUser doit etre appelé avec en paramètre perlinpinpin et renvoyer un objet MerlinLenchanteur et ne doit être utilisée qu’une fois sinon la terre explose »:
$merlin = new MerlinLenchanteur();
$user = Mockery::mock('pmsipilotUser');
$user->shouldReceive('tartenpion')->with('perlinpinpin')->once()->andReturn($merlin);
(ok je triche encore un peu on ne peut pas vraiment faire exploser la terre) Par contre si on veut vérifier que tout a bien été appelé comme on l’a défini, il ne faut pas oublier de le dire a la fin: Mockery::close(); Un dernier exemple pour vous convaincre de la puissance de Mockery. On rencontre souvent du code qui viole la Loi de Demeter, et qui de fait devient difficile à tester. Point de chichi avec Mockery, car il permet aussi de Mocker une chaine d’appels complète très simplement: $mock = Mockery::mock('demeter'); $mock->shouldReceive('foo->bar->quux->roflpwn->demeter')->andReturn('omgponies'); Magique non ? Pour encore plus de trucs magiques, je vous invite à lire la doc sur la page github de mockery. Voilà j’espère que ça vous aura donné envie d’utiliser un peu plus de Mocks dans vos tests :-)

Exemple: disons qu’on a un fichier /tmp/foobar.csv avec le contenu suivant:

foo;bar
baz;quux

Nous pouvons simplement parcourir les lignes avec le code suivant:

<?php

$file = new SplFileObject('/tmp/foobar.csv');

foreach($file as $line)
{
  $data = explode(';', $line);
}

C’est pas mal non ?

On pourra arguer qu’on arrive au même genre de résultat en utilisant file, mais l’empreinte mémoire est loin d’être la même (sans compter l’approche object de SplFileObject qui va nous permettre plus tard de faire des trucs rigolos).

Mais il y a encore mieux. SplFileObject gère le format CSV, de deux manières différentes. Vous pouvez au choix utiliser SplFileObject#fgetcsv ou bien (pour garder une API cohérente) expliquer à SplFileObject que le fichier que vous traitez est un CSV.

Attardons nous sur cette dernière méthode (toujours avec le même /tmp/foobar.csv):

<?php

$file = new SplFileObject('/tmp/foobar.csv');
$file->setFlags(SplFileObject::READ_CSV);
$file->setCsvControl(';');

foreach($file as $data)
{
  var_dump($data);
}

Magique !

Vous pouvez bien évidemment (comme on le disait plus haut) surclasser SplFileObject pour lui ajouter tout un tas de trucs rigolos (par exemple chez nous on détecte les BOM, et on configure automatiquement les fichiers CSV).

Bien sur, SplFileObject permet bien d’autres choses, quasiment tout ce que vous pouvez faire avec les fonctions utilisant un handler fopen est disponible (à quelques exceptions près, comme par exemple fread) (globalement c’est assez orienté lecture ligne par ligne).

Il y a deux ans, le projet a connu un virage technologique important. D’une technologie obsolète et d’un conception un peu hasardeuse (PHP4, framesets, …), un projet de refonte vers symfony 1.2 a été lancé. Cette refonte a durée un peu plus de 20 mois / homme et a abouti à un seul projet symfony contenant plusieurs applications. PMSIpilot a, dès le début, pris le parti d’utiliser les conventions de symfony et le paradigme MVC.

En chiffres, aujourd’hui

• 12 développeurs (en constante augmentation !),
• 410 déploiements,
• environ 50 000 utilisateurs répartis sur toutes ces instances,
• environ 220 000 lignes de code PHP écrites et maintenues,
• la prochaine release importante intégrera symfony 1.4.

Comment symfony nous aide

La documentation et la communauté

La documentation de symfony nous permet d’intégrer rapidement une nouvelle recrue. Le fait de trouver de nombreuses contributions au framework (sur symfony-project.org ou bien ailleurs) est très facilitant également. Nous utilisons de nombreux plugins comme :

• sfPropelPlugin
• dwJpGraphPlugin
• sfDB4toPropelPlugin
• sfPropelSqlDiffPlugin
• sfJqueryReloadedPlugin
• sfValidatorHtmlPlugin
• sfJobQueuePlugin
• sfFormExtraPlugin
• …

Parfois tels quels, parfois un peu modifiés pour nos besoins.

Le développement

• le framework nous donne une bonne structure de base pour l’organisation du code source ainsi que des normes de codage, sur une équipe aussi importante cela permet de gagner du temps,
• mis à part i18n et l’admin générator, on utilise la quasi totalité de la stack de symfony : MVC, le cache, les filtres, les évènements, le système de gestion des utilisateur, l’ORM, les tasks, …
• symfony est suffisament flexible pour que nous puissions facilement l’adapter à nos besoins parfois bien spécifiques ; aujourd’hui nous avons étendu : sessionStorage, viewCacheManager, ApplicationConfiguration et ProjectConfiguration, sfAction, sfComponent, sfView, frontWebController, plein de choses autour de propel …
• nous ajoutons des fonctionnalités tous les jours, et une grande réactivité nous est demandé pour suivre l’évolution de la réglementation ; le framework de test nous permet de garantir aucune régression et de livrer des correctifs très rapidement si nécessaire (voir plus bas),
• le système de task, de gestions des fixtures, d’environnement de développement permet de proposer des méthodes faciles d’accès au développeur permettant d’instancier un environnement de dev complet et identique à ses collègues.

L’intégration continue

Dès la refonte du projet, sécuriser ces mises à jours a été une priorité. C’est pourquoi nous avons investi dans le développements d’un grand nombre de tests fonctionnels et leur suivi au sein d’une instance Hudson.

Les tests fonctionnels chez PMSIpilot vérifient des cas d’utilisation standards mais sont surtout utilisés pour valider la totalité des données chiffrées présentées. A ce titre les quelques 250 000 tests fonctionnels sont majoritairement générés (on ne teste pas vraiment « les tests » mais « les étalons des tests »).

La création de nouvelles classes a entrainé l’écriture d’une centaine de tests unitaires (avec des efforts pour optimiser la couverture du code).

Enfin, nous avons commencé l’écriture de tests d’interface avec Selenium (une dizaine aujourd’hui) afin de gérer plus facilement les différents navigateurs (parfois un peu datés) de nos clients. (oui, ça n’a rien à voir avec symfony …)

Pour boucler la boucle on a, en projet, l’intégration automatique de tests de montée en charge. (un outil à suggérer ?)

Comment nous tentons d’aider symfony

Dans la mesure du possible, nous tentons de faire bénéficier la communauté symfony de nos travaux, en remontant des bugs et en proposant des plugins . Nous essayons également de partager les slides des réunions techniques organisées au sein de l’équipe afin de diffuser les compétences au sein des équipes.

Chez PMSIpilot, on aime symfony. Si c’est votre cas aussi, n’hésitez pas à proposer votre candidature au service technique.

tickets :

• #1776 (fixed) [PATCH] checkbox arrays not checked with fillin enabled
• #2916 (wontfix) Bug in Spyc : losing data with successive load/dump
• #3792 (fixed) i18n XLIFF do not handle entities correctly
• #4306 (wontfix) sfDebugConnection does not handle correctly logging of nested transactions
• #4620 (fixed) [PATCH] – PHP5NestedSetBuilder generate a require to lib/model/om/Basesometing
• #5374 (fixed) Changeset [14122] breaks config parsing
• #5402 (fixed) sf doc refer to the getDispatcher() method but it doesnt exist
• #5964 (fixed) Notice sur sfTestBrowser
• #7446 (fixed) sfOutputEscaper::unescape($vars) appelé systématiquement dans _call_component
• #6639 (fixed) Option passed to a widget schema do not get passed through to an sfWidgetFormChoice’s renderer
• #8365 (opened) sfFinder not_name does not works if results intersects due to #6589
• #8384 (opened) sfPropelPluginConfiguration::filterAutoloadConfig should exclude vendor dir just like in the plugin autoload file
• #8402 (opened) sfPropelBaseTask should add propel include path by using sf_propel_runtime_path
• #8457 (fixed) sort the parameters in order to compute the cache key the same way when parameters are submited in different order

plugins :

• omCredentialsMapPlugin : This plugin allows you to check  all the credentials in your project and detect action not or badly securised
• omCrossAppUrlPlugin : Simple helper to get cross app urls
• elXHProfPlugin : Integration of XHProf profiler (simple API to profile + web debug toolbar integration)

Au menu :

• le code coupling
• les tests unitaires (en attendant un point complet là dessus le prochain mois)
• les tests d’interface que je propose de réaliser avec Selenium