Просмотр исходного кода

Force resolution of request User object when logging an object deletion (resolves intermittent test failures)

Jeremy Stretch 7 лет назад
Родитель
Сommit
6f5c35c278
1 измененных файлов с 5 добавлено и 1 удалено
  1. 5 1
      netbox/extras/middleware.py

+ 5 - 1
netbox/extras/middleware.py

@@ -29,7 +29,11 @@ def cache_changed_object(instance, **kwargs):
 
 
 def _record_object_deleted(request, instance, **kwargs):
 def _record_object_deleted(request, instance, **kwargs):
 
 
-    # Record that the object was deleted.
+    # Force resolution of request.user in case it's still a SimpleLazyObject. This seems to happen
+    # occasionally during tests, but haven't been able to determine why.
+    assert request.user.is_authenticated
+
+    # Record that the object was deleted
     if hasattr(instance, 'log_change'):
     if hasattr(instance, 'log_change'):
         instance.log_change(request.user, request.id, OBJECTCHANGE_ACTION_DELETE)
         instance.log_change(request.user, request.id, OBJECTCHANGE_ACTION_DELETE)