Ensured that `makemessages` doesn't leave any temporary file over if the parsing of a template file fails. Refs #8536.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@17240 bcc190cf-cafb-0310-a4f2-bffc1f526a37

1 files changed, 5 insertions, 7 deletions

diff --git a/tests/regressiontests/i18n/commands/extraction.py b/tests/regressiontests/i18n/commands/extraction.py
index 6566ef4808..b1fc0025c7 100644
--- a/tests/regressiontests/i18n/commands/extraction.py
+++ b/tests/regressiontests/i18n/commands/extraction.py
@@ -108,16 +108,14 @@ class BasicExtractorTests(ExtractorTests):
         os.chdir(self.test_dir)
         shutil.copyfile('./templates/template_with_error.tpl', './templates/template_with_error.html')
         self.assertRaises(SyntaxError, management.call_command, 'makemessages', locale=LOCALE, verbosity=0)
-        try:
+        with self.assertRaises(SyntaxError) as context_manager:
             management.call_command('makemessages', locale=LOCALE, verbosity=0)
-        except SyntaxError, e:
-            self.assertRegexpMatches(
-                str(e),
+        self.assertRegexpMatches(str(context_manager.exception),
                 r'Translation blocks must not include other block tags: blocktrans \(file templates[/\\]template_with_error\.html, line 3\)'
             )
-        finally:
-            os.remove('./templates/template_with_error.html')
-            os.remove('./templates/template_with_error.html.py') # Waiting for #8536 to be fixed
+        os.remove('./templates/template_with_error.html')
+        # Check that the temporary file was cleaned up
+        self.assertFalse(os.path.exists('./templates/template_with_error.html.py'))
 
     def test_template_message_context_extractor(self):
         """