[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

ogr/services/forgejo/commit_flag.py

@@ -1,12 +1,14 @@
 # Copyright Contributors to the Packit project.
 # SPDX-License-Identifier: MIT
 
+from datetime import datetime
+from typing import Any, List
+
 import requests
 
-from datetime import datetime
-from typing import Any, List, Optional
 from ogr.abstract import CommitFlag, CommitStatus
 
+
 class ForgejoCommitFlag(CommitFlag):
     """
     CommitFlag implementation for Forgejo.
@@ -18,17 +20,20 @@ def _state_from_str(cls, state: str) -> CommitStatus:
         state = state.lower()
         if state == "success":
             return CommitStatus.success
-        elif state == "failure":
+        if state == "failure":
             return CommitStatus.failure
-        elif state == "pending":
+        if state == "pending":
             return CommitStatus.pending
-        else:
-            raise ValueError(f"Unknown commit state from Forgejo: {state}")
+        raise ValueError(f"Unknown commit state from Forgejo: {state}")
 
     @classmethod
     def _validate_state(cls, state: CommitStatus) -> CommitStatus:
         # Validate that the provided state is acceptable for Forgejo.
-        valid_states = {CommitStatus.success, CommitStatus.failure, CommitStatus.pending}
+        valid_states = {
+            CommitStatus.success,
+            CommitStatus.failure,
+            CommitStatus.pending,
+        }
         if state in valid_states:
             return state
         raise ValueError(f"Invalid commit state for Forgejo: {state}")
@@ -62,7 +67,11 @@ def get(project: Any, commit: str) -> List["CommitFlag"]:
         response = requests.get(url, headers=headers)
         flags = []
         for raw_flag in response.json():
-            flags.append(ForgejoCommitFlag(raw_commit_flag=raw_flag, project=project, commit=commit))
+            flags.append(
+                ForgejoCommitFlag(
+                    raw_commit_flag=raw_flag, project=project, commit=commit,
+                ),
+            )
         return flags
 
     @staticmethod
@@ -86,7 +95,9 @@ def set(
         }
         headers = project.get_auth_header()
         response = requests.post(url, json=payload, headers=headers)
-        return ForgejoCommitFlag(raw_commit_flag=response.json(), project=project, commit=commit)
+        return ForgejoCommitFlag(
+            raw_commit_flag=response.json(), project=project, commit=commit,
+        )
 
     @property
     def created(self) -> datetime:

tests/integration/forgejo/test_commit_flag.py

@@ -2,14 +2,14 @@
 # SPDX-License-Identifier: MIT
 
 import datetime
+from typing import Dict, List
+
 import responses
-import pytest
-from typing import Any, Dict, List, Optional
-import requests
 
-from ogr.abstract import CommitStatus, CommitFlag
+from ogr.abstract import CommitFlag, CommitStatus
 from ogr.services.forgejo.commit_flag import ForgejoCommitFlag
 
+
 class MockProject:
     forge_api_url = "http://dummy-forgejo/api/v1"
     owner = "dummy_owner"
@@ -18,28 +18,31 @@ class MockProject:
     def get_auth_header(self) -> Dict[str, str]:
         return {"Authorization": "Bearer dummy_token"}
 
+
 @responses.activate
 def test_get_commit_flag_integration():
     project = MockProject()
     commit = "abcdef123456"
     url = f"{project.forge_api_url}/repos/{project.owner}/{project.repo}/commits/{commit}/statuses"
-    
+
     # Dummy response data simulating Forgejo API output.
-    dummy_response = [{
-        "commit": commit,
-        "state": "success",
-        "context": "CI",
-        "comment": "All tests passed",
-        "id": "123",
-        "url": "http://dummy-forgejo/commit/abcdef123456/status",
-        "created": "2023-01-01T12:00:00Z",
-        "updated": "2023-01-01T12:30:00Z"
-    }]
+    dummy_response = [
+        {
+            "commit": commit,
+            "state": "success",
+            "context": "CI",
+            "comment": "All tests passed",
+            "id": "123",
+            "url": "http://dummy-forgejo/commit/abcdef123456/status",
+            "created": "2023-01-01T12:00:00Z",
+            "updated": "2023-01-01T12:30:00Z",
+        },
+    ]
     responses.add(responses.GET, url, json=dummy_response, status=200)
-    
+
     # Call the method under test.
     flags: List[CommitFlag] = ForgejoCommitFlag.get(project, commit)
-    
+
     # Assertions using CommitStatus from packit.ogr.abstract.
     assert len(flags) == 1
     flag = flags[0]
@@ -48,18 +51,23 @@ def test_get_commit_flag_integration():
     assert flag.context == "CI"
     assert flag.comment == "All tests passed"
     assert flag.uid == "123"
-    
-    expected_created = datetime.datetime.strptime("2023-01-01T12:00:00Z", "%Y-%m-%dT%H:%M:%SZ")
-    expected_updated = datetime.datetime.strptime("2023-01-01T12:30:00Z", "%Y-%m-%dT%H:%M:%SZ")
+
+    expected_created = datetime.datetime.strptime(
+        "2023-01-01T12:00:00Z", "%Y-%m-%dT%H:%M:%SZ",
+    )
+    expected_updated = datetime.datetime.strptime(
+        "2023-01-01T12:30:00Z", "%Y-%m-%dT%H:%M:%SZ",
+    )
     assert flag.created == expected_created
     assert flag.edited == expected_updated
 
+
 @responses.activate
 def test_set_commit_flag_integration():
     project = MockProject()
     commit = "abcdef123456"
     url = f"{project.forge_api_url}/repos/{project.owner}/{project.repo}/commits/{commit}/statuses"
-    
+
     # Dummy response for setting a commit status.
     dummy_response = {
         "commit": commit,
@@ -69,28 +77,32 @@ def test_set_commit_flag_integration():
         "id": "456",
         "url": "http://dummy-forgejo/commit/abcdef123456/status",
         "created": "2023-02-01T12:00:00Z",
-        "updated": "2023-02-01T12:30:00Z"
+        "updated": "2023-02-01T12:30:00Z",
     }
     responses.add(responses.POST, url, json=dummy_response, status=200)
-    
+
     # Call the set method to create a new commit flag.
     flag = ForgejoCommitFlag.set(
         project=project,
         commit=commit,
         state=CommitStatus.success,
         target_url="http://dummy-target",
         description="Build succeeded",
-        context="CI"
+        context="CI",
     )
-    
+
     # Assertions to verify correct mapping using CommitStatus.
     assert flag.commit == commit
     assert flag.state == CommitStatus.success
     assert flag.context == "CI"
     assert flag.comment == "Build succeeded"
     assert flag.uid == "456"
-    
-    expected_created = datetime.datetime.strptime("2023-02-01T12:00:00Z", "%Y-%m-%dT%H:%M:%SZ")
-    expected_updated = datetime.datetime.strptime("2023-02-01T12:30:00Z", "%Y-%m-%dT%H:%M:%SZ")
+
+    expected_created = datetime.datetime.strptime(
+        "2023-02-01T12:00:00Z", "%Y-%m-%dT%H:%M:%SZ",
+    )
+    expected_updated = datetime.datetime.strptime(
+        "2023-02-01T12:30:00Z", "%Y-%m-%dT%H:%M:%SZ",
+    )
     assert flag.created == expected_created
     assert flag.edited == expected_updated