⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 244705 in webkit


Ignore:
Timestamp:
Apr 26, 2019, 1:47:29 PM (7 years ago)
Author:
ap@apple.com
Message:

Update inactive reviewers' status to 'contributor'.

  • Scripts/webkitpy/common/config/contributors.json: The change.
  • Scripts/webkitpy/common/checkout/changelog_unittest.py:
  • Scripts/webkitpy/common/checkout/checkout_mock.py:
  • Scripts/webkitpy/common/checkout/checkout_unittest.py:
  • Scripts/webkitpy/common/net/bugzilla/bugzilla_mock.py:
  • Scripts/webkitpy/tool/bot/irc_command_unittest.py:
  • Scripts/webkitpy/tool/bot/ircbot_unittest.py:
  • Scripts/webkitpy/tool/commands/queries_unittest.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:

Sadly, regression tests operate on a crazy mix of mock and live data for
contributors.json. It would be a substantial project to mock all code paths,
so changing the tests instead.

Location:
trunk/Tools
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/Tools/ChangeLog

    r244704 r244705  
     12019-04-26  Alexey Proskuryakov  <ap@apple.com>
     2
     3        Update inactive reviewers' status to 'contributor'.
     4
     5        * Scripts/webkitpy/common/config/contributors.json: The change.
     6
     7        * Scripts/webkitpy/common/checkout/changelog_unittest.py:
     8        * Scripts/webkitpy/common/checkout/checkout_mock.py:
     9        * Scripts/webkitpy/common/checkout/checkout_unittest.py:
     10        * Scripts/webkitpy/common/net/bugzilla/bugzilla_mock.py:
     11        * Scripts/webkitpy/tool/bot/irc_command_unittest.py:
     12        * Scripts/webkitpy/tool/bot/ircbot_unittest.py:
     13        * Scripts/webkitpy/tool/commands/queries_unittest.py:
     14        * Scripts/webkitpy/tool/commands/upload_unittest.py:
     15        Sadly, regression tests operate on a crazy mix of mock and live data for
     16        contributors.json. It would be a substantial project to mock all code paths,
     17        so changing the tests instead.
     18
    1192019-04-26  Eric Carlson  <eric.carlson@apple.com>
    220

  • trunk/Tools/Scripts/webkitpy/common/checkout/changelog_unittest.py

    r234663 r244705  
    4242    _example_entry = u'''2009-08-17  Peter Kasting  <pkasting@google.com>
    4343
    44         Reviewed by Tor Arne Vestb\xf8.
     44        Reviewed by Fr\u00e9d\u00e9ric Wang.
    4545
    4646        https://bugs.webkit.org/show_bug.cgi?id=27323
    … …  
    5757
    5858    # More example text than we need.  Eventually we need to support parsing this all and write tests for the parsing.
    59     _example_changelog = u"""2009-08-17  Tor Arne Vestb\xf8  <vestbo@webkit.org>
     59    _example_changelog = u"""2009-08-17  Fr\u00e9d\u00e9ric Wang  <fred.wang@free.fr>
    6060
    6161        <http://webkit.org/b/28393> check-webkit-style: add check for use of std::max()/std::min() instead of MAX()/MIN()
    … …  
    238238        parsed_entries = list(ChangeLog.parse_entries_from_file(changelog_file))
    239239        self.assertEqual(len(parsed_entries), 9)
    240         self.assertEqual(parsed_entries[0].date_line(), u"2009-08-17  Tor Arne Vestb\xf8  <vestbo@webkit.org>")
     240        self.assertEqual(parsed_entries[0].date_line(), u"2009-08-17  Fr\u00e9d\u00e9ric Wang  <fred.wang@free.fr>")
    241241        self.assertEqual(parsed_entries[0].date(), "2009-08-17")
    242242        self.assertEqual(parsed_entries[0].reviewer_text(), "David Levin")
    … …  
    580580
    581581    def test_has_valid_reviewer(self):
    582         self._assert_has_valid_reviewer("Reviewed by Eric Seidel.", True)
    583         self._assert_has_valid_reviewer("Reviewed by Eric Seidel", True)  # Not picky about the '.'
    584         self._assert_has_valid_reviewer("Reviewed by Eric.", False)
    585         self._assert_has_valid_reviewer("Reviewed by Eric C Seidel.", False)
    586         self._assert_has_valid_reviewer("Rubber-stamped by Eric.", False)
    587         self._assert_has_valid_reviewer("Rubber-stamped by Eric Seidel.", True)
    588         self._assert_has_valid_reviewer("Rubber stamped by Eric.", False)
    589         self._assert_has_valid_reviewer("Rubber stamped by Eric Seidel.", True)
     582        self._assert_has_valid_reviewer("Reviewed by Darin Adler.", True)
     583        self._assert_has_valid_reviewer("Reviewed by Darin Adler", True)  # Not picky about the '.'
     584        self._assert_has_valid_reviewer("Reviewed by Darin.", False)
     585        self._assert_has_valid_reviewer("Reviewed by Darin B Adler.", False)
     586        self._assert_has_valid_reviewer("Rubber-stamped by Darin.", False)
     587        self._assert_has_valid_reviewer("Rubber-stamped by Darin Adler.", True)
     588        self._assert_has_valid_reviewer("Rubber stamped by Darin.", False)
     589        self._assert_has_valid_reviewer("Rubber stamped by Darin Adler.", True)
    590590        self._assert_has_valid_reviewer("Unreviewed build fix.", True)
    591591        self._assert_has_valid_reviewer("Reviewed by Gabor Rapcsanyi.", False)
    … …  
    642642        self.assertEqual(latest_entry.author_name(), "Peter Kasting")
    643643        self.assertEqual(latest_entry.author_email(), "pkasting@google.com")
    644         self.assertEqual(latest_entry.reviewer_text(), u"Tor Arne Vestb\xf8")
     644        self.assertEqual(latest_entry.reviewer_text(), u"Fr\u00e9d\u00e9ric Wang")
    645645        touched_files = ["DumpRenderTree/win/DumpRenderTree.vcproj", "DumpRenderTree/win/ImageDiff.vcproj", "DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.vcproj"]
    646646        self.assertEqual(latest_entry.touched_files(), touched_files)

  • trunk/Tools/Scripts/webkitpy/common/checkout/checkout_mock.py

    r202362 r244705  
    4343
    4444mock_revisions = {
    45     1: CommitInfo(852, "eric@webkit.org", {
     45    1: CommitInfo(852, "ap@apple.com", {
    4646        "bug_id": 50000,
    4747        "author_name": "Adam Barth",
    … …  
    5656        "bug_description": "Example description of bug 50000.",
    5757    }),
    58     963: CommitInfo(963, "eric@webkit.org", {
     58    963: CommitInfo(963, "ap@apple.com", {
    5959        "bug_id": 50005,
    6060        "author_name": "Carol Szabo",
    … …  
    6969        "bug_description": "Example description of bug 50005.",
    7070    }),
    71     987: CommitInfo(987, "eric@webkit.org", {
     71    987: CommitInfo(987, "ap@apple.com", {
    7272        "bug_id": 50006,
    7373        "author_name": "Adam Barth",

  • trunk/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py

    r228303 r244705  
    4646
    4747
    48 _changelog1entry1 = u"""2010-03-25  Tor Arne Vestb\u00f8  <vestbo@webkit.org>
     48_changelog1entry1 = u"""2010-03-25  Fr\u00e9d\u00e9ric Wang  <fred.wang@free.fr>
    4949
    5050        Unreviewed build fix to un-break webkit-patch land.
    … …  
    6565"""
    6666_changelog1 = u"\n".join([_changelog1entry1, _changelog1entry2])
    67 _changelog2 = u"""2010-03-25  Tor Arne Vestb\u00f8  <vestbo@webkit.org>
     67_changelog2 = u"""2010-03-25  Fr\u00e9d\u00e9ric Wang  <fred.wang@free.fr>
    6868
    6969        Unreviewed build fix to un-break webkit-patch land.
    7070
    71         Second part of this complicated change by me, Tor Arne Vestb\u00f8!
     71        Second part of this complicated change by me, Fr\u00e9d\u00e9ric Wang!
    7272
    7373        * Path/To/Complicated/File: Added.
    … …  
    162162            # invoking commit-log-editor correctly.
    163163            env = os.environ.copy()
    164             env['CHANGE_LOG_EMAIL_ADDRESS'] = 'vestbo@webkit.org'
     164            env['CHANGE_LOG_EMAIL_ADDRESS'] = 'fred.wang@free.fr'
    165165            kwargs['env'] = env
    166166            return executive.run_command(*args, **kwargs)
    … …  
    191191LayoutTests:
    192192
    193 Second part of this complicated change by me, Tor Arne Vestb\u00f8!
     193Second part of this complicated change by me, Fr\u00e9d\u00e9ric Wang!
    194194
    195195* Path/To/Complicated/File: Added.
    … …  
    378378        self.assertEqual(commitinfo.bug_id(), 36629)
    379379        self.assertEqual(commitinfo.bug_description(), "Unreviewed build fix to un-break webkit-patch land.")
    380         self.assertEqual(commitinfo.author_name(), u"Tor Arne Vestb\u00f8")
    381         self.assertEqual(commitinfo.author_email(), "vestbo@webkit.org")
     380        self.assertEqual(commitinfo.author_name(), u"Fr\u00e9d\u00e9ric Wang")
     381        self.assertEqual(commitinfo.author_email(), "fred.wang@free.fr")
    382382        self.assertIsNone(commitinfo.reviewer_text())
    383383        self.assertIsNone(commitinfo.reviewer())
    … …  
    386386        self.assertEqual(commitinfo.to_json(), {
    387387            'bug_id': 36629,
    388             'author_email': 'vestbo@webkit.org',
     388            'author_email': 'fred.wang@free.fr',
    389389            'changed_files': [
    390390                'path/to/file',
    … …  
    392392            ],
    393393            'reviewer_text': None,
    394             'author_name': u'Tor Arne Vestb\xf8',
     394            'author_name': u'Fr\u00e9d\u00e9ric Wang',
    395395            'bug_description': 'Unreviewed build fix to un-break webkit-patch land.',
    396396        })
    … …  
    439439        reviewers = checkout.suggested_reviewers(git_commit=None)
    440440        reviewer_names = [reviewer.full_name for reviewer in reviewers]
    441         self.assertEqual(reviewer_names, [u'Tor Arne Vestb\xf8'])
     441        self.assertEqual(reviewer_names, [u'Fr\u00e9d\u00e9ric Wang'])
    442442
    443443    def test_apply_patch(self):

  • trunk/Tools/Scripts/webkitpy/common/config/contributors.json

    r243815 r244705  
    4444         "inferno-sec"
    4545      ],
    46       "status" : "reviewer"
     46      "status" : "contributor"
    4747   },
    4848   "Ada Chan" : {
    … …  
    5454         "chanada"
    5555      ],
    56       "status" : "reviewer"
     56      "status" : "contributor"
    5757   },
    5858   "Adam Barth" : {
    … …  
    112112         "aroben"
    113113      ],
    114       "status" : "reviewer"
     114      "status" : "contributor"
    115115   },
    116116   "Adam Treat" : {
    … …  
    124124         "manyoso"
    125125      ],
    126       "status" : "reviewer"
     126      "status" : "contributor"
    127127   },
    128128   "Adele Peterson" : {
    … …  
    134134         "adele"
    135135      ],
    136       "status" : "reviewer"
     136      "status" : "contributor"
    137137   },
    138138   "Ademar de Souza Reis Jr" : {
    … …  
    191191         "enne"
    192192      ],
    193       "status" : "reviewer"
     193      "status" : "contributor"
    194194   },
    195195   "Aharon Lanin" : {
    … …  
    327327         "apavlov"
    328328      ],
    329       "status" : "reviewer"
     329      "status" : "contributor"
    330330   },
    331331   "Alexandre Elias" : {
    … …  
    347347         "achicu"
    348348      ],
    349       "status" : "reviewer"
     349      "status" : "contributor"
    350350   },
    351351   "Alexey Marinichev" : {
    … …  
    379379         "darktears"
    380380      ],
    381       "status" : "reviewer"
     381      "status" : "contributor"
    382382   },
    383383   "Ali Juma" : {
    … …  
    408408         "aliu"
    409409      ],
    410       "status" : "reviewer"
     410      "status" : "contributor"
    411411   },
    412412   "Alicia Boya Garcia" : {
    … …  
    432432         "carewolf"
    433433      ],
    434       "status" : "reviewer"
     434      "status" : "contributor"
    435435   },
    436436   "Alok Priyadarshi" : {
    … …  
    453453         "alp"
    454454      ],
    455       "status" : "reviewer"
     455      "status" : "contributor"
    456456   },
    457457   "Ami Fischman" : {
    … …  
    526526         "abucur"
    527527      ],
    528       "status" : "reviewer"
     528      "status" : "contributor"
    529529   },
    530530   "Andrei Popescu" : {
    … …  
    774774         "ariya"
    775775      ],
    776       "status" : "reviewer"
     776      "status" : "contributor"
    777777   },
    778778   "Arko Saha" : {
    … …  
    875875         "bemjb"
    876876      ],
    877       "status" : "reviewer"
     877      "status" : "contributor"
    878878   },
    879879   "Ben Murdoch" : {
    … …  
    10961096         "cmarcelo"
    10971097      ],
    1098       "status" : "reviewer"
     1098      "status" : "contributor"
    10991099   },
    11001100   "Caitlin Potter" : {
    … …  
    11331133         "clopez"
    11341134      ],
    1135       "status" : "reviewer"
     1135      "status" : "contributor"
    11361136   },
    11371137   "Carlos Eduardo Ramalho" : {
    … …  
    11871187         "cshu"
    11881188      ],
    1189       "status" : "reviewer"
     1189      "status" : "contributor"
    11901190   },
    11911191   "ChangSeok Oh" : {
    … …  
    12331233         "cblu"
    12341234      ],
    1235       "status" : "reviewer"
     1235      "status" : "contributor"
    12361236   },
    12371237   "Chris Dumez" : {
    … …  
    12891289         "cjerdonek"
    12901290      ],
    1291       "status" : "reviewer"
     1291      "status" : "contributor"
    12921292   },
    12931293   "Chris Marrin" : {
    … …  
    12981298         "cmarrin"
    12991299      ],
    1300       "status" : "reviewer"
     1300      "status" : "contributor"
    13011301   },
    13021302   "Chris Nardi" : {
    … …  
    13261326         "crogers"
    13271327      ],
    1328       "status" : "reviewer"
     1328      "status" : "contributor"
    13291329   },
    13301330   "Christian Biesinger" : {
    … …  
    14151415         "ossy"
    14161416      ],
    1417       "status" : "reviewer"
     1417      "status" : "contributor"
    14181418   },
    14191419   "Dan Bernstein" : {
    … …  
    15021502         "fishd"
    15031503      ],
    1504       "status" : "reviewer"
     1504      "status" : "contributor"
    15051505   },
    15061506   "Dave Barton" : {
    … …  
    15721572         "harrison"
    15731573      ],
    1574       "status" : "reviewer"
     1574      "status" : "contributor"
    15751575   },
    15761576   "David Hyatt" : {
    … …  
    15861586         "hyatt"
    15871587      ],
    1588       "status" : "reviewer"
     1588      "status" : "contributor"
    15891589   },
    15901590   "David Jonathan Ross" : {
    … …  
    16171617         "dave_levin"
    16181618      ],
    1619       "status" : "reviewer"
     1619      "status" : "contributor"
    16201620   },
    16211621   "David Michael Barr" : {
    … …  
    17421742         "dglazkov"
    17431743      ],
    1744       "status" : "reviewer"
     1744      "status" : "contributor"
    17451745   },
    17461746   "Dinu Jacob" : {
    … …  
    17631763         "dpranke"
    17641764      ],
    1765       "status" : "reviewer"
     1765      "status" : "contributor"
    17661766   },
    17671767   "Dirk Schulze" : {
    … …  
    18031803         "dimich"
    18041804      ],
    1805       "status" : "reviewer"
     1805      "status" : "contributor"
    18061806   },
    18071807   "Dominic Cooney" : {
    … …  
    18551855         "gramps"
    18561856      ],
    1857       "status" : "reviewer"
     1857      "status" : "contributor"
    18581858   },
    18591859   "Don Olmstead" : {
    … …  
    19271927         "dumi"
    19281928      ],
    1929       "status" : "reviewer"
     1929      "status" : "contributor"
    19301930   },
    19311931   "D\u00e1niel B\u00e1tyai" : {
    … …  
    19691969         "esprehn"
    19701970      ],
    1971       "status" : "reviewer"
     1971      "status" : "contributor"
    19721972   },
    19731973   "Emil A Eklund" : {
    … …  
    19821982         "eae"
    19831983      ],
    1984       "status" : "reviewer"
     1984      "status" : "contributor"
    19851985   },
    19861986   "Emilio Cobos Alvarez" : {
    … …  
    20012001         "enrica"
    20022002      ],
    2003       "status" : "reviewer"
     2003      "status" : "contributor"
    20042004   },
    20052005   "Enrique Oca\u00f1a Gonz\u00e1lez" : {
    … …  
    20502050         "eseidel"
    20512051      ],
    2052       "status" : "reviewer"
     2052      "status" : "contributor"
    20532053   },
    20542054   "Eric Uhrhane" : {
    … …  
    22472247         "gbarra"
    22482248      ],
    2249       "status" : "reviewer"
     2249      "status" : "contributor"
    22502250   },
    22512251   "Gavin Peters" : {
    … …  
    22802280      ],
    22812281      "expertise" : "Core KHTML Contributor, The QtWebKit Port",
    2282       "status" : "reviewer"
     2282      "status" : "contributor"
    22832283   },
    22842284   "Gergo Balogh" : {
    … …  
    24302430         "morrita"
    24312431      ],
    2432       "status" : "reviewer"
     2432      "status" : "contributor"
    24332433   },
    24342434   "Hans Muller" : {
    … …  
    25152515         "zecke"
    25162516      ],
    2517       "status" : "reviewer"
     2517      "status" : "contributor"
    25182518   },
    25192519   "Hugo Parente Lima" : {
    … …  
    27432743         "jamesr"
    27442744      ],
    2745       "status" : "reviewer"
     2745      "status" : "contributor"
    27462746   },
    27472747   "James Simonsen" : {
    … …  
    27632763         "janm"
    27642764      ],
    2765       "status" : "reviewer"
     2765      "status" : "contributor"
    27662766   },
    27672767   "Janos Badics" : {
    … …  
    29032903         "jorlow"
    29042904      ],
    2905       "status" : "reviewer"
     2905      "status" : "contributor"
    29062906   },
    29072907   "Jessie Berlin" : {
    … …  
    29412941         "jianli"
    29422942      ],
    2943       "status" : "reviewer"
     2943      "status" : "contributor"
    29442944   },
    29452945   "Jiewen Tan" : {
    … …  
    29992999         "jturcotte"
    30003000      ],
    3001       "status" : "reviewer"
     3001      "status" : "contributor"
    30023002   },
    30033003   "Jochen Eisinger" : {
    … …  
    30093009         "jochen__"
    30103010      ],
    3011       "status" : "reviewer"
     3011      "status" : "contributor"
    30123012   },
    30133013   "Joe Thomas" : {
    … …  
    30933093         "sullivan"
    30943094      ],
    3095       "status" : "reviewer"
     3095      "status" : "contributor"
    30963096   },
    30973097   "John Wilander" : {
    … …  
    31253125         "jhoneycutt"
    31263126      ],
    3127       "status" : "reviewer"
     3127      "status" : "contributor"
    31283128   },
    31293129   "Jon Lee" : {
    … …  
    32923292         "jchaffraix"
    32933293      ],
    3294       "status" : "reviewer"
     3294      "status" : "contributor"
    32953295   },
    32963296   "Jungshik Shin" : {
    … …  
    33153315         "justing"
    33163316      ],
    3317       "status" : "reviewer"
     3317      "status" : "contributor"
    33183318   },
    33193319   "Justin Michaud" : {
    … …  
    34403440         "kocienda@apple.com"
    34413441      ],
    3442       "status" : "reviewer"
     3442      "status" : "contributor"
    34433443   },
    34443444   "Kenichi Ishibashi" : {
    … …  
    34763476         "kenne"
    34773477      ],
    3478       "status" : "reviewer"
     3478      "status" : "contributor"
    34793479   },
    34803480   "Kenneth Russell" : {
    … …  
    34913491         "kbrgg"
    34923492      ],
    3493       "status" : "reviewer"
     3493      "status" : "contributor"
    34943494   },
    34953495   "Kent Hansen" : {
    … …  
    35123512         "tkent"
    35133513      ],
    3514       "status" : "reviewer"
     3514      "status" : "contributor"
    35153515   },
    35163516   "Kentaro Hara" : {
    … …  
    35233523         "haraken"
    35243524      ],
    3525       "status" : "reviewer"
     3525      "status" : "contributor"
    35263526   },
    35273527   "Kevin Decker" : {
    … …  
    35333533         "superkevin"
    35343534      ],
    3535       "status" : "reviewer"
     3535      "status" : "contributor"
    35363536   },
    35373537   "Kevin McCullough" : {
    … …  
    35433543         "maculloch"
    35443544      ],
    3545       "status" : "reviewer"
     3545      "status" : "contributor"
    35463546   },
    35473547   "Kevin Ollivier" : {
    … …  
    35543554         "kollivier"
    35553555      ],
    3556       "status" : "reviewer"
     3556      "status" : "contributor"
    35573557   },
    35583558   "Kihong Kwon" : {
    … …  
    37103710         "lars"
    37113711      ],
    3712       "status" : "reviewer"
     3712      "status" : "contributor"
    37133713   },
    37143714   "Laszlo Gombos" : {
    … …  
    37233723         "lgombos"
    37243724      ],
    3725       "status" : "reviewer"
     3725      "status" : "contributor"
    37263726   },
    37273727   "Laszlo Vidacs" : {
    … …  
    37863786         "leviw"
    37873787      ],
    3788       "status" : "reviewer"
     3788      "status" : "contributor"
    37893789   },
    37903790   "Li Yin" : {
    … …  
    38553855         "lca"
    38563856      ],
    3857       "status" : "reviewer"
     3857      "status" : "contributor"
    38583858   },
    38593859   "Luke Macpherson" : {
    … …  
    39553955         "mhahnenberg"
    39563956      ],
    3957       "status" : "reviewer"
     3957      "status" : "contributor"
    39583958   },
    39593959   "Mark Lam" : {
    … …  
    39843984         "bdash"
    39853985      ],
    3986       "status" : "reviewer"
     3986      "status" : "contributor"
    39873987   },
    39883988   "Martin Hock" : {
    … …  
    40144014         "mrobinson"
    40154015      ],
    4016       "status" : "reviewer"
     4016      "status" : "contributor"
    40174017   },
    40184018   "Mary Wu" : {
    … …  
    42614261         "mihaip"
    42624262      ],
    4263       "status" : "reviewer"
     4263      "status" : "contributor"
    42644264   },
    42654265   "Mihai Tica" : {
    … …  
    42814281         "mihnea"
    42824282      ],
    4283       "status" : "reviewer"
     4283      "status" : "contributor"
    42844284   },
    42854285   "Mike Belshe" : {
    … …  
    44404440         "natechapin"
    44414441      ],
    4442       "status" : "reviewer"
     4442      "status" : "contributor"
    44434443   },
    44444444   "Nayan Kumar K" : {
    … …  
    44714471         "thakis"
    44724472      ],
    4473       "status" : "reviewer"
     4473      "status" : "contributor"
    44744474   },
    44754475   "Nikita Vasilyev" : {
    … …  
    44994499         "wildfox"
    45004500      ],
    4501       "status" : "reviewer"
     4501      "status" : "contributor"
    45024502   },
    45034503   "Nils Barth" : {
    … …  
    45294529         "noamr"
    45304530      ],
    4531       "status" : "reviewer"
     4531      "status" : "contributor"
    45324532   },
    45334533   "Noel Gordon" : {
    … …  
    45514551         "ojan"
    45524552      ],
    4553       "status" : "reviewer"
     4553      "status" : "contributor"
    45544554   },
    45554555   "Oliver Hunt" : {
    … …  
    45614561         "olliej"
    45624562      ],
    4563       "status" : "reviewer"
     4563      "status" : "contributor"
    45644564   },
    45654565   "Pablo Flouret" : {
    … …  
    46084608         "pfeldman"
    46094609      ],
    4610       "status" : "reviewer"
     4610      "status" : "contributor"
    46114611   },
    46124612   "Pavel Podivilov" : {
    … …  
    47094709         "pdr"
    47104710      ],
    4711       "status" : "reviewer"
     4711      "status" : "contributor"
    47124712   },
    47134713   "Philippe Normand" : {
    … …  
    47614761         "psolanki"
    47624762      ],
    4763       "status" : "reviewer"
     4763      "status" : "contributor"
    47644764   },
    47654765   "Pravin D" : {
    … …  
    48944894         "rjw"
    48954895      ],
    4896       "status" : "reviewer"
     4896      "status" : "contributor"
    48974897   },
    48984898   "Ricky Mondello" : {
    … …  
    52015201         "hamaji"
    52025202      ],
    5203       "status" : "reviewer"
     5203      "status" : "contributor"
    52045204   },
    52055205   "Shinya Kawanaka" : {
    … …  
    52735273         "tronical"
    52745274      ],
    5275       "status" : "reviewer"
     5275      "status" : "contributor"
    52765276   },
    52775277   "Simon Pena" : {
    … …  
    53135313         "schenney"
    53145314      ],
    5315       "status" : "reviewer"
     5315      "status" : "contributor"
    53165316   },
    53175317   "Stephen White" : {
    … …  
    53235323         "senorblanco"
    53245324      ],
    5325       "status" : "reviewer"
     5325      "status" : "contributor"
    53265326   },
    53275327   "Steve Block" : {
    … …  
    53345334         "steveblock"
    53355335      ],
    5336       "status" : "reviewer"
     5336      "status" : "contributor"
    53375337   },
    53385338   "Steve Falkenburg" : {
    … …  
    53445344         "sfalken"
    53455345      ],
    5346       "status" : "reviewer"
     5346      "status" : "contributor"
    53475347   },
    53485348   "Steve Lacey" : {
    … …  
    55825582         "timo@apple.com"
    55835583      ],
    5584       "status" : "reviewer"
     5584      "status" : "contributor"
    55855585   },
    55865586   "Tim Volodine" : {
    … …  
    56625662         "tony^work"
    56635663      ],
    5664       "status" : "reviewer"
     5664      "status" : "contributor"
    56655665   },
    56665666   "Tony Gentilcore" : {
    … …  
    56725672         "tonyg-cr"
    56735673      ],
    5674       "status" : "reviewer"
     5674      "status" : "contributor"
    56755675   },
    56765676   "Tor Arne Vestb\u00f8" : {
    … …  
    56845684         "torarne"
    56855685      ],
    5686       "status" : "reviewer"
     5686      "status" : "contributor"
    56875687   },
    56885688   "Trey Matteson" : {
    … …  
    57375737         "vicki@apple.com"
    57385738      ],
    5739       "status" : "reviewer"
     5739      "status" : "contributor"
    57405740   },
    57415741   "Victor Carbune" : {
    … …  
    58235823         "vsevik"
    58245824      ],
    5825       "status" : "reviewer"
     5825      "status" : "contributor"
    58265826   },
    58275827   "W. James MacLean" : {
    … …  
    59875987         "yong"
    59885988      ],
    5989       "status" : "reviewer"
     5989      "status" : "contributor"
    59905990   },
    59915991   "Yongjun Zhang" : {
    … …  
    60506050         "yurys"
    60516051      ],
    6052       "status" : "reviewer"
     6052      "status" : "contributor"
    60536053   },
    60546054   "Yusuke Suzuki" : {
    … …  
    60726072         "yutak"
    60736073      ],
    6074       "status" : "reviewer"
     6074      "status" : "contributor"
    60756075   },
    60766076   "Yuzo Fujishima" : {
    … …  
    60916091         "zackr"
    60926092      ],
    6093       "status" : "reviewer"
     6093      "status" : "contributor"
    60946094   },
    60956095   "Zalan Bujtas" : {
    … …  
    61536153         "zherczeg"
    61546154      ],
    6155       "status" : "reviewer"
     6155      "status" : "contributor"
    61566156   },
    61576157   "Zoltan Horvath" : {
    … …  
    61666166         "zoltan"
    61676167      ],
    6168       "status" : "reviewer"
     6168      "status" : "contributor"
    61696169   },
    61706170   "Zsolt Borbely" : {

  • trunk/Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla_mock.py

    r234856 r244705  
    7474    "commit-queue": "+",
    7575    "committer_email": "non-committer@example.com",
    76     "attacher_email": "eric@webkit.org",
     76    "attacher_email": "ap@webkit.org",
    7777}
    7878
    … …  
    115115    "review": "+",
    116116    "reviewer_email": "foo@bar.com",
    117     "attacher_email": "eric@webkit.org",
     117    "attacher_email": "ap@webkit.org",
    118118}
    119119

  • trunk/Tools/Scripts/webkitpy/tool/bot/irc_command_unittest.py

    r243321 r244705  
    4747        self.assertEqual("tom: Sorry, I don't know any contributors matching 'unknown@example.com'.",
    4848                          whois.execute("tom", ["unknown@example.com"], None, None))
    49         self.assertEqual('tom: tonyg@chromium.org is "Tony Gentilcore" <tonyg@chromium.org> (:tonyg-cr) (r). Why do you ask?',
     49        self.assertEqual('tom: tonyg@chromium.org is "Tony Gentilcore" <tonyg@chromium.org> (:tonyg-cr). Why do you ask?',
    5050                          whois.execute("tom", ["tonyg@chromium.org"], None, None))
    51         self.assertEqual('tom: TonyG@Chromium.org is "Tony Gentilcore" <tonyg@chromium.org> (:tonyg-cr) (r). Why do you ask?',
     51        self.assertEqual('tom: TonyG@Chromium.org is "Tony Gentilcore" <tonyg@chromium.org> (:tonyg-cr). Why do you ask?',
    5252                          whois.execute("tom", ["TonyG@Chromium.org"], None, None))
    5353        self.assertEqual('tom: rniwa is "Ryosuke Niwa" <rniwa@webkit.org> (:rniwa) (r). Why do you ask?',
    … …  
    5555        self.assertEqual('tom: Xan Lopez is "Xan Lopez" <xan.lopez@gmail.com> (:xan) (r). Why do you ask?',
    5656                          whois.execute("tom", ["Xan", "Lopez"], None, None))
    57         self.assertEqual(u'tom: Osztrogon\u00e1c is "Csaba Osztrogon\u00e1c" <ossy@webkit.org> (:ossy) (r). Why do you ask?',
     57        self.assertEqual(u'tom: Osztrogon\u00e1c is "Csaba Osztrogon\u00e1c" <ossy@webkit.org> (:ossy). Why do you ask?',
    5858                          whois.execute("tom", [u'Osztrogon\u00e1c'], None, None))
    5959        self.assertEqual('tom: "Vicki Murley" <vicki@apple.com> hasn\'t told me their nick. Boo hoo :-(',
    6060                          whois.execute("tom", ["vicki@apple.com"], None, None))
    61         self.assertEqual('tom: I\'m not sure who you mean?  "Gavin Barraclough" <barraclough@apple.com> (:gbarra) (r) or "Gavin Peters" <gavinp@chromium.org> (:gavinp) could be \'Gavin\'.',
     61        self.assertEqual('tom: I\'m not sure who you mean?  "Gavin Barraclough" <barraclough@apple.com> (:gbarra) or "Gavin Peters" <gavinp@chromium.org> (:gavinp) could be \'Gavin\'.',
    6262                          whois.execute("tom", ["Gavin"], None, None))
    6363        self.assertEqual('tom: More than 5 contributors match \'david\', could you be more specific?',

  • trunk/Tools/Scripts/webkitpy/tool/bot/ircbot_unittest.py

    r223177 r244705  
    9696
    9797    def test_rollout(self):
    98         expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, darin, eseidel: Created rollout: http://example.com/36936\n"
     98        expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, ap, darin: Created rollout: http://example.com/36936\n"
    9999        OutputCapture().assert_outputs(self, run, args=["rollout 21654 This patch broke the world"], expected_logs=expected_logs)
    100100
    101101    def test_revert(self):
    102         expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, darin, eseidel: Created rollout: http://example.com/36936\n"
     102        expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, ap, darin: Created rollout: http://example.com/36936\n"
    103103        OutputCapture().assert_outputs(self, run, args=["revert 21654 This patch broke the world"], expected_logs=expected_logs)
    104104
    105105    def test_multi_rollout(self):
    106         expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654, https://trac.webkit.org/changeset/21655, and https://trac.webkit.org/changeset/21656 ...\nMOCK: irc.post: mock_nick, abarth, darin, eseidel: Created rollout: http://example.com/36936\n"
     106        expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654, https://trac.webkit.org/changeset/21655, and https://trac.webkit.org/changeset/21656 ...\nMOCK: irc.post: mock_nick, abarth, ap, darin: Created rollout: http://example.com/36936\n"
    107107        OutputCapture().assert_outputs(self, run, args=["rollout 21654 21655 21656 This 21654 patch broke the world"], expected_logs=expected_logs)
    108108
    109109    def test_rollout_with_r_in_svn_revision(self):
    110         expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, darin, eseidel: Created rollout: http://example.com/36936\n"
     110        expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...\nMOCK: irc.post: mock_nick, abarth, ap, darin: Created rollout: http://example.com/36936\n"
    111111        OutputCapture().assert_outputs(self, run, args=["rollout r21654 This patch broke the world"], expected_logs=expected_logs)
    112112
    113113    def test_multi_rollout_with_r_in_svn_revision(self):
    114         expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654, https://trac.webkit.org/changeset/21655, and https://trac.webkit.org/changeset/21656 ...\nMOCK: irc.post: mock_nick, abarth, darin, eseidel: Created rollout: http://example.com/36936\n"
     114        expected_logs = "MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654, https://trac.webkit.org/changeset/21655, and https://trac.webkit.org/changeset/21656 ...\nMOCK: irc.post: mock_nick, abarth, ap, darin: Created rollout: http://example.com/36936\n"
    115115        OutputCapture().assert_outputs(self, run, args=["rollout r21654 21655 r21656 This r21654 patch broke the world"], expected_logs=expected_logs)
    116116
    … …  
    129129        # FIXME: I'm slightly confused as to why this doesn't return the USAGE message.
    130130        expected_logs = """MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654 ...
    131 MOCK: irc.post: mock_nick, abarth, darin, eseidel: Failed to create rollout patch:
     131MOCK: irc.post: mock_nick, abarth, ap, darin: Failed to create rollout patch:
    132132MOCK: irc.post: The rollout reason may not begin with - (\"-bad (Requested by mock_nick on #webkit).\").
    133133"""
    … …  
    138138    def test_multi_rollout_invalidate_reason(self):
    139139        expected_logs = """MOCK: irc.post: mock_nick: Preparing rollout for https://trac.webkit.org/changeset/21654, https://trac.webkit.org/changeset/21655, and https://trac.webkit.org/changeset/21656 ...
    140 MOCK: irc.post: mock_nick, abarth, darin, eseidel: Failed to create rollout patch:
     140MOCK: irc.post: mock_nick, abarth, ap, darin: Failed to create rollout patch:
    141141MOCK: irc.post: The rollout reason may not begin with - (\"-bad (Requested by mock_nick on #webkit).\").
    142142"""

  • trunk/Tools/Scripts/webkitpy/tool/commands/queries_unittest.py

    r232979 r244705  
    7676    def test_patches_to_commit_queue(self):
    7777        expected_stdout = "http://example.com/10003&action=edit\n"
    78         expected_logs = "10000 already has cq=+\n10001 already has cq=+\n10004 committer = \"Eric Seidel\" <eric@webkit.org>\n"
     78        expected_logs = "10000 already has cq=+\n10001 already has cq=+\n10004 committer = \"Alexey Proskuryakov\" <ap@webkit.org>\n"
    7979        options = Mock()
    8080        options.bugs = False

  • trunk/Tools/Scripts/webkitpy/tool/commands/upload_unittest.py

    r237037 r244705  
    4646        tool = MockTool()
    4747        expected_logs = """Warning, attachment 10001 on bug 50000 has invalid committer (non-committer@example.com)
    48 MOCK reassign_bug: bug_id=50000, assignee=eric@webkit.org
     48MOCK reassign_bug: bug_id=50000, assignee=ap@webkit.org
    4949-- Begin comment --
    50 Attachment 10001 was posted by a committer and has review+, assigning to Eric Seidel for commit.
     50Attachment 10001 was posted by a committer and has review+, assigning to Alexey Proskuryakov for commit.
    5151-- End comment --
    5252Bug 50003 is already assigned to foo@foo.com (None).
Note: See TracChangeset for help on using the changeset viewer.