Bug #42368
Corriger un test qui plante aléatoirement
0%
Fichiers
Demandes liées
Révisions associées
Revert "tests: do not suppress ResourceLog to keep order (#42368)"
This reverts commit 83863f5efa3ddb38a3a12cac246a671e882cd488.
tests: sort ResourceLog by id to keep order (#42368)
Historique
Mis à jour par Nicolas Roche il y a environ 4 ans
- Lié à Bug #42187: Ne pas faire dépendre les tests ProxyLogger d'un connecteur spécifique. ajouté
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-tests-do-not-suppress-ResourceLog-to-keep-order-4236.patch 0001-tests-do-not-suppress-ResourceLog-to-keep-order-4236.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
- https://jenkins.entrouvert.org/job/passerelle/1356/
- https://jenkins.entrouvert.org/job/passerelle/1358/
- https://jenkins.entrouvert.org/job/passerelle-wip/job/wip%252F39650-sms-to-number-masks/1/
J'ai repris le comportement du test initial qui ne posait pas problème, à savoir de ne pas supprimer les logs au fur et à mesure.
Je ne suis pas complètement convaincu par mon patch mais jusqu'à présent (11 builds) je n'ai pas eu d'erreur.
(désolé pour la fausse manip : commit+revert sur master)
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à En cours
Bon, ça a planté au 13ème (c'était un peu évident).
https://jenkins.entrouvert.org/job/passerelle-wip/job/wip%252F42368-test-correction/7/
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
C'est juste que tu ne définis pas d'ordre, tu ne peux pas supposer que la base va te renvoyer les lignes de la table dans un ordre précis, tente avec un .order_by('id') (et crée un qs = ResourceLog.objects.order_by('id') plus haut).
Mis à jour par Frédéric Péters il y a environ 4 ans
Nicolas, tu peux trier par défaut,
class Meta:
+ ordering = ('id',)
permissions = (
('view_resourcelog', 'Can view resource logs'),
)
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-tests-sort-ResourceLog-by-id-to-keep-order-42368.patch 0001-tests-sort-ResourceLog-by-id-to-keep-order-42368.patch ajouté
- Statut changé de En cours à Solution proposée
Merci pour le patch et pour l'explication, je n'avais pas trouvé !
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1626b133e8c877d5d9531cd5c595725dbd4a7d9f
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Sat May 2 12:31:57 2020 +0200
tests: sort ResourceLog by id to keep order (#42368)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Lié à Bug #50727: Rendre la queryset Job.objects.all() déterministe ajouté
tests: do not suppress ResourceLog to keep order (#42368)