IRC commit-bot: Make svn revision appear in the beginning of bot message.

Tools/Scripts/webkitpy/tool/bot/commitannouncer_unittest.py

 # 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
@@ skipping 16 matching lines @@
 
 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_password")
         self.assertEqual(
-           'authorABC@chromium.org committed "Commit test subject line" '
-           'http://crrev.com/123456 r456789',
+           'r456789 http://crrev.com/123456 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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-            'authorABC@chromium.org committed "Commit test subject line" '
-            'https://chromium.googlesource.com/chromium/blink/+/1234comm '
-            'r456789',
+            'r456789 https://chromium.googlesource.com/chromium/blink/+/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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-            'authorABC@chromium.org committed "Commit test subject line" '
-            'http://crrev.com/123456 ',
+            'http://crrev.com/123456 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(
-            'authorABC@chromium.org committed "Commit test subject line" '
-            'https://chromium.googlesource.com/chromium/blink/+/1234comm ',
+            'https://chromium.googlesource.com/chromium/blink/+/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(
-            'authorABC@chromium.org committed "Commit test subject line" '
-            'http://crrev.com/123456 r456789',
+            'r456789 http://crrev.com/123456 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
 git-svn-id: svn://svn.chromium.org/blink/trunk@000000 Fake-SVN-number
 
 BUG=654321
 
 Review URL: https://codereview.chromium.org/123456
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-           'authorABC@chromium.org committed "Commit test subject line" '
-           'http://crrev.com/123456 r456789 '
+           'r456789 http://crrev.com/123456 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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-           'authorABC@chromium.org committed "Commit test subject line" '
-           'http://crrev.com/123456 r456789 '
+           'r456789 http://crrev.com/123456 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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-           'authorABC@chromium.org committed "Commit test subject line" '
-           'http://crrev.com/123456 r456789 '
+           'r456789 http://crrev.com/123456 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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
         self.assertEqual(
-           'authorABC@chromium.org committed "Commit test subject line" '
-           'http://crrev.com/123456 r456789 '
+           'r456789 http://crrev.com/123456 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
 
 git-svn-id: svn://svn.chromium.org/blink/trunk@456789 bbb929c8-8fbe-4397-9dbb-9b2b20218538
 """))
 
     def test_sanitize_string(self):
         bot = CommitAnnouncer(MockTool(), "test_password")
         self.assertEqual('normal ascii', bot._sanitize_string('normal ascii'))
         self.assertEqual('uni\\u0441ode!', bot._sanitize_string(u'uni\u0441ode!'))