@ngocsangairvds/vsaf 4.0.6 → 4.0.8

package/skills/vds-skill/vds-scripts/jira_viettelmoney_orchestrator/tests/test_agile.py

@@ -0,0 +1,334 @@
+"""Tests for JIRA Agile/Scrum operations."""
+
+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 VDSClientError
+
+
+@pytest.fixture
+def runner() -> CliRunner:
+    """Create a CLI runner for testing."""
+    return CliRunner()
+
+
+@pytest.fixture
+def sample_board() -> dict:
+    """Sample agile board response."""
+    return {
+        "id": 1,
+        "name": "Test Board",
+        "type": "scrum",
+        "location": {"projectId": 10000, "displayName": "Test Project"},
+    }
+
+
+@pytest.fixture
+def sample_epic() -> dict:
+    """Sample epic response."""
+    return {
+        "id": 1,
+        "key": "TEST-100",
+        "name": "Test Epic",
+        "summary": "Test Epic Summary",
+        "done": False,
+    }
+
+
+class TestAgileBoardOperations:
+    """Test suite for Agile board operations."""
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_list(
+        self,
+        mock_settings: Mock,
+        mock_adapter_class: Mock,
+        runner: CliRunner,
+        sample_board: dict,
+    ) -> None:
+        """Test board list command."""
+        mock_adapter = Mock()
+        mock_adapter.get_all_agile_boards.return_value = [sample_board]
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "list", "--limit", "10"])
+        assert result.exit_code == 0
+        mock_adapter.get_all_agile_boards.assert_called_once_with(
+            project_key=None, board_type=None, start=0, limit=10
+        )
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_get_by_id(
+        self,
+        mock_settings: Mock,
+        mock_adapter_class: Mock,
+        runner: CliRunner,
+        sample_board: dict,
+    ) -> None:
+        """Test board get by ID command."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board.return_value = sample_board
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "get", "--board-id", "1"])
+        assert result.exit_code == 0
+        mock_adapter.get_agile_board.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_get_by_filter(
+        self,
+        mock_settings: Mock,
+        mock_adapter_class: Mock,
+        runner: CliRunner,
+        sample_board: dict,
+    ) -> None:
+        """Test board get by filter ID command."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board_by_filter_id.return_value = sample_board
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "get", "--filter-id", "100"])
+        assert result.exit_code == 0
+        mock_adapter.get_agile_board_by_filter_id.assert_called_once_with(100)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_create(
+        self,
+        mock_settings: Mock,
+        mock_adapter_class: Mock,
+        runner: CliRunner,
+        sample_board: dict,
+    ) -> None:
+        """Test board create command."""
+        mock_adapter = Mock()
+        mock_adapter.create_agile_board.return_value = sample_board
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(
+            app,
+            [
+                "board",
+                "create",
+                "--name",
+                "Test Board",
+                "--type",
+                "scrum",
+                "--filter-id",
+                "100",
+                "--yes",
+            ],
+        )
+        assert result.exit_code == 0
+        mock_adapter.create_agile_board.assert_called_once_with(
+            "Test Board", "scrum", 100
+        )
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_create_requires_yes(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board create requires --yes flag."""
+        result = runner.invoke(
+            app,
+            [
+                "board",
+                "create",
+                "--name",
+                "Test Board",
+                "--type",
+                "scrum",
+                "--filter-id",
+                "100",
+            ],
+        )
+        assert result.exit_code != 0
+        assert "Refusing to create without --yes" in result.stderr
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_delete(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board delete command."""
+        mock_adapter = Mock()
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "delete", "--board-id", "1", "--yes"])
+        assert result.exit_code == 0
+        mock_adapter.delete_agile_board.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_config(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board config command."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board_configuration.return_value = {"config": "test"}
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "config", "--board-id", "1"])
+        assert result.exit_code == 0
+        mock_adapter.get_agile_board_configuration.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_issues(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board issues command."""
+        mock_adapter = Mock()
+        mock_adapter.get_issues_for_board.return_value = []
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "issues", "--board-id", "1"])
+        assert result.exit_code == 0
+        mock_adapter.get_issues_for_board.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_properties(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board properties command."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board_properties.return_value = [
+            {"key": "prop1", "value": "val1"}
+        ]
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "properties", "--board-id", "1"])
+        assert result.exit_code == 0
+        mock_adapter.get_agile_board_properties.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_velocity(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board velocity command."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board_refined_velocity.return_value = {"velocity": 10}
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "velocity", "--board-id", "1"])
+        assert result.exit_code == 0
+        mock_adapter.get_agile_board_refined_velocity.assert_called_once_with(1)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_board_error_handling(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test board command error handling."""
+        mock_adapter = Mock()
+        mock_adapter.get_agile_board.side_effect = VDSClientError("Test error")
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["board", "get", "--board-id", "1"])
+        assert result.exit_code != 0
+
+
+class TestEpicOperations:
+    """Test suite for Epic operations."""
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_epic_issues(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test epic issues command."""
+        mock_adapter = Mock()
+        mock_adapter.epic_issues.return_value = []
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["epic", "issues", "--epic-key", "TEST-100"])
+        assert result.exit_code == 0
+        mock_adapter.epic_issues.assert_called_once_with("TEST-100")
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_epic_list(
+        self,
+        mock_settings: Mock,
+        mock_adapter_class: Mock,
+        runner: CliRunner,
+        sample_epic: dict,
+    ) -> None:
+        """Test epic list command."""
+        mock_adapter = Mock()
+        mock_adapter.get_epics.return_value = [sample_epic]
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(
+            app, ["epic", "list", "--board-id", "1", "--limit", "10"]
+        )
+        assert result.exit_code == 0
+        mock_adapter.get_epics.assert_called_once_with(1, done=False, start=0, limit=10)
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_epic_issues_for_epic(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test epic issues-for-epic command."""
+        mock_adapter = Mock()
+        mock_adapter.get_issues_for_epic.return_value = []
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(
+            app,
+            [
+                "epic",
+                "issues-for-epic",
+                "--board-id",
+                "1",
+                "--epic-id",
+                "100",
+                "--limit",
+                "10",
+            ],
+        )
+        assert result.exit_code == 0
+        mock_adapter.get_issues_for_epic.assert_called_once_with(
+            1,
+            100,
+            jql="",
+            validate_query="",
+            fields="*all",
+            expand="",
+            start=0,
+            limit=10,
+        )
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_epic_requires_epic_key(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test epic issues requires epic-key."""
+        result = runner.invoke(app, ["epic", "issues"])
+        assert result.exit_code != 0
+        assert "--epic-key required" in result.stderr
+
+    @patch("vds_jira_orchestrator.cli.JiraAdapter")
+    @patch("vds_jira_orchestrator.cli.JiraSettings")
+    def test_epic_error_handling(
+        self, mock_settings: Mock, mock_adapter_class: Mock, runner: CliRunner
+    ) -> None:
+        """Test epic command error handling."""
+        mock_adapter = Mock()
+        mock_adapter.epic_issues.side_effect = VDSClientError("Test error")
+        mock_adapter_class.return_value = mock_adapter
+
+        result = runner.invoke(app, ["epic", "issues", "--epic-key", "TEST-100"])
+        assert result.exit_code != 0

package/skills/vds-skill/vds-scripts/jira_viettelmoney_orchestrator/tests/test_application_properties.py

@@ -0,0 +1,261 @@
+from __future__ import annotations
+
+from unittest.mock import MagicMock, Mock, patch
+
+import pytest
+from typer.testing import CliRunner
+from vds_jira_orchestrator.adapter import JiraAdapter, VDSClientError
+from vds_jira_orchestrator.cli import app
+
+
+@pytest.fixture
+def mock_adapter() -> JiraAdapter:
+    """Fixture for a JiraAdapter."""
+    with patch("vds_jira_orchestrator.adapter.Jira") as mock_jira_class:
+        mock_client = MagicMock()
+        mock_jira_class.return_value = mock_client
+        adapter = JiraAdapter(
+            url="https://jira.example.com", username="user", password="pass"
+        )
+        adapter._client = mock_client
+        return adapter
+
+
+@pytest.fixture
+def runner() -> CliRunner:
+    return CliRunner()
+
+
+@pytest.fixture
+def mock_settings_class() -> MagicMock:
+    """Mock JiraSettings for CLI tests."""
+    mock_settings = Mock()
+    mock_settings.base_url = "https://jira.example.com"
+    mock_settings.username = "testuser"
+    mock_settings.password = "testpass"
+    mock_settings.token = None
+    mock_settings.require_basic_or_token = Mock()
+    with patch("vds_jira_orchestrator.cli.JiraSettings") as mock_class:
+        mock_class.return_value = mock_settings
+        yield mock_class
+
+
+@pytest.fixture
+def mock_adapter_class() -> MagicMock:
+    """Mock JiraAdapter for CLI tests."""
+    with patch("vds_jira_orchestrator.cli.JiraAdapter") as mock_class:
+        yield mock_class
+
+
+# ---------------------------------------------------------------------------
+# Adapter method tests
+# ---------------------------------------------------------------------------
+
+
+def test_get_property_calls_client(mock_adapter: JiraAdapter) -> None:
+    """Test that get_property calls the SDK client with correct parameters."""
+    mock_adapter._client.get_property.return_value = {
+        "key": "test.key",
+        "value": "test_value",
+    }
+    result = mock_adapter.get_property(key="test.key")
+    mock_adapter._client.get_property.assert_called_once_with(
+        key="test.key", permission_level=None, key_filter=None
+    )
+    assert result["key"] == "test.key"
+
+
+def test_get_property_with_filters(mock_adapter: JiraAdapter) -> None:
+    """Test get_property with permission level and key filter."""
+    mock_adapter._client.get_property.return_value = {
+        "key": "test.key",
+        "value": "test_value",
+    }
+    result = mock_adapter.get_property(
+        key="test.key", permission_level="SYSTEM", key_filter="test.*"
+    )
+    mock_adapter._client.get_property.assert_called_once_with(
+        key="test.key", permission_level="SYSTEM", key_filter="test.*"
+    )
+    assert result["key"] == "test.key"
+
+
+def test_get_property_handles_error(mock_adapter: JiraAdapter) -> None:
+    """Test that get_property handles errors from the SDK client."""
+    mock_adapter._client.get_property.side_effect = Exception("API Error")
+    with pytest.raises(VDSClientError, match="API Error"):
+        mock_adapter.get_property(key="test.key")
+
+
+def test_set_property_calls_client(mock_adapter: JiraAdapter) -> None:
+    """Test that set_property calls the SDK client."""
+    mock_adapter._client.set_property.return_value = None
+    mock_adapter.set_property("prop.id", "new_value")
+    mock_adapter._client.set_property.assert_called_once_with("prop.id", "new_value")
+
+
+def test_set_property_handles_error(mock_adapter: JiraAdapter) -> None:
+    """Test that set_property handles errors from the SDK client."""
+    mock_adapter._client.set_property.side_effect = Exception("API Error")
+    with pytest.raises(VDSClientError, match="API Error"):
+        mock_adapter.set_property("prop.id", "value")
+
+
+def test_get_advanced_settings_calls_client(mock_adapter: JiraAdapter) -> None:
+    """Test that get_advanced_settings calls the SDK client."""
+    mock_adapter._client.get_advanced_settings.return_value = {
+        "setting1": "value1",
+        "setting2": "value2",
+    }
+    result = mock_adapter.get_advanced_settings()
+    mock_adapter._client.get_advanced_settings.assert_called_once()
+    assert "setting1" in result
+
+
+def test_get_advanced_settings_handles_error(mock_adapter: JiraAdapter) -> None:
+    """Test that get_advanced_settings handles errors from the SDK client."""
+    mock_adapter._client.get_advanced_settings.side_effect = Exception("API Error")
+    with pytest.raises(VDSClientError, match="API Error"):
+        mock_adapter.get_advanced_settings()
+
+
+# ---------------------------------------------------------------------------
+# CLI command tests
+# ---------------------------------------------------------------------------
+
+
+def test_property_get_success(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test successful property get."""
+    mock_adapter = MagicMock()
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.get_property.return_value = {"key": "test.key", "value": "test_value"}
+
+    result = runner.invoke(app, ["property", "get", "--key", "test.key"])
+    assert result.exit_code == 0
+    assert "test.key" in result.stdout
+    mock_adapter.get_property.assert_called_once_with(
+        key="test.key", permission_level=None, key_filter=None
+    )
+
+
+def test_property_get_with_filters(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test property get with filters."""
+    mock_adapter = MagicMock()
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.get_property.return_value = {"key": "test.key", "value": "test_value"}
+
+    result = runner.invoke(
+        app,
+        [
+            "property",
+            "get",
+            "--key",
+            "test.key",
+            "--permission-level",
+            "SYSTEM",
+            "--key-filter",
+            "test.*",
+        ],
+    )
+    assert result.exit_code == 0
+    mock_adapter.get_property.assert_called_once_with(
+        key="test.key", permission_level="SYSTEM", key_filter="test.*"
+    )
+
+
+def test_property_set_success(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test successful property set."""
+    mock_adapter = MagicMock()
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.set_property.return_value = None
+
+    result = runner.invoke(
+        app,
+        [
+            "property",
+            "set",
+            "--property-id",
+            "prop.id",
+            "--value",
+            "new_value",
+            "--yes",
+        ],
+    )
+    assert result.exit_code == 0
+    assert "set" in result.stdout
+    mock_adapter.set_property.assert_called_once_with("prop.id", "new_value")
+
+
+def test_property_set_requires_yes(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test property set requires --yes flag."""
+    result = runner.invoke(
+        app, ["property", "set", "--property-id", "prop.id", "--value", "new_value"]
+    )
+    assert result.exit_code != 0
+    assert "Refusing to set property without --yes" in result.stderr
+    mock_adapter_class.return_value.set_property.assert_not_called()
+
+
+def test_property_set_requires_property_id(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test property set requires --property-id."""
+    result = runner.invoke(app, ["property", "set", "--value", "new_value", "--yes"])
+    assert result.exit_code != 0
+    assert "--property-id required for set" in result.stderr
+    mock_adapter_class.return_value.set_property.assert_not_called()
+
+
+def test_property_set_requires_value(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test property set requires --value."""
+    result = runner.invoke(
+        app, ["property", "set", "--property-id", "prop.id", "--yes"]
+    )
+    assert result.exit_code != 0
+    assert "--value required for set" in result.stderr
+    mock_adapter_class.return_value.set_property.assert_not_called()
+
+
+def test_settings_advanced_success(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test successful advanced settings get."""
+    mock_adapter = MagicMock()
+    mock_adapter_class.return_value = mock_adapter
+    mock_adapter.get_advanced_settings.return_value = {
+        "setting1": "value1",
+        "setting2": "value2",
+    }
+
+    result = runner.invoke(app, ["settings", "advanced"])
+    assert result.exit_code == 0
+    assert "setting1" in result.stdout
+    mock_adapter.get_advanced_settings.assert_called_once()
+
+
+def test_property_invalid_action(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test property command with invalid action."""
+    result = runner.invoke(app, ["property", "invalid"])
+    assert result.exit_code != 0
+    assert "Unknown action: invalid" in result.stderr
+
+
+def test_settings_invalid_action(
+    runner: CliRunner, mock_settings_class: MagicMock, mock_adapter_class: MagicMock
+) -> None:
+    """Test settings command with invalid action."""
+    result = runner.invoke(app, ["settings", "invalid"])
+    assert result.exit_code != 0
+    assert "Unknown action: invalid" in result.stderr

package/skills/vds-skill/vds-scripts/jira_viettelmoney_orchestrator/tests/test_backlog.py

@@ -0,0 +1,91 @@
+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 VDSClientError
+
+
+def _runner() -> CliRunner:
+    return CliRunner()
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_move_requires_yes(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    result = runner.invoke(app, ["backlog", "move", "--issues", "NTTC-1,NTTC-2"])
+    assert result.exit_code != 0
+    assert "Refusing to modify backlog without --yes" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_move_success(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["backlog", "move", "--issues", "NTTC-1,NTTC-2", "--yes"])
+    assert result.exit_code == 0
+    mock_adapter.move_issues_to_backlog.assert_called_once_with(["NTTC-1", "NTTC-2"])
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_move_trims_whitespace(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["backlog", "move", "--issues", "NTTC-1 , NTTC-2 ", "--yes"])
+    assert result.exit_code == 0
+    mock_adapter.move_issues_to_backlog.assert_called_once_with(["NTTC-1", "NTTC-2"])
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_add(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["backlog", "add", "--issues", "NTTC-1,NTTC-2", "--yes"])
+    assert result.exit_code == 0
+    mock_adapter.add_issues_to_backlog.assert_called_once_with(["NTTC-1", "NTTC-2"])
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_add_adapter_error(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.add_issues_to_backlog.side_effect = VDSClientError("kaboom")
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["backlog", "add", "--issues", "NTTC-1", "--yes"])
+    assert result.exit_code != 0
+    assert "Error: kaboom" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_requires_issues(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    result = runner.invoke(app, ["backlog", "add", "--yes"])
+    assert result.exit_code != 0
+    assert "Missing option '--issues'" in result.stderr
+
+
+@patch("vds_jira_orchestrator.cli.JiraAdapter")
+@patch("vds_jira_orchestrator.cli.JiraSettings")
+def test_backlog_move_adapter_error(mock_settings: Mock, mock_adapter_cls: Mock) -> None:
+    runner = _runner()
+    mock_adapter = Mock()
+    mock_adapter.move_issues_to_backlog.side_effect = VDSClientError("boom")
+    mock_adapter_cls.return_value = mock_adapter
+
+    result = runner.invoke(app, ["backlog", "move", "--issues", "NTTC-1", "--yes"])
+    assert result.exit_code != 0
+    assert "Error: boom" in result.stderr