@ngocsangairvds/vsaf 3.1.27 → 3.2.2

package/tools/vds-scripts/jira_orchestrator/tests/test_reindex.py

@@ -0,0 +1,81 @@
+from __future__ import annotations
+
+from unittest.mock import Mock, patch
+
+from typer.testing import CliRunner
+from vds_jira_orchestrator.cli import app
+from vds_jira_orchestrator.errors import JiraAdapterError
+
+
+def _runner() -> CliRunner:
+    return CliRunner()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_status(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.reindex_status.return_value = {"state": "IDLE"}
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["reindex", "status"])
+    assert result.exit_code == 0
+    mock_adapter.reindex_status.assert_called_once()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_start_requires_yes(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    result = runner.invoke(app, ["reindex", "start"])
+    assert result.exit_code != 0
+    assert "Refusing to start reindex without --yes" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_start_success(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.reindex.return_value = {"result": "OK"}
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["reindex", "start", "--yes"])
+    assert result.exit_code == 0
+    mock_adapter.reindex.assert_called_once_with()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_with_type(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.reindex_with_type.return_value = {"result": "OK"}
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["reindex", "with-type", "--type", "BACKGROUND", "--yes"])
+    assert result.exit_code == 0
+    mock_adapter.reindex_with_type.assert_called_once_with("BACKGROUND")
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_with_type_requires_type(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    result = runner.invoke(app, ["reindex", "with-type", "--yes"])
+    assert result.exit_code != 0
+    assert "--type required for with-type" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_reindex_status_adapter_error(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.reindex_status.side_effect = JiraAdapterError("failure")
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["reindex", "status"])
+    assert result.exit_code != 0
+    assert "Error: failure" in result.stderr

package/tools/vds-scripts/jira_orchestrator/tests/test_remote_links.py

@@ -0,0 +1,254 @@
+from __future__ import annotations
+
+import json
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+import pytest
+from typer.testing import CliRunner
+from vds_jira_orchestrator.cli import app
+
+
+@pytest.fixture
+def runner() -> CliRunner:
+    return CliRunner()
+
+
+def _configure_mocks(mock_settings_class: Mock, mock_adapter_class: Mock, adapter: Mock) -> None:
+    settings = Mock()
+    settings.base_url = "https://jira.example.com"
+    settings.username = "user"
+    settings.password = "pass"
+    settings.token = None
+    settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = settings
+    mock_adapter_class.return_value = adapter
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_list_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    adapter.get_issue_remote_links.return_value = [{"id": 1}, {"id": 2}]
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(app, ["remote-link", "list", "--issue-key", "NTTC-1"])
+
+    assert result.exit_code == 0
+    adapter.get_issue_remote_links.assert_called_once_with("NTTC-1")
+    assert "remote_links" in result.stdout
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_get_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    adapter.get_issue_remote_link_by_id.return_value = {"id": 5}
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        ["remote-link", "get", "--issue-key", "NTTC-2", "--remote-link-id", "5"],
+    )
+
+    assert result.exit_code == 0
+    adapter.get_issue_remote_link_by_id.assert_called_once_with("NTTC-2", 5)
+    assert "id" in result.stdout
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_create_requires_yes(
+    mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner
+) -> None:
+    adapter = Mock()
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        ["remote-link", "create", "--issue-key", "NTTC-3", "--url", "https://example.com"],
+    )
+
+    assert result.exit_code != 0
+    assert "--yes required" in result.stderr or "--yes required" in result.stdout
+    adapter.create_or_update_issue_remote_link.assert_not_called()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_create_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    adapter.create_or_update_issue_remote_link.return_value = {"id": 10}
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        [
+            "remote-link",
+            "create",
+            "--issue-key",
+            "NTTC-4",
+            "--url",
+            "https://example.com",
+            "--title",
+            "Sample",
+            "--relationship",
+            "relates to",
+            "--yes",
+        ],
+    )
+
+    assert result.exit_code == 0
+    adapter.create_or_update_issue_remote_link.assert_called_once()
+    call_args = adapter.create_or_update_issue_remote_link.call_args[0]
+    assert call_args[0] == "NTTC-4"
+    payload = call_args[1]
+    assert payload["object"]["url"] == "https://example.com"
+    assert payload["object"]["title"] == "Sample"
+    assert payload["relationship"] == "relates to"
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_create_with_data_file(
+    mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, tmp_path: Path
+) -> None:
+    adapter = Mock()
+    adapter.create_or_update_issue_remote_link.return_value = {"id": 11}
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    data = {"object": {"url": "https://example.com", "title": "From file"}}
+    data_file = tmp_path / "remote_link.json"
+    data_file.write_text(json.dumps(data))
+
+    result = runner.invoke(
+        app,
+        [
+            "remote-link",
+            "create",
+            "--issue-key",
+            "NTTC-5",
+            "--data-file",
+            str(data_file),
+            "--yes",
+        ],
+    )
+
+    assert result.exit_code == 0
+    adapter.create_or_update_issue_remote_link.assert_called_once_with("NTTC-5", data)
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_update_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    adapter.update_issue_remote_link_by_id.return_value = {"updated": True}
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        [
+            "remote-link",
+            "update",
+            "--issue-key",
+            "NTTC-6",
+            "--remote-link-id",
+            "14",
+            "--url",
+            "https://example.com/updated",
+            "--yes",
+        ],
+    )
+
+    assert result.exit_code == 0
+    adapter.update_issue_remote_link_by_id.assert_called_once_with(
+        "NTTC-6",
+        14,
+        {"object": {"url": "https://example.com/updated"}},
+    )
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_delete_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        [
+            "remote-link",
+            "delete",
+            "--issue-key",
+            "NTTC-7",
+            "--remote-link-id",
+            "20",
+            "--yes",
+        ],
+    )
+
+    assert result.exit_code == 0
+    adapter.delete_issue_remote_link_by_id.assert_called_once_with("NTTC-7", 20)
+    assert "deleted" in result.stdout
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_update_requires_id(
+    mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner
+) -> None:
+    adapter = Mock()
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        ["remote-link", "update", "--issue-key", "NTTC-8", "--url", "https://example.com", "--yes"],
+    )
+
+    assert result.exit_code != 0
+    assert "remote-link-id required" in result.stderr or "remote-link-id required" in result.stdout
+    adapter.update_issue_remote_link_by_id.assert_not_called()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_application_validation(
+    mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner
+) -> None:
+    adapter = Mock()
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        [
+            "remote-link",
+            "create",
+            "--issue-key",
+            "NTTC-9",
+            "--url",
+            "https://example.com",
+            "--application-type",
+            "com.atlassian.confluence",
+            "--yes",
+        ],
+    )
+
+    assert result.exit_code != 0
+    assert "application-type and --application-name must be provided together" in result.stderr or result.stdout
+    adapter.create_or_update_issue_remote_link.assert_not_called()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_remote_link_unknown_action(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    adapter = Mock()
+    _configure_mocks(mock_settings_class, mock_adapter_class, adapter)
+
+    result = runner.invoke(
+        app,
+        ["remote-link", "unknown", "--issue-key", "NTTC-10"],
+    )
+
+    assert result.exit_code != 0
+    assert "Unknown action" in result.stderr or "Unknown action" in result.stdout
+

package/tools/vds-scripts/jira_orchestrator/tests/test_security_schemes.py

@@ -0,0 +1,170 @@
+"""Tests for JIRA issue security scheme CLI commands."""
+
+from __future__ import annotations
+
+from unittest.mock import Mock, patch
+
+import pytest
+from typer.testing import CliRunner
+from vds_jira_orchestrator.cli import app
+from vds_jira_orchestrator.errors import JiraAdapterError, JiraNotFound
+
+
+@pytest.fixture
+def runner() -> CliRunner:
+    """Create a CLI runner for testing."""
+    return CliRunner()
+
+
+@pytest.fixture
+def mock_adapter() -> Mock:
+    """Create a mock adapter."""
+    adapter = Mock()
+    adapter.get_issue_security_schemes.return_value = [
+        {"id": "10000", "name": "Default Security Scheme", "description": "Default scheme"},
+        {"id": "10001", "name": "Project Security Scheme", "description": "Project-specific scheme"},
+    ]
+    adapter.get_issue_security_scheme.return_value = {
+        "id": "10000",
+        "name": "Default Security Scheme",
+        "description": "Default scheme",
+        "levels": [{"id": "10000", "name": "Level 1"}],
+    }
+    return adapter
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_list_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, mock_adapter: Mock) -> None:
+    """Test security-scheme list command success."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+    mock_adapter_class.return_value = mock_adapter
+
+    result = runner.invoke(app, ["security-scheme", "list"])
+
+    assert result.exit_code == 0
+    assert "security_schemes" in result.stdout
+    assert "Default Security Scheme" in result.stdout
+    mock_adapter.get_issue_security_schemes.assert_called_once()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_get_success(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, mock_adapter: Mock) -> None:
+    """Test security-scheme get command success."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+    mock_adapter_class.return_value = mock_adapter
+
+    result = runner.invoke(app, ["security-scheme", "get", "--scheme-id", "10000"])
+
+    assert result.exit_code == 0
+    assert "Default Security Scheme" in result.stdout
+    mock_adapter.get_issue_security_scheme.assert_called_once_with("10000", only_levels=False)
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_get_with_only_levels(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, mock_adapter: Mock) -> None:
+    """Test security-scheme get command with --only-levels flag."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+    mock_adapter_class.return_value = mock_adapter
+
+    result = runner.invoke(app, ["security-scheme", "get", "--scheme-id", "10000", "--only-levels"])
+
+    assert result.exit_code == 0
+    mock_adapter.get_issue_security_scheme.assert_called_once_with("10000", only_levels=True)
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_get_missing_id(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    """Test security-scheme get command with missing scheme-id."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+
+    result = runner.invoke(app, ["security-scheme", "get"])
+
+    assert result.exit_code != 0
+    assert "scheme-id required" in result.stdout or "scheme-id required" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_unknown_action(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner) -> None:
+    """Test security-scheme command with unknown action."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+
+    result = runner.invoke(app, ["security-scheme", "unknown"])
+
+    assert result.exit_code != 0
+    assert "Unknown action" in result.stdout or "Unknown action" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_error_handling(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, mock_adapter: Mock) -> None:
+    """Test security-scheme command error handling."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.get_issue_security_scheme.side_effect = JiraNotFound("Security scheme not found")
+
+    result = runner.invoke(app, ["security-scheme", "get", "--scheme-id", "99999"])
+
+    assert result.exit_code != 0
+    assert "Error" in result.stdout or "Error" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_security_scheme_adapter_error(mock_settings_class: Mock, mock_adapter_class: Mock, runner: CliRunner, mock_adapter: Mock) -> None:
+    """Test security-scheme command adapter error handling."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "test"
+    mock_settings.password = "test"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    mock_settings_class.return_value = mock_settings
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.get_issue_security_schemes.side_effect = JiraAdapterError("Adapter error")
+
+    result = runner.invoke(app, ["security-scheme", "list"])
+
+    assert result.exit_code != 0
+    assert "Error" in result.stdout or "Error" in result.stderr
+

package/tools/vds-scripts/jira_orchestrator/tests/test_transitions_changelog.py

@@ -0,0 +1,114 @@
+from __future__ import annotations
+
+from unittest.mock import Mock, patch
+
+from typer.testing import CliRunner
+from vds_jira_orchestrator.cli import app
+from vds_jira_orchestrator.errors import JiraAdapterError
+
+
+def _runner() -> CliRunner:
+    return CliRunner()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_transitions_success(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test successful transitions get."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_transitions.return_value = [
+        {"id": "1", "name": "To Do"},
+        {"id": "2", "name": "In Progress"},
+    ]
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["transitions", "NTTC-1"])
+    assert result.exit_code == 0
+    mock_adapter.get_issue_transitions.assert_called_once_with("NTTC-1")
+    assert "To Do" in result.stdout or "In Progress" in result.stdout
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_transitions_adapter_error(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test that adapter errors are handled."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_transitions.side_effect = JiraAdapterError("Adapter error")
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["transitions", "NTTC-1"])
+    assert result.exit_code != 0
+    assert "Error" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_changelog_get_success(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test successful changelog get."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_changelog.return_value = {
+        "histories": [{"id": "1", "created": "2025-11-12T10:00:00Z"}]
+    }
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["changelog", "get", "NTTC-1"])
+    assert result.exit_code == 0
+    mock_adapter.get_issue_changelog.assert_called_once_with("NTTC-1", start=None, limit=None)
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_changelog_get_with_pagination(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test changelog get with pagination."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_changelog.return_value = {"histories": []}
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["changelog", "get", "NTTC-1", "--start", "10", "--limit", "20"])
+    assert result.exit_code == 0
+    mock_adapter.get_issue_changelog.assert_called_once_with("NTTC-1", start=10, limit=20)
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_changelog_status_success(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test successful changelog status."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_status_changelog.return_value = {
+        "histories": [{"id": "1", "items": [{"field": "status"}]}]
+    }
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["changelog", "status", "NTTC-1"])
+    assert result.exit_code == 0
+    mock_adapter.get_issue_status_changelog.assert_called_once_with("NTTC-1")
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_changelog_unknown_action(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test that unknown action raises error."""
+    runner = _runner()
+    result = runner.invoke(app, ["changelog", "unknown", "NTTC-1"])
+    assert result.exit_code != 0
+    assert "Unknown action" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_changelog_adapter_error(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    """Test that adapter errors are handled."""
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.get_issue_changelog.side_effect = JiraAdapterError("Adapter error")
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["changelog", "get", "NTTC-1"])
+    assert result.exit_code != 0
+    assert "Error" in result.stderr
+