@ngocsangairvds/vsaf 4.1.7 → 4.1.9

package/skills/vds-skill/vds-scripts/bitbucket_orchestrator/tests/test_repo_ops_archive.py

@@ -1,169 +0,0 @@
-"""Tests for repo_ops archive functions."""
-
-from __future__ import annotations
-
-from pathlib import Path
-from unittest.mock import MagicMock, patch
-
-import pytest
-from vds_bitbucket_orchestrator.repo_ops import (
-    ArchiveResult,
-    RepoRef,
-    archive_repo,
-    download_rest_archive,
-)
-
-
-@pytest.fixture
-def ref() -> RepoRef:
-    return RepoRef(project_key="TEST", repo_slug="test-repo", source_format="shorthand")
-
-
-class TestDownloadRestArchive:
-    """REST archive download via BitbucketClient."""
-
-    def test_success(self, ref: RepoRef, tmp_path: Path) -> None:
-        output = tmp_path / "test-repo.zip"
-        client = MagicMock()
-
-        def write_archive(proj, repo, fd, **kwargs):
-            fd.write(b"PK\x03\x04fake-zip-content")
-
-        client.download_archive.side_effect = write_archive
-
-        result = download_rest_archive(client, ref, output)
-
-        assert result is not None
-        assert result.method == "rest_api"
-        assert result.format == "zip"
-        assert result.size_bytes > 0
-        assert output.exists()
-
-    def test_returns_none_on_failure(self, ref: RepoRef, tmp_path: Path) -> None:
-        output = tmp_path / "test-repo.zip"
-        client = MagicMock()
-        client.download_archive.side_effect = RuntimeError("404 Not Found")
-
-        result = download_rest_archive(client, ref, output)
-
-        assert result is None
-        assert not output.exists()
-
-    def test_returns_none_on_empty_archive(self, ref: RepoRef, tmp_path: Path) -> None:
-        output = tmp_path / "test-repo.zip"
-        client = MagicMock()
-        # download_archive writes nothing → empty file
-        client.download_archive.side_effect = lambda *a, **kw: None
-
-        result = download_rest_archive(client, ref, output)
-
-        assert result is None
-
-    def test_passes_format_and_at(self, ref: RepoRef, tmp_path: Path) -> None:
-        from unittest.mock import ANY
-
-        output = tmp_path / "test-repo.tar.gz"
-        client = MagicMock()
-
-        def write_archive(_proj, _repo, fd, **_kwargs):
-            fd.write(b"fake-tar-gz")
-
-        client.download_archive.side_effect = write_archive
-
-        download_rest_archive(client, ref, output, format="tar.gz", at="develop")
-
-        client.download_archive.assert_called_once_with(
-            "TEST",
-            "test-repo",
-            ANY,
-            at="develop",
-            format="tar.gz",
-            path=None,
-        )
-
-
-class TestArchiveRepo:
-    """Full archive workflow: REST-first with fallback."""
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.download_rest_archive")
-    def test_rest_success_skips_clone(self, mock_rest, ref: RepoRef, tmp_path: Path) -> None:
-        output = tmp_path / "out.zip"
-        mock_rest.return_value = ArchiveResult(
-            archive_path=output,
-            format="zip",
-            size_bytes=100,
-            method="rest_api",
-            ref=ref,
-        )
-        client = MagicMock()
-
-        result = archive_repo(
-            ref=ref,
-            client=client,
-            clone_url="http://bb/scm/TEST/test-repo.git",
-            cache_root=tmp_path / "cache",
-            output_path=output,
-        )
-
-        assert result.method == "rest_api"
-        mock_rest.assert_called_once()
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.clone_repo")
-    @patch("vds_bitbucket_orchestrator.repo_ops.download_rest_archive")
-    def test_rest_failure_falls_back_to_clone_zip(
-        self,
-        mock_rest,
-        mock_clone,
-        ref: RepoRef,
-        tmp_path: Path,
-    ) -> None:
-        output = tmp_path / "out.zip"
-        mock_rest.return_value = None  # REST failed
-
-        # Create a fake cloned repo
-        clone_dir = tmp_path / "cache" / "repos" / "TEST-project" / "test-repo"
-        clone_dir.mkdir(parents=True)
-        (clone_dir / ".git").mkdir()
-        (clone_dir / "README.md").write_text("hello")
-        (clone_dir / ".git" / "config").write_text("gitconfig")
-
-        mock_clone.return_value = MagicMock(path=clone_dir)
-
-        result = archive_repo(
-            ref=ref,
-            client=MagicMock(),
-            clone_url="http://bb/scm/TEST/test-repo.git",
-            cache_root=tmp_path / "cache",
-            output_path=output,
-        )
-
-        assert result.method == "local_zip"
-        assert output.exists()
-        assert result.size_bytes > 0
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.download_rest_archive")
-    def test_include_git_skips_rest(self, mock_rest, ref: RepoRef, tmp_path: Path) -> None:
-        """When include_git=True, REST path is skipped since it never includes .git/."""
-        output = tmp_path / "out.zip"
-
-        # Create fake repo
-        clone_dir = tmp_path / "cache" / "repos" / "TEST-project" / "test-repo"
-        clone_dir.mkdir(parents=True)
-        (clone_dir / ".git").mkdir()
-        (clone_dir / ".git" / "HEAD").write_text("ref: refs/heads/master")
-        (clone_dir / "file.txt").write_text("content")
-
-        with patch("vds_bitbucket_orchestrator.repo_ops.clone_repo") as mock_clone:
-            mock_clone.return_value = MagicMock(path=clone_dir)
-
-            result = archive_repo(
-                ref=ref,
-                client=MagicMock(),
-                clone_url="http://bb/scm/TEST/test-repo.git",
-                cache_root=tmp_path / "cache",
-                output_path=output,
-                include_git=True,
-            )
-
-        mock_rest.assert_not_called()
-        assert result.method == "local_zip"

