chore: linter checks

Author: Skanislav

packages/valory/skills/staking_abci/behaviours.py

@@ -360,8 +360,8 @@ def _get_liveness_ratio(self) -> WaitableConditionType:
             placeholder=get_name(CallCheckpointBehaviour.liveness_ratio),
         )
         return status
-    
-    def ensure_service_id(self) -> WaitableConditionType:
+
+    def ensure_service_id(self) -> bool:
         """Ensure that the service id is set."""
         service_id = self.params.on_chain_service_id
         if service_id is None:
@@ -388,7 +388,7 @@ def _get_agent_ids(self) -> WaitableConditionType:
         """Get the service information to retrieve the agent ids."""
         if not self.ensure_service_id():
             return True
-        
+
         status = yield from self._staking_contract_interact(
             contract_callable="get_agent_ids",
             placeholder=get_name(CallCheckpointBehaviour.agent_ids),

packages/valory/skills/trader_abci/handlers.py

@@ -23,7 +23,7 @@
 import json
 from http import HTTPStatus
 from pathlib import Path
-from typing import Any, Dict, List, Union
+from typing import Any, Dict, List, Optional, Union
 from urllib.parse import urlparse
 
 from packages.valory.protocols.http.message import HttpMessage
@@ -146,13 +146,11 @@ def _send_ok_response(
         http_msg: HttpMessage,
         http_dialogue: HttpDialogue,
         data: Union[str, Dict, List, bytes],
-        content_type: str = None,
+        content_type: Optional[str] = None,
     ) -> None:
         """Send an OK response with the provided data"""
         headers = content_type or (
-            CONTENT_TYPES[".json"]
-            if isinstance(data, (dict, list))
-            else DEFAULT_HEADER
+            CONTENT_TYPES[".json"] if isinstance(data, (dict, list)) else DEFAULT_HEADER
         )
         headers += http_msg.headers
 
@@ -216,7 +214,9 @@ def _handle_get_static_file(
                 content_type = self._get_content_type(file_path)
 
                 # Send the file content as a response
-                self._send_ok_response(http_msg, http_dialogue, file_content, content_type)
+                self._send_ok_response(
+                    http_msg, http_dialogue, file_content, content_type
+                )
             else:
                 # If the file doesn't exist or is not a file, return the index.html file
                 with open(

packages/valory/skills/trader_abci/tests/tests_handlers.py

@@ -19,9 +19,9 @@
 """This module contains the tests for the handlers for the trader abci."""
 
 import json
+from pathlib import Path
 from typing import Any, Dict
 from unittest.mock import MagicMock, patch
-from pathlib import Path
 
 import pytest
 from aea.configurations.data_types import PublicId
@@ -56,15 +56,15 @@
     HandleTestCase,
 )
 from packages.valory.skills.trader_abci.handlers import (
+    CONTENT_TYPES,
     ContractApiHandler,
+    DEFAULT_HEADER,
     HttpHandler,
     IpfsHandler,
     LedgerApiHandler,
     SigningHandler,
     TendermintHandler,
     TraderHandler,
-    CONTENT_TYPES,
-    DEFAULT_HEADER,
 )
 
 
@@ -120,13 +120,19 @@ def test_get_content_type(self) -> None:
         """Test _get_content_type method."""
         # Test known extensions
         assert self.handler._get_content_type(Path("test.js")) == CONTENT_TYPES[".js"]
-        assert self.handler._get_content_type(Path("test.html")) == CONTENT_TYPES[".html"]
-        assert self.handler._get_content_type(Path("test.json")) == CONTENT_TYPES[".json"]
+        assert (
+            self.handler._get_content_type(Path("test.html")) == CONTENT_TYPES[".html"]
+        )
+        assert (
+            self.handler._get_content_type(Path("test.json")) == CONTENT_TYPES[".json"]
+        )
         assert self.handler._get_content_type(Path("test.css")) == CONTENT_TYPES[".css"]
         assert self.handler._get_content_type(Path("test.png")) == CONTENT_TYPES[".png"]
         assert self.handler._get_content_type(Path("test.jpg")) == CONTENT_TYPES[".jpg"]
-        assert self.handler._get_content_type(Path("test.jpeg")) == CONTENT_TYPES[".jpeg"]
-        
+        assert (
+            self.handler._get_content_type(Path("test.jpeg")) == CONTENT_TYPES[".jpeg"]
+        )
+
         # Test unknown extension
         assert self.handler._get_content_type(Path("test.xyz")) == DEFAULT_HEADER