Discussion :

[grunk]

Bonjour à tous,

après moultes hésitations je me décide enfin à mettre en place des tests unitaires , au moins pour mes classes les plus importantes. Mais quelques questions me viennent :

Prenons ce test que je viens d'écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public function testIsIp() {
        $ip     = '172.16.12.82';
        $ip2    = '172.256.32.44';
        $ip3    = 'abc.def.cc.qq';
        $ip4    = '123654789';
        $this->assertIsA(Util_String::isIp($ip),'bool');
        $this->assertTrue(Util_String::isIp($ip));
        $this->assertFalse(Util_String::isIp($ip2));
        $this->assertFalse(Util_String::isIp($ip3));
        $this->assertFalse(Util_String::isIp($ip4));
}

Est il correct ? Parce que finalement il y'a une infinité de tests possible pour vérifier que ma méthode isIp marche correctement. Dois je simplement vérifier qu'elle me renvoie true sur une ip correcte ou au contraire faire plus de test sur de mauvaise ip ?

Dans le même esprit , comment puis je tester une méthode sensé me retourner une chaine aléatoire ? A part un assertIsA($chaine,'string') , je peux pas tester grand chose et dans ce cas précis j'ai l'impression de tester si l'eau sa mouille ...

Dernière question mais pas des moindre , comment justifiez vous du temps passer à écrire des tests ? Parce que ca prend limite plus de temps que de construire la classe testé, et du coup pour mes supérieurs non technique tant qu'il ne vois pas un nouveau bouton sur leur écran , j'ai rien foutu ... Et entre "code plus robuste", "économie sur le long terme" ," temps de développement plus long" il ne retiennent évidement que le dernier.

Merci pour vos réponses

[Petibidon]

Question bête en voyant ton code : Pourquoi ne pas te reposer sur la fonction assert() fournie par PHP ? C'est pratique et ça te laisse la possibilité (certes mauvaise, faut pas, mais parfois on oublie des trucs) d'oublier de virer tes assertions lors de la mise en prod de ton code.

Après pour les tests, tout dépend de ce que tu attends de ta fonction. Se limite-t'elle à matcher [1-255].[0-255].[0-255].[0-255] ou est elle capable d'identifier les réseaux privés, les IP multicast etc ? Le nombre de tests va dépendre de cela.

Sinon pour le temps à y passer, tout dépend de la certitude que tu as concernant ce que va te renvoyer ta méthode. Faire des tests sur des algos complexes pour lesquels on a des résultats "types" qui devraient prouver l'exactitude des calculs, oui. Mais tester qu'une méthode bateau renvoie une chaîne aléatoire, ça vaut pas vraiment le coup.

[grunk]

Question bête en voyant ton code : Pourquoi ne pas te reposer sur la fonction assert() fournie par PHP ? C'est pratique et ça te laisse la possibilité (certes mauvaise, faut pas, mais parfois on oublie des trucs) d'oublier de virer tes assertions lors de la mise en prod de ton code.

J'ai pas d'assertion dans mon code de production, la méthode de test si dessus est dans une classe de test qui hérite de simpletest qui ne fait absolument pas partie de mon projet.

Après pour les tests, tout dépend de ce que tu attends de ta fonction. Se limite-t'elle à matcher [1-255].[0-255].[0-255].[0-255] ou est elle capable d'identifier les réseaux privés, les IP multicast etc ? Le nombre de tests va dépendre de cela.

Elle ne match effectivement que le pattern d'une ip, donc a priori mon nombre de test semble suffisant.

Merci pour ta réponse