package/skills/vds-skill/vds-scripts/bitbucket_orchestrator/tests/test_repo_ops_clone.py

@@ -1,115 +0,0 @@
-"""Tests for repo_ops clone_repo function."""
-
-from __future__ import annotations
-
-from pathlib import Path
-from unittest.mock import patch
-
-import pytest
-from vds_bitbucket_orchestrator.repo_ops import RepoRef, clone_repo
-
-
-@pytest.fixture
-def ref() -> RepoRef:
-    return RepoRef(project_key="TEST", repo_slug="test-repo", source_format="shorthand")
-
-
-class TestCloneRepo:
-    """Git clone/pull subprocess tests."""
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_fresh_clone(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "repos" / "TEST-project" / "test-repo"
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        result = clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target)
-
-        assert result.operation == "clone"
-        assert result.ref == ref
-        cmd = mock_run.call_args[0][0]
-        assert "clone" in cmd
-        assert "http://bb.vn/scm/TEST/test-repo.git" in cmd
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_pull_when_exists(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "existing-repo"
-        target.mkdir(parents=True)
-        (target / ".git").mkdir()
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        result = clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target)
-
-        assert result.operation == "pull"
-        cmd = mock_run.call_args[0][0]
-        assert "pull" in cmd
-        assert "--ff-only" in cmd
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_shallow_clone(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "shallow-repo"
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target, shallow=True)
-
-        cmd = mock_run.call_args[0][0]
-        assert "--depth" in cmd
-        assert "1" in cmd
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_clone_with_branch(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "branched-repo"
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        result = clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target, branch="develop")
-
-        assert result.branch == "develop"
-        cmd = mock_run.call_args[0][0]
-        assert "--branch" in cmd
-        assert "develop" in cmd
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.shutil.rmtree")
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_force_reclone(self, mock_run, mock_rmtree, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "existing-repo"
-        target.mkdir(parents=True)
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        result = clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target, force_reclone=True)
-
-        assert result.operation == "reclone"
-        mock_rmtree.assert_called_once_with(target)
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_clone_failure_raises(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "fail-repo"
-        mock_run.return_value.returncode = 128
-        mock_run.return_value.stderr = "fatal: repository not found"
-
-        with pytest.raises(RuntimeError, match="git clone failed"):
-            clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target)
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_pull_failure_raises(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "existing-repo"
-        target.mkdir(parents=True)
-        (target / ".git").mkdir()
-        mock_run.return_value.returncode = 1
-        mock_run.return_value.stderr = "fatal: not a git repository"
-
-        with pytest.raises(RuntimeError, match="git pull failed"):
-            clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target)
-
-    @patch("vds_bitbucket_orchestrator.repo_ops.subprocess.run")
-    def test_creates_parent_directories(self, mock_run, ref: RepoRef, tmp_path: Path) -> None:
-        target = tmp_path / "deep" / "nested" / "path" / "repo"
-        mock_run.return_value.returncode = 0
-        mock_run.return_value.stderr = ""
-
-        clone_repo(ref, "http://bb.vn/scm/TEST/test-repo.git", target)
-
-        assert target.parent.exists()

package/skills/vds-skill/vds-scripts/bitbucket_orchestrator/tests/test_repo_ops_parsing.py

