Merge lp:~dholbach/help-app/1435772 into lp:help-app

[Daniel Holbach]

add test to make sure we don't have markup tags in msgid of .po and .pot files

[lp:~dholbach/help-app/1435772]

122. By Daniel Holbach on 2015-03-24

remove code leftover

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

FAILED: Continuous integration, rev:122
http://91.189.93.70:8080/job/help-app-ci/6/
Executed test runs:
    FAILURE: http://91.189.93.70:8080/job/help-app-utopic-amd64-ci/6/console
    FAILURE: http://91.189.93.70:8080/job/help-app-vivid-amd64-ci/6/console

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/help-app-ci/6/rebuild

review: Needs Fixing (continuous-integration)

[Nicholas Skaggs (nskaggs)]

LGTM

review: Approve

=== modified file 'internals/tests/test_translations.py'
--- internals/tests/test_translations.py	2015-03-20 10:40:27 +0000
+++ internals/tests/test_translations.py	2015-03-24 12:45:22 +0000
@@ -1,6 +1,7 @@
 from unittest import TestCase
 
 from translations.build import Translations
+from pelicanconf import HIDE_FROM_POT
 
 
 class HelpTestCase(TestCase):
@@ -30,3 +31,14 @@
                     self.assertEqual(1, len(link_msgstr))
                     self.assertIn(po.langs[po_fn]['gettext_code'],
                                   link_msgstr[0])
+
+    def test_markup_in_po_or_pot_files(self):
+        po = self.translations.po
+        files = [ po.pot_file_ob ]
+        for po_fn in po.langs:
+            po.load_pofile(po_fn)
+            files += [ po.langs[po_fn]['pofile'] ]
+        for po_file in files:
+            for tag in HIDE_FROM_POT:
+                self.assertEqual(po_file.find_in_msgid(tag), [])
+