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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py

Issue 2064313003: Rename files in webkitpy/tool to have underscores between words (for consistency). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 6 months 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: third_party/WebKit/Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py
deleted file mode 100644
index 74876d99a2afb5e76249f2e8c04639be285842b4..0000000000000000000000000000000000000000
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py
+++ /dev/null
@@ -1,205 +0,0 @@
-# Copyright (C) 2013 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.
-#
-# 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 unittest
-
-from webkitpy.tool.bot.commitannouncer import CommitAnnouncer
-from webkitpy.tool.mocktool import MockTool
-
-
-class CommitAnnouncerTest(unittest.TestCase):
-
- def test_format_commit(self):
- tool = MockTool()
- bot = CommitAnnouncer(tool, "test/directory", "test_password")
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line"',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 '
- 'authorABC@chromium.org committed "Commit test subject line"',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/1234comm authorABC@chromium.org committed "Commit test subject line"',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-
-Review URL: https://codereview.chromium.org/123456
-"""))
-
- self.assertEqual(
- 'https://crrev.com/1234comm authorABC@chromium.org committed "Commit test subject line"',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line"',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-Review URL: http://fake.review.url
-Cr-Commit-Position: refs/heads/master@{#000000}
-
-BUG=654321
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line" '
- '\x037TBR=reviewerDEF@chromium.org\x03',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-TBR=reviewerDEF@chromium.org
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line" '
- '\x037NOTRY=true\x03',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-NOTRY=true
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line" '
- '\x037NOTRY=true TBR=reviewerDEF@chromium.org\x03',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-NOTRY=true
-BUG=654321
-TBR=reviewerDEF@chromium.org
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- self.assertEqual(
- 'https://crrev.com/456789 authorABC@chromium.org committed "Commit test subject line" '
- '\x037tbr=reviewerDEF@chromium.org, reviewerGHI@chromium.org, reviewerJKL@chromium.org notry=TRUE\x03',
- bot._format_commit_detail("""\
-1234commit1234
-authorABC@chromium.org
-Commit test subject line
-Multiple
-lines
-of
-description.
-
-BUG=654321
-tbr=reviewerDEF@chromium.org, reviewerGHI@chromium.org, reviewerJKL@chromium.org
-notry=TRUE
-
-Review URL: https://codereview.chromium.org/123456
-
-Cr-Commit-Position: refs/heads/master@{#456789}
-"""))
-
- def test_sanitize_string(self):
- bot = CommitAnnouncer(MockTool(), "test/directory", "test_password")
- self.assertEqual('normal ascii', bot._sanitize_string('normal ascii'))
- self.assertEqual('uni\\u0441ode!', bot._sanitize_string(u'uni\u0441ode!'))

Powered by Google App Engine
This is Rietveld 408576698