Fixed #27175 -- Deprecated silencing exceptions from the {% include %} template tag.

Thanks Tim Graham for the review.

2 files changed, 30 insertions, 9 deletions

diff --git a/tests/template_tests/syntax_tests/test_include.py b/tests/template_tests/syntax_tests/test_include.py
index fc1554a17e..ebf3e8e7ab 100644
--- a/tests/template_tests/syntax_tests/test_include.py
+++ b/tests/template_tests/syntax_tests/test_include.py
@@ -1,7 +1,10 @@
+import warnings
+
 from django.template import (
     Context, Engine, TemplateDoesNotExist, TemplateSyntaxError,
 )
-from django.test import SimpleTestCase
+from django.test import SimpleTestCase, ignore_warnings
+from django.utils.deprecation import RemovedInDjango21Warning
 
 from ..utils import setup
 from .test_basic import basic_templates
@@ -41,9 +44,20 @@ class IncludeTagTests(SimpleTestCase):
             with self.assertRaises(TemplateDoesNotExist):
                 template.render(Context({}))
         else:
-            output = template.render(Context({}))
+            with warnings.catch_warnings(record=True) as warns:
+                warnings.simplefilter('always')
+                output = template.render(Context({}))
+
             self.assertEqual(output, "ab")
 
+            self.assertEqual(len(warns), 1)
+            self.assertEqual(
+                str(warns[0].message),
+                "Rendering {% include 'include04' %} raised "
+                "TemplateDoesNotExist. In Django 2.1, this exception will be "
+                "raised rather than silenced and rendered as an empty string.",
+            )
+
     @setup({
         'include 05': 'template with a space',
         'include06': '{% include "include 05"%}',
@@ -169,7 +183,8 @@ class IncludeTagTests(SimpleTestCase):
             with self.assertRaises(RuntimeError):
                 template.render(Context())
         else:
-            self.assertEqual(template.render(Context()), '')
+            with ignore_warnings(category=RemovedInDjango21Warning):
+                self.assertEqual(template.render(Context()), '')
 
     @setup({'include-error08': '{% include "include-fail2" %}'}, include_fail_templates)
     def test_include_error08(self):
@@ -179,7 +194,8 @@ class IncludeTagTests(SimpleTestCase):
             with self.assertRaises(TemplateSyntaxError):
                 template.render(Context())
         else:
-            self.assertEqual(template.render(Context()), '')
+            with ignore_warnings(category=RemovedInDjango21Warning):
+                self.assertEqual(template.render(Context()), '')
 
     @setup({'include-error09': '{% include failed_include %}'}, include_fail_templates)
     def test_include_error09(self):
@@ -190,7 +206,8 @@ class IncludeTagTests(SimpleTestCase):
             with self.assertRaises(RuntimeError):
                 template.render(context)
         else:
-            self.assertEqual(template.render(context), '')
+            with ignore_warnings(category=RemovedInDjango21Warning):
+                self.assertEqual(template.render(context), '')
 
     @setup({'include-error10': '{% include failed_include %}'}, include_fail_templates)
     def test_include_error10(self):
@@ -201,7 +218,8 @@ class IncludeTagTests(SimpleTestCase):
             with self.assertRaises(TemplateSyntaxError):
                 template.render(context)
         else:
-            self.assertEqual(template.render(context), '')
+            with ignore_warnings(category=RemovedInDjango21Warning):
+                self.assertEqual(template.render(context), '')
 
 
 class IncludeTests(SimpleTestCase):
diff --git a/tests/template_tests/test_logging.py b/tests/template_tests/test_logging.py
index c231fb28d6..a11af3c02d 100644
--- a/tests/template_tests/test_logging.py
+++ b/tests/template_tests/test_logging.py
@@ -3,7 +3,8 @@ from __future__ import unicode_literals
 import logging
 
 from django.template import Context, Engine, Variable, VariableDoesNotExist
-from django.test import SimpleTestCase
+from django.test import SimpleTestCase, ignore_warnings
+from django.utils.deprecation import RemovedInDjango21Warning
 
 
 class TestHandler(logging.Handler):
@@ -104,7 +105,8 @@ class IncludeNodeLoggingTests(BaseTemplateLoggingTestCase):
     def test_logs_exceptions_during_rendering_with_debug_disabled(self):
         template = self.engine.from_string('{% include "child" %}')
         template.name = 'template_name'
-        self.assertEqual(template.render(self.ctx), '')
+        with ignore_warnings(category=RemovedInDjango21Warning):
+            self.assertEqual(template.render(self.ctx), '')
         self.assertEqual(
             self.test_handler.log_record.getMessage(),
             "Exception raised while rendering {% include %} for template "
@@ -115,7 +117,8 @@ class IncludeNodeLoggingTests(BaseTemplateLoggingTestCase):
 
     def test_logs_exceptions_during_rendering_with_no_template_name(self):
         template = self.engine.from_string('{% include "child" %}')
-        self.assertEqual(template.render(self.ctx), '')
+        with ignore_warnings(category=RemovedInDjango21Warning):
+            self.assertEqual(template.render(self.ctx), '')
         self.assertEqual(
             self.test_handler.log_record.getMessage(),
             "Exception raised while rendering {% include %} for template "