Rename plugins to nox_task_plugins

Author: ArBridgeman

doc/changes/unreleased.md

@@ -10,4 +10,4 @@ and replaces them with `format:fix` and `format:check`.
 ## Feature
 
 * #614: Replaced `path_filters` with `BaseConfig.add_to_excluded_python_paths` and `BaseConfig.excluded_python_paths`
-* #626: Moved `plugins` into `BaseConfig`
+* #626: Replaced `plugins` with `BaseConfig.nox_task_plugins`

doc/user_guide/customization.rst

@@ -65,14 +65,14 @@ Plugin Registration
 
 Once the plugin class has been defined, it must be registered in the Nox configuration. This is done by adding the class to the `plugins` list within the `Config` data class.
 
-In the Nox `PROJECT_CONFIG`, you should amend the `plugins` list to include the new plugin:
+In the Nox `PROJECT_CONFIG`, you should amend the `nox_task_plugins` tuple to include the new plugin:
 
 .. code-block:: python
 
     from exasol.toolbox.config import BaseConfig
 
     PROJECT_CONFIG = BaseConfig(
-        plugins=(UpdateTemplates,), # register the plugin
+        nox_task_plugins=(UpdateTemplates,), # register the plugin
     )
 
 When Nox runs, it will instantiate `UpdateTemplates` with no arguments and integrate the hooks defined by the plugin into the execution lifecycle. All registered plugins’ hooks are called at their designated points in the Nox workflow.

exasol/toolbox/config.py

@@ -95,7 +95,7 @@ class BaseConfig(BaseModel):
         `exasol.toolbox.config.DEFAULT_EXCLUDED_PATHS`.
         """,
     )
-    plugins: tuple[ValidPluginHook, ...] = Field(
+    nox_task_plugins: tuple[ValidPluginHook, ...] = Field(
         default=(),
         description="""
         This is used to provide hooks to extend one or more of the Nox tasks provided

exasol/toolbox/nox/plugin.py

@@ -108,7 +108,7 @@ def post_integration_tests_hook(self, session, config, context):
     def plugin_manager(config) -> pluggy.PluginManager:
         pm = pluggy.PluginManager(_PLUGIN_MARKER)
         pm.add_hookspecs(NoxTasks)
-        for plugin in getattr(config, "plugins", []):
+        for plugin in getattr(config, "nox_task_plugins", []):
             pm.register(plugin())
         return pm
 

noxconfig.py

@@ -71,5 +71,5 @@ class Config(BaseConfig):
     # The PTB does not have integration tests run with an Exasol DB,
     # so for running in the CI, we take the first element.
     exasol_versions=(BaseConfig().exasol_versions[0],),
-    plugins=(UpdateTemplates,),
+    nox_task_plugins=(UpdateTemplates,),
 )

test/unit/config_test.py

@@ -110,20 +110,20 @@ def prepare_release_update_version(self, session, config, version: Version) -> N
 class TestPlugins:
     @staticmethod
     def test_works_when_empty():
-        BaseConfig(plugins=())
+        BaseConfig(nox_task_plugins=())
 
     @staticmethod
     def test_works_for_hook(capsys):
-        BaseConfig(plugins=(WithHook,))
+        BaseConfig(nox_task_plugins=(WithHook,))
 
     @staticmethod
     def test_raises_exception_method_with_hook_not_specified():
         with pytest.raises(ValidationError) as ex:
-            BaseConfig(plugins=(WithNotSpecifiedHook,))
+            BaseConfig(nox_task_plugins=(WithNotSpecifiedHook,))
         assert "Method `not_specified_anywhere`" in str(ex.value)
 
     @staticmethod
     def test_raises_exception_without_hook():
         with pytest.raises(ValidationError) as ex:
-            BaseConfig(plugins=(WithoutHook,))
+            BaseConfig(nox_task_plugins=(WithoutHook,))
         assert "No methods in `WithoutHook`" in str(ex.value)