implemented test_003_refuse_manual_merge #156

Merged
M0ses merged 1 commits from tests_by_M0ses into main 2026-03-05 09:36:35 +01:00
Owner

Fixes: git-workflow/tests #19

Fixes: git-workflow/tests #19
M0ses added 1 commit 2026-03-04 17:06:00 +01:00
implemented test_003_refuse_manual_merge
Some checks failed
Integration tests / t (pull_request) Failing after 18m12s
4c481720af
Fixes: git-workflow/tests #19
M0ses requested review from andriinikitin 2026-03-04 17:06:14 +01:00
M0ses requested review from atartamo 2026-03-04 17:06:15 +01:00
andriinikitin requested changes 2026-03-04 17:52:46 +01:00
andriinikitin left a comment
Member

We need to tweak the comment on line 211, otherwise looks good.
Also use t: prefix in the commit message, (the devs use various prefixes to quckly understand what it touches), e.g.
t: add test for unauthorized user tries to merge PR with a comment

We need to tweak the comment on line 211, otherwise looks good. Also use t: prefix in the commit message, (the devs use various prefixes to quckly understand what it touches), e.g. t: add test for unauthorized user tries to merge PR with a comment
@@ -140,0 +208,4 @@
assert all_requested, f"Timed out waiting for all expected reviewers {expected_reviewers} to approve. Current: {current_reviewers}"
print("Both PRs have all required approvals but are not merged (as expected with ManualMergeOnly).")
# 4. Comment "merge ok" from a requested reviewer (usera)
Member

This comment should be tweaked

This comment should be tweaked
Author
Owner

Hope my new PR is correct.

Hope my new PR is correct.
andriinikitin marked this conversation as resolved
M0ses force-pushed tests_by_M0ses from 4c481720af to e47f3a2f07 2026-03-05 08:42:43 +01:00 Compare
andriinikitin force-pushed tests_by_M0ses from e47f3a2f07 to 2531238a05 2026-03-05 09:11:42 +01:00 Compare
M0ses force-pushed tests_by_M0ses from 2531238a05 to 7e7d65bbd1 2026-03-05 09:17:16 +01:00 Compare
andriinikitin force-pushed tests_by_M0ses from 7e7d65bbd1 to f5a32792e0 2026-03-05 09:20:26 +01:00 Compare
M0ses merged commit f5a32792e0 into main 2026-03-05 09:36:35 +01:00
M0ses deleted branch tests_by_M0ses 2026-03-05 09:36:35 +01:00
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: git-workflow/autogits#156