git_footers_test.py 11 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272
  1. #!/usr/bin/env vpython3
  2. """Tests for git_footers."""
  3. from io import StringIO
  4. import json
  5. import os
  6. import sys
  7. import unittest
  8. from unittest import mock
  9. sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
  10. import gclient_utils
  11. import git_footers
  12. class GitFootersTest(unittest.TestCase):
  13. _message = """
  14. This is my commit message. There are many like it, but this one is mine.
  15. My commit message is my best friend. It is my life.
  16. """
  17. _position = 'refs/heads/main@{#292272}'
  18. _position_footer = 'Cr-Commit-Position: %s\n' % _position
  19. def testFootersBasic(self):
  20. self.assertEqual(git_footers.split_footers('Not-A: footer'),
  21. (['Not-A: footer'], [], []))
  22. self.assertEqual(
  23. git_footers.split_footers('Header\n\nActual: footer'),
  24. (['Header', ''], ['Actual: footer'], [('Actual', 'footer')]))
  25. self.assertEqual(git_footers.split_footers('\nActual: footer'),
  26. ([''], ['Actual: footer'], [('Actual', 'footer')]))
  27. self.assertEqual(
  28. git_footers.split_footers('H\n\nBug:\nAlso: footer'),
  29. (['H', ''], ['Bug:', 'Also: footer'], [('Bug', ''),
  30. ('Also', 'footer')]))
  31. self.assertEqual(git_footers.split_footers('H\n\nBug: '),
  32. (['H', ''], ['Bug:'], [('Bug', '')]))
  33. self.assertEqual(git_footers.split_footers('H\n\nBug: 1234 '),
  34. (['H', ''], ['Bug: 1234'], [('Bug', '1234')]))
  35. self.assertEqual(
  36. git_footers.split_footers('H\n\nBug: 1234\nChange-Id: Ib4321 '),
  37. (['H', ''], ['Bug: 1234', 'Change-Id: Ib4321'
  38. ], [('Bug', '1234'), ('Change-Id', 'Ib4321')]))
  39. self.assertEqual(git_footers.parse_footers(self._message), {})
  40. self.assertEqual(
  41. git_footers.parse_footers(self._message + self._position_footer),
  42. {'Cr-Commit-Position': [self._position]})
  43. self.assertEqual(
  44. git_footers.parse_footers(self._message + self._position_footer +
  45. self._position_footer),
  46. {'Cr-Commit-Position': [self._position, self._position]})
  47. self.assertEqual(
  48. git_footers.parse_footers(self._message + 'Bug:\n' +
  49. self._position_footer),
  50. {
  51. 'Bug': [''],
  52. 'Cr-Commit-Position': [self._position]
  53. })
  54. def testSkippingBadFooterLines(self):
  55. message = ('Title.\n'
  56. '\n'
  57. 'Last: paragraph starts\n'
  58. 'It-may: contain\n'
  59. 'bad lines, which should be skipped\n'
  60. 'For: example\n'
  61. '(cherry picked from)\n'
  62. 'And-only-valid: footers taken')
  63. self.assertEqual(git_footers.split_footers(message), (['Title.', ''], [
  64. 'Last: paragraph starts', 'It-may: contain',
  65. 'bad lines, which should be skipped', 'For: example',
  66. '(cherry picked from)', 'And-only-valid: footers taken'
  67. ], [('Last', 'paragraph starts'), ('It-may', 'contain'),
  68. ('For', 'example'), ('And-only-valid', 'footers taken')]))
  69. self.assertEqual(
  70. git_footers.parse_footers(message), {
  71. 'Last': ['paragraph starts'],
  72. 'It-May': ['contain'],
  73. 'For': ['example'],
  74. 'And-Only-Valid': ['footers taken']
  75. })
  76. def testAvoidingURLs(self):
  77. message = ('Someone accidentally put a URL in the footers.\n'
  78. '\n'
  79. 'Followed: by\n'
  80. 'http://domain.tld\n'
  81. 'Some: footers')
  82. self.assertEqual(
  83. git_footers.split_footers(message),
  84. (['Someone accidentally put a URL in the footers.', ''], [
  85. 'Followed: by', 'http://domain.tld', 'Some: footers'
  86. ], [('Followed', 'by'), ('Some', 'footers')]))
  87. self.assertEqual(git_footers.parse_footers(message), {
  88. 'Followed': ['by'],
  89. 'Some': ['footers']
  90. })
  91. def testSplittingLastParagraph(self):
  92. message = ('Title.\n'
  93. '\n'
  94. 'The final paragraph has some normal text first.\n'
  95. 'Followed: by\n'
  96. 'nonsense trailers and\n'
  97. 'Some: footers')
  98. self.assertEqual(git_footers.split_footers(message), ([
  99. 'Title.', '', 'The final paragraph has some normal text first.', ''
  100. ], ['Followed: by', 'nonsense trailers and', 'Some: footers'
  101. ], [('Followed', 'by'), ('Some', 'footers')]))
  102. self.assertEqual(git_footers.parse_footers(message), {
  103. 'Followed': ['by'],
  104. 'Some': ['footers']
  105. })
  106. def testGetFooterChangeId(self):
  107. msg = '\n'.join([
  108. 'whatever',
  109. '',
  110. 'Change-Id: ignored',
  111. '', # Above is ignored because of this empty line.
  112. 'Change-Id: Ideadbeaf'
  113. ])
  114. self.assertEqual(['Ideadbeaf'], git_footers.get_footer_change_id(msg))
  115. self.assertEqual([],
  116. git_footers.get_footer_change_id(
  117. 'desc\nBUG=not-a-valid-footer\nChange-Id: Ixxx'))
  118. self.assertEqual(['Ixxx'],
  119. git_footers.get_footer_change_id(
  120. 'desc\nBUG=not-a-valid-footer\n\nChange-Id: Ixxx'))
  121. def testAddFooterChangeId(self):
  122. with self.assertRaises(AssertionError):
  123. git_footers.add_footer_change_id('Already has\n\nChange-Id: Ixxx',
  124. 'Izzz')
  125. self.assertEqual(
  126. git_footers.add_footer_change_id('header-only', 'Ixxx'),
  127. 'header-only\n\nChange-Id: Ixxx')
  128. self.assertEqual(
  129. git_footers.add_footer_change_id('header\n\nsome: footer', 'Ixxx'),
  130. 'header\n\nsome: footer\nChange-Id: Ixxx')
  131. self.assertEqual(
  132. git_footers.add_footer_change_id('header\n\nBUG: yy', 'Ixxx'),
  133. 'header\n\nBUG: yy\nChange-Id: Ixxx')
  134. self.assertEqual(
  135. git_footers.add_footer_change_id('header\n\nBUG: yy\nPos: 1',
  136. 'Ixxx'),
  137. 'header\n\nBUG: yy\nChange-Id: Ixxx\nPos: 1')
  138. self.assertEqual(
  139. git_footers.add_footer_change_id('header\n\nBUG: yy\n\nPos: 1',
  140. 'Ixxx'),
  141. 'header\n\nBUG: yy\n\nPos: 1\nChange-Id: Ixxx')
  142. # Special case: first line is never a footer, even if it looks line one.
  143. self.assertEqual(
  144. git_footers.add_footer_change_id('header: like footer', 'Ixxx'),
  145. 'header: like footer\n\nChange-Id: Ixxx')
  146. self.assertEqual(
  147. git_footers.add_footer_change_id('Header.\n\nBug: v8\nN=t\nT=z',
  148. 'Ix'),
  149. 'Header.\n\nBug: v8\nChange-Id: Ix\nN=t\nT=z')
  150. def testAddFooter(self):
  151. with self.assertRaises(ValueError):
  152. git_footers.add_footer('', 'Invalid Footer', 'Value')
  153. self.assertEqual(git_footers.add_footer('', 'Key', 'Value'),
  154. '\nKey: Value')
  155. self.assertEqual(
  156. git_footers.add_footer('Header with empty line.\n\n', 'Key',
  157. 'Value'),
  158. 'Header with empty line.\n\nKey: Value')
  159. self.assertEqual(
  160. git_footers.add_footer('Top\n\nSome: footer', 'Key', 'value'),
  161. 'Top\n\nSome: footer\nKey: value')
  162. self.assertEqual(
  163. git_footers.add_footer('Top\n\nSome: footer',
  164. 'Key',
  165. 'value',
  166. after_keys=['Any']),
  167. 'Top\n\nSome: footer\nKey: value')
  168. self.assertEqual(
  169. git_footers.add_footer('Top\n\nSome: footer',
  170. 'Key',
  171. 'value',
  172. after_keys=['Some']),
  173. 'Top\n\nSome: footer\nKey: value')
  174. self.assertEqual(
  175. git_footers.add_footer('Top\n\nSome: footer\nOther: footer',
  176. 'Key',
  177. 'value',
  178. after_keys=['Some']),
  179. 'Top\n\nSome: footer\nKey: value\nOther: footer')
  180. self.assertEqual(
  181. git_footers.add_footer('Top\n\nSome: footer\nOther: footer',
  182. 'Key',
  183. 'value',
  184. before_keys=['Other']),
  185. 'Top\n\nSome: footer\nKey: value\nOther: footer')
  186. self.assertEqual(
  187. git_footers.add_footer(
  188. 'Top\n\nSome: footer\nOther: footer\nFinal: footer',
  189. 'Key',
  190. 'value',
  191. after_keys=['Some'],
  192. before_keys=['Final']),
  193. 'Top\n\nSome: footer\nKey: value\nOther: footer\nFinal: footer')
  194. self.assertEqual(
  195. git_footers.add_footer(
  196. 'Top\n\nSome: footer\nOther: footer\nFinal: footer',
  197. 'Key',
  198. 'value',
  199. after_keys=['Other'],
  200. before_keys=['Some']),
  201. 'Top\n\nSome: footer\nOther: footer\nKey: value\nFinal: footer')
  202. def testRemoveFooter(self):
  203. self.assertEqual(git_footers.remove_footer('message', 'Key'), 'message')
  204. self.assertEqual(
  205. git_footers.remove_footer('message\n\nSome: footer', 'Key'),
  206. 'message\n\nSome: footer')
  207. self.assertEqual(
  208. git_footers.remove_footer('message\n\nSome: footer\nKey: value',
  209. 'Key'), 'message\n\nSome: footer')
  210. self.assertEqual(
  211. git_footers.remove_footer(
  212. 'message\n\nKey: value\nSome: footer\nKey: value', 'Key'),
  213. 'message\n\nSome: footer')
  214. @mock.patch('sys.stdout', StringIO())
  215. @mock.patch(
  216. 'sys.stdin',
  217. StringIO('line\r\notherline\r\n\r\n\r\nFoo: baz\r\nStill: footer'))
  218. def testReadStdin(self):
  219. self.assertEqual(git_footers.main([]), 0)
  220. self.assertEqual(sys.stdout.getvalue(), 'Still: footer\nFoo: baz\n')
  221. @mock.patch('sys.stdin',
  222. StringIO('line\r\nany spaces\r\n\r\n\r\nFoo: 1\nBar: 2\nFoo: 3')
  223. )
  224. def testToJson(self):
  225. with gclient_utils.temporary_file() as tmp:
  226. self.assertEqual(git_footers.main(['--json', tmp]), 0)
  227. with open(tmp) as f:
  228. js = json.load(f)
  229. self.assertEqual(js, {'Foo': ['3', '1'], 'Bar': ['2']})
  230. if __name__ == '__main__':
  231. unittest.main()