Improve TranslationTests to catch stray percent signs. (#6174)

Author: dbrant

app/src/main/res/values-ce/strings.xml

@@ -168,7 +168,7 @@
   <string name="create_account_button">Аккаунт кхолла</string>
   <string name="create_account_name_unavailable">Декъашхочун цӀе \"%s\" мегаш йац. Кхечу кепара цӀе йазйе.</string>
   <string name="create_account_ip_block_message">Бехк ма билла, хьан IP-адрес блоктоьхна ду керла аккаунт кхолларх.</string>
-  <string name="create_account_ip_block_help_url">https://ce.wikipedia.org/w/index.php?title=%D0%92%D0%B8%D0%BA%D0%B8%D0%BF%D0%B5%D0%B4%D0%B8:%D0%91%D0%BB%D0%BE%D0%BA%D1%82%D0%BE%D1%85%D0%B0%D1%80%D1%88&amp;action=edit&amp;redlink=1</string>
+  <string name="create_account_ip_block_help_url">https://ce.wikipedia.org/wiki/Википеди:Блоктохарш</string>
   <string name="create_account_ip_block_details">Мадарра</string>
   <string name="preferences_general_heading">Йукъара</string>
   <string name="preferences_account_heading">Аккаунт</string>

app/src/main/res/values-cs/strings.xml

@@ -479,9 +479,9 @@
   <string name="error_blocked_start">Začátek blokování: %s</string>
   <string name="error_blocked_expiry">Konec blokování: %s</string>
   <string name="error_blocked_id">ID blokování: %s</string>
-  <string name="error_blocked_footer">Můžete kontaktovat &lt;a href=\"%2$s\"&gt;%1$s&lt;/a&gt; nebo jiného &lt;a href=\"https://cs.wikipedia.org/wiki/Wikipedie%%3ASpr%%C3%%A1vci\"&gt;správce&lt;/a&gt; a prodiskutovat s ním blok.</string>
+  <string name="error_blocked_footer">Můžete kontaktovat &lt;a href=\"%2$s\"&gt;%1$s&lt;/a&gt; nebo jiného &lt;a href=\"https://cs.wikipedia.org/wiki/Wikipedie:Správci\"&gt;správce&lt;/a&gt; a prodiskutovat s ním blok.</string>
   <string name="theme_chooser_menu_item_short_tooltip">Přizpůsobte si panel nástrojů</string>
-  <string name="error_blocked_footer_no_blocker">Můžete kontaktovat &lt;a href=\"https://cs.wikipedia.org/wiki/Wikipedie%3ASpr%C3%A1vci\"&gt;správce&lt;/a&gt; a prodiskutovat s ním blok.</string>
+  <string name="error_blocked_footer_no_blocker">Můžete kontaktovat &lt;a href=\"https://cs.wikipedia.org/wiki/Wikipedie:Správci\"&gt;správce&lt;/a&gt; a prodiskutovat s ním blok.</string>
   <string name="temp_account_not_logged_in">Nejste přihlášen(a)</string>
   <string name="temp_account_anon_ip_dialog_body">Když provedete editaci, vaše IP adresa bude přiřazena k vašim editacím.&lt;br /&gt;&lt;br /&gt;&lt;a href=\"https://#login\"&gt;Přihlaste se&lt;/a&gt; nebo si &lt;a href=\"https://#createaccount\"&gt;vytvořte účet&lt;/a&gt;, abyste získal(a) ocenění za další příspěvky a dostal(a) přístup k dalším funkcím.</string>
   <string name="temp_account_anon_dialog_body">Jakmile provedete úpravu, bude pro vás vytvořen &lt;b&gt;dočasný účet,&lt;/b&gt; který bude chránit vaše soukromí. &lt;a href=\"%1$s\"&gt;Learn more&lt;/a&gt; .&lt;br /&gt;&lt;br /&gt; &lt;a href=\"https://#login\"&gt;Přihlaste se&lt;/a&gt; nebo &lt;a href=\"https://#createaccount\"&gt;si vytvořte účet,&lt;/a&gt; abyste získali uznání za budoucí úpravy a přístup k dalším funkcím.</string>

app/src/main/res/values-de/strings.xml

@@ -2226,7 +2226,7 @@
   <string name="donation_reminders_prompt_dismiss_snackbar">Spendenerinnerungen können jederzeit in den Einstellungen angepasst werden.</string>
   <string name="donation_settings_section_header">Spenden</string>
   <string name="donation_reminders_settings_title" fuzzy="true">Spendenerinnerungen</string>
-  <string name="donation_reminders_settings_thank_you_message" fuzzy="true">Schließe dich den 2 % der Leser an, die spenden, damit Wikipedia werbefrei, aktuell und für alle zugänglich bleibt. Richte jetzt deine Spendenerinnerung ein.</string>
+  <string name="donation_reminders_settings_thank_you_message">Schließe dich den 2 %% der Leser an, die spenden, damit Wikipedia werbefrei, aktuell und für alle zugänglich bleibt. Richte jetzt deine Spendenerinnerung ein.</string>
   <string name="donation_reminders_settings_donation_info" fuzzy="true">Die Spenden gehen an die Wikimedia Foundation und zugehörige Organisationen, die stolzen Betreiber von Wikipedia und ihren Schwesterseiten.</string>
   <string name="donation_reminders_settings_article_frequency_label">Wenn ich lese</string>
   <string name="donation_reminders_settings_amount_label">Erinnere mich daran, zu spenden</string>

app/src/main/res/values-hi/strings.xml

@@ -453,9 +453,9 @@
   <string name="error_blocked_start">प्रतिबंध की शुरुआत: %s</string>
   <string name="error_blocked_expiry">प्रतिबंध की समाप्ति: %s</string>
   <string name="error_blocked_id">ब्लॉक आईडी: %s</string>
-  <string name="error_blocked_footer">आप प्रतिबंध पर चर्चा करने के लिए &lt;a href=\"%2$s\"&gt;%1$s&lt;/a&gt; या फिर किसी दूसरे &lt;a href=\"https://hi.wikipedia.org/wiki/%E0%A4%B5%E0%A4%BF%E0%A4%95%E0%A4%BF%E0%A4%AA%E0%A5%80%E0%A4%A1%E0%A4%BF%E0%A4%AF%E0%A4%BE:%E0%A4%AA%E0%A5%8D%E0%A4%B0%E0%A4%AC%E0%A4%A8%E0%A5%8D%E0%A4%A7%E0%A4%95\"&gt;प्रबंधक&lt;/a&gt; से संपर्क कर सकते हैं।</string>
+  <string name="error_blocked_footer">आप प्रतिबंध पर चर्चा करने के लिए &lt;a href=\"%2$s\"&gt;%1$s&lt;/a&gt; या फिर किसी दूसरे &lt;a href=\"https://hi.wikipedia.org/wiki/विकिपीडिया:प्रबन्धक\"&gt;प्रबंधक&lt;/a&gt; से संपर्क कर सकते हैं।</string>
   <string name="theme_chooser_menu_item_short_tooltip">अपने टूलबार में परिवर्तन करें</string>
-  <string name="error_blocked_footer_no_blocker">प्रतिबंध पर चर्चा करने के लिए आप किसी &lt;a href=\"https://hi.wikipedia.org/wiki/%E0%A4%B5%E0%A4%BF%E0%A4%95%E0%A4%BF%E0%A4%AA%E0%A5%80%E0%A4%A1%E0%A4%BF%E0%A4%AF%E0%A4%BE:%E0%A4%AA%E0%A5%8D%E0%A4%B0%E0%A4%AC%E0%A4%A8%E0%A5%8D%E0%A4%A7%E0%A4%95\"&gt;प्रबन्धक&lt;/a&gt; से संपर्क कर सकते हैं।</string>
+  <string name="error_blocked_footer_no_blocker">प्रतिबंध पर चर्चा करने के लिए आप किसी &lt;a href=\"https://hi.wikipedia.org/wiki/विकिपीडिया:प्रबन्धक\"&gt;प्रबन्धक&lt;/a&gt; से संपर्क कर सकते हैं।</string>
   <string name="temp_account_not_logged_in">आप लॉग इन नहीं हैं</string>
   <string name="temp_account_anon_ip_dialog_body">जैसे ही आप कोई संपादन करते हैं, आपका आईपी पता आपके संपादनों से जुड़ जाएगा।&lt;br /&gt;&lt;br /&gt;&lt;a href=\"https://#login\"&gt;लॉग इन करें&lt;/a&gt; या &lt;a href=\"https://#createaccount\"&gt;खाता बनाएं&lt;/a&gt; ताकि भविष्य के संपादनों का श्रेय मिल सके और अन्य सुविधाओं तक पहुंच प्राप्त हो सके।</string>
   <string name="temp_account_anon_dialog_body">जैसे ही आप कोई संपादन करते हैं, आपकी गोपनीयता की सुरक्षा के लिए एक &lt;b&gt;अस्थायी खाता&lt;/b&gt; बनाया जाएगा। &lt;a href=\"%1$s\"&gt;अधिक जानें&lt;/a&gt;।&lt;br /&gt;&lt;br /&gt;&lt;a href=\"https://#login\"&gt;लॉग इन करें&lt;/a&gt; या &lt;a href=\"https://#createaccount\"&gt;खाता बनाएं&lt;/a&gt; ताकि भविष्य के संपादनों का श्रेय मिल सके और अन्य सुविधाओं तक पहुंच प्राप्त हो सके।</string>

app/src/main/res/values-zh/strings.xml

@@ -213,7 +213,7 @@
   <string name="search_reading_lists_no_results">在阅读列表中找不到结果</string>
   <string name="search_history_no_results">在历史中找不到结果</string>
   <string name="edit_section_captcha_message">为了防止自动化垃圾信息，请输入下方显示的文字</string>
-  <string name="edit_section_captcha_request_an_account_message">无法显示图片？请&lt;a href=\"https://zh.wikipedia.org/wiki/Wikipedia:%E8%B4%A6%E5%8F%B7%E8%AF%B7%E6%B1%82\"&gt;申请账号&lt;/a&gt;</string>
+  <string name="edit_section_captcha_request_an_account_message">无法显示图片？请&lt;a href=\"https://zh.wikipedia.org/wiki/Wikipedia:维基百科:账号请求\"&gt;申请账号&lt;/a&gt;</string>
   <string name="edit_section_captcha_hint">请输入上方的文字</string>
   <string name="title_captcha">输入验证码</string>
   <string name="edit_section_captcha_reload">点按验证码重新加载</string>

app/src/test/java/org/wikipedia/language/TranslationTests.kt

@@ -34,7 +34,7 @@ class TranslationTests {
             val lang =
                 if (dir.name.contains("-")) dir.name.substring(dir.name.indexOf("-") + 1) else "en"
             val targetStringsXml = File(dir, STRINGS_XML_NAME)
-            val targetMap = findMatchedParamsInXML(targetStringsXml, POSSIBLE_PARAMS, true)
+            val targetMap = findMatchedParamsInXML(targetStringsXml, POSSIBLE_PARAMS, quote = true, ignorePluralOne = true)
 
             // compare the counts inside the maps
             targetMap.forEach { (targetKey, targetList) ->
@@ -142,7 +142,7 @@ class TranslationTests {
             // Skip "qq" since it contains a lot of {{Identical}} tags
             if (lang != "qq") {
                 val targetStringsXml = File(dir, STRINGS_XML_NAME)
-                val targetMap = findMatchedParamsInXML(targetStringsXml, UNSUPPORTED_TEXTS_REGEX, false)
+                val targetMap = findMatchedParamsInXML(targetStringsXml, UNSUPPORTED_TEXTS_REGEX, quote = false)
 
                 // compare the counts inside the maps
                 targetMap.forEach { (targetKey, targetList) ->
@@ -209,7 +209,7 @@ class TranslationTests {
     }
 
     @Throws(Throwable::class)
-    private fun findMatchedParamsInXML(xmlPath: File, params: List<String>, quote: Boolean): Map<String, List<Int>> {
+    private fun findMatchedParamsInXML(xmlPath: File, params: List<String>, quote: Boolean, ignorePluralOne: Boolean = false): Map<String, List<Int>> {
         val map = mutableMapOf<String, List<Int>>()
         val document = Jsoup.parse(xmlPath, "UTF-8")
 
@@ -252,7 +252,7 @@ class TranslationTests {
             for (subElement in pluralElements) {
                 val subName = subElement.attr("quantity")
                 val subValue = subElement.text()
-                if (subName == "one") {
+                if (subName == "one" && ignorePluralOne) {
                     continue
                 }
 
@@ -291,7 +291,7 @@ class TranslationTests {
             "\\[\\[.*?\\]\\]",
             "\\*\\*.*?\\*\\*",
             "''.*?''",
-            "[^%]%[ .,;?][^d]"
+            "(?<!%)%(?!%|,d|\\d|d|s|f)"
         )
         private val BAD_NAMES = listOf("ldrtl", "sw360dp", "sw600dp", "sw720dp", "v19", "v21", "v23", "land", "night")