Followup PR to integrate nits of #207

Author: MaximilianSoerenPollak

src/extensions/score_source_code_linker/tests/test_codelink.py

@@ -124,75 +124,6 @@ def git_repo(temp_dir):
     return git_dir
 
 
-@pytest.fixture
-def git_repo_with_https_remote(temp_dir):
-    """Create a git repository with HTTPS remote for testing."""
-    git_dir = temp_dir / "test_repo_https"
-    git_dir.mkdir()
-
-    # Initialize git repo
-    subprocess.run(["git", "init"], cwd=git_dir, check=True, capture_output=True)
-    subprocess.run(
-        ["git", "config", "user.email", "[email protected]"], cwd=git_dir, check=True
-    )
-    subprocess.run(["git", "config", "user.name", "Test User"], cwd=git_dir, check=True)
-
-    # Create a test file and commit
-    test_file = git_dir / "test_file.py"
-    test_file.write_text("# Test file\nprint('hello')\n")
-    subprocess.run(["git", "add", "."], cwd=git_dir, check=True)
-    subprocess.run(["git", "commit", "-m", "Initial commit"], cwd=git_dir, check=True)
-
-    # Add HTTPS remote
-    subprocess.run(
-        [
-            "git",
-            "remote",
-            "add",
-            "origin",
-            "https://github.com/test-user/test-repo.git",
-        ],
-        cwd=git_dir,
-        check=True,
-    )
-
-    return git_dir
-
-
-@pytest.fixture
-def git_repo_multiple_remotes(temp_dir):
-    """Create a git repository with multiple remotes for testing."""
-    git_dir = temp_dir / "test_repo_multiple"
-    git_dir.mkdir()
-
-    # Initialize git repo
-    subprocess.run(["git", "init"], cwd=git_dir, check=True, capture_output=True)
-    subprocess.run(
-        ["git", "config", "user.email", "[email protected]"], cwd=git_dir, check=True
-    )
-    subprocess.run(["git", "config", "user.name", "Test User"], cwd=git_dir, check=True)
-
-    # Create a test file and commit
-    test_file = git_dir / "test_file.py"
-    test_file.write_text("# Test file\nprint('hello')\n")
-    subprocess.run(["git", "add", "."], cwd=git_dir, check=True)
-    subprocess.run(["git", "commit", "-m", "Initial commit"], cwd=git_dir, check=True)
-
-    # Add multiple remotes
-    subprocess.run(
-        ["git", "remote", "add", "upstream", "[email protected]:upstream/test-repo.git"],
-        cwd=git_dir,
-        check=True,
-    )
-    subprocess.run(
-        ["git", "remote", "add", "origin", "[email protected]:test-user/test-repo.git"],
-        cwd=git_dir,
-        check=True,
-    )
-
-    return git_dir
-
-
 @pytest.fixture
 def sample_needlinks():
     """Create sample NeedLink objects for testing."""
@@ -354,75 +285,6 @@ def test_group_by_need_empty_list():
     assert len(result) == 0
 
 
-# Test git-related functions
-def test_parse_git_output_ssh_format():
-    """Test parsing git remote output in SSH format."""
-    git_line = "origin	[email protected]:test-user/test-repo.git (fetch)"
-    result = parse_remote_git_output(git_line)
-    assert result == "test-user/test-repo"
-
-
-def test_parse_git_output_https_format():
-    """Test parsing git remote output in HTTPS format."""
-    git_line = "origin	https://github.com/test-user/test-repo.git (fetch)"
-    result = parse_remote_git_output(git_line)
-    assert result == "test-user/test-repo"
-
-
-def test_parse_git_output_ssh_format_without_git_suffix():
-    """Test parsing git remote output in SSH format without .git suffix."""
-    git_line = "origin	[email protected]:test-user/test-repo (fetch)"
-    result = parse_remote_git_output(git_line)
-    assert result == "test-user/test-repo"
-
-
-def test_parse_git_output_invalid_format():
-    """Test parsing invalid git remote output."""
-    git_line = "invalid"
-    result = parse_remote_git_output(git_line)
-    assert result == ""
-
-
-def test_parse_git_output_empty_string():
-    """Test parsing empty git remote output."""
-    git_line = ""
-    result = parse_remote_git_output(git_line)
-    assert result == ""
-
-
-def test_get_github_repo_info_ssh_remote(git_repo):
-    """Test getting GitHub repository information with SSH remote."""
-    result = get_github_repo_info(git_repo)
-    assert result == "test-user/test-repo"
-
-
-def test_get_github_repo_info_https_remote(git_repo_with_https_remote):
-    """Test getting GitHub repository information with HTTPS remote."""
-    result = get_github_repo_info(git_repo_with_https_remote)
-    assert result == "test-user/test-repo"
-
-
-def test_get_github_repo_info_multiple_remotes(git_repo_multiple_remotes):
-    """Test GitHub repo info retrieval with multiple remotes (origin preferred)."""
-    result = get_github_repo_info(git_repo_multiple_remotes)
-    assert result == "test-user/test-repo"
-
-
-def test_get_current_git_hash(git_repo):
-    """Test getting current git hash."""
-    result = get_current_git_hash(git_repo)
-
-    # Verify it's a valid git hash (40 hex characters)
-    assert len(result) == 40
-    assert all(c in "0123456789abcdef" for c in result)
-
-
-def test_get_current_git_hash_invalid_repo(temp_dir):
-    """Test getting git hash from invalid repository."""
-    with pytest.raises(subprocess.CalledProcessError):
-        get_current_git_hash(temp_dir)
-
-
 def test_get_github_link_with_real_repo(git_repo):
     """Test generating GitHub link with real repository."""
     # Create a needlink