InseeFr / InseeFr/Magma

Refactor certains tests ?

Open
#421 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
1
Forks
6
Avg merge
10m
Merged PRs (30d)
4

Description

Fabrice : Pour les tests end to end : vérifier la conformité du json (mockMvc.perform(get("/")).andExpect(content().json("{...}", JsonCompareMode.LENIENT));). Avec Spring boot 4, on peut utiliser RestTestClient qui est plus riche. Je pense que les tests seront beaucoup plus aisés à écrire.

Guylène : A voir car je ne teste que certaines rubriques, une par type de rubrique (pas nécessaire de les tester toutes). En faisant cela, on teste tout le Json. Or je me base sur des données de la base de dev, et le Json est gros. Comme je ne vois pas l'intérêt de tester tout le Json, j'avais trouvé cette solution.

cf commentaires sur cette PR : https://github.com/InseeFr/Magma/pull/417

Vu avec Fabrice : à prévoir sur les tests où on teste tous les champs (petit Json en sortie), par ex geo/pays/{code} ?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.