Project

General

Profile

Bug #83710

Instabilité du test de ratelimiting des vues provoquant l’envoi de SMS

Added by Paul Marillonnet 11 days ago. Updated about 17 hours ago.

Status:
Résolu (à déployer)
Priority:
Normal
Category:
-
Target version:
-
Start date:
20 November 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

tests/test_views.py::test_views_sms_ratelimit qui casse de façon assez aléatoire, notamment dans le build principal.

Associated revisions

Revision 9ee9d0ff (diff)
Added by Benjamin Dauvergne about 17 hours ago

tests: use deterministic suffixes (#83710)

History

#1

Updated by Paul Marillonnet 11 days ago

  • Status changed from Nouveau to En cours
  • Assignee set to Paul Marillonnet
#2

Updated by Benjamin Dauvergne 11 days ago

Ça arrive quand le build prend plus longtemps que la normale il me semble (ralentissement sur la machine jenkins), j'aurai pensé à une histoire de dépassement de durée mais on utilise freezer, donc ça ne vient pas du code python lui même, pour lui le temps est figé.

Un cas : https://jenkins.entrouvert.org/job/gitea/job/authentic/job/main/210/testReport/junit/py3-oldstable.tests/test_views/test_views_sms_ratelimit_phone_change_/

Une autre possibilité serait que le comportement change quand on a des collisions sur les numéros de téléphone qui sont supposément random mais un doublon peut quand même arriver, on pourrait remplacer randint(0,1000), par numbers = range(0, 1000);... next(numbers)

#4

Updated by Robot Gitea about 20 hours ago

  • Assignee changed from Paul Marillonnet to Benjamin Dauvergne

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea about 17 hours ago

  • Status changed from En cours to Solution proposée
#6

Updated by Robot Gitea about 17 hours ago

  • Status changed from Solution proposée to Solution validée

Paul Marillonnet (pmarillonnet) a approuvé une pull request sur Gitea concernant cette demande :

#7

Updated by Robot Gitea about 17 hours ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

Also available in: Atom PDF