[Java - Junit] Des tests trop poussés, sont-ils mauvais ?

Bonjour,

Je me pose une question. Faire des tests trop avancés, j'entends par là, vérifier que tous les mocks sont bien appelés, un certain nombre de fois, avec des captor, etc. (exemple :

Code:

---

1 2 3

when(myMock.maFonction(captor)).thenReturn(...); verify(myMock, times(4).maFonction(captor)); assertEquals(captor.getValue(), moObjet);

---

)

Est-ce une mauvaise pratique ? Souvent, ce genre de tests ne sont qu'une "duplication" du code de production et quand ce dernier change, on est souvent obligés de changer le code de tests lui-même ce qui fait perdre de l'intérêt au process. En somme, est-ce qu'écrire des tests trop poussés n'est-il pas contreproductif?

Je n'arrive pas vraiment à trouver de documentation sur ça, si vous avez une idée

Bonjour,

Vous ne trouverez pas de "documentation" à proprement parler, c'est de l'appréciation.

// ! Avis personnel
Je ne dirais pas que des tests trop poussés sont mauvais. Mais plutôt qu'il faut tester ce qui est utile.
Comem vous le dites, vos tests sont de la duplication de la logique. Est-ce vraiment important de savoir que votre méthode va appeler 4 fois une autre méthode ? Je ne crois pas. Ce qui est iportant c'est de vérifier que la sortie soit correcte.

Imaginons que `isBirthDay` doive retourner retourner true ou false ou lancer une exception si la date de naissance est invalide.

Code:

---

1 2 3 4 5 6

// Pseudo code boolean isBirthDay(Person p) { validator.validateOrThrow(p.getBirthDay()); return today().equals(p.getBirthDay()) }

---

On à pas besoin de vérifier que `validator` ou `p.getBirthDay()` soient appelées. Ce que l'on veut tester c'est les entrées et sorties :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

// Pseudo code must_return_true_when_birth_date_is_today() { Person p = new Person(dateOfToday()) assertTrue(isBirthDay(p)) }   must_return_false_when_birth_date_is_not_today() { Person p = new Person(notToday()) assertTrue(isBirthDay(p)) }   must_fail_when_birth_day_is_absent() { Person p = new Person("") assertFailWithException(isBirthDay(p)) }

---

Le problème avec votre exemple est que, si vous changez la façon de valider la date, il faudra changer vos tests. C'est pourtaht l'objectif inverse qui est visé, idéalement vous devriez pouvoir changer l'implémentation sans modifier les tests.

Merci pour ta réponse. Je comprends ce que tu veux dire :)