wip

.github/workflows/python_lint_test.yml

@@ -33,4 +33,6 @@ jobs:
         flake8 reusable_workflows/
 
     - name: Run all tests
-      run: pytest --runslow reusable_workflows/
+      run: pytest --integration-tests reusable_workflows/
+      env:
+        GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.gitignore

@@ -1,3 +1,4 @@
 sorting/target
 **/__pycache__
 .env/**
+.venv/**

reusable_workflows/tests/conftest.py

@@ -3,19 +3,19 @@
 
 def pytest_addoption(parser):
     parser.addoption(
-        "--runslow", action="store_true", default=False, help="run slow tests"
+        "--integration-tests", action="store_true", default=False, help="run integration tests"
     )
 
 
 def pytest_configure(config):
-    config.addinivalue_line("markers", "slow: mark test as slow to run")
+    config.addinivalue_line("markers", "integration: mark test as integration test")
 
 
 def pytest_collection_modifyitems(config, items):
-    if config.getoption("--runslow"):
-        # --runslow given in cli: do not skip slow tests
+    if config.getoption("--integration-tests"):
+        # --integration-tests given in cli: do not skip integration tests
         return
-    skip_slow = pytest.mark.skip(reason="need --runslow option to run")
+    skip_integration = pytest.mark.skip(reason="need --integration-tests option to run")
     for item in items:
-        if "slow" in item.keywords:
-            item.add_marker(skip_slow)
+        if "integration" in item.keywords:
+            item.add_marker(skip_integration)

reusable_workflows/tests/test_check_external_contrib.py

@@ -1,6 +1,7 @@
 import os
 from unittest import mock
 
+import github3
 import pytest
 
 from check_membership.check_external_contrib import (
@@ -53,3 +54,9 @@ def test_github_token_not_passed_in(github_login_mock):
     assert (
         str(exc.value) == "github login failed - maybe GH_TOKEN was not correctly set"
     )
+
+@pytest.mark.integration
+def test_check_repos_open_to_contributions_accessible():
+    gh = github3.login(token=os.getenv("GH_TOKEN"))
+
+    get_repos_open_to_contributions(gh)

reusable_workflows/tests/test_utils.py

@@ -19,7 +19,7 @@ def test_download_file_succeeds_first_try():
     assert repo.file_contents.call_count == 1
 
 
-@pytest.mark.slow
+@pytest.mark.integration
 def test_download_file_succeeds_third_try():
     repo = mock.MagicMock()
     file_content_obj = mock.Mock()
@@ -35,7 +35,7 @@ def test_download_file_succeeds_third_try():
     assert repo.file_contents.call_count == 3
 
 
-@pytest.mark.slow
+@pytest.mark.integration
 @mock.patch("requests.get")
 def test_download_file_fails(mock_get):
     repo = mock.MagicMock()