diff --git a/zerver/webhooks/github/tests.py b/zerver/webhooks/github/tests.py index 72a4a02249..146c865b18 100644 --- a/zerver/webhooks/github/tests.py +++ b/zerver/webhooks/github/tests.py @@ -240,13 +240,13 @@ class GitHubWebhookTest(WebhookTestCase): self.check_webhook("status__with_target_url", TOPIC_REPO, expected_message) def test_pull_request_review_msg(self) -> None: - expected_message = "baxterthehacker submitted [PR review](https://github.com/baxterthehacker/public-repo/pull/1#pullrequestreview-2626884)." + expected_message = "baxterthehacker submitted [PR review](https://github.com/baxterthehacker/public-repo/pull/1#pullrequestreview-2626884):\n\n~~~ quote\nLooks great!\n~~~" self.check_webhook("pull_request_review", TOPIC_PR, expected_message) def test_pull_request_review_msg_with_custom_topic_in_url(self) -> None: self.url = self.build_webhook_url(topic="notifications") expected_topic = "notifications" - expected_message = "baxterthehacker submitted [PR review for #1 Update the README with new information](https://github.com/baxterthehacker/public-repo/pull/1#pullrequestreview-2626884)." + expected_message = "baxterthehacker submitted [PR review for #1 Update the README with new information](https://github.com/baxterthehacker/public-repo/pull/1#pullrequestreview-2626884):\n\n~~~ quote\nLooks great!\n~~~" self.check_webhook("pull_request_review", expected_topic, expected_message) def test_pull_request_review_comment_msg(self) -> None: diff --git a/zerver/webhooks/github/view.py b/zerver/webhooks/github/view.py index ea22a580ea..8b414adcac 100644 --- a/zerver/webhooks/github/view.py +++ b/zerver/webhooks/github/view.py @@ -490,6 +490,7 @@ def get_pull_request_review_body(helper: Helper) -> str: url=payload["review"]["html_url"].tame(check_string), type="PR review", title=title if include_title else None, + message=payload["review"]["body"].tame(check_string), )