linter: Multiple fixes across the project to satisfy linters

nuclearcat · nuclearcat · commit 26904b74b777 · 2026-03-13T11:25:00.000Z
Signed-off-by: Denys Fedoryshchenko &lt;denys.f@collabora.com&gt;
diff --git a/.pylintrc b/.pylintrc
@@ -56,7 +56,7 @@ py-version=3.10
 
 # When enabled, pylint would attempt to guess common misconfiguration and emit
 # user-friendly hints instead of false-positive error messages.
-suggestion-mode=yes
+# suggestion-mode is removed in pylint 4.x
 
 # Allow loading of arbitrary C extensions. Extensions are imported into the
 # active Python interpreter and may run arbitrary code.
@@ -564,5 +564,5 @@ min-public-methods=2
 
 # Exceptions that will emit a warning when being caught. Defaults to
 # "BaseException, Exception".
-overgeneral-exceptions=BaseException,
-                       Exception
+overgeneral-exceptions=builtins.BaseException,
+                       builtins.Exception
diff --git a/api/db.py b/api/db.py
@@ -73,31 +73,31 @@ async def get_kv(self, namespace, key):
         Get value from redis key-value store
         Create a keyname by concatenating namespace and key
         """
-        keyname = f"{namespace}:{key}"
+        keyname = ":".join([namespace, key])
         return await self._redis.get(keyname)
 
     async def set_kv(self, namespace, key, value):
         """
         Set value in redis key-value store
         Create a keyname by concatenating namespace and key
         """
-        keyname = f"{namespace}:{key}"
+        keyname = ":".join([namespace, key])
         return await self._redis.set(keyname, value)
 
     async def del_kv(self, namespace, key):
         """
         Delete key from redis key-value store
         Create a keyname by concatenating namespace and key
         """
-        keyname = f"{namespace}:{key}"
+        keyname = ":".join([namespace, key])
         return await self._redis.delete(keyname)
 
     async def exists_kv(self, namespace, key):
         """
         Check if key exists in redis key-value store
         Create a keyname by concatenating namespace and key
         """
-        keyname = f"{namespace}:{key}"
+        keyname = ":".join([namespace, key])
         return await self._redis.exists(keyname)
 
     async def create_indexes(self):
diff --git a/api/main.py b/api/main.py
@@ -306,7 +306,7 @@ def _is_proxy_addr() -> bool:
 
     if forwarded_host:
         scheme = forwarded_proto or request.url.scheme
-        return f"{scheme}://{forwarded_host}".rstrip("/")
+        return (scheme + "://" + forwarded_host).rstrip("/")
 
     return str(request.base_url).rstrip("/")
 
@@ -746,8 +746,8 @@ def _user_can_edit_node(user: User, node: Node) -> bool:
         return True
     runtime = _get_node_runtime(node)
     if runtime:
-        runtime_editor = f'runtime:{runtime}:node-editor'
-        runtime_admin = f'runtime:{runtime}:node-admin'
+        runtime_editor = ":".join(['runtime', runtime, 'node-editor'])
+        runtime_admin = ":".join(['runtime', runtime, 'node-admin'])
         if (runtime_editor in user_group_names
                 or runtime_admin in user_group_names):
             return True
diff --git a/api/pubsub.py b/api/pubsub.py
@@ -43,7 +43,8 @@ def __init__(self, host=None, db_number=None):
         if db_number is None:
             db_number = self._settings.redis_db_number
         self._redis = aioredis.from_url(
-            f'redis://{host}/{db_number}', health_check_interval=30
+            'redis://' + host + '/' + str(db_number),
+            health_check_interval=30
         )
         # self._subscriptions is a dict that matches a subscription id
         # (key) with a Subscription object ('sub') and a redis
diff --git a/api/pubsub_mongo.py b/api/pubsub_mongo.py
@@ -77,7 +77,8 @@ def __init__(self, mongo_client=None, host=None, db_number=None,
             db_number = self._settings.redis_db_number
 
         self._redis = aioredis.from_url(
-            f'redis://{host}/{db_number}', health_check_interval=30
+            'redis://' + host + '/' + str(db_number),
+            health_check_interval=30
         )
 
         # MongoDB setup
diff --git a/scripts/usermanager.py b/scripts/usermanager.py
@@ -294,8 +294,10 @@ def main():
         ("update-user", "Patch user by id/email/username"),
         ("whoami", "Show current user"),
     ]
-    command_list = "\n".join(f"  {name:<18} {desc}" for name, desc in command_help)
-    default_paths = "\n".join(f"  - {path}" for path in DEFAULT_CONFIG_PATHS)
+    command_list = "\n".join(
+        "  {:<18} {}".format(name, desc) for name, desc in command_help)
+    default_paths = "\n".join(
+        "  - {}".format(path) for path in DEFAULT_CONFIG_PATHS)
     parser = argparse.ArgumentParser(
         description="KernelCI API user management helper",
         epilog=(
diff --git a/tests/e2e_tests/conftest.py b/tests/e2e_tests/conftest.py
@@ -10,9 +10,10 @@
 from httpx import AsyncClient
 from motor.motor_asyncio import AsyncIOMotorClient
 
-from api.main import versioned_app
 from kernelci.api.models import Node, Regression
 
+from api.main import versioned_app
+
 BASE_URL = 'http://api:8000/latest/'
 DB_URL = 'mongodb://db:27017'
 DB_NAME = 'kernelci'
diff --git a/tests/e2e_tests/test_pipeline.py b/tests/e2e_tests/test_pipeline.py
@@ -37,8 +37,9 @@ async def test_node_pipeline(test_async_client):
     """
 
     # Create Task to listen pubsub event on 'node' channel
