troubadix 25.3.3__tar.gz → 25.3.4__tar.gz

{troubadix-25.3.3 → troubadix-25.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: troubadix
-Version: 25.3.3
+Version: 25.3.4
 Summary: A linting and QA check tool for NASL files
 License: GPL-3.0-or-later
 Author: Greenbone

{troubadix-25.3.3 → troubadix-25.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "troubadix"
-version = "25.3.3"
+version = "25.3.4"
 description = "A linting and QA check tool for NASL files"
 authors = ["Greenbone <info@greenbone.net>"]
 license = "GPL-3.0-or-later"

{troubadix-25.3.3 → troubadix-25.3.4}/tests/test_runner.py

@@ -351,6 +351,18 @@ class TestRunner(unittest.TestCase):
                 root=self.root,
             )
 
+    def test_unknown_plugins_with_suggestion(self):
+        with self.assertRaisesRegex(
+            ValueError,
+            r"Unknown plugins: 'check_badwards' \(Did you mean 'check_badwords'\?\)",
+        ):
+            Runner(
+                n_jobs=1,
+                reporter=self._reporter,
+                included_plugins=["check_badwards"],
+                root=self.root,
+            )
+
     def test_runner_log_file(self):
         included_plugins = [
             CheckDuplicateOID.name,

{troubadix-25.3.3 → troubadix-25.3.4}/troubadix/__version__.py

@@ -2,4 +2,4 @@
 
 # THIS IS AN AUTOGENERATED FILE. DO NOT TOUCH!
 
-__version__ = "25.3.3"
+__version__ = "25.3.4"

{troubadix-25.3.3 → troubadix-25.3.4}/troubadix/plugins/__init__.py

@@ -178,9 +178,7 @@ class StandardPlugins(Plugins):
         files_plugins = _FILES_PLUGINS
 
         if excluded_plugins:
-            self._check_unknown_plugins(
-                _FILE_PLUGINS + _FILES_PLUGINS, excluded_plugins
-            )
+            self._check_unknown_plugins(excluded_plugins)
 
             file_plugins = self._exclude_plugins(excluded_plugins, file_plugins)
             files_plugins = self._exclude_plugins(
@@ -188,9 +186,7 @@ class StandardPlugins(Plugins):
             )
 
         if included_plugins:
-            self._check_unknown_plugins(
-                _FILE_PLUGINS + _FILES_PLUGINS, included_plugins
-            )
+            self._check_unknown_plugins(included_plugins)
 
             file_plugins = self._include_plugins(included_plugins, file_plugins)
             files_plugins = self._include_plugins(
@@ -220,33 +216,25 @@ class StandardPlugins(Plugins):
         ]
 
     @staticmethod
-    def _check_unknown_plugins(found_plugins, selected_plugins):
-        if len(found_plugins) == len(selected_plugins):
-            return
-
+    def _check_unknown_plugins(selected_plugins: list[str]):
         all_plugin_names = {
             name
             for plugin in _FILE_PLUGINS + _FILES_PLUGINS
             for name in (plugin.name, plugin.__name__)
         }
-        found_plugin_names = {
-            name
-            for plugin in found_plugins
-            for name in (plugin.name, plugin.__name__)
-        }
 
-        unknown_plugins = set(selected_plugins).difference(found_plugin_names)
+        unknown_plugins = set(selected_plugins).difference(all_plugin_names)
 
         if not unknown_plugins:
             return
 
-        messages = []
-        for plugin in sorted(unknown_plugins):
-            message = f"'{plugin}'"
+        def build_message(plugin: str):
             match = difflib.get_close_matches(plugin, all_plugin_names, n=1)
-            if match:
-                message += f" (Did you mean '{match[0]}'?)"
-
-            messages.append(message)
+            return (
+                f"'{plugin}' (Did you mean '{match[0]}'?)"
+                if match
+                else f"'{plugin}'"
+            )
 
+        messages = [build_message(plugin) for plugin in sorted(unknown_plugins)]
         raise ValueError(f"Unknown plugins: {', '.join(messages)}")