From 661a526109af425330f26224f9075e4df797b92d Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Fri, 8 Mar 2019 15:52:59 +0100 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(-) diff --git a/hobo/deploy/signals.py b/hobo/deploy/signals.py index b8507f2..a2ca1eb 100644 --- a/hobo/deploy/signals.py +++ b/hobo/deploy/signals.py @@ -20,7 +20,7 @@ from celery import Celery from kombu.common import Broadcast from django.conf import settings -from django.core.signals import request_finished +from django.core.signals import request_finished, request_started from django.db.models.signals import post_save from django.dispatch import receiver @@ -43,6 +43,11 @@ def post_environment_save(sender, instance, **kwargs): tls.MUST_NOTIFY = True +@receiver(request_started) +def reset_must_notify(sender, **kwargs): + tls.MUST_NOTIFY = False + + @receiver(request_finished) def notify_agents(sender, **kwargs): if not tls.MUST_NOTIFY: diff --git a/tests_schemas/example_env.json b/tests_schemas/example_env.json index ac97274..2048670 100644 --- a/tests_schemas/example_env.json +++ b/tests_schemas/example_env.json @@ -224,4 +224,4 @@ "theme": "publik", "theme_color": "#E80E89" } -} \ No newline at end of file +} -- 2.23.0