gh-381: Add test and fix when base branch isn't a maintenance branch. Fixes #381.

Author: jaraco

bedevere/backport.py

@@ -67,6 +67,14 @@ async def manage_labels(event, gh, *args, **kwargs):
     await _copy_over_labels(gh, original_issue, backport_issue)
 
 
+def targets_maintenance_branch(ref):
+    """
+    Return True if the ref refers to a maintenance branch.
+    """
+    maintenance_branch_pattern = r'\d+\.\d+'
+    return bool(re.fullmatch(maintenance_branch_pattern, ref))
+
+
 @router.register("pull_request", action="opened")
 @router.register("pull_request", action="reopened")
 @router.register("pull_request", action="edited")
@@ -84,7 +92,7 @@ async def validate_maintenance_branch_pr(event, gh, *args, **kwargs):
     pull_request = event.data["pull_request"]
     base_branch = pull_request["base"]["ref"]
 
-    if base_branch == "main":
+    if not targets_maintenance_branch(base_branch):
         return
 
     title = util.normalize_title(pull_request["title"],

tests/test_backport.py

@@ -328,6 +328,41 @@ async def test_not_valid_maintenance_branch_pr_title(action):
     assert post[1]['target_url'] == 'https://devguide.python.org/committing/#backport-pr-title'
 
 
+@pytest.mark.parametrize('action', ['opened', 'reopened', 'edited', 'synchronize'])
+async def test_not_maintenance_branch_pr_title(action):
+    title = 'Fix some typo'
+    data = {
+        'action': action,
+        'number': 2248,
+        'pull_request': {
+            'title': title,
+            'body': '',
+            'issue_url': 'https://api.github.com/issue/2248',
+            'base': {
+                'ref': 'gh-1234/dependent-change',
+            },
+            'statuses_url': 'https://api.github.com/repos/python/cpython/statuses/somehash',
+        },
+        'repository': {'issues_url': 'https://api.github.com/issue{/number}'},
+        'changes': {'title': title},
+    }
+    event = sansio.Event(data, event='pull_request',
+                        delivery_id='1')
+    getitem = {
+        'https://api.github.com/issue/1234':
+            {'labels': [{'name': 'CLA signed'}]},
+        'https://api.github.com/issue/2248': {},
+    }
+    gh = FakeGH(getitem=getitem)
+    await backport.router.dispatch(event, gh)
+    post = gh.post_[0]
+    assert post[0] == 'https://api.github.com/repos/python/cpython/statuses/somehash'
+    assert not any(
+        post_item['context'] == 'bedevere/maintenance-branch-pr'
+        for post_item in post
+    )
+
+
 @pytest.mark.parametrize('action', ['opened', 'reopened', 'edited', 'synchronize'])
 async def test_maintenance_branch_pr_status_not_posted_on_main(action):
     title = 'Fix some typo'