git cl upload for Gerit no-squash: correct detect missing Change-Id.

tests/git_footers_test.py

 #!/usr/bin/env python
 
 """Tests for git_footers."""
 
 import os
 import sys
 import unittest
 
 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
@@ skipping 55 matching lines @@
         git_footers.get_position(footers),
         ('refs/branch-heads/blabble', None))
 
   def testGetFooterChangeId(self):
     msg = '\n'.join(['whatever',
                      '',
                      'Change-Id: ignored',
                      '',  # Above is ignored because of this empty line.
                      'Change-Id: Ideadbeaf'])
     self.assertEqual(['Ideadbeaf'], git_footers.get_footer_change_id(msg))
+    self.assertEqual([], git_footers.get_footer_change_id(
+        'desc\nBUG=not-a-valid-footer\nChange-Id: Ixxx'))
+    self.assertEqual(['Ixxx'], git_footers.get_footer_change_id(
+        'desc\nBUG=not-a-valid-footer\n\nChange-Id: Ixxx'))
 
   def testAddFooterChangeId(self):
     self.assertEqual(
         git_footers.add_footer_change_id('header-only', 'Ixxx'),
         'header-only\n\nChange-Id: Ixxx')
 
     self.assertEqual(
         git_footers.add_footer_change_id('header\n\nsome: footter', 'Ixxx'),
         'header\n\nChange-Id: Ixxx\nsome: footter')
 
     self.assertEqual(
         git_footers.add_footer_change_id('header\n\nBUG: yy', 'Ixxx'),
         'header\n\nBUG: yy\nChange-Id: Ixxx')
 
     self.assertEqual(
         git_footers.add_footer_change_id('header\n\nBUG: yy\nPos: 1', 'Ixxx'),
         'header\n\nBUG: yy\nChange-Id: Ixxx\nPos: 1')
 
     self.assertEqual(
         git_footers.add_footer_change_id('header\n\nBUG: yy\n\nPos: 1', 'Ixxx'),
         'header\n\nBUG: yy\n\nChange-Id: Ixxx\nPos: 1')
 
 
 if __name__ == '__main__':
   unittest.main()