Projet

Général

Profil

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

Benjamin Dauvergne, 07 juin 2019 22:20

Télécharger (1,58 ko)

Voir les différences: 

Subject: [PATCH 10/10] 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 ++++++-
 tests_schemas/example_env.json | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)
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:

  		












  

    
      tests_schemas/example_env.json
    
  





  224
  224
  
    
        "theme": "publik",

  		




  225
  225
  
    
        "theme_color": "#E80E89"

  		




  226
  226
  
    
    }

  		




  227
  
  
    
}

  		




  
  227
  
    
}

  		




  228
  
  
    
-