Skip to content

Commit

Permalink
add test condition
Browse files Browse the repository at this point in the history
  • Loading branch information
2bndy5 committed Feb 10, 2024
1 parent 4eeb2c9 commit 0c71946
Showing 1 changed file with 14 additions and 9 deletions.
23 changes: 14 additions & 9 deletions tests/reviews/test_pr_review.py
Expand Up @@ -13,18 +13,19 @@


@pytest.mark.parametrize(
"is_draft,is_closed,with_token,force_approved,tidy_review,format_review,changes",
"is_draft,is_closed,with_token,force_approved,tidy_review,format_review,changes,summary_only",
[
(True, False, True, False, False, True, 2),
(False, True, True, False, False, True, 2),
(True, False, True, False, False, True, 2, False),
(False, True, True, False, False, True, 2, False),
pytest.param(
False, False, False, False, False, True, 2, marks=pytest.mark.xfail
False, False, False, False, False, True, 2, False, marks=pytest.mark.xfail
),
(False, False, True, True, False, True, 2),
(False, False, True, False, True, False, 2),
(False, False, True, False, False, True, 2),
(False, False, True, False, True, True, 1),
(False, False, True, False, True, True, 0),
(False, False, True, True, False, True, 2, False),
(False, False, True, False, True, False, 2, False),
(False, False, True, False, False, True, 2, False),
(False, False, True, False, True, True, 1, False),
(False, False, True, False, True, True, 0, False),
(False, False, True, False, True, True, 0, True),
],
ids=[
"draft",
Expand All @@ -35,6 +36,7 @@
"format", # changes == diff_chunks only
"lines_added",
"all_lines",
"summary_only",
],
)
def test_post_review(
Expand All @@ -47,6 +49,7 @@ def test_post_review(
format_review: bool,
force_approved: bool,
changes: int,
summary_only: bool,
):
"""A mock test of posting PR reviews"""
# patch env vars
Expand All @@ -57,6 +60,8 @@ def test_post_review(
monkeypatch.setenv("CI", "true")
if with_token:
monkeypatch.setenv("GITHUB_TOKEN", "123456")
if summary_only:
monkeypatch.setenv("CPP_LINTER_PR_REVIEW_SUMMARY_ONLY", "true")
monkeypatch.chdir(str(tmp_path))
(tmp_path / "src").mkdir()
demo_dir = Path(__file__).parent.parent / "demo"
Expand Down

0 comments on commit 0c71946

Please sign in to comment.