even more ruff fixes

src/plugins/analysis/elf_analysis/test/test_plugin_elf_analysis.py

@@ -68,9 +68,7 @@ def test_get_color_code(self, analysis_plugin, tag, tag_color):
             (['a', 'b', 'c'], 'b', ['d', 'e'], ['x'], ['x']),
         ],
     )
-    def test_get_tags_from_library_list(  # noqa: PLR0913
-        self, analysis_plugin, indicators, behaviour_class, libraries, tags, expected
-    ):
+    def test_get_tags_from_library_list(self, analysis_plugin, indicators, behaviour_class, libraries, tags, expected):
         analysis_plugin._get_tags_from_library_list(libraries, behaviour_class, indicators, tags)
         assert tags == expected
 
@@ -85,7 +83,7 @@ def test_get_tags_from_library_list(  # noqa: PLR0913
             (['a', 'b'], 'c', ['b'], ['d', 'e'], ['d', 'e', 'c']),
         ],
     )
-    def test_get_tags_from_function_list(  # noqa: PLR0913
+    def test_get_tags_from_function_list(
         self, analysis_plugin, functions, behaviour_class, indicators, tags, expected_result
     ):
         analysis_plugin._get_tags_from_function_list(functions, behaviour_class, indicators, tags)

src/plugins/analysis/kernel_config/internal/kernel_config_hardening_check.py

@@ -150,7 +150,7 @@ def _add_protection_info(hardening_result: list[list[str]]) -> list[HardeningChe
     for single_result in hardening_result:
         config_key = single_result[0]
         actual_value = _detach_actual_value_from_result(single_result)
-        protection_info = PROTECTS_AGAINST[config_key] if config_key in PROTECTS_AGAINST else []
+        protection_info = PROTECTS_AGAINST.get(config_key, [])
         full_result.append(HardeningCheckResult(*single_result, actual_value, protection_info))
     return full_result
 

src/plugins/analysis/known_vulnerabilities/internal/rulebook.py

@@ -17,7 +17,7 @@ class BadRuleError(ValueError):
 
 
 class Vulnerability:
-    def __init__(self, rule, description, reliability, score, link, short_name):  # noqa: PLR0913
+    def __init__(self, rule, description, reliability, score, link, short_name):
         try:
             self.reliability = str(int(reliability))
             self.score = score

src/storage/db_interface_frontend.py

@@ -171,7 +171,7 @@ def get_latest_comments(self, limit=10):
 
     # --- generic search ---
 
-    def generic_search(  # noqa: PLR0913
+    def generic_search(
         self,
         search_dict: dict,
         skip: int = 0,
@@ -252,7 +252,7 @@ def generate_file_tree_nodes_for_uid_list(
         for entry in file_tree_data:
             yield from self.generate_file_tree_level(entry.uid, root_uid, parent_uid, whitelist, entry)
 
-    def generate_file_tree_level(  # noqa: PLR0913
+    def generate_file_tree_level(
         self,
         uid: str,
         root_uid: str,
@@ -308,7 +308,7 @@ def _get_included_files_for_uid_list(session, uid_list: list[str]) -> dict[str,
 
     # --- REST ---
 
-    def rest_get_firmware_uids(  # noqa: PLR0913
+    def rest_get_firmware_uids(
         self, offset: int, limit: int, query: Optional[dict] = None, recursive=False, inverted=False
     ):
         if query is None:

src/storage/entry_conversion.py

@@ -23,7 +23,7 @@ def firmware_from_entry(fw_entry: FirmwareEntry, analysis_filter: list[str] | No
     return firmware
 
 
-def file_object_from_entry(  # noqa: PLR0913
+def file_object_from_entry(
     fo_entry: FileObjectEntry,
     analysis_filter: list[str] | None = None,
     included_files: set[str] | None = None,
@@ -43,7 +43,7 @@ def _convert_vfp_entries_to_dict(vfp_list: list[VirtualFilePath]) -> dict[str, l
     return result
 
 
-def _populate_fo_data(  # noqa: PLR0913
+def _populate_fo_data(
     fo_entry: FileObjectEntry,
     file_object: FileObject,
     analysis_filter: list[str] | None = None,

src/test/acceptance/test_misc.py

@@ -98,7 +98,7 @@ def _add_comment(self, test_client):
         items_to_analyze=4 * 2 * 2,
     )
     @pytest.mark.usefixtures('intercom_backend_binding')
-    def test_misc(  # noqa: PLR0913
+    def test_misc(
         self,
         test_client,
         workload_statistic,

src/test/unit/scheduler/test_analysis.py

@@ -310,7 +310,7 @@ def setup_class(cls):
             (' 1.0', '1.1', '1.1', '1.0', False),  # invalid version string
         ],
     )
-    def test_analysis_is_already_in_db_and_up_to_date(  # noqa: PLR0913
+    def test_analysis_is_already_in_db_and_up_to_date(
         self, plugin_version, plugin_system_version, analysis_plugin_version, analysis_system_version, expected_output
     ):
         plugin = 'foo'

src/test/unit/web_interface/test_app_advanced_search.py

@@ -6,7 +6,7 @@
 
 class DbMock(CommonDatabaseMock):
     @staticmethod
-    def generic_search(  # noqa: PLR0913
+    def generic_search(
         search_dict: dict,
         skip: int = 0,  # noqa: ARG004
         limit: int = 0,  # noqa: ARG004

src/test/unit/web_interface/test_plugin_routes.py

@@ -15,7 +15,7 @@
 
 
 class PluginRoutesMock(PluginRoutes):
-    def __init__(self, app, db=None, intercom=None, api=None, status=None):  # noqa: PLR0913
+    def __init__(self, app, db=None, intercom=None, api=None, status=None):
         self._app = app
         self._api = api
         self.db = db

src/test/unit/web_interface/test_quick_search.py

@@ -6,7 +6,7 @@
 
 class DbMock(CommonDatabaseMock):
     @staticmethod
-    def generic_search(  # noqa: PLR0913
+    def generic_search(
         search_dict: dict,
         skip: int = 0,  # noqa: ARG004
         limit: int = 0,  # noqa: ARG004