diff options
Diffstat (limited to 'Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py')
| -rw-r--r-- | Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py | 260 |
1 files changed, 0 insertions, 260 deletions
diff --git a/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py b/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py deleted file mode 100644 index 587798e77..000000000 --- a/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py +++ /dev/null @@ -1,260 +0,0 @@ -# Copyright (C) 2010 Google Inc. All rights reserved. -# -# Redistribution and use in source and binary forms, with or without -# modification, are permitted provided that the following conditions are -# met: -# -# * Redistributions of source code must retain the above copyright -# notice, this list of conditions and the following disclaimer. -# * Redistributions in binary form must reproduce the above -# copyright notice, this list of conditions and the following disclaimer -# in the documentation and/or other materials provided with the -# distribution. -# * Neither the name of Google Inc. nor the names of its -# contributors may be used to endorse or promote products derived from -# this software without specific prior written permission. -# -# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS -# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT -# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR -# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT -# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, -# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT -# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, -# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY -# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT -# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE -# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - -import codecs -import os -import shutil -import tempfile -import unittest2 as unittest - -from .checkout import Checkout -from .changelog import ChangeLogEntry -from .scm import CommitMessage, SCMDetector -from .scm.scm_mock import MockSCM -from webkitpy.common.webkit_finder import WebKitFinder -from webkitpy.common.system.executive import Executive, ScriptError -from webkitpy.common.system.filesystem import FileSystem # FIXME: This should not be needed. -from webkitpy.common.system.filesystem_mock import MockFileSystem -from webkitpy.common.system.executive_mock import MockExecutive -from webkitpy.common.system.outputcapture import OutputCapture -from webkitpy.thirdparty.mock import Mock - - -_changelog1entry1 = u"""2010-03-25 Tor Arne Vestb\u00f8 <vestbo@webkit.org> - - Unreviewed build fix to un-break webkit-patch land. - - Move commit_message_for_this_commit from scm to checkout - https://bugs.webkit.org/show_bug.cgi?id=36629 - - * Scripts/webkitpy/common/checkout/api.py: import scm.CommitMessage -""" -_changelog1entry2 = u"""2010-03-25 Adam Barth <abarth@webkit.org> - - Reviewed by Eric Seidel. - - Move commit_message_for_this_commit from scm to checkout - https://bugs.webkit.org/show_bug.cgi?id=36629 - - * Scripts/webkitpy/common/checkout/api.py: -""" -_changelog1 = u"\n".join([_changelog1entry1, _changelog1entry2]) -_changelog2 = u"""2010-03-25 Tor Arne Vestb\u00f8 <vestbo@webkit.org> - - Unreviewed build fix to un-break webkit-patch land. - - Second part of this complicated change by me, Tor Arne Vestb\u00f8! - - * Path/To/Complicated/File: Added. - -2010-03-25 Adam Barth <abarth@webkit.org> - - Reviewed by Eric Seidel. - - Filler change. -""" - -class CommitMessageForThisCommitTest(unittest.TestCase): - expected_commit_message = u"""Unreviewed build fix to un-break webkit-patch land. - -Tools: - -Move commit_message_for_this_commit from scm to checkout -https://bugs.webkit.org/show_bug.cgi?id=36629 - -* Scripts/webkitpy/common/checkout/api.py: import scm.CommitMessage - -LayoutTests: - -Second part of this complicated change by me, Tor Arne Vestb\u00f8! - -* Path/To/Complicated/File: Added. -""" - - def setUp(self): - # FIXME: This should not need to touch the filesystem, however - # ChangeLog is difficult to mock at current. - self.filesystem = FileSystem() - self.temp_dir = str(self.filesystem.mkdtemp(suffix="changelogs")) - self.old_cwd = self.filesystem.getcwd() - self.filesystem.chdir(self.temp_dir) - self.webkit_base = WebKitFinder(self.filesystem).webkit_base() - - # Trick commit-log-editor into thinking we're in a Subversion working copy so it won't - # complain about not being able to figure out what SCM is in use. - # FIXME: VCSTools.pm is no longer so easily fooled. It logs because "svn info" doesn't - # treat a bare .svn directory being part of an svn checkout. - self.filesystem.maybe_make_directory(".svn") - - self.changelogs = map(self.filesystem.abspath, (self.filesystem.join("Tools", "ChangeLog"), self.filesystem.join("LayoutTests", "ChangeLog"))) - for path, contents in zip(self.changelogs, (_changelog1, _changelog2)): - self.filesystem.maybe_make_directory(self.filesystem.dirname(path)) - self.filesystem.write_text_file(path, contents) - - def tearDown(self): - self.filesystem.rmtree(self.temp_dir) - self.filesystem.chdir(self.old_cwd) - - def test_commit_message_for_this_commit(self): - executive = Executive() - - def mock_run(*args, **kwargs): - # Note that we use a real Executive here, not a MockExecutive, so we can test that we're - # invoking commit-log-editor correctly. - env = os.environ.copy() - env['CHANGE_LOG_EMAIL_ADDRESS'] = 'vestbo@webkit.org' - kwargs['env'] = env - return executive.run_command(*args, **kwargs) - - detector = SCMDetector(self.filesystem, executive) - real_scm = detector.detect_scm_system(self.webkit_base) - - mock_scm = MockSCM() - mock_scm.run = mock_run - mock_scm.script_path = real_scm.script_path - - checkout = Checkout(mock_scm) - checkout.modified_changelogs = lambda git_commit, changed_files=None: self.changelogs - commit_message = checkout.commit_message_for_this_commit(git_commit=None, return_stderr=True) - # Throw away the first line - a warning about unknown VCS root. - commit_message.message_lines = commit_message.message_lines[1:] - self.assertMultiLineEqual(commit_message.message(), self.expected_commit_message) - - -class CheckoutTest(unittest.TestCase): - def _make_checkout(self): - return Checkout(scm=MockSCM(), filesystem=MockFileSystem(), executive=MockExecutive()) - - def test_latest_entry_for_changelog_at_revision(self): - def mock_contents_at_revision(changelog_path, revision): - self.assertEqual(changelog_path, "foo") - self.assertEqual(revision, "bar") - # contents_at_revision is expected to return a byte array (str) - # so we encode our unicode ChangeLog down to a utf-8 stream. - # The ChangeLog utf-8 decoding should ignore invalid codepoints. - invalid_utf8 = "\255" - return _changelog1.encode("utf-8") + invalid_utf8 - checkout = self._make_checkout() - checkout._scm.contents_at_revision = mock_contents_at_revision - entry = checkout._latest_entry_for_changelog_at_revision("foo", "bar") - self.assertMultiLineEqual(entry.contents(), _changelog1entry1) # Pylint is confused about this line, pylint: disable=E1101 - - # FIXME: This tests a hack around our current changed_files handling. - # Right now changelog_entries_for_revision tries to fetch deleted files - # from revisions, resulting in a ScriptError exception. Test that we - # recover from those and still return the other ChangeLog entries. - def test_changelog_entries_for_revision(self): - checkout = self._make_checkout() - checkout._scm.changed_files_for_revision = lambda revision: ['foo/ChangeLog', 'bar/ChangeLog'] - - def mock_latest_entry_for_changelog_at_revision(path, revision): - if path == "foo/ChangeLog": - return 'foo' - raise ScriptError() - - checkout._latest_entry_for_changelog_at_revision = mock_latest_entry_for_changelog_at_revision - - # Even though fetching one of the entries failed, the other should succeed. - entries = checkout.changelog_entries_for_revision(1) - self.assertEqual(len(entries), 1) - self.assertEqual(entries[0], 'foo') - - def test_commit_info_for_revision(self): - checkout = self._make_checkout() - checkout._scm.changed_files_for_revision = lambda revision: ['path/to/file', 'another/file'] - checkout._scm.committer_email_for_revision = lambda revision, changed_files=None: "committer@example.com" - checkout.changelog_entries_for_revision = lambda revision, changed_files=None: [ChangeLogEntry(_changelog1entry1)] - commitinfo = checkout.commit_info_for_revision(4) - self.assertEqual(commitinfo.bug_id(), 36629) - self.assertEqual(commitinfo.author_name(), u"Tor Arne Vestb\u00f8") - self.assertEqual(commitinfo.author_email(), "vestbo@webkit.org") - self.assertIsNone(commitinfo.reviewer_text()) - self.assertIsNone(commitinfo.reviewer()) - self.assertEqual(commitinfo.committer_email(), "committer@example.com") - self.assertIsNone(commitinfo.committer()) - self.assertEqual(commitinfo.to_json(), { - 'bug_id': 36629, - 'author_email': 'vestbo@webkit.org', - 'changed_files': [ - 'path/to/file', - 'another/file', - ], - 'reviewer_text': None, - 'author_name': u'Tor Arne Vestb\xf8', - }) - - checkout.changelog_entries_for_revision = lambda revision, changed_files=None: [] - self.assertIsNone(checkout.commit_info_for_revision(1)) - - def test_bug_id_for_revision(self): - checkout = self._make_checkout() - checkout._scm.committer_email_for_revision = lambda revision: "committer@example.com" - checkout.changelog_entries_for_revision = lambda revision, changed_files=None: [ChangeLogEntry(_changelog1entry1)] - self.assertEqual(checkout.bug_id_for_revision(4), 36629) - - def test_bug_id_for_this_commit(self): - checkout = self._make_checkout() - checkout.commit_message_for_this_commit = lambda git_commit, changed_files=None: CommitMessage(ChangeLogEntry(_changelog1entry1).contents().splitlines()) - self.assertEqual(checkout.bug_id_for_this_commit(git_commit=None), 36629) - - def test_modified_changelogs(self): - checkout = self._make_checkout() - checkout._scm.checkout_root = "/foo/bar" - checkout._scm.changed_files = lambda git_commit: ["file1", "ChangeLog", "relative/path/ChangeLog"] - expected_changlogs = ["/foo/bar/ChangeLog", "/foo/bar/relative/path/ChangeLog"] - self.assertEqual(checkout.modified_changelogs(git_commit=None), expected_changlogs) - - def test_suggested_reviewers(self): - def mock_changelog_entries_for_revision(revision, changed_files=None): - if revision % 2 == 0: - return [ChangeLogEntry(_changelog1entry1)] - return [ChangeLogEntry(_changelog1entry2)] - - def mock_revisions_changing_file(path, limit=5): - if path.endswith("ChangeLog"): - return [3] - return [4, 8] - - checkout = self._make_checkout() - checkout._scm.checkout_root = "/foo/bar" - checkout._scm.changed_files = lambda git_commit: ["file1", "file2", "relative/path/ChangeLog"] - checkout._scm.revisions_changing_file = mock_revisions_changing_file - checkout.changelog_entries_for_revision = mock_changelog_entries_for_revision - reviewers = checkout.suggested_reviewers(git_commit=None) - reviewer_names = [reviewer.full_name for reviewer in reviewers] - self.assertEqual(reviewer_names, [u'Tor Arne Vestb\xf8']) - - def test_apply_patch(self): - checkout = self._make_checkout() - checkout._executive = MockExecutive(should_log=True) - checkout._scm.script_path = lambda script: script - mock_patch = Mock() - mock_patch.contents = lambda: "foo" - mock_patch.reviewer = lambda: None - expected_logs = "MOCK run_command: ['svn-apply', '--force'], cwd=/mock-checkout, input=foo\n" - OutputCapture().assert_outputs(self, checkout.apply_patch, [mock_patch], expected_logs=expected_logs) |