Projet

Général

Profil

Development #41025

Ajout de tests sur les vues et les commandes

Ajouté par Nicolas Roche il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
25 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour essayer d'augmenter un petit peu la couverture.


Fichiers

Révisions associées

Révision 6e48d379 (diff)
Ajouté par Nicolas Roche il y a environ 4 ans

tests: add tests on views and commands (#41025)

Historique

#1

Mis à jour par Nicolas Roche il y a environ 4 ans

Suite de #40990.
J'ai beaucoup de mal à gagner en couverture, et là je pense que j'ai tout donné.
(J'ai profité des tests pour corriger une typo dans une commande de "management".)

#2

Mis à jour par Frédéric Péters il y a environ 4 ans

Je pense qu'il y a encore quelques trucs faciles dans nanterre/utils.py (en regardant les fichiers longs et mal couverts).

#3

Mis à jour par Thomas Noël il y a environ 4 ans

... et aussi le Jenkinsfile lance un « tox -rv -e coverage » ce qui ne lance que les tests py3, pour se rassurer il faudrait pousser qlq chose genre ça dans ta branche :

diff --git a/Jenkinsfile b/Jenkinsfile
index 5496721..d9651bd 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -5,7 +5,7 @@ pipeline {
     stages {
         stage('Unit Tests') {
             steps {
-                sh 'tox -rv -e coverage'
+                sh 'tox -rv'
             }
             post {
                 always {
diff --git a/tox.ini b/tox.ini
index 083c252..de91cbc 100644
--- a/tox.ini
+++ b/tox.ini
@@ -5,7 +5,7 @@

 [tox]
 toxworkdir = {env:TMPDIR:/tmp}/tox-{env:USER}/zoo/{env:BRANCH_NAME:}
-envlist = {py27,py3}
+envlist = {py27,py3}-coverage

 [testenv]
 # django.contrib.auth is not tested it does not work with our templates

#4

Mis à jour par Nicolas Roche il y a environ 4 ans

  • Statut changé de Solution proposée à En cours

le Jenkinsfile lance un « tox -rv -e coverage » ce qui ne lance que les tests py3

Merci, j'ai corrigé dans #40911 (et je me suis rebasé dessus ici).
Je continue les tests.

#5

Mis à jour par Nicolas Roche il y a environ 4 ans

pas fini mais je peux arrêter :)

#6

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée
#7

Mis à jour par Nicolas Roche il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 6e48d37985296f132c2aecd549287c56b65404bb
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Mar 25 15:22:07 2020 +0100

    tests: add tests on views and commands (#41025)
#8

Mis à jour par Nicolas Roche il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF