Don’t check settings until it’s been configured

Author: Pamela McA'Nulty

django_coverage_plugin/plugin.py

@@ -43,6 +43,7 @@ def check_debug():
     to do its work.  Check that the setting is correct, and raise an exception
     if it is not.
 
+    Returns True if the debug check was performed, False otherwise
     """
     # The settings for templates changed in Django 1.8 from TEMPLATE_DEBUG to
     # TEMPLATES[..]['debug'].  Django 1.9 tolerated both forms, 1.10 insists on
@@ -51,14 +52,17 @@ def check_debug():
 
     from django.conf import settings
 
+    if not settings.configured:
+        return False
+
     try:
         templates = getattr(settings, 'TEMPLATES', [])
     except ImproperlyConfigured:
         # Maybe there are no settings at all.  We are fine with this.  Our
         # code will need settings, but if this program we're in runs without
         # settings, then it must be that it never uses templates, and our code
         # will never try to use the settings anyway.
-        return
+        return True
 
     if templates:
         for template_settings in templates:
@@ -75,6 +79,8 @@ def check_debug():
                 "Template debugging must be enabled in settings."
             )
 
+    return True
+
 
 if django.VERSION >= (1, 9):
     # Since we are grabbing at internal details, we have to adapt as they
@@ -151,8 +157,7 @@ def sys_info(self):
     def file_tracer(self, filename):
         if filename.startswith(self.django_template_dir):
             if not self.debug_checked:
-                check_debug()
-                self.debug_checked = True
+                self.debug_checked = check_debug()
 
             return self
         return None