Adding files to fix an issue with creating incidents (#585)

* Adding files to fix an issue with creating incidents * Adding the AWS_SSO_ROLE_ARN to the unit tests so that they dont fail
cds-snc · Jul 19, 2024 · 4c88a74 · 4c88a74
1 parent c6196eb
commit 4c88a74
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/.github/workflows/ci_code.yml b/.github/workflows/ci_code.yml
@@ -56,4 +56,5 @@ jobs:
           NOTIFY_SRE_USER_NAME: ${{ secrets.NOTIFY_SRE_USER_NAME }}
           NOTIFY_SRE_CLIENT_SECRET: ${{ secrets.NOTIFY_SRE_CLIENT_SECRET }}
           NOTIFY_OPS_CHANNEL_ID: ${{ secrets.NOTIFY_OPS_CHANNEL_ID }}
-          SRE_BOT_EMAIL: ${{ secrets.SRE_BOT_EMAIL }}
+          SRE_BOT_EMAIL: ${{ secrets.SRE_BOT_EMAIL }}
+          AWS_SSO_ROLE_ARN: ${{ secrets.AWS_SSO_ROLE_ARN }}
diff --git a/app/modules/incident/incident.py b/app/modules/incident/incident.py
@@ -286,7 +286,7 @@ def submit(ack, view, say, body, client, logger):
     )
 
     # Set the description
-    client.conversations_setDescription(channel=channel_id, description=f"{name}")
+    client.conversations_setPurpose(channel=channel_id, purpose=f"{name}")
 
     # Announce incident
     user_id = body["user"]["id"]

diff --git a/app/tests/modules/incident/test_incident.py b/app/tests/modules/incident/test_incident.py
@@ -580,8 +580,8 @@ def test_incident_submit_creates_channel_sets_description(
     }
     incident.submit(ack, view, say, body, client, logger)
     client.conversations_create.assert_called_once_with(name=f"incident-{DATE}-name")
-    client.conversations_setDescription.assert_called_once_with(
-        channel="channel_id", description="name"
+    client.conversations_setPurpose.assert_called_once_with(
+        channel="channel_id", purpose="name"
     )