@@ -1,149 +0,0 @@
-"""Tests for repo_ops URL parsing, clone URL building, and default paths."""
-
-from __future__ import annotations
-
-from pathlib import Path
-
-import pytest
-from vds_bitbucket_orchestrator.repo_ops import (
-    RepoRef,
-    build_clone_url,
-    default_clone_dir,
-    parse_repo_identifier,
-)
-
-
-class TestParseRepoIdentifier:
-    """URL parsing for all supported formats."""
-
-    @pytest.mark.parametrize(
-        "identifier,expected_project,expected_repo,expected_format",
-        [
-            # Shorthand
-            ("INSURANCE-AI/ntbh-tech-standards", "INSURANCE-AI", "ntbh-tech-standards", "shorthand"),
-            ("TEST/repo", "TEST", "repo", "shorthand"),
-            ("my-project/my.repo", "my-project", "my.repo", "shorthand"),
-            # Browse URL
-            (
-                "http://bitbucket.digital.vn/projects/INSURANCE-AI/repos/ntbh-tech-standards/browse",
-                "INSURANCE-AI",
-                "ntbh-tech-standards",
-                "browse_url",
-            ),
-            (
-                "https://bb.vn/projects/PAR/repos/merchant-billing/browse?at=refs%2Fheads%2Fmaster",
-                "PAR",
-                "merchant-billing",
-                "browse_url",
-            ),
-            # Clone URL
-            (
-                "http://bitbucket.digital.vn/scm/INSURANCE-AI/ntbh-tech-standards.git",
-                "INSURANCE-AI",
-                "ntbh-tech-standards",
-                "clone_url",
-            ),
-            (
-                "https://bb.vn/scm/PAR/merchant-report.git",
-                "PAR",
-                "merchant-report",
-                "clone_url",
-            ),
-            # Clone URL without .git suffix
-            (
-                "http://bb.vn/scm/TEST/repo",
-                "TEST",
-                "repo",
-                "clone_url",
-            ),
-            # SSH URL
-            (
-                "ssh://git@bitbucket.digital.vn/INSURANCE-AI/ntbh-tech-standards.git",
-                "INSURANCE-AI",
-                "ntbh-tech-standards",
-                "ssh_url",
-            ),
-            # SSH with port
-            (
-                "ssh://git@bb.vn:7999/PAR/merchant-billing.git",
-                "PAR",
-                "merchant-billing",
-                "ssh_url",
-            ),
-        ],
-    )
-    def test_valid_identifiers(
-        self,
-        identifier: str,
-        expected_project: str,
-        expected_repo: str,
-        expected_format: str,
-    ) -> None:
-        ref = parse_repo_identifier(identifier)
-        assert ref.project_key == expected_project
-        assert ref.repo_slug == expected_repo
-        assert ref.source_format == expected_format
-
-    def test_strips_whitespace(self) -> None:
-        ref = parse_repo_identifier("  TEST/repo  ")
-        assert ref.project_key == "TEST"
-        assert ref.repo_slug == "repo"
-
-    @pytest.mark.parametrize(
-        "invalid_input",
-        [
-            "not-a-valid-identifier",
-            "",
-            "   ",
-            "http://example.com/random/path",
-            "just-a-word",
-            "/leading/slash",
-        ],
-    )
-    def test_invalid_identifiers(self, invalid_input: str) -> None:
-        with pytest.raises(ValueError, match="Cannot parse repository identifier"):
-            parse_repo_identifier(invalid_input)
-
-
-class TestBuildCloneUrl:
-    """Clone URL construction with auth injection."""
-
-    def test_with_token(self) -> None:
-        ref = RepoRef(project_key="TEST", repo_slug="repo", source_format="shorthand")
-        url = build_clone_url(ref, "http://bitbucket.digital.vn", token="mytoken")
-        assert url == "http://x-token-auth:mytoken@bitbucket.digital.vn/scm/TEST/repo.git"
-
-    def test_with_username(self) -> None:
-        ref = RepoRef(project_key="TEST", repo_slug="repo", source_format="shorthand")
-        url = build_clone_url(ref, "http://bitbucket.digital.vn", username="admin")
-        assert url == "http://admin@bitbucket.digital.vn/scm/TEST/repo.git"
-
-    def test_no_auth(self) -> None:
-        ref = RepoRef(project_key="TEST", repo_slug="repo", source_format="shorthand")
-        url = build_clone_url(ref, "http://bitbucket.digital.vn")
-        assert url == "http://bitbucket.digital.vn/scm/TEST/repo.git"
-
-    def test_https_scheme(self) -> None:
-        ref = RepoRef(project_key="PRJ", repo_slug="my-repo", source_format="shorthand")
-        url = build_clone_url(ref, "https://bb.example.com", token="tok")
-        assert url == "https://x-token-auth:tok@bb.example.com/scm/PRJ/my-repo.git"
-
-    def test_token_takes_precedence_over_username(self) -> None:
-        ref = RepoRef(project_key="TEST", repo_slug="repo", source_format="shorthand")
-        url = build_clone_url(ref, "http://bb.vn", username="admin", token="tok")
-        assert "x-token-auth:tok@" in url
-        assert "admin" not in url
-
-
-class TestDefaultCloneDir:
-    """Standard VDS cache path for cloned repos."""
-
-    def test_standard_path(self) -> None:
-        ref = RepoRef(project_key="INSURANCE-AI", repo_slug="ntbh-tech-standards", source_format="shorthand")
-        result = default_clone_dir(ref, Path("/home/user/.vds/cache"))
-        assert result == Path("/home/user/.vds/cache/repos/INSURANCE-AI-project/ntbh-tech-standards")
-
-    def test_different_project(self) -> None:
-        ref = RepoRef(project_key="PAR", repo_slug="merchant-billing", source_format="browse_url")
-        result = default_clone_dir(ref, Path("/tmp/cache"))
-        assert result == Path("/tmp/cache/repos/PAR-project/merchant-billing")