From 9c19aff7c7561e3a82978a272ecdaad40dda5c00 Mon Sep 17 00:00:00 2001 From: django-bot Date: Thu, 3 Feb 2022 20:24:19 +0100 Subject: Refs #33476 -- Reformatted code with Black. --- tests/i18n/test_extraction.py | 657 +++++++++++++++++++++++++----------------- 1 file changed, 395 insertions(+), 262 deletions(-) (limited to 'tests/i18n/test_extraction.py') diff --git a/tests/i18n/test_extraction.py b/tests/i18n/test_extraction.py index 98932fdcbb..c4aeef7339 100644 --- a/tests/i18n/test_extraction.py +++ b/tests/i18n/test_extraction.py @@ -13,9 +13,8 @@ from admin_scripts.tests import AdminScriptTestCase from django.core import management from django.core.management import execute_from_command_line from django.core.management.base import CommandError -from django.core.management.commands.makemessages import ( - Command as MakeMessagesCommand, write_pot_file, -) +from django.core.management.commands.makemessages import Command as MakeMessagesCommand +from django.core.management.commands.makemessages import write_pot_file from django.core.management.utils import find_command from django.test import SimpleTestCase, override_settings from django.test.utils import captured_stderr, captured_stdout @@ -24,22 +23,26 @@ from django.utils.translation import TranslatorCommentWarning from .utils import POFileAssertionMixin, RunInTmpDirMixin, copytree -LOCALE = 'de' -has_xgettext = find_command('xgettext') +LOCALE = "de" +has_xgettext = find_command("xgettext") gettext_version = MakeMessagesCommand().gettext_version if has_xgettext else None -requires_gettext_019 = skipIf(has_xgettext and gettext_version < (0, 19), 'gettext 0.19 required') +requires_gettext_019 = skipIf( + has_xgettext and gettext_version < (0, 19), "gettext 0.19 required" +) -@skipUnless(has_xgettext, 'xgettext is mandatory for extraction tests') +@skipUnless(has_xgettext, "xgettext is mandatory for extraction tests") class ExtractorTests(POFileAssertionMixin, RunInTmpDirMixin, SimpleTestCase): - work_subdir = 'commands' + work_subdir = "commands" - PO_FILE = 'locale/%s/LC_MESSAGES/django.po' % LOCALE + PO_FILE = "locale/%s/LC_MESSAGES/django.po" % LOCALE def _run_makemessages(self, **options): out = StringIO() - management.call_command('makemessages', locale=[LOCALE], verbosity=2, stdout=out, **options) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=2, stdout=out, **options + ) output = out.getvalue() self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: @@ -47,26 +50,30 @@ class ExtractorTests(POFileAssertionMixin, RunInTmpDirMixin, SimpleTestCase): return output, po_contents def assertMsgIdPlural(self, msgid, haystack, use_quotes=True): - return self._assertPoKeyword('msgid_plural', msgid, haystack, use_quotes=use_quotes) + return self._assertPoKeyword( + "msgid_plural", msgid, haystack, use_quotes=use_quotes + ) def assertMsgStr(self, msgstr, haystack, use_quotes=True): - return self._assertPoKeyword('msgstr', msgstr, haystack, use_quotes=use_quotes) + return self._assertPoKeyword("msgstr", msgstr, haystack, use_quotes=use_quotes) def assertNotMsgId(self, msgid, s, use_quotes=True): if use_quotes: msgid = '"%s"' % msgid msgid = re.escape(msgid) - return self.assertTrue(not re.search('^msgid %s' % msgid, s, re.MULTILINE)) + return self.assertTrue(not re.search("^msgid %s" % msgid, s, re.MULTILINE)) - def _assertPoLocComment(self, assert_presence, po_filename, line_number, *comment_parts): + def _assertPoLocComment( + self, assert_presence, po_filename, line_number, *comment_parts + ): with open(po_filename) as fp: po_contents = fp.read() - if os.name == 'nt': + if os.name == "nt": # #: .\path\to\file.html:123 - cwd_prefix = '%s%s' % (os.curdir, os.sep) + cwd_prefix = "%s%s" % (os.curdir, os.sep) else: # #: path/to/file.html:123 - cwd_prefix = '' + cwd_prefix = "" path = os.path.join(cwd_prefix, *comment_parts) parts = [path] @@ -74,21 +81,28 @@ class ExtractorTests(POFileAssertionMixin, RunInTmpDirMixin, SimpleTestCase): if isinstance(line_number, str): line_number = self._get_token_line_number(path, line_number) if line_number is not None: - parts.append(':%d' % line_number) + parts.append(":%d" % line_number) - needle = ''.join(parts) - pattern = re.compile(r'^\#\:.*' + re.escape(needle), re.MULTILINE) + needle = "".join(parts) + pattern = re.compile(r"^\#\:.*" + re.escape(needle), re.MULTILINE) if assert_presence: - return self.assertRegex(po_contents, pattern, '"%s" not found in final .po file.' % needle) + return self.assertRegex( + po_contents, pattern, '"%s" not found in final .po file.' % needle + ) else: - return self.assertNotRegex(po_contents, pattern, '"%s" shouldn\'t be in final .po file.' % needle) + return self.assertNotRegex( + po_contents, pattern, '"%s" shouldn\'t be in final .po file.' % needle + ) def _get_token_line_number(self, path, token): with open(path) as f: for line, content in enumerate(f, 1): if token in content: return line - self.fail("The token '%s' could not be found in %s, please check the test config" % (token, path)) + self.fail( + "The token '%s' could not be found in %s, please check the test config" + % (token, path) + ) def assertLocationCommentPresent(self, po_filename, line_number, *comment_parts): r""" @@ -128,123 +142,147 @@ class ExtractorTests(POFileAssertionMixin, RunInTmpDirMixin, SimpleTestCase): class BasicExtractorTests(ExtractorTests): - @override_settings(USE_I18N=False) def test_use_i18n_false(self): """ makemessages also runs successfully when USE_I18N is False. """ - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) - with open(self.PO_FILE, encoding='utf-8') as fp: + with open(self.PO_FILE, encoding="utf-8") as fp: po_contents = fp.read() # Check two random strings - self.assertIn('#. Translators: One-line translator comment #1', po_contents) + self.assertIn("#. Translators: One-line translator comment #1", po_contents) self.assertIn('msgctxt "Special trans context #1"', po_contents) def test_no_option(self): # One of either the --locale, --exclude, or --all options is required. msg = "Type 'manage.py help makemessages' for usage information." with mock.patch( - 'django.core.management.commands.makemessages.sys.argv', - ['manage.py', 'makemessages'], + "django.core.management.commands.makemessages.sys.argv", + ["manage.py", "makemessages"], ): with self.assertRaisesRegex(CommandError, msg): - management.call_command('makemessages') + management.call_command("makemessages") def test_valid_locale(self): out = StringIO() - management.call_command('makemessages', locale=['de'], stdout=out, verbosity=1) - self.assertNotIn('invalid locale de', out.getvalue()) - self.assertIn('processing locale de', out.getvalue()) + management.call_command("makemessages", locale=["de"], stdout=out, verbosity=1) + self.assertNotIn("invalid locale de", out.getvalue()) + self.assertIn("processing locale de", out.getvalue()) self.assertIs(Path(self.PO_FILE).exists(), True) def test_invalid_locale(self): out = StringIO() - management.call_command('makemessages', locale=['pl-PL'], stdout=out, verbosity=1) - self.assertIn('invalid locale pl-PL, did you mean pl_PL?', out.getvalue()) - self.assertNotIn('processing locale pl-PL', out.getvalue()) - self.assertIs(Path('locale/pl-PL/LC_MESSAGES/django.po').exists(), False) + management.call_command( + "makemessages", locale=["pl-PL"], stdout=out, verbosity=1 + ) + self.assertIn("invalid locale pl-PL, did you mean pl_PL?", out.getvalue()) + self.assertNotIn("processing locale pl-PL", out.getvalue()) + self.assertIs(Path("locale/pl-PL/LC_MESSAGES/django.po").exists(), False) def test_comments_extractor(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) - with open(self.PO_FILE, encoding='utf-8') as fp: + with open(self.PO_FILE, encoding="utf-8") as fp: po_contents = fp.read() - self.assertNotIn('This comment should not be extracted', po_contents) + self.assertNotIn("This comment should not be extracted", po_contents) # Comments in templates - self.assertIn('#. Translators: This comment should be extracted', po_contents) + self.assertIn( + "#. Translators: This comment should be extracted", po_contents + ) self.assertIn( "#. Translators: Django comment block for translators\n#. " "string's meaning unveiled", - po_contents + po_contents, + ) + self.assertIn("#. Translators: One-line translator comment #1", po_contents) + self.assertIn( + "#. Translators: Two-line translator comment #1\n#. continued here.", + po_contents, + ) + self.assertIn("#. Translators: One-line translator comment #2", po_contents) + self.assertIn( + "#. Translators: Two-line translator comment #2\n#. continued here.", + po_contents, + ) + self.assertIn("#. Translators: One-line translator comment #3", po_contents) + self.assertIn( + "#. Translators: Two-line translator comment #3\n#. continued here.", + po_contents, + ) + self.assertIn("#. Translators: One-line translator comment #4", po_contents) + self.assertIn( + "#. Translators: Two-line translator comment #4\n#. continued here.", + po_contents, ) - self.assertIn('#. Translators: One-line translator comment #1', po_contents) - self.assertIn('#. Translators: Two-line translator comment #1\n#. continued here.', po_contents) - self.assertIn('#. Translators: One-line translator comment #2', po_contents) - self.assertIn('#. Translators: Two-line translator comment #2\n#. continued here.', po_contents) - self.assertIn('#. Translators: One-line translator comment #3', po_contents) - self.assertIn('#. Translators: Two-line translator comment #3\n#. continued here.', po_contents) - self.assertIn('#. Translators: One-line translator comment #4', po_contents) - self.assertIn('#. Translators: Two-line translator comment #4\n#. continued here.', po_contents) self.assertIn( - '#. Translators: One-line translator comment #5 -- with ' - 'non ASCII characters: áéíóúö', - po_contents + "#. Translators: One-line translator comment #5 -- with " + "non ASCII characters: áéíóúö", + po_contents, ) self.assertIn( - '#. Translators: Two-line translator comment #5 -- with ' - 'non ASCII characters: áéíóúö\n#. continued here.', - po_contents + "#. Translators: Two-line translator comment #5 -- with " + "non ASCII characters: áéíóúö\n#. continued here.", + po_contents, ) def test_special_char_extracted(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) - with open(self.PO_FILE, encoding='utf-8') as fp: + with open(self.PO_FILE, encoding="utf-8") as fp: po_contents = fp.read() self.assertMsgId("Non-breaking space\u00a0:", po_contents) def test_blocktranslate_trimmed(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() # should not be trimmed - self.assertNotMsgId('Text with a few line breaks.', po_contents) + self.assertNotMsgId("Text with a few line breaks.", po_contents) # should be trimmed - self.assertMsgId("Again some text with a few line breaks, this time should be trimmed.", po_contents) + self.assertMsgId( + "Again some text with a few line breaks, this time should be trimmed.", + po_contents, + ) # #21406 -- Should adjust for eaten line numbers self.assertMsgId("Get my line number", po_contents) - self.assertLocationCommentPresent(self.PO_FILE, 'Get my line number', 'templates', 'test.html') + self.assertLocationCommentPresent( + self.PO_FILE, "Get my line number", "templates", "test.html" + ) def test_extraction_error(self): msg = ( - 'Translation blocks must not include other block tags: blocktranslate ' - '(file %s, line 3)' % os.path.join('templates', 'template_with_error.tpl') + "Translation blocks must not include other block tags: blocktranslate " + "(file %s, line 3)" % os.path.join("templates", "template_with_error.tpl") ) with self.assertRaisesMessage(SyntaxError, msg): - management.call_command('makemessages', locale=[LOCALE], extensions=['tpl'], verbosity=0) + management.call_command( + "makemessages", locale=[LOCALE], extensions=["tpl"], verbosity=0 + ) # The temporary files were cleaned up. - self.assertFalse(os.path.exists('./templates/template_with_error.tpl.py')) - self.assertFalse(os.path.exists('./templates/template_0_with_no_error.tpl.py')) + self.assertFalse(os.path.exists("./templates/template_with_error.tpl.py")) + self.assertFalse(os.path.exists("./templates/template_0_with_no_error.tpl.py")) def test_unicode_decode_error(self): - shutil.copyfile('./not_utf8.sample', './not_utf8.txt') + shutil.copyfile("./not_utf8.sample", "./not_utf8.txt") out = StringIO() - management.call_command('makemessages', locale=[LOCALE], stdout=out) - self.assertIn("UnicodeDecodeError: skipped file not_utf8.txt in .", out.getvalue()) + management.call_command("makemessages", locale=[LOCALE], stdout=out) + self.assertIn( + "UnicodeDecodeError: skipped file not_utf8.txt in .", out.getvalue() + ) def test_unicode_file_name(self): - open(os.path.join(self.test_dir, 'vidéo.txt'), 'a').close() - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + open(os.path.join(self.test_dir, "vidéo.txt"), "a").close() + management.call_command("makemessages", locale=[LOCALE], verbosity=0) def test_extraction_warning(self): """test xgettext warning about multiple bare interpolation placeholders""" - shutil.copyfile('./code.sample', './code_sample.py') + shutil.copyfile("./code.sample", "./code_sample.py") out = StringIO() - management.call_command('makemessages', locale=[LOCALE], stdout=out) + management.call_command("makemessages", locale=[LOCALE], stdout=out) self.assertIn("code_sample.py:4", out.getvalue()) def test_template_message_context_extractor(self): @@ -252,7 +290,7 @@ class BasicExtractorTests(ExtractorTests): Message contexts are correctly extracted for the {% translate %} and {% blocktranslate %} template tags (#14806). """ - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() @@ -265,9 +303,15 @@ class BasicExtractorTests(ExtractorTests): self.assertMsgId("Translatable literal #7c", po_contents) # {% translate %} with a filter - for minor_part in 'abcdefgh': # Iterate from #7.1a to #7.1h template markers - self.assertIn('msgctxt "context #7.1{}"'.format(minor_part), po_contents) - self.assertMsgId('Translatable literal #7.1{}'.format(minor_part), po_contents) + for ( + minor_part + ) in "abcdefgh": # Iterate from #7.1a to #7.1h template markers + self.assertIn( + 'msgctxt "context #7.1{}"'.format(minor_part), po_contents + ) + self.assertMsgId( + "Translatable literal #7.1{}".format(minor_part), po_contents + ) # {% blocktranslate %} self.assertIn('msgctxt "Special blocktranslate context #1"', po_contents) @@ -282,11 +326,11 @@ class BasicExtractorTests(ExtractorTests): self.assertMsgId("Translatable literal #8d %(a)s", po_contents) # {% trans %} and {% blocktrans %} - self.assertMsgId('trans text', po_contents) - self.assertMsgId('blocktrans text', po_contents) + self.assertMsgId("trans text", po_contents) + self.assertMsgId("blocktrans text", po_contents) def test_context_in_single_quotes(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() @@ -295,16 +339,24 @@ class BasicExtractorTests(ExtractorTests): self.assertIn('msgctxt "Context wrapped in single quotes"', po_contents) # {% blocktranslate %} - self.assertIn('msgctxt "Special blocktranslate context wrapped in double quotes"', po_contents) - self.assertIn('msgctxt "Special blocktranslate context wrapped in single quotes"', po_contents) + self.assertIn( + 'msgctxt "Special blocktranslate context wrapped in double quotes"', + po_contents, + ) + self.assertIn( + 'msgctxt "Special blocktranslate context wrapped in single quotes"', + po_contents, + ) def test_template_comments(self): """Template comment tags on the same line of other constructs (#19552)""" # Test detection/end user reporting of old, incorrect templates # translator comments syntax with warnings.catch_warnings(record=True) as ws: - warnings.simplefilter('always') - management.call_command('makemessages', locale=[LOCALE], extensions=['thtml'], verbosity=0) + warnings.simplefilter("always") + management.call_command( + "makemessages", locale=[LOCALE], extensions=["thtml"], verbosity=0 + ) self.assertEqual(len(ws), 3) for w in ws: self.assertTrue(issubclass(w.category, TranslatorCommentWarning)) @@ -312,55 +364,55 @@ class BasicExtractorTests(ExtractorTests): str(ws[0].message), r"The translator-targeted comment 'Translators: ignored i18n " r"comment #1' \(file templates[/\\]comments.thtml, line 4\) " - r"was ignored, because it wasn't the last item on the line\." + r"was ignored, because it wasn't the last item on the line\.", ) self.assertRegex( str(ws[1].message), r"The translator-targeted comment 'Translators: ignored i18n " r"comment #3' \(file templates[/\\]comments.thtml, line 6\) " - r"was ignored, because it wasn't the last item on the line\." + r"was ignored, because it wasn't the last item on the line\.", ) self.assertRegex( str(ws[2].message), r"The translator-targeted comment 'Translators: ignored i18n " r"comment #4' \(file templates[/\\]comments.thtml, line 8\) " - r"was ignored, because it wasn't the last item on the line\." + r"was ignored, because it wasn't the last item on the line\.", ) # Now test .po file contents self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() - self.assertMsgId('Translatable literal #9a', po_contents) - self.assertNotIn('ignored comment #1', po_contents) + self.assertMsgId("Translatable literal #9a", po_contents) + self.assertNotIn("ignored comment #1", po_contents) - self.assertNotIn('Translators: ignored i18n comment #1', po_contents) + self.assertNotIn("Translators: ignored i18n comment #1", po_contents) self.assertMsgId("Translatable literal #9b", po_contents) - self.assertNotIn('ignored i18n comment #2', po_contents) - self.assertNotIn('ignored comment #2', po_contents) - self.assertMsgId('Translatable literal #9c', po_contents) + self.assertNotIn("ignored i18n comment #2", po_contents) + self.assertNotIn("ignored comment #2", po_contents) + self.assertMsgId("Translatable literal #9c", po_contents) - self.assertNotIn('ignored comment #3', po_contents) - self.assertNotIn('ignored i18n comment #3', po_contents) - self.assertMsgId('Translatable literal #9d', po_contents) + self.assertNotIn("ignored comment #3", po_contents) + self.assertNotIn("ignored i18n comment #3", po_contents) + self.assertMsgId("Translatable literal #9d", po_contents) - self.assertNotIn('ignored comment #4', po_contents) - self.assertMsgId('Translatable literal #9e', po_contents) - self.assertNotIn('ignored comment #5', po_contents) + self.assertNotIn("ignored comment #4", po_contents) + self.assertMsgId("Translatable literal #9e", po_contents) + self.assertNotIn("ignored comment #5", po_contents) - self.assertNotIn('ignored i18n comment #4', po_contents) - self.assertMsgId('Translatable literal #9f', po_contents) - self.assertIn('#. Translators: valid i18n comment #5', po_contents) + self.assertNotIn("ignored i18n comment #4", po_contents) + self.assertMsgId("Translatable literal #9f", po_contents) + self.assertIn("#. Translators: valid i18n comment #5", po_contents) - self.assertMsgId('Translatable literal #9g', po_contents) - self.assertIn('#. Translators: valid i18n comment #6', po_contents) - self.assertMsgId('Translatable literal #9h', po_contents) - self.assertIn('#. Translators: valid i18n comment #7', po_contents) - self.assertMsgId('Translatable literal #9i', po_contents) + self.assertMsgId("Translatable literal #9g", po_contents) + self.assertIn("#. Translators: valid i18n comment #6", po_contents) + self.assertMsgId("Translatable literal #9h", po_contents) + self.assertIn("#. Translators: valid i18n comment #7", po_contents) + self.assertMsgId("Translatable literal #9i", po_contents) - self.assertRegex(po_contents, r'#\..+Translators: valid i18n comment #8') - self.assertRegex(po_contents, r'#\..+Translators: valid i18n comment #9') + self.assertRegex(po_contents, r"#\..+Translators: valid i18n comment #8") + self.assertRegex(po_contents, r"#\..+Translators: valid i18n comment #9") self.assertMsgId("Translatable literal #9j", po_contents) def test_makemessages_find_files(self): @@ -368,24 +420,24 @@ class BasicExtractorTests(ExtractorTests): find_files only discover files having the proper extensions. """ cmd = MakeMessagesCommand() - cmd.ignore_patterns = ['CVS', '.*', '*~', '*.pyc'] + cmd.ignore_patterns = ["CVS", ".*", "*~", "*.pyc"] cmd.symlinks = False - cmd.domain = 'django' - cmd.extensions = ['html', 'txt', 'py'] + cmd.domain = "django" + cmd.extensions = ["html", "txt", "py"] cmd.verbosity = 0 cmd.locale_paths = [] - cmd.default_locale_path = os.path.join(self.test_dir, 'locale') + cmd.default_locale_path = os.path.join(self.test_dir, "locale") found_files = cmd.find_files(self.test_dir) found_exts = {os.path.splitext(tfile.file)[1] for tfile in found_files} - self.assertEqual(found_exts.difference({'.py', '.html', '.txt'}), set()) + self.assertEqual(found_exts.difference({".py", ".html", ".txt"}), set()) - cmd.extensions = ['js'] - cmd.domain = 'djangojs' + cmd.extensions = ["js"] + cmd.domain = "djangojs" found_files = cmd.find_files(self.test_dir) found_exts = {os.path.splitext(tfile.file)[1] for tfile in found_files} - self.assertEqual(found_exts.difference({'.js'}), set()) + self.assertEqual(found_exts.difference({".js"}), set()) - @mock.patch('django.core.management.commands.makemessages.popen_wrapper') + @mock.patch("django.core.management.commands.makemessages.popen_wrapper") def test_makemessages_gettext_version(self, mocked_popen_wrapper): # "Normal" output: mocked_popen_wrapper.return_value = ( @@ -394,21 +446,28 @@ class BasicExtractorTests(ExtractorTests): "License GPLv3+: GNU GPL version 3 or later \n" "This is free software: you are free to change and redistribute it.\n" "There is NO WARRANTY, to the extent permitted by law.\n" - "Written by Ulrich Drepper.\n", '', 0) + "Written by Ulrich Drepper.\n", + "", + 0, + ) cmd = MakeMessagesCommand() self.assertEqual(cmd.gettext_version, (0, 18, 1)) # Version number with only 2 parts (#23788) mocked_popen_wrapper.return_value = ( - "xgettext (GNU gettext-tools) 0.17\n", '', 0) + "xgettext (GNU gettext-tools) 0.17\n", + "", + 0, + ) cmd = MakeMessagesCommand() self.assertEqual(cmd.gettext_version, (0, 17)) # Bad version output - mocked_popen_wrapper.return_value = ( - "any other return value\n", '', 0) + mocked_popen_wrapper.return_value = ("any other return value\n", "", 0) cmd = MakeMessagesCommand() - with self.assertRaisesMessage(CommandError, "Unable to get gettext version. Is it installed?"): + with self.assertRaisesMessage( + CommandError, "Unable to get gettext version. Is it installed?" + ): cmd.gettext_version def test_po_file_encoding_when_updating(self): @@ -416,50 +475,53 @@ class BasicExtractorTests(ExtractorTests): Update of PO file doesn't corrupt it with non-UTF-8 encoding on Windows (#23271). """ - BR_PO_BASE = 'locale/pt_BR/LC_MESSAGES/django' - shutil.copyfile(BR_PO_BASE + '.pristine', BR_PO_BASE + '.po') - management.call_command('makemessages', locale=['pt_BR'], verbosity=0) - self.assertTrue(os.path.exists(BR_PO_BASE + '.po')) - with open(BR_PO_BASE + '.po', encoding='utf-8') as fp: + BR_PO_BASE = "locale/pt_BR/LC_MESSAGES/django" + shutil.copyfile(BR_PO_BASE + ".pristine", BR_PO_BASE + ".po") + management.call_command("makemessages", locale=["pt_BR"], verbosity=0) + self.assertTrue(os.path.exists(BR_PO_BASE + ".po")) + with open(BR_PO_BASE + ".po", encoding="utf-8") as fp: po_contents = fp.read() self.assertMsgStr("Größe", po_contents) def test_pot_charset_header_is_utf8(self): """Content-Type: ... charset=CHARSET is replaced with charset=UTF-8""" msgs = ( - '# SOME DESCRIPTIVE TITLE.\n' - '# (some lines truncated as they are not relevant)\n' + "# SOME DESCRIPTIVE TITLE.\n" + "# (some lines truncated as they are not relevant)\n" '"Content-Type: text/plain; charset=CHARSET\\n"\n' '"Content-Transfer-Encoding: 8bit\\n"\n' - '\n' - '#: somefile.py:8\n' + "\n" + "#: somefile.py:8\n" 'msgid "mañana; charset=CHARSET"\n' 'msgstr ""\n' ) with tempfile.NamedTemporaryFile() as pot_file: pot_filename = pot_file.name write_pot_file(pot_filename, msgs) - with open(pot_filename, encoding='utf-8') as fp: + with open(pot_filename, encoding="utf-8") as fp: pot_contents = fp.read() - self.assertIn('Content-Type: text/plain; charset=UTF-8', pot_contents) - self.assertIn('mañana; charset=CHARSET', pot_contents) + self.assertIn("Content-Type: text/plain; charset=UTF-8", pot_contents) + self.assertIn("mañana; charset=CHARSET", pot_contents) class JavaScriptExtractorTests(ExtractorTests): - PO_FILE = 'locale/%s/LC_MESSAGES/djangojs.po' % LOCALE + PO_FILE = "locale/%s/LC_MESSAGES/djangojs.po" % LOCALE def test_javascript_literals(self): - _, po_contents = self._run_makemessages(domain='djangojs') - self.assertMsgId('This literal should be included.', po_contents) - self.assertMsgId('gettext_noop should, too.', po_contents) - self.assertMsgId('This one as well.', po_contents) - self.assertMsgId(r'He said, \"hello\".', po_contents) + _, po_contents = self._run_makemessages(domain="djangojs") + self.assertMsgId("This literal should be included.", po_contents) + self.assertMsgId("gettext_noop should, too.", po_contents) + self.assertMsgId("This one as well.", po_contents) + self.assertMsgId(r"He said, \"hello\".", po_contents) self.assertMsgId("okkkk", po_contents) self.assertMsgId("TEXT", po_contents) self.assertMsgId("It's at http://example.com", po_contents) self.assertMsgId("String", po_contents) - self.assertMsgId("/* but this one will be too */ 'cause there is no way of telling...", po_contents) + self.assertMsgId( + "/* but this one will be too */ 'cause there is no way of telling...", + po_contents, + ) self.assertMsgId("foo", po_contents) self.assertMsgId("bar", po_contents) self.assertMsgId("baz", po_contents) @@ -470,91 +532,115 @@ class JavaScriptExtractorTests(ExtractorTests): """ Regression test for #23583. """ - with override_settings(STATIC_ROOT=os.path.join(self.test_dir, 'static/'), - MEDIA_ROOT=os.path.join(self.test_dir, 'media_root/')): - _, po_contents = self._run_makemessages(domain='djangojs') - self.assertMsgId("Static content inside app should be included.", po_contents) - self.assertNotMsgId("Content from STATIC_ROOT should not be included", po_contents) + with override_settings( + STATIC_ROOT=os.path.join(self.test_dir, "static/"), + MEDIA_ROOT=os.path.join(self.test_dir, "media_root/"), + ): + _, po_contents = self._run_makemessages(domain="djangojs") + self.assertMsgId( + "Static content inside app should be included.", po_contents + ) + self.assertNotMsgId( + "Content from STATIC_ROOT should not be included", po_contents + ) - @override_settings(STATIC_ROOT=None, MEDIA_ROOT='') + @override_settings(STATIC_ROOT=None, MEDIA_ROOT="") def test_default_root_settings(self): """ Regression test for #23717. """ - _, po_contents = self._run_makemessages(domain='djangojs') + _, po_contents = self._run_makemessages(domain="djangojs") self.assertMsgId("Static content inside app should be included.", po_contents) class IgnoredExtractorTests(ExtractorTests): - def test_ignore_directory(self): - out, po_contents = self._run_makemessages(ignore_patterns=[ - os.path.join('ignore_dir', '*'), - ]) + out, po_contents = self._run_makemessages( + ignore_patterns=[ + os.path.join("ignore_dir", "*"), + ] + ) self.assertIn("ignoring directory ignore_dir", out) - self.assertMsgId('This literal should be included.', po_contents) - self.assertNotMsgId('This should be ignored.', po_contents) + self.assertMsgId("This literal should be included.", po_contents) + self.assertNotMsgId("This should be ignored.", po_contents) def test_ignore_subdirectory(self): - out, po_contents = self._run_makemessages(ignore_patterns=[ - 'templates/*/ignore.html', - 'templates/subdir/*', - ]) + out, po_contents = self._run_makemessages( + ignore_patterns=[ + "templates/*/ignore.html", + "templates/subdir/*", + ] + ) self.assertIn("ignoring directory subdir", out) - self.assertNotMsgId('This subdir should be ignored too.', po_contents) + self.assertNotMsgId("This subdir should be ignored too.", po_contents) def test_ignore_file_patterns(self): - out, po_contents = self._run_makemessages(ignore_patterns=[ - 'xxx_*', - ]) + out, po_contents = self._run_makemessages( + ignore_patterns=[ + "xxx_*", + ] + ) self.assertIn("ignoring file xxx_ignored.html", out) - self.assertNotMsgId('This should be ignored too.', po_contents) + self.assertNotMsgId("This should be ignored too.", po_contents) def test_media_static_dirs_ignored(self): - with override_settings(STATIC_ROOT=os.path.join(self.test_dir, 'static/'), - MEDIA_ROOT=os.path.join(self.test_dir, 'media_root/')): + with override_settings( + STATIC_ROOT=os.path.join(self.test_dir, "static/"), + MEDIA_ROOT=os.path.join(self.test_dir, "media_root/"), + ): out, _ = self._run_makemessages() self.assertIn("ignoring directory static", out) self.assertIn("ignoring directory media_root", out) class SymlinkExtractorTests(ExtractorTests): - def setUp(self): super().setUp() - self.symlinked_dir = os.path.join(self.test_dir, 'templates_symlinked') + self.symlinked_dir = os.path.join(self.test_dir, "templates_symlinked") def test_symlink(self): if symlinks_supported(): - os.symlink(os.path.join(self.test_dir, 'templates'), self.symlinked_dir) + os.symlink(os.path.join(self.test_dir, "templates"), self.symlinked_dir) else: - self.skipTest("os.symlink() not available on this OS + Python version combination.") - management.call_command('makemessages', locale=[LOCALE], verbosity=0, symlinks=True) + self.skipTest( + "os.symlink() not available on this OS + Python version combination." + ) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, symlinks=True + ) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() - self.assertMsgId('This literal should be included.', po_contents) - self.assertLocationCommentPresent(self.PO_FILE, None, 'templates_symlinked', 'test.html') + self.assertMsgId("This literal should be included.", po_contents) + self.assertLocationCommentPresent( + self.PO_FILE, None, "templates_symlinked", "test.html" + ) class CopyPluralFormsExtractorTests(ExtractorTests): - PO_FILE_ES = 'locale/es/LC_MESSAGES/django.po' + PO_FILE_ES = "locale/es/LC_MESSAGES/django.po" def test_copy_plural_forms(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() - self.assertIn('Plural-Forms: nplurals=2; plural=(n != 1)', po_contents) + self.assertIn("Plural-Forms: nplurals=2; plural=(n != 1)", po_contents) def test_override_plural_forms(self): """Ticket #20311.""" - management.call_command('makemessages', locale=['es'], extensions=['djtpl'], verbosity=0) + management.call_command( + "makemessages", locale=["es"], extensions=["djtpl"], verbosity=0 + ) self.assertTrue(os.path.exists(self.PO_FILE_ES)) - with open(self.PO_FILE_ES, encoding='utf-8') as fp: + with open(self.PO_FILE_ES, encoding="utf-8") as fp: po_contents = fp.read() - found = re.findall(r'^(?P"Plural-Forms.+?\\n")\s*$', po_contents, re.MULTILINE | re.DOTALL) + found = re.findall( + r'^(?P"Plural-Forms.+?\\n")\s*$', + po_contents, + re.MULTILINE | re.DOTALL, + ) self.assertEqual(1, len(found)) def test_translate_and_plural_blocktranslate_collision(self): @@ -563,30 +649,42 @@ class CopyPluralFormsExtractorTests(ExtractorTests): found inside a {% translate %} tag and also in another file inside a {% blocktranslate %} with a plural (#17375). """ - management.call_command('makemessages', locale=[LOCALE], extensions=['html', 'djtpl'], verbosity=0) + management.call_command( + "makemessages", locale=[LOCALE], extensions=["html", "djtpl"], verbosity=0 + ) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() - self.assertNotIn("#-#-#-#-# django.pot (PACKAGE VERSION) #-#-#-#-#\\n", po_contents) - self.assertMsgId('First `translate`, then `blocktranslate` with a plural', po_contents) - self.assertMsgIdPlural('Plural for a `translate` and `blocktranslate` collision case', po_contents) + self.assertNotIn( + "#-#-#-#-# django.pot (PACKAGE VERSION) #-#-#-#-#\\n", po_contents + ) + self.assertMsgId( + "First `translate`, then `blocktranslate` with a plural", po_contents + ) + self.assertMsgIdPlural( + "Plural for a `translate` and `blocktranslate` collision case", + po_contents, + ) class NoWrapExtractorTests(ExtractorTests): - def test_no_wrap_enabled(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0, no_wrap=True) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, no_wrap=True + ) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() self.assertMsgId( - 'This literal should also be included wrapped or not wrapped ' - 'depending on the use of the --no-wrap option.', - po_contents + "This literal should also be included wrapped or not wrapped " + "depending on the use of the --no-wrap option.", + po_contents, ) def test_no_wrap_disabled(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0, no_wrap=False) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, no_wrap=False + ) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() @@ -594,64 +692,82 @@ class NoWrapExtractorTests(ExtractorTests): '""\n"This literal should also be included wrapped or not ' 'wrapped depending on the "\n"use of the --no-wrap option."', po_contents, - use_quotes=False + use_quotes=False, ) class LocationCommentsTests(ExtractorTests): - def test_no_location_enabled(self): """Behavior is correct if --no-location switch is specified. See #16903.""" - management.call_command('makemessages', locale=[LOCALE], verbosity=0, no_location=True) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, no_location=True + ) self.assertTrue(os.path.exists(self.PO_FILE)) - self.assertLocationCommentNotPresent(self.PO_FILE, None, 'test.html') + self.assertLocationCommentNotPresent(self.PO_FILE, None, "test.html") def test_no_location_disabled(self): """Behavior is correct if --no-location switch isn't specified.""" - management.call_command('makemessages', locale=[LOCALE], verbosity=0, no_location=False) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, no_location=False + ) self.assertTrue(os.path.exists(self.PO_FILE)) # #16903 -- Standard comment with source file relative path should be present - self.assertLocationCommentPresent(self.PO_FILE, 'Translatable literal #6b', 'templates', 'test.html') + self.assertLocationCommentPresent( + self.PO_FILE, "Translatable literal #6b", "templates", "test.html" + ) def test_location_comments_for_templatized_files(self): """ Ensure no leaky paths in comments, e.g. #: path\to\file.html.py:123 Refs #21209/#26341. """ - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE)) with open(self.PO_FILE) as fp: po_contents = fp.read() - self.assertMsgId('#: templates/test.html.py', po_contents) - self.assertLocationCommentNotPresent(self.PO_FILE, None, '.html.py') - self.assertLocationCommentPresent(self.PO_FILE, 5, 'templates', 'test.html') + self.assertMsgId("#: templates/test.html.py", po_contents) + self.assertLocationCommentNotPresent(self.PO_FILE, None, ".html.py") + self.assertLocationCommentPresent(self.PO_FILE, 5, "templates", "test.html") @requires_gettext_019 def test_add_location_full(self): """makemessages --add-location=full""" - management.call_command('makemessages', locale=[LOCALE], verbosity=0, add_location='full') + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, add_location="full" + ) self.assertTrue(os.path.exists(self.PO_FILE)) # Comment with source file relative path and line number is present. - self.assertLocationCommentPresent(self.PO_FILE, 'Translatable literal #6b', 'templates', 'test.html') + self.assertLocationCommentPresent( + self.PO_FILE, "Translatable literal #6b", "templates", "test.html" + ) @requires_gettext_019 def test_add_location_file(self): """makemessages --add-location=file""" - management.call_command('makemessages', locale=[LOCALE], verbosity=0, add_location='file') + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, add_location="file" + ) self.assertTrue(os.path.exists(self.PO_FILE)) # Comment with source file relative path is present. - self.assertLocationCommentPresent(self.PO_FILE, None, 'templates', 'test.html') + self.assertLocationCommentPresent(self.PO_FILE, None, "templates", "test.html") # But it should not contain the line number. - self.assertLocationCommentNotPresent(self.PO_FILE, 'Translatable literal #6b', 'templates', 'test.html') + self.assertLocationCommentNotPresent( + self.PO_FILE, "Translatable literal #6b", "templates", "test.html" + ) @requires_gettext_019 def test_add_location_never(self): """makemessages --add-location=never""" - management.call_command('makemessages', locale=[LOCALE], verbosity=0, add_location='never') + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, add_location="never" + ) self.assertTrue(os.path.exists(self.PO_FILE)) - self.assertLocationCommentNotPresent(self.PO_FILE, None, 'test.html') + self.assertLocationCommentNotPresent(self.PO_FILE, None, "test.html") - @mock.patch('django.core.management.commands.makemessages.Command.gettext_version', new=(0, 18, 99)) + @mock.patch( + "django.core.management.commands.makemessages.Command.gettext_version", + new=(0, 18, 99), + ) def test_add_location_gettext_version_check(self): """ CommandError is raised when using makemessages --add-location with @@ -659,34 +775,40 @@ class LocationCommentsTests(ExtractorTests): """ msg = "The --add-location option requires gettext 0.19 or later. You have 0.18.99." with self.assertRaisesMessage(CommandError, msg): - management.call_command('makemessages', locale=[LOCALE], verbosity=0, add_location='full') + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, add_location="full" + ) class KeepPotFileExtractorTests(ExtractorTests): - POT_FILE = 'locale/django.pot' + POT_FILE = "locale/django.pot" def test_keep_pot_disabled_by_default(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) self.assertFalse(os.path.exists(self.POT_FILE)) def test_keep_pot_explicitly_disabled(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0, keep_pot=False) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, keep_pot=False + ) self.assertFalse(os.path.exists(self.POT_FILE)) def test_keep_pot_enabled(self): - management.call_command('makemessages', locale=[LOCALE], verbosity=0, keep_pot=True) + management.call_command( + "makemessages", locale=[LOCALE], verbosity=0, keep_pot=True + ) self.assertTrue(os.path.exists(self.POT_FILE)) class MultipleLocaleExtractionTests(ExtractorTests): - PO_FILE_PT = 'locale/pt/LC_MESSAGES/django.po' - PO_FILE_DE = 'locale/de/LC_MESSAGES/django.po' - PO_FILE_KO = 'locale/ko/LC_MESSAGES/django.po' - LOCALES = ['pt', 'de', 'ch'] + PO_FILE_PT = "locale/pt/LC_MESSAGES/django.po" + PO_FILE_DE = "locale/de/LC_MESSAGES/django.po" + PO_FILE_KO = "locale/ko/LC_MESSAGES/django.po" + LOCALES = ["pt", "de", "ch"] def test_multiple_locales(self): - management.call_command('makemessages', locale=['pt', 'de'], verbosity=0) + management.call_command("makemessages", locale=["pt", "de"], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE_PT)) self.assertTrue(os.path.exists(self.PO_FILE_DE)) @@ -696,19 +818,19 @@ class MultipleLocaleExtractionTests(ExtractorTests): are considered as language directories, except if the directory doesn't start with two letters (which excludes __pycache__, .gitignore, etc.). """ - os.mkdir(os.path.join('locale', '_do_not_pick')) + os.mkdir(os.path.join("locale", "_do_not_pick")) # Excluding locales that do not compile - management.call_command('makemessages', exclude=['ja', 'es_AR'], verbosity=0) + management.call_command("makemessages", exclude=["ja", "es_AR"], verbosity=0) self.assertTrue(os.path.exists(self.PO_FILE_KO)) - self.assertFalse(os.path.exists('locale/_do_not_pick/LC_MESSAGES/django.po')) + self.assertFalse(os.path.exists("locale/_do_not_pick/LC_MESSAGES/django.po")) class ExcludedLocaleExtractionTests(ExtractorTests): - work_subdir = 'exclude' + work_subdir = "exclude" - LOCALES = ['en', 'fr', 'it'] - PO_FILE = 'locale/%s/LC_MESSAGES/django.po' + LOCALES = ["en", "fr", "it"] + PO_FILE = "locale/%s/LC_MESSAGES/django.po" def _set_times_for_all_po_files(self): """ @@ -719,7 +841,7 @@ class ExcludedLocaleExtractionTests(ExtractorTests): def setUp(self): super().setUp() - copytree('canned_locale', 'locale') + copytree("canned_locale", "locale") self._set_times_for_all_po_files() def test_command_help(self): @@ -727,36 +849,40 @@ class ExcludedLocaleExtractionTests(ExtractorTests): # `call_command` bypasses the parser; by calling # `execute_from_command_line` with the help subcommand we # ensure that there are no issues with the parser itself. - execute_from_command_line(['django-admin', 'help', 'makemessages']) + execute_from_command_line(["django-admin", "help", "makemessages"]) def test_one_locale_excluded(self): - management.call_command('makemessages', exclude=['it'], verbosity=0) - self.assertRecentlyModified(self.PO_FILE % 'en') - self.assertRecentlyModified(self.PO_FILE % 'fr') - self.assertNotRecentlyModified(self.PO_FILE % 'it') + management.call_command("makemessages", exclude=["it"], verbosity=0) + self.assertRecentlyModified(self.PO_FILE % "en") + self.assertRecentlyModified(self.PO_FILE % "fr") + self.assertNotRecentlyModified(self.PO_FILE % "it") def test_multiple_locales_excluded(self): - management.call_command('makemessages', exclude=['it', 'fr'], verbosity=0) - self.assertRecentlyModified(self.PO_FILE % 'en') - self.assertNotRecentlyModified(self.PO_FILE % 'fr') - self.assertNotRecentlyModified(self.PO_FILE % 'it') + management.call_command("makemessages", exclude=["it", "fr"], verbosity=0) + self.assertRecentlyModified(self.PO_FILE % "en") + self.assertNotRecentlyModified(self.PO_FILE % "fr") + self.assertNotRecentlyModified(self.PO_FILE % "it") def test_one_locale_excluded_with_locale(self): - management.call_command('makemessages', locale=['en', 'fr'], exclude=['fr'], verbosity=0) - self.assertRecentlyModified(self.PO_FILE % 'en') - self.assertNotRecentlyModified(self.PO_FILE % 'fr') - self.assertNotRecentlyModified(self.PO_FILE % 'it') + management.call_command( + "makemessages", locale=["en", "fr"], exclude=["fr"], verbosity=0 + ) + self.assertRecentlyModified(self.PO_FILE % "en") + self.assertNotRecentlyModified(self.PO_FILE % "fr") + self.assertNotRecentlyModified(self.PO_FILE % "it") def test_multiple_locales_excluded_with_locale(self): - management.call_command('makemessages', locale=['en', 'fr', 'it'], exclude=['fr', 'it'], verbosity=0) - self.assertRecentlyModified(self.PO_FILE % 'en') - self.assertNotRecentlyModified(self.PO_FILE % 'fr') - self.assertNotRecentlyModified(self.PO_FILE % 'it') + management.call_command( + "makemessages", locale=["en", "fr", "it"], exclude=["fr", "it"], verbosity=0 + ) + self.assertRecentlyModified(self.PO_FILE % "en") + self.assertNotRecentlyModified(self.PO_FILE % "fr") + self.assertNotRecentlyModified(self.PO_FILE % "it") class CustomLayoutExtractionTests(ExtractorTests): - work_subdir = 'project_dir' + work_subdir = "project_dir" def test_no_locale_raises(self): msg = ( @@ -765,15 +891,15 @@ class CustomLayoutExtractionTests(ExtractorTests): "or LOCALE_PATHS setting is set." ) with self.assertRaisesMessage(management.CommandError, msg): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) # Working files are cleaned up on an error. - self.assertFalse(os.path.exists('./app_no_locale/test.html.py')) + self.assertFalse(os.path.exists("./app_no_locale/test.html.py")) def test_project_locale_paths(self): - self._test_project_locale_paths(os.path.join(self.test_dir, 'project_locale')) + self._test_project_locale_paths(os.path.join(self.test_dir, "project_locale")) def test_project_locale_paths_pathlib(self): - self._test_project_locale_paths(Path(self.test_dir) / 'project_locale') + self._test_project_locale_paths(Path(self.test_dir) / "project_locale") def _test_project_locale_paths(self, locale_path): """ @@ -781,43 +907,50 @@ class CustomLayoutExtractionTests(ExtractorTests): * translations outside of that app are in LOCALE_PATHS[0] """ with override_settings(LOCALE_PATHS=[locale_path]): - management.call_command('makemessages', locale=[LOCALE], verbosity=0) + management.call_command("makemessages", locale=[LOCALE], verbosity=0) project_de_locale = os.path.join( - self.test_dir, 'project_locale', 'de', 'LC_MESSAGES', 'django.po') + self.test_dir, "project_locale", "de", "LC_MESSAGES", "django.po" + ) app_de_locale = os.path.join( - self.test_dir, 'app_with_locale', 'locale', 'de', 'LC_MESSAGES', 'django.po') + self.test_dir, + "app_with_locale", + "locale", + "de", + "LC_MESSAGES", + "django.po", + ) self.assertTrue(os.path.exists(project_de_locale)) self.assertTrue(os.path.exists(app_de_locale)) with open(project_de_locale) as fp: po_contents = fp.read() - self.assertMsgId('This app has no locale directory', po_contents) - self.assertMsgId('This is a project-level string', po_contents) + self.assertMsgId("This app has no locale directory", po_contents) + self.assertMsgId("This is a project-level string", po_contents) with open(app_de_locale) as fp: po_contents = fp.read() - self.assertMsgId('This app has a locale directory', po_contents) + self.assertMsgId("This app has a locale directory", po_contents) -@skipUnless(has_xgettext, 'xgettext is mandatory for extraction tests') +@skipUnless(has_xgettext, "xgettext is mandatory for extraction tests") class NoSettingsExtractionTests(AdminScriptTestCase): def test_makemessages_no_settings(self): - out, err = self.run_django_admin(['makemessages', '-l', 'en', '-v', '0']) + out, err = self.run_django_admin(["makemessages", "-l", "en", "-v", "0"]) self.assertNoOutput(err) self.assertNoOutput(out) class UnchangedPoExtractionTests(ExtractorTests): - work_subdir = 'unchanged' + work_subdir = "unchanged" def setUp(self): super().setUp() po_file = Path(self.PO_FILE) - po_file_tmp = Path(self.PO_FILE + '.tmp') - if os.name == 'nt': + po_file_tmp = Path(self.PO_FILE + ".tmp") + if os.name == "nt": # msgmerge outputs Windows style paths on Windows. po_contents = po_file_tmp.read_text().replace( - '#: __init__.py', - '#: .\\__init__.py', + "#: __init__.py", + "#: .\\__init__.py", ) po_file.write_text(po_contents) else: @@ -831,10 +964,10 @@ class UnchangedPoExtractionTests(ExtractorTests): def test_po_changed_with_new_strings(self): """PO files are updated when new changes are detected.""" - Path('models.py.tmp').rename('models.py') + Path("models.py.tmp").rename("models.py") _, po_contents = self._run_makemessages() self.assertNotEqual(po_contents, self.original_po_contents) self.assertMsgId( - 'This is a hitherto undiscovered translatable string.', + "This is a hitherto undiscovered translatable string.", po_contents, ) -- cgit v1.2.1