refactor: simplify init command and improve test structure

1. Removed config_file option from init command
2. Converted test_base_command.py from class-based to function-based tests
3. Renamed TestCommand to DummyCommand in test_registry.py to avoid warnings
4. Updated test assertions to match new command output

Author: ritwik-g

helm_values_manager/cli.py

@@ -28,15 +28,9 @@ def main(ctx: typer.Context):
 @app.command()
 def init(
     release_name: str = typer.Option(..., "--release", "-r", help="Name of the Helm release"),
-    config_file: str = typer.Option(
-        "values-manager.yaml",
-        "--config",
-        "-c",
-        help="Path to the values manager configuration file",
-    ),
 ):
     """Initialize a new values manager configuration."""
-    typer.echo(f"Initializing values manager with config file: {config_file}, for the release: {release_name}.")
+    typer.echo(f"Initializing values manager for the release: {release_name}.")
     # TODO: Implement initialization logic
 
 

helm_values_manager/commands/registry.py

@@ -0,0 +1,71 @@
+"""Command registry for managing helm-values commands."""
+
+from typing import Dict, Type
+
+from helm_values_manager.commands.base_command import BaseCommand
+
+
+class CommandRegistry:
+    """Registry for managing helm-values commands.
+
+    This class provides a central registry for all available commands in the
+    helm-values plugin. It handles command registration and lookup.
+    """
+
+    _instance = None
+    _commands: Dict[str, Type[BaseCommand]] = {}
+
+    def __new__(cls):
+        """Create a new singleton instance of CommandRegistry."""
+        if cls._instance is None:
+            cls._instance = super(CommandRegistry, cls).__new__(cls)
+        return cls._instance
+
+    @classmethod
+    def register(cls, name: str, command_class: Type[BaseCommand]) -> None:
+        """Register a new command.
+
+        Args:
+            name: The name of the command
+            command_class: The command class to register
+
+        Raises:
+            ValueError: If a command with the same name already exists
+        """
+        if name in cls._commands:
+            raise ValueError(f"Command {name} already registered")
+        cls._commands[name] = command_class
+
+    @classmethod
+    def get_command(cls, name: str) -> Type[BaseCommand]:
+        """Get a command by name.
+
+        Args:
+            name: The name of the command to get
+
+        Returns:
+            The command class
+
+        Raises:
+            KeyError: If the command is not found
+        """
+        if name not in cls._commands:
+            raise KeyError(f"Command {name} not found")
+        return cls._commands[name]
+
+    @classmethod
+    def list_commands(cls) -> Dict[str, Type[BaseCommand]]:
+        """List all registered commands.
+
+        Returns:
+            A dictionary mapping command names to command classes
+        """
+        return cls._commands.copy()
+
+    @classmethod
+    def clear(cls) -> None:
+        """Clear all registered commands.
+
+        This is primarily used for testing.
+        """
+        cls._commands.clear()

tests/integration/test_cli_integration.py

@@ -70,13 +70,13 @@ def test_init_help_command(plugin_install):
 
     assert returncode == 0
     assert "Initialize a new values manager configuration" in stdout
-    assert "--config" in stdout
-    assert "-c" in stdout
+    assert "--release" in stdout
+    assert "-r" in stdout
 
 
 def test_init_command(plugin_install, tmp_path):
-    """Test that the init command works with default config file."""
+    """Test that the init command works."""
     stdout, stderr, returncode = run_helm_command(["values-manager", "init", "-r", "test"])
 
     assert returncode == 0
-    assert "Initializing values manager with config file: values-manager.yaml, for the release: test." in stdout
+    assert "Initializing values manager for the release: test." in stdout