Fix: Formatting

Ref.: DEVOPS-1269
greenbone · Nov 19, 2024 · 1ee82be · 1ee82be
1 parent 7943353
commit 1ee82be
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 23 deletions.
diff --git a/mattermost_notify/git.py b/mattermost_notify/git.py
@@ -26,8 +26,10 @@
     "{highlight}"
 )
 
-SHORT_TEMPLATE = ("{status_emoji} {status_text}: {workflow} | {repository} "
-                  "(b {branch}) {highlight}")
+SHORT_TEMPLATE = (
+    "{status_emoji} {status_text}: {workflow} | {repository} "
+    "(b {branch}) {highlight}"
+)
 
 DEFAULT_GIT = "https://github.com"
 

diff --git a/mattermost_notify/post.py b/mattermost_notify/post.py
@@ -2,13 +2,13 @@
 #
 # SPDX-License-Identifier: GPL-3.0-or-later
 
-from enum import Enum
 import httpx
 from pontos.typing import SupportsStr
 
 from mattermost_notify.errors import MattermostNotifyError
 
-class Colors(Enum):
+
+class Colors:
     """
     Colors namespace for mattermost notifications.
     The colors are borrowed from bootstrap 5.
@@ -19,11 +19,9 @@ class Colors(Enum):
     WARNING = "#e0a800"
     DANGER = "#c82333"
 
+
 def post(
-    url: str,
-    channel: str,
-    text: SupportsStr,
-    color: str = Colors.SECONDARY
+    url: str, channel: str, text: SupportsStr, color: str = Colors.SECONDARY
 ) -> None:
     """
     Post a message to a Mattermost channel.
@@ -38,14 +36,20 @@ def post(
     Raises:
         MattermostNotifyError: If the HTTP request fails.
     """
-
-    response = httpx.post(url=url, json={"channel": channel, "attachments": [
-            {
-                "color": color,
-                "text": text,
-                "fallback": text,
-            }
-        ]})
+
+    response = httpx.post(
+        url=url,
+        json={
+            "channel": channel,
+            "attachments": [
+                {
+                    "color": color,
+                    "text": text,
+                    "fallback": text,
+                }
+            ],
+        },
+    )
 
     if not response.is_success:
         raise MattermostNotifyError(

diff --git a/tests/test_post.py b/tests/test_post.py
@@ -15,13 +15,25 @@ def test_success(self, post_mock: MagicMock):
         response = post_mock.return_value
         response.is_success = True
 
-        post("https://some.mattermost.url", "FooChannel", "Some Message", color=Colors.SUCCESS)
+        post(
+            "https://some.mattermost.url",
+            "FooChannel",
+            "Some Message",
+            color=Colors.SUCCESS,
+        )
 
         post_mock.assert_called_once_with(
             url="https://some.mattermost.url",
-            json={'channel': 'FooChannel', 'attachments': [
-                    {'color': '#28a745', 'text': 'Some Message', 'fallback': 'Some Message'}
-                ]},
+            json={
+                "channel": "FooChannel",
+                "attachments": [
+                    {
+                        "color": "#28a745",
+                        "text": "Some Message",
+                        "fallback": "Some Message",
+                    }
+                ],
+            },
         )
 
     @patch("mattermost_notify.post.httpx.post", autospec=True)
@@ -38,7 +50,14 @@ def test_failure(self, post_mock: MagicMock):
 
         post_mock.assert_called_once_with(
             url="https://some.mattermost.url",
-            json={'channel': 'FooChannel', 'attachments': [
-                    {'color': '#6c757d', 'text': 'Some Message', 'fallback': 'Some Message'}
-                ]},
+            json={
+                "channel": "FooChannel",
+                "attachments": [
+                    {
+                        "color": "#6c757d",
+                        "text": "Some Message",
+                        "fallback": "Some Message",
+                    }
+                ],
+            },
         )