-    task_listen = create_listen_task(test_async_client,
-                                     pytest.node_channel_subscription_id)  # pylint: disable=no-member
+    task_listen = create_listen_task(
+        test_async_client,
+        pytest.node_channel_subscription_id)  # pylint: disable=no-member
 
     # Create a node
     node = {
@@ -48,8 +49,10 @@ async def test_node_pipeline(test_async_client):
         "data": {
             "kernel_revision": {
                 "tree": "mainline",
-                "url": ("https://git.kernel.org/pub/scm/linux/kernel/git/"
-                       "torvalds/linux.git"),
+                "url": (
+                    "https://git.kernel.org/pub/scm/"
+                    "linux/kernel/git/torvalds/linux.git"
+                ),
                 "branch": "master",
                 "commit": "2a987e65025e2b79c6d453b78cb5985ac6e5eb28",
                 "describe": "v5.16-rc4-31-g2a987e65025e"
@@ -73,8 +76,9 @@ async def test_node_pipeline(test_async_client):
     node = response.json()
 
     # Create Task to listen 'updated' event on 'node' channel
-    task_listen = create_listen_task(test_async_client,
-                                     pytest.node_channel_subscription_id)  # pylint: disable=no-member
+    task_listen = create_listen_task(
+        test_async_client,
+        pytest.node_channel_subscription_id)  # pylint: disable=no-member
 
     # Update node.state
     node.update({"state": "done"})
diff --git a/tests/e2e_tests/test_pubsub_handler.py b/tests/e2e_tests/test_pubsub_handler.py
@@ -23,8 +23,9 @@ async def test_pubsub_handler(test_async_client):
     Use pubsub listener task to verify published event message.
     """
     # Create Task to listen pubsub event on 'test_channel' channel
-    task_listen = create_listen_task(test_async_client,
-                                     pytest.test_channel_subscription_id)  # pylint: disable=no-member
+    task_listen = create_listen_task(
+        test_async_client,
+        pytest.test_channel_subscription_id)  # pylint: disable=no-member
 
     # Created and publish CloudEvent
     attributes = {
diff --git a/tests/unit_tests/conftest.py b/tests/unit_tests/conftest.py
@@ -38,7 +38,7 @@
 t3bAE-pHSzZaSHp7FMlImqgYvL6f_0xDUD-nQwxEm3k'
 
 API_VERSION = 'latest'
-BASE_URL = f'http://testserver/{API_VERSION}/'
+BASE_URL = 'http://testserver/' + API_VERSION + '/'
 
 
 def mock_get_current_user(request: Request):
@@ -90,6 +90,7 @@ def mock_get_current_admin_user(request: Request):
         is_verified=True
     )
 
+
 # Mock dependency callables for getting current user
 app.dependency_overrides[get_current_user] = mock_get_current_user
 app.dependency_overrides[get_current_superuser] = mock_get_current_admin_user
diff --git a/tests/unit_tests/test_events_handler.py b/tests/unit_tests/test_events_handler.py
@@ -34,7 +34,7 @@ def test_get_events_filter_by_ids(mock_db_find_by_attributes, test_client):
     oid1, oid2 = ObjectId(), ObjectId()
     mock_db_find_by_attributes.return_value = []
 
-    resp = test_client.get(f"events?ids={oid1},{oid2}")
+    resp = test_client.get("events?ids=" + oid1 + "," + oid2)
 
     assert resp.status_code == 200
     called_model, called_query = mock_db_find_by_attributes.call_args.args
diff --git a/tests/unit_tests/test_node_handler.py b/tests/unit_tests/test_node_handler.py
@@ -12,9 +12,10 @@
 
 import json
 
-from tests.unit_tests.conftest import BEARER_TOKEN
 from kernelci.api.models import Node, Revision
+
 from api.models import PageModel
+from tests.unit_tests.conftest import BEARER_TOKEN
 
 
 def test_create_node_endpoint(mock_db_create, mock_publish_cloudevent,
@@ -40,7 +41,7 @@ def test_create_node_endpoint(mock_db_create, mock_publish_cloudevent,
         name="checkout",
         path=["checkout"],
         group="debug",
-        data= {'kernel_revision': revision_obj},
+        data={'kernel_revision': revision_obj},
         parent=None,
         state="closing",
         result=None,
@@ -216,7 +217,7 @@ def test_get_node_by_id_endpoint(mock_db_find_by_id,
         name="checkout",
         path=["checkout"],
         group="blah",
-        data = {'kernel_revision': revision_obj},
+        data={'kernel_revision': revision_obj},
         parent=None,
         state="closing",
         result=None,
diff --git a/tests/unit_tests/test_pubsub.py b/tests/unit_tests/test_pubsub.py
@@ -99,16 +99,24 @@ async def test_pubsub_publish_couldevent(mock_pubsub_publish):
     """
 
     data = 'validate json'
-    attributes = { "specversion": "1.0",  "id": "6878b661-96dc-4e93-8c92-26eb9ff8db64",
-    "source": "https://api.kernelci.org/", "type": "api.kernelci.org",
-    "time": "2022-01-31T21:29:29.675593+00:00"}
+    attributes = {
+        "specversion": "1.0",
+        "id": "6878b661-96dc-4e93-8c92-26eb9ff8db64",
+        "source": "https://api.kernelci.org/",
+        "type": "api.kernelci.org",
+        "time": "2022-01-31T21:29:29.675593+00:00",
+    }
 
     await mock_pubsub_publish.publish_cloudevent('CHANNEL1', data, attributes)
 
-    expected_json = str.encode('{"specversion": "1.0", '\
-    '"id": "6878b661-96dc-4e93-8c92-26eb9ff8db64", "source": "https://api.kernelci.org/", '\
-    '"type": "api.kernelci.org", "time": "2022-01-31T21:29:29.675593+00:00", '\
-    '"data": "validate json"}')
+    expected_json = str.encode(
+        '{"specversion": "1.0", '
+        '"id": "6878b661-96dc-4e93-8c92-26eb9ff8db64", '
+        '"source": "https://api.kernelci.org/", '
+        '"type": "api.kernelci.org", '
+        '"time": "2022-01-31T21:29:29.675593+00:00", '
+        '"data": "validate json"}'
+    )
 
     json_arg = mock_pubsub_publish._redis.execute_command.call_args.args[2]
 
diff --git a/tests/unit_tests/test_user_handler.py b/tests/unit_tests/test_user_handler.py
@@ -129,7 +129,7 @@ async def test_create_user_endpoint_negative(test_async_client):
 
 @pytest.mark.asyncio
 async def test_create_user_with_group(test_async_client, mock_db_find_one,
-                                      mock_db_update,mock_db_find_by_id):
+                                      mock_db_update, mock_db_find_by_id):
     """
     Test Case : Test KernelCI API /user/register endpoint to create a user
     with a user group

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,8 @@ def __init__(self, mongo_client=None, host=None, db_number=None,`
`77`	`77`	`db_number = self._settings.redis_db_number`
`78`	`78`
`79`	`79`	`self._redis = aioredis.from_url(`
`80`		`- f'redis://{host}/{db_number}', health_check_interval=30`
	`80`	`+ 'redis://' + host + '/' + str(db_number),`
	`81`	`+ health_check_interval=30`
`81`	`82`	`)`
`82`	`83`
`83`	`84`	`# MongoDB setup`