mirror of
https://github.com/TencentCloud/Octop.git
synced 2026-10-21 06:29:24 +08:00
HUANG Cheng
34f483b8ef
Lazy CLI lookup used getattr(..., "acp") but the Click command is acp_cmd, so every `octop acp` invocation crashed before help parsing. Fixes #486
30 lines
1.0 KiB
Python
30 lines
1.0 KiB
Python
"""Lazy CLI registry integrity — every COMMANDS attr must resolve."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import importlib
|
|
|
|
import click
|
|
from click.testing import CliRunner
|
|
|
|
from octop.cli.main import cli
|
|
from octop.cli.registry import COMMANDS
|
|
|
|
|
|
def test_all_registry_attrs_are_click_commands() -> None:
|
|
for name, (module_path, attr, _help) in COMMANDS.items():
|
|
mod = importlib.import_module(module_path, package="octop.cli")
|
|
assert hasattr(mod, attr), f"{name}: module {module_path} missing attr {attr!r}"
|
|
result = getattr(mod, attr)
|
|
assert isinstance(result, click.Command), (
|
|
f"{name}: {module_path}.{attr} is {type(result).__name__}, not click.Command"
|
|
)
|
|
|
|
|
|
def test_acp_help_loads() -> None:
|
|
"""Regression for #486: registry must point at acp_cmd, not missing attr acp."""
|
|
runner = CliRunner()
|
|
result = runner.invoke(cli, ["acp", "--help"])
|
|
assert result.exit_code == 0, result.output or result.exception
|
|
assert "ACP" in result.output or "acp" in result.output.lower()
|