From 6f2097926862f918a850b56820fdac3eb4bee6dd Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Tue, 16 Jun 2026 05:10:20 +0000 Subject: [PATCH] Add tests for split_repo in pr_review_merge_scheduler --- tests/test_pr_review_merge_scheduler.py | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 tests/test_pr_review_merge_scheduler.py diff --git a/tests/test_pr_review_merge_scheduler.py b/tests/test_pr_review_merge_scheduler.py new file mode 100644 index 0000000..3a16137 --- /dev/null +++ b/tests/test_pr_review_merge_scheduler.py @@ -0,0 +1,24 @@ +import sys +from pathlib import Path +import pytest + +sys.path.insert(0, str(Path(__file__).parent.parent / "scripts" / "ci")) +import pr_review_merge_scheduler + +def test_split_repo_success(): + assert pr_review_merge_scheduler.split_repo("owner/repo") == ("owner", "repo") + +def test_split_repo_success_multiple_slashes(): + assert pr_review_merge_scheduler.split_repo("owner/repo/extra") == ("owner", "repo/extra") + +def test_split_repo_invalid(): + with pytest.raises(ValueError, match="repo must be owner/name, got 'invalid'"): + pr_review_merge_scheduler.split_repo("invalid") + +def test_split_repo_empty_owner(): + with pytest.raises(ValueError, match="repo must be owner/name, got '/repo'"): + pr_review_merge_scheduler.split_repo("/repo") + +def test_split_repo_empty_repo(): + with pytest.raises(ValueError, match="repo must be owner/name, got 'owner/'"): + pr_review_merge_scheduler.split_repo("owner/")