Ignore new comments

This removes the handling of `r?`.

Author: Mark-Simulacrum

highfive/newpr.py

@@ -68,9 +68,6 @@ def run(self, event):
         elif event == "pull_request" and self.payload["action"] == "opened":
             self.new_pr()
             return 'OK\n'
-        elif event == "issue_comment" and self.payload["action"] == "created":
-            self.new_comment()
-            return 'OK\n'
         else:
             return 'Unsupported webhook event.\n'
 
@@ -398,37 +395,3 @@ def new_pr(self):
 
         if self.repo_config.get("new_pr_labels"):
             self.add_labels(owner, repo, issue)
-
-    def new_comment(self):
-        # Check the issue is a PR and is open.
-        if self.payload['issue', 'state'] != 'open' \
-                or 'pull_request' not in self.payload['issue']:
-            return
-
-        commenter = self.payload['comment', 'user', 'login']
-        # Ignore our own comments.
-        if commenter == self.integration_user:
-            return
-
-        owner = self.payload['repository', 'owner', 'login']
-        repo = self.payload['repository', 'name']
-
-        # Check the commenter is the submitter of the PR or the previous assignee.
-        author = self.payload['issue', 'user', 'login']
-        if not (author == commenter or (
-                self.payload['issue', 'assignee'] \
-                and commenter == self.payload['issue', 'assignee', 'login']
-        )):
-            # Check if commenter is a collaborator.
-            if not self.is_collaborator(commenter, owner, repo):
-                return
-
-        # Check for r? and set the assignee.
-        msg = self.payload['comment', 'body']
-        reviewer = self.find_reviewer(msg)
-        if reviewer:
-            issue = str(self.payload['issue', 'number'])
-            self.set_assignee(
-                reviewer, owner, repo, issue, self.integration_user,
-                author, None
-            )

highfive/tests/fakes.py

@@ -65,12 +65,6 @@ def get_global_configs():
 
 
 class Payload(object):
-    @staticmethod
-    def new_comment():
-        with open(get_fake_filename('create-comment.payload'), 'r') as fin:
-            p = json.load(fin)
-        return payload.Payload(p)
-
     @staticmethod
     def new_pr(
             number=7, pr_body='The PR comment.', pr_url='https://the.url/',

highfive/tests/fakes/create-comment.payload

@@ -258,45 +258,3 @@ def make_mocks(cls, patcherize):
         config_mock = mock.Mock()
         config_mock.get.side_effect = ('integration-user', 'integration-token')
         cls.mocks['ConfigParser'].RawConfigParser.return_value = config_mock
-
-    def test_author_is_commenter(self):
-        payload = fakes.Payload.new_comment()
-        handler = newpr.HighfiveHandler(payload, dummy_config())
-        api_req_mock = ApiReqMocker([
-            (
-                (
-                    'PATCH', newpr.issue_url % ('rust-lang', 'rust', '1'),
-                    {'assignee': 'davidalber'}
-                ),
-                {'body': {}},
-            ),
-        ])
-        handler.new_comment()
-        api_req_mock.verify_calls()
-
-    def test_author_not_commenter_is_collaborator(self):
-        payload = fakes.Payload.new_comment()
-        payload._payload['issue']['user']['login'] = 'foouser'
-
-        handler = newpr.HighfiveHandler(payload, dummy_config())
-        api_req_mock = ApiReqMocker([
-            (
-                (
-                    "GET",
-                    newpr.user_collabo_url % (
-                        'rust-lang', 'rust', 'davidalber'
-                    ),
-                    None
-                ),
-                {'body': {}},
-            ),
-            (
-                (
-                    'PATCH', newpr.issue_url % ('rust-lang', 'rust', '1'),
-                    {'assignee': 'davidalber'}
-                ),
-                {'body': {}},
-            ),
-        ])
-        handler.new_comment()
-        api_req_mock.verify_calls()