Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3781)

Unified Diff: appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py

Issue 2435863003: [Findit] Add local git parsers. (Closed)
Patch Set: Rebase. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py
diff --git a/appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py b/appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py
new file mode 100644
index 0000000000000000000000000000000000000000..51dd646014bafe32a460773da0d8bb27ba3d99e8
--- /dev/null
+++ b/appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py
@@ -0,0 +1,247 @@
+# 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 setUp(self):
+ super(LocalGitParsersTest, self).setUp()
+ self.blame_parser = local_git_parsers.GitBlameParser()
+
+ 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')
+ # TODO(crbug.com/663445): Add methods in blame for this.
+ for expected_region in expected_regions:
+ expected_blame.AddRegion(expected_region)
+
+ blame_result = self.blame_parser(output, '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):
+ blame_result = self.blame_parser('', 'src/core/SkFont.h', 'rev')
+ self.assertIsNone(blame_result)
+
+ def testGitBlameParserDummyOutput(self):
+ blame_result = self.blame_parser('Dummy',
+ '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):
+ self.assertIsNone(local_git_parsers.GitChangeLogParser()(None, '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):
+ self.assertIsNone(local_git_parsers.GitChangeLogsParser()(None, 'repo'))
+
+ def testGitChangeLogsParserWithEmptyChangelog(self):
+ output = '**Changelog start**\nblablabla'
+ self.assertEqual(local_git_parsers.GitChangeLogsParser()(output,
+ 'http://repo'), [])
+
+ def testGitDiffParser(self):
+ self.assertEqual('output', local_git_parsers.GitDiffParser()('output'))
+
+
+if __name__ == '__main__':
+ unittest.main()
« no previous file with comments | « appengine/findit/util_scripts/git_checkout/test/__init__.py ('k') | appengine/findit/util_scripts/iterator.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698