[Findit] Add local git parsers.

appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py

+# Copyright 2014 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.
+
+from datetime import datetime
+from datetime import timedelta
+import os
+import sys
+import textwrap
+import unittest
+
+_SCRIPT_DIR = os.path.join(os.path.dirname(__file__),
+                           os.path.pardir, os.path.pardir)
+sys.path.insert(1, _SCRIPT_DIR)
+import script_util
+script_util.SetUpSystemPaths()
+
+from git_checkout import local_git_parsers
+from lib.gitiles import blame
+from lib.gitiles import change_log
+
+
+class LocalGitParsersTest(unittest.TestCase):
+
+  def testGitBlameParser(self):
+    output = textwrap.dedent(
+        """
+        revision_hash 18 18 3
+        author test@google.com
+        author-mail <test@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
+        author-time 1363032816
+        author-tz +0000
+        committer test@google.com
+        committer-mail <test@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
+        committer-time 1363032816
+        committer-tz +0000
+        summary add (mac) test for ttcindex in SkFontStream
+        previous fe7533eebe777cc66c7f8fa7a03f00572755c5b4 src/core/SkFont.h
+        filename src/core/SkFont.h
+                     *  blabla line 1
+        revision_hash 19 19
+                     *  blabla line 2
+        revision_hash 20 20
+                     *  blabla line 3
+
+        revision_hash 29 29 2
+                     *  blabla line 4
+        """
+    )
+
+    expected_regions = [blame.Region(18, 3, 'revision_hash', 'test@google.com',
+                                     'test@google.com',
+                                     datetime(2013, 03, 11, 17, 13, 36)),
+                        blame.Region(29, 2, 'revision_hash', 'test@google.com',
+                                     'test@google.com',
+                                     datetime(2013, 03, 11, 17, 13, 36))]
+    expected_blame = blame.Blame('src/core/SkFont.h', 'rev')
+    for expected_region in expected_regions:
+      expected_blame.AddRegion(expected_region)

[wrengr, 2016/11/08 19:05:16]

Can you add a TODO(crbug.com/663445)? resolving that can be done in a future cl

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+
+    blame_result = local_git_parsers.GitBlameParser()(output,

[wrengr, 2016/11/08 19:05:16]

should prolly cache the local_git_parsers.GitBlameParser() and
local_git_parsers.GitChangeLogParser() during setUp

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+                                                      'src/core/SkFont.h',
+                                                      'rev')
+    self.assertTrue(blame_result.revision, expected_blame.revision)
+    self.assertTrue(blame_result.path, expected_blame.path)
+    for region, expected_region in zip(blame_result, expected_blame):
+      self.assertTrue(region.ToDict(), expected_region.ToDict())
+
+  def testGitBlameParserEmptyOutput(self):
+    output = ''

[wrengr, 2016/11/08 19:05:16]

since output and blame_result are only used once (and are relatively short),
they should probably be inlined like they are in testGetFileChangeInfo

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+    blame_result = local_git_parsers.GitBlameParser()(output,
+                                                      'src/core/SkFont.h',
+                                                      'rev')
+    self.assertIsNone(blame_result)
+
+  def testGitBlameParserDummyOutput(self):
+    output = 'Dummy'

[wrengr, 2016/11/08 19:05:16]

ditto

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+    blame_result = local_git_parsers.GitBlameParser()(output,
+                                                      'src/core/SkFont.h',
+                                                      'rev')
+    self.assertIsNone(blame_result)
+
+  def testGetFileChangeInfo(self):
+    self.assertIsNone(local_git_parsers.GetFileChangeInfo('change type',
+                                                          None, None))
+
+  def testGitChangeLogParser(self):
+    output = textwrap.dedent(
+        """
+        commit revision
+        tree tree_revision
+        parents parent_revision
+
+        author Test
+        author-mail test@google.com
+        author-time 2016-07-13 20:37:06
+
+        committer Test
+        committer-mail test@google.com
+        committer-time 2016-07-13 20:37:06
+
+        --Message start--
+        Revert commit messages...
+        > Committed: https://c.com/+/c9cc182781484f9010f062859cda048afefefefe
+        Review-Url: https://codereview.chromium.org/2391763002
+        Cr-Commit-Position: refs/heads/master@{#425880}
+        --Message end--
+
+        :100644 100644 25f95f c766f1 M      src/a/b.py
+        """
+    )
+
+    message = ('Revert commit messages...\n'
+               '> Committed: https://c.com/+/'
+               'c9cc182781484f9010f062859cda048afefefefe\n'
+               'Review-Url: https://codereview.chromium.org/2391763002\n'
+               'Cr-Commit-Position: refs/heads/master@{#425880}')
+
+    expected_changelog = change_log.ChangeLog(
+        'Test', 'test@google.com', datetime(2016, 7, 13, 20, 37, 6),
+        'Test', 'test@google.com', datetime(2016, 7, 13, 20, 37, 6),
+        'revision', 425880, message, [change_log.FileChangeInfo(
+            'modify', 'src/a/b.py', 'src/a/b.py')],
+        'https://repo/+/revision',
+        'https://codereview.chromium.org/2391763002',
+        'c9cc182781484f9010f062859cda048afefefefe')
+
+    changelog = local_git_parsers.GitChangeLogParser()(output, 'https://repo')
+    self.assertTrue(expected_changelog.ToDict(), changelog.ToDict())
+
+  def testGitChangeLogParserParseEmptyOutput(self):
+    output = None

