Fixing a bug in GRIT where messages with substitutions would

grit/clique_unittest.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 '''Unit tests for grit.clique'''
 
 import os
 import sys
 if __name__ == '__main__':
@@ skipping 171 matching lines @@
       pass  # expected case - 'Bingo bongo' does not start with 'jjj'
 
     message = tclib.Message(text='jjjBingo bongo')
     c = factory.MakeClique(message)
     c.SetCustomType(util.NewClassInstance(
       'grit.clique_unittest.DummyCustomType', clique.CustomType))
     translation = tclib.Translation(id=message.GetId(), text='Bilingo bolongo')
     c.AddTranslation(translation, 'fr')
     self.failUnless(c.MessageForLanguage('fr').GetRealContent().startswith('jjj'))
 
+  def testWhitespaceMessagesAreNontranslateable(self):
+    factory = clique.UberClique()
+
+    message = tclib.Message(text=' \t')
+    c = factory.MakeClique(message, translateable=True)
+    self.failIf(c.IsTranslateable())
+
+    message = tclib.Message(text='\n \n ')
+    c = factory.MakeClique(message, translateable=True)
+    self.failIf(c.IsTranslateable())
+
+    message = tclib.Message(text='\n hello')
+    c = factory.MakeClique(message, translateable=True)
+    self.failUnless(c.IsTranslateable())
+
 
 class DummyCustomType(clique.CustomType):
   def Validate(self, message):
     return message.GetRealContent().startswith('jjj')
   def ValidateAndModify(self, lang, translation):
     is_ok = self.Validate(translation)
     self.ModifyEachTextPart(lang, translation)
   def ModifyTextPart(self, lang, text):
     return 'jjj%s' % text
 
 
 if __name__ == '__main__':
   unittest.main()