Fix: Keep original text when mentee is claimed in mentors-intenral

Author: AllenAnthes

pybot/endpoints/slack/message_templates/mentor_request.py

@@ -178,6 +178,7 @@ def __init__(
         super().__init__(raw_action)
         self.slack = slack
         self.airtable = airtable
+        self.text = raw_action["original_message"]["text"]
         self.attachment = raw_action["original_message"]["attachments"][0]
         self.should_update = True
 
@@ -256,6 +257,7 @@ async def update_message(self):
         response = {
             "channel": self["channel"]["id"],
             "ts": self["message_ts"],
+            "text": self.text,
             "attachments": self["original_message"]["attachments"],
         }
         await self.slack.query(methods.CHAT_UPDATE, response)

tests/test_slack_actions.py

@@ -5,6 +5,22 @@
 async def test_claim_mentee_response_attachment_is_list(
     action: dict, aiohttp_client, bot: SirBot
 ):
+    client, slack_mock = await create_mocks(aiohttp_client, bot)
+
+    await client.post("/slack/actions", data=action)
+    assert isinstance(slack_mock.call_args[0][1]["attachments"], list)
+
+
+async def test_claim_mentee_response_contains_original_text(
+    action: dict, aiohttp_client, bot: SirBot
+):
+    client, slack_mock = await create_mocks(aiohttp_client, bot)
+    await client.post("/slack/actions", data=action)
+    request_payload = slack_mock.call_args[0][1]
+    assert request_payload["text"] is not None
+
+
+async def create_mocks(aiohttp_client, bot):
     slack_mock = CoroutineMock(
         return_value={"user": {"profile": {"email": "email@email.com"}}}
     )
@@ -14,7 +30,5 @@ async def test_claim_mentee_response_attachment_is_list(
     bot["plugins"]["airtable"].api.update_request = airtable_mock
     bot["plugins"]["airtable"].api.get_name_from_record_id = airtable_mock
     bot["plugins"]["airtable"].api.get_row_from_record_id = airtable_mock
-
     client = await aiohttp_client(bot)
-    await client.post("/slack/actions", data=action)
-    assert isinstance(slack_mock.call_args[0][1]["attachments"], list)
+    return client, slack_mock