[wrengr, 2016/11/08 19:05:16]

ditto

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+    self.assertIsNone(local_git_parsers.GitChangeLogParser()(output, 'repo'))
+
+  def testGitChangeLogsParser(self):
+    output = textwrap.dedent(
+        """
+        **Changelog start**
+        commit rev1
+        tree 27b0421273ed4aea25e497c6d26d9c7db6481852
+        parents rev22c9e
+
+        author author1
+        author-mail author1@chromium.org
+        author-time 2016-06-02 10:55:38
+
+        committer Commit bot
+        committer-mail commit-bot@chromium.org
+        committer-time 2016-06-02 10:57:13
+
+        --Message start--
+        Message 1
+        --Message end--
+
+        :100644 100644 28e117 f12d3 D      a/b.py
+
+
+        **Changelog start**
+        commit rev2
+        tree d22d3786e135b83183cfeba5f3d8913959f56299
+        parents ac7ee4ce7b8d39b22a710c58d110e0039c11cf9a
+
+        author author2
+        author-mail author2@chromium.org
+        author-time 2016-06-02 10:53:03
+
+        committer Commit bot
+        committer-mail commit-bot@chromium.org
+        committer-time 2016-06-02 10:54:14
+
+        --Message start--
+        Message 2
+        --Message end--
+
+        :100644 100644 7280f df186 A      b/c.py
+
+        **Changelog start**
+        commit rev3
+        tree d22d3786e135b83183cfeba5f3d8913959f56299
+        parents ac7ee4ce7b8d39b22a710c58d110e0039c11cf9a
+
+        author author3
+        author-mail author3@chromium.org
+        author-time 2016-06-02 10:53:03
+
+        committer Commit bot
+        committer-mail commit-bot@chromium.org
+        committer-time 2016-06-02 10:54:14
+
+        --Message start--
+        Message 3
+        --Message end--
+
+        :100644 100644 3f2e 20a5 R078 b/c.py b/cc.py
+        """
+    )
+
+    expected_changelogs = [
+        change_log.ChangeLog('author1',
+                             'author1@chromium.org',
+                             datetime(2016, 6, 2, 10, 55, 38),
+                             'Commit bot',
+                             'commit-bot@chromium.org',
+                             datetime(2016, 6, 2, 10, 57, 13),
+                             'rev1', None,
+                             'Message 1', [change_log.FileChangeInfo(
+                                 'delete', 'a/b.py', None)],
+                             'http://repo/+/rev1', None, None),
+        change_log.ChangeLog('author2',
+                             'author2@chromium.org',
+                             datetime(2016, 6, 2, 10, 53, 3),
+                             'Commit bot',
+                             'commit-bot@chromium.org',
+                             datetime(2016, 6, 2, 10, 54, 14),
+                             'rev2', None,
+                             'Message 2', [change_log.FileChangeInfo(
+                                 'add', None, 'b/c.py')],
+                             'http://repo/+/rev2', None, None),
+        change_log.ChangeLog('author3',
+                             'author3@chromium.org',
+                             datetime(2016, 6, 2, 10, 53, 3),
+                             'Commit bot',
+                             'commit-bot@chromium.org',
+                             datetime(2016, 6, 2, 10, 54, 14),
+                             'rev3', None,
+                             'Message 3', [change_log.FileChangeInfo(
+                                 'rename', 'b/c.py', 'b/cc.py')],
+                             'http://repo/+/rev3', None, None),
+    ]
+
+    changelogs = local_git_parsers.GitChangeLogsParser()(output, 'http://repo')
+    for changelog, expected_changelog in zip(changelogs, expected_changelogs):
+      self.assertEqual(changelog.ToDict(), expected_changelog.ToDict())
+
+
+  def testGitChangeLogsParserParseEmptyOutput(self):
+    output = None

[wrengr, 2016/11/08 19:05:16]

ditto

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+    self.assertIsNone(local_git_parsers.GitChangeLogsParser()(output, 'repo'))
+
+  def testGitChangeLogsParserWithEmptyChangelog(self):
+    output = '**Changelog start**\nblablabla'
+    self.assertEqual(local_git_parsers.GitChangeLogsParser()(output,
+                                                             'http://repo'), [])
+
+  def testGitDiffParser(self):
+    output = 'output'

[wrengr, 2016/11/08 19:05:16]

ditto

[Sharu Jiang, 2016/11/10 22:28:38]

Done.

+    self.assertEqual(output, local_git_parsers.GitDiffParser()(output))
+
+  def testGitSourceParser(self):
+    output = 'output'

[wrengr, 2016/11/08 19:05:16]

ditto

[Sharu Jiang, 2016/11/10 22:28:37]

Done.

+    self.assertEqual(output, local_git_parsers.GitSourceParser()(output))
+
+
+if __name__ == '__main__':
+  unittest.main()