@ngocsangairvds/vsaf 3.1.27 → 3.2.1

package/tools/vds-scripts/bitbucket_orchestrator/tests/test_code_insights.py

@@ -0,0 +1,335 @@
+from __future__ import annotations
+
+from pathlib import Path
+from tempfile import NamedTemporaryFile
+from unittest.mock import Mock, patch
+
+from typer.testing import CliRunner
+from vds_bitbucket_orchestrator.cli import app
+from vds_bitbucket_orchestrator.errors import BitbucketClientError
+
+
+def _runner() -> CliRunner:
+    return CliRunner()
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_requires_cloud_mode(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that Code Insights requires Cloud mode."""
+    runner = _runner()
+    settings = Mock()
+    settings.is_cloud_mode = False
+    mock_load.return_value = settings
+
+    result = runner.invoke(app, ["code-insights", "get", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code != 0
+    # Exit code 1 indicates Cloud mode requirement failed
+    mock_build.assert_not_called()
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_get_requires_report_key(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that get requires --report-key."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(app, ["code-insights", "get", "PRJ", "repo", "abc123"])
+    assert result.exit_code != 0
+    assert "--report-key required" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_get_success(mock_load: Mock, mock_build: Mock) -> None:
+    """Test successful Code Insights get."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    client.get_code_insights_report.return_value = {"title": "Test Report", "result": "PASSED"}
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(app, ["code-insights", "get", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code == 0
+    client.get_code_insights_report.assert_called_once_with("PRJ", "repo", "abc123", "test-report")
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_create_requires_yes(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that create requires --yes."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(
+        app,
+        [
+            "code-insights",
+            "create",
+            "PRJ",
+            "repo",
+            "abc123",
+            "--report-key",
+            "test-report",
+            "--title",
+            "Test",
+            "--reporter",
+            "test",
+            "--result",
+            "PASSED",
+        ],
+    )
+    assert result.exit_code != 0
+    assert "Refusing to create report without --yes" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_create_requires_title(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that create requires --title."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(
+        app,
+        [
+            "code-insights",
+            "create",
+            "PRJ",
+            "repo",
+            "abc123",
+            "--report-key",
+            "test-report",
+            "--reporter",
+            "test",
+            "--result",
+            "PASSED",
+            "--yes",
+        ],
+    )
+    assert result.exit_code != 0
+    assert "--title required" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_create_success(mock_load: Mock, mock_build: Mock) -> None:
+    """Test successful Code Insights create."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    client.create_code_insights_report.return_value = {"title": "Test Report", "result": "PASSED"}
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(
+        app,
+        [
+            "code-insights",
+            "create",
+            "PRJ",
+            "repo",
+            "abc123",
+            "--report-key",
+            "test-report",
+            "--title",
+            "Test Report",
+            "--reporter",
+            "test-reporter",
+            "--result",
+            "PASSED",
+            "--yes",
+        ],
+    )
+    assert result.exit_code == 0
+    client.create_code_insights_report.assert_called_once()
+    call_kwargs = client.create_code_insights_report.call_args[1]
+    assert call_kwargs["project_key"] == "PRJ"
+    assert call_kwargs["repository_slug"] == "repo"
+    assert call_kwargs["commit_id"] == "abc123"
+    assert call_kwargs["report_key"] == "test-report"
+    assert call_kwargs["report_title"] == "Test Report"
+    assert call_kwargs["reporter"] == "test-reporter"
+    assert call_kwargs["result"] == "PASSED"
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_create_with_data_file(mock_load: Mock, mock_build: Mock) -> None:
+    """Test create with data file."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    client.create_code_insights_report.return_value = {"title": "Test Report"}
+    mock_build.return_value.__enter__.return_value = client
+
+    with NamedTemporaryFile(mode="w", suffix=".json", delete=False) as f:
+        f.write('[{"title": "Coverage", "type": "NUMBER", "value": 85}]')
+        data_file = Path(f.name)
+
+    try:
+        result = runner.invoke(
+            app,
+            [
+                "code-insights",
+                "create",
+                "PRJ",
+                "repo",
+                "abc123",
+                "--report-key",
+                "test-report",
+                "--title",
+                "Test Report",
+                "--reporter",
+                "test",
+                "--result",
+                "PASSED",
+                "--data-file",
+                str(data_file),
+                "--yes",
+            ],
+        )
+        assert result.exit_code == 0
+        call_kwargs = client.create_code_insights_report.call_args[1]
+        assert call_kwargs["data"] == [{"title": "Coverage", "type": "NUMBER", "value": 85}]
+    finally:
+        data_file.unlink()
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_annotate_requires_yes(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that annotate requires --yes."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(app, ["code-insights", "annotate", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code != 0
+    assert "Refusing to add annotations without --yes" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_annotate_requires_file(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that annotate requires --annotations-file."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(
+        app, ["code-insights", "annotate", "PRJ", "repo", "abc123", "--report-key", "test-report", "--yes"]
+    )
+    assert result.exit_code != 0
+    assert "--annotations-file required" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_annotate_success(mock_load: Mock, mock_build: Mock) -> None:
+    """Test successful Code Insights annotate."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    client.add_code_insights_annotations_to_report.return_value = {"annotations": []}
+    mock_build.return_value.__enter__.return_value = client
+
+    with NamedTemporaryFile(mode="w", suffix=".json", delete=False) as f:
+        f.write('[{"path": "file.py", "line": 10, "severity": "HIGH"}]')
+        annotations_file = Path(f.name)
+
+    try:
+        result = runner.invoke(
+            app,
+            [
+                "code-insights",
+                "annotate",
+                "PRJ",
+                "repo",
+                "abc123",
+                "--report-key",
+                "test-report",
+                "--annotations-file",
+                str(annotations_file),
+                "--yes",
+            ],
+        )
+        assert result.exit_code == 0
+        client.add_code_insights_annotations_to_report.assert_called_once_with(
+            "PRJ", "repo", "abc123", "test-report", [{"path": "file.py", "line": 10, "severity": "HIGH"}]
+        )
+    finally:
+        annotations_file.unlink()
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_delete_requires_yes(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that delete requires --yes."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(app, ["code-insights", "delete", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code != 0
+    assert "Refusing to delete report without --yes" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_delete_success(mock_load: Mock, mock_build: Mock) -> None:
+    """Test successful Code Insights delete."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(
+        app, ["code-insights", "delete", "PRJ", "repo", "abc123", "--report-key", "test-report", "--yes"]
+    )
+    assert result.exit_code == 0
+    client.delete_code_insights_report.assert_called_once_with("PRJ", "repo", "abc123", "test-report")
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_unknown_action(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that unknown action raises error."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    result = runner.invoke(app, ["code-insights", "unknown", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code != 0
+    assert "Unknown action" in result.stderr
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_code_insights_client_error(mock_load: Mock, mock_build: Mock) -> None:
+    """Test that client errors are handled."""
+    runner = _runner()
+    settings = Mock(is_cloud_mode=True)
+    mock_load.return_value = settings
+
+    client = Mock()
+    client.get_code_insights_report.side_effect = BitbucketClientError("Client error", context={})
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(app, ["code-insights", "get", "PRJ", "repo", "abc123", "--report-key", "test-report"])
+    assert result.exit_code != 0
+    assert "Error" in result.stderr
+

package/tools/vds-scripts/bitbucket_orchestrator/tests/test_conditions.py

@@ -0,0 +1,147 @@
+from __future__ import annotations
+
+import json
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+from typer.testing import CliRunner
+from vds_bitbucket_orchestrator.cli import app
+from vds_bitbucket_orchestrator.errors import BitbucketClientError
+
+
+def _runner() -> CliRunner:
+    return CliRunner()
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_list_project(mock_load: Mock, mock_build: Mock) -> None:
+    runner = _runner()
+    client = Mock()
+    client.get_project_conditions.return_value = []
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(app, ["conditions", "list", "PRJ"])
+    assert result.exit_code == 0
+    client.get_project_conditions.assert_called_once_with("PRJ")
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_list_repo(mock_load: Mock, mock_build: Mock) -> None:
+    runner = _runner()
+    client = Mock()
+    client.get_repo_conditions.return_value = []
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(app, ["conditions", "list", "PRJ", "--repo", "repo"])
+    assert result.exit_code == 0
+    client.get_repo_conditions.assert_called_once_with("PRJ", "repo")
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_get_repo(mock_load: Mock, mock_build: Mock) -> None:
+    runner = _runner()
+    client = Mock()
+    client.get_repo_condition.return_value = {"id": 5}
+    mock_build.return_value.__enter__.return_value = client
+
+    result = runner.invoke(app, ["conditions", "get", "PRJ", "--repo", "repo", "--condition-id", "5"])
+    assert result.exit_code == 0
+    client.get_repo_condition.assert_called_once_with("PRJ", "repo", 5)
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_create_project(mock_load: Mock, mock_build: Mock, tmp_path: Path) -> None:
+    runner = _runner()
+    client = Mock()
+    mock_build.return_value.__enter__.return_value = client
+
+    payload = {"sourceMatcher": {"id": "any"}}
+    condition_file = tmp_path / "condition.json"
+    condition_file.write_text(json.dumps(payload))
+
+    result = runner.invoke(
+        app,
+        ["conditions", "create", "PRJ", "--condition-file", str(condition_file), "--yes"],
+    )
+    assert result.exit_code == 0
+    client.create_project_condition.assert_called_once_with("PRJ", payload)
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_create_requires_file(mock_load: Mock, mock_build: Mock) -> None:
+    runner = _runner()
+
+    result = runner.invoke(
+        app,
+        ["conditions", "create", "PRJ", "--yes"],
+    )
+    assert result.exit_code != 0
+    assert "--condition-file required for create" in result.stdout
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_create_project_error(mock_load: Mock, mock_build: Mock, tmp_path: Path) -> None:
+    runner = _runner()
+    client = Mock()
+    client.create_project_condition.side_effect = BitbucketClientError("fail")
+    mock_build.return_value.__enter__.return_value = client
+
+    payload = {"sourceMatcher": {"id": "any"}}
+    condition_file = tmp_path / "condition.json"
+    condition_file.write_text(json.dumps(payload))
+
+    result = runner.invoke(
+        app,
+        ["conditions", "create", "PRJ", "--condition-file", str(condition_file), "--yes"],
+    )
+    assert result.exit_code != 0
+    assert "bitbucket_error" in result.stdout
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_update_repo(mock_load: Mock, mock_build: Mock, tmp_path: Path) -> None:
+    runner = _runner()
+    client = Mock()
+    mock_build.return_value.__enter__.return_value = client
+
+    payload = {"targetMatcher": {"id": "refs/heads/main"}}
+    condition_file = tmp_path / "condition.json"
+    condition_file.write_text(json.dumps(payload))
+
+    result = runner.invoke(
+        app,
+        [
+            "conditions",
+            "update",
+            "PRJ",
+            "--repo",
+            "repo",
+            "--condition-id",
+            "11",
+            "--condition-file",
+            str(condition_file),
+            "--yes",
+        ],
+    )
+    assert result.exit_code == 0
+    client.update_repo_condition.assert_called_once_with("PRJ", "repo", payload, 11)
+
+
+@patch("vds_bitbucket_orchestrator.cli._build_client")
+@patch("vds_bitbucket_orchestrator.cli.load_settings")
+def test_conditions_delete_requires_id(mock_load: Mock, mock_build: Mock) -> None:
+    runner = _runner()
+
+    result = runner.invoke(
+        app,
+        ["conditions", "delete", "PRJ", "--yes"],
+    )
+    assert result.exit_code != 0
+    assert "--condition-id required for delete" in result.stdout

package/tools/vds-scripts/bitbucket_orchestrator/tests/test_config.py

@@ -0,0 +1,131 @@
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+from vds_bitbucket_orchestrator.config import load_settings
+
+
+def write_env(tmp_path: Path, content: str) -> Path:
+    """Write a temporary .env file for testing."""
+    env_path = tmp_path / ".env"
+    env_path.write_text(content)
+    return env_path
+
+
+def clear_runtime_env(monkeypatch: pytest.MonkeyPatch) -> None:
+    """Clear runtime environment variables for testing."""
+    for key in (
+        "VDS_USERNAME",
+        "VDS_PASSWORD",
+        "BITBUCKET_TOKEN",
+        "BITBUCKET_BASE_URL",
+        "BITBUCKET_CLOUD_WORKSPACE",
+        "BITBUCKET_MODE",
+    ):
+        monkeypatch.delenv(key, raising=False)
+
+
+def test_load_settings_prefers_env_over_file(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None:
+    """Test that environment variables take precedence over file settings."""
+    clear_runtime_env(monkeypatch)
+    env_path = write_env(
+        tmp_path,
+        """
+VDS_USERNAME=file_user
+VDS_PASSWORD=file_pass
+BITBUCKET_TOKEN=file_token
+BITBUCKET_MODE=server
+""".strip(),
+    )
+
+    monkeypatch.setenv("VDS_USERNAME", "env_user")
+    settings = load_settings(env_path=env_path)
+
+    assert settings.username == "env_user"
+    assert settings.password == "file_pass"
+    assert settings.token == "file_token"
+    assert settings.mode == "server"
+
+
+def test_load_settings_strict_requires_credentials(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None:
+    """Test that strict mode requires either basic auth or token."""
+    clear_runtime_env(monkeypatch)
+    env_path = write_env(
+        tmp_path,
+        """
+VDS_USERNAME=file_user
+VDS_PASSWORD=file_pass
+""".strip(),
+    )
+
+    # Should work with basic auth
+    settings = load_settings(env_path=env_path, strict=True)
+    assert settings.username == "file_user"
+    assert settings.password == "file_pass"
+
+    # Should fail with no credentials
+    clear_runtime_env(monkeypatch)
+    empty_env_path = write_env(tmp_path, "")
+    with pytest.raises(ValueError, match="Missing credentials"):
+        load_settings(env_path=empty_env_path, strict=True)
+
+
+def test_cloud_mode_configuration(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None:
+    """Test cloud mode configuration validation."""
+    clear_runtime_env(monkeypatch)
+    env_path = write_env(
+        tmp_path,
+        """
+BITBUCKET_MODE=cloud
+BITBUCKET_CLOUD_WORKSPACE=testworkspace
+BITBUCKET_TOKEN=cloud_token
+""".strip(),
+    )
+
+    settings = load_settings(env_path=env_path, strict=True)
+    assert settings.is_cloud_mode
+    assert settings.cloud_workspace == "testworkspace"
+    assert settings.token == "cloud_token"
+
+    # Test URL generation for cloud mode
+    url = settings.get_url()
+    assert "testworkspace" in url
+    assert "api.bitbucket.org" in url
+
+
+def test_server_mode_configuration(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None:
+    """Test server mode configuration validation."""
+    clear_runtime_env(monkeypatch)
+    env_path = write_env(
+        tmp_path,
+        """
+BITBUCKET_MODE=server
+BITBUCKET_BASE_URL=https://bitbucket.company.com
+VDS_USERNAME=server_user
+VDS_PASSWORD=server_pass
+""".strip(),
+    )
+
+    settings = load_settings(env_path=env_path, strict=True)
+    assert settings.is_server_mode
+    assert str(settings.base_url) == "https://bitbucket.company.com"
+    assert settings.username == "server_user"
+    assert settings.password == "server_pass"
+
+
+def test_auth_kwargs_generation(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None:
+    """Test authentication kwargs generation for different auth methods."""
+    clear_runtime_env(monkeypatch)
+
+    # Test token auth
+    token_env_path = write_env(tmp_path, "BITBUCKET_TOKEN=test_token\n")
+    settings = load_settings(env_path=token_env_path)
+    auth_kwargs = settings.get_auth_kwargs()
+    assert auth_kwargs == {"token": "test_token"}
+
+    # Test basic auth
+    basic_env_path = write_env(tmp_path, "VDS_USERNAME=test_user\nVDS_PASSWORD=test_pass\n")
+    settings = load_settings(env_path=basic_env_path)
+    auth_kwargs = settings.get_auth_kwargs()
+    assert auth_kwargs == {"username": "test_user", "password": "test_pass"}