Commit

Author: StanFromIreland

bedevere/prtype.py

@@ -51,6 +51,8 @@ async def classify_by_filepaths(gh, pull_request, filenames):
             return pr_labels
     if tests:
         pr_labels = [Labels.tests]
+        if not news and not docs:
+            pr_labels.append(Labels.skip_news)
     elif docs:
         if news:
             pr_labels = [Labels.docs]

tests/test_filepaths.py

@@ -139,10 +139,12 @@ async def test_tests_only(author_association):
         gh.getiter_url == "https://api.github.com/repos/cpython/python/pulls/1234/files"
     )
     assert gh.getitem_url == "https://api.github.com/repos/cpython/python/issue/1234"
-    assert len(gh.post_url) == 3 if author_association == "NONE" else 2
+    assert len(gh.post_url) == 1
     assert gh.post_url.pop(0) == "https://api.github.com/some/label"
-    assert gh.post_data.pop(0) == [Labels.tests.value]
-    check_n_pop_nonews_events(gh, author_association == "NONE")
+    assert gh.post_data.pop(0) == [Labels.tests.value, Labels.skip_news.value]
+    # Don't post any comment
+    assert gh.post_url == []
+    assert gh.post_data == []
 
 
 async def test_docs_and_tests():
@@ -176,7 +178,7 @@ async def test_docs_and_tests():
         gh.getiter_url == "https://api.github.com/repos/cpython/python/pulls/1234/files"
     )
     assert gh.getitem_url == "https://api.github.com/repos/cpython/python/issue/1234"
-    # Only creates type-tests label.
+    # Only creates tests label.
     assert len(gh.post_url) == 2
     assert gh.post_url[0] == "https://api.github.com/some/label"
     assert gh.post_data[0] == [Labels.tests.value]

tests/test_prtype.py

@@ -142,7 +142,7 @@ async def test_tests_only():
     assert gh.getitem_url == "https://api.github.com/repos/cpython/python/issue/1234"
     assert len(gh.post_url) == 1
     assert gh.post_url[0] == "https://api.github.com/some/label"
-    assert gh.post_data[0] == [Labels.tests.value]
+    assert gh.post_data[0] == [Labels.tests.value, Labels.skip_news.value]
 
 
 async def test_tests_and_testmods_only():
@@ -162,7 +162,7 @@ async def test_tests_and_testmods_only():
     assert gh.getitem_url == "https://api.github.com/repos/cpython/python/issue/1234"
     assert len(gh.post_url) == 1
     assert gh.post_url[0] == "https://api.github.com/some/label"
-    assert gh.post_data[0] == [Labels.tests.value]
+    assert gh.post_data[0] == [Labels.tests.value, Labels.skip_news.value]
 
 
 async def test_docs_and_tests():