Projet

Général

Profil

0009-reset-MUST_NOTIFY-on-request-start-29240.patch

Benjamin Dauvergne, 13 mars 2019 18:38

Télécharger (1,23 ko)

Voir les différences: 

Subject: [PATCH 9/9] reset MUST_NOTIFY on request start (#29240)

It breaks test tests_hobo/test_manager.py:test_unlogged_access which do
not expect any DB access but the flag seems to still be True from a
previous test.

 hobo/deploy/signals.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
hobo/deploy/signals.py
20 20 
from kombu.common import Broadcast
21 21 

  		




  22
  22
  
    
from django.conf import settings

  		




  23
  
  
    
from django.core.signals import request_finished

  		




  
  23
  
    
from django.core.signals import request_finished, request_started

  		




  24
  24
  
    
from django.db.models.signals import post_save

  		




  25
  25
  
    
from django.dispatch import receiver

  		




  26
  26
  
    

  		




  ......




  43
  43
  
    
    tls.MUST_NOTIFY = True

  		




  44
  44
  
    

  		




  45
  45
  
    

  		




  
  46
  
    
@receiver(request_started)

  		




  
  47
  
    
def reset_must_notify(sender, **kwargs):

  		




  
  48
  
    
    tls.MUST_NOTIFY = False

  		




  
  49
  
    

  		




  
  50
  
    

  		




  46
  51
  
    
@receiver(request_finished)

  		




  47
  52
  
    
def notify_agents(sender, **kwargs):

  		




  48
  53
  
    
    if not tls.MUST_NOTIFY:

  		




  49
  
  
    
-