Refactorisation de quelques très gros fichiers

[Arnaud-D]

Participe à la résolution d'un des items de #6246, à savoir la séparation de certains très gros fichiers (et la correction de restes de la dernière refacto du genre, oups, mais ça fait gagner 2 min à la CI).

Contrôle qualité

La CI devrait suffire, mais n'hésitez pas à vous promener sur le site.

[coveralls]

Coverage increased (+0.08%) to 87.248% when pulling 2fa53c4 on Arnaud-D:coupe_coupe into 44624b9 on zestedesavoir:dev.

[Situphen]

Tu as supprimé le fichier zds/tutorialv2/tests/tests_views/tests_content.py ! C'est une erreur ?

Sinon, j'ai réussi à lister les fonctions et les classes des vues et des tests sur la branche upstream/dev et sur ta PR pour vérifier que tu n'en as pas oublié et effectivement tu n'en as pas oublié !

Mémo pour moi-même ou quelqu'un d'autre plus tard :

grep -h "def test_" `find zds/ -name "*.py"` | sort -d > fichier.txt

[Arnaud-D]

Oui, j'ai supprimé une bonne partie (mais pas la totalité) de zds/tutorialv2/tests/tests_views/tests_content.py. Non, ce n'est pas une erreur. Il ne fait que contenir des tests dupliqués par accident par le passé. On retrouve toute la classe supprimée à l'identique dans zds/tutorialv2/tests/tests_views/tests_published.py.

Je pense avoir vraiment bien vérifié, mais faut pas hésiter à reregarder.

[Situphen]

Je comprends mieux, merci. J'ai listé les fonctions avant et après avec ma petite commande : les doublons sont bien supprimés mais il y a aussi des fonctions uniques qui ont disparues. En voici la liste :

test_add_help_opinion()
test_add_help_tuto()
test_save_no_redirect()

Elles étaient toutes les trois à la toute fin du fichier tests_content.py.

[Arnaud-D]

Oui, effectivement, je confirme que ces trois là sont passées à la trappe. Je corrige ça.

Ça explique aussi les chutes de coverage.

[Situphen]

QA OK