fix: remove unneeded github auth check when displaying available plugins (#287)

Author: antazoey

CONTRIBUTING.md

@@ -32,6 +32,17 @@ pre-commit install
 
 Committing will now automatically run the local hooks and ensure that your commit passes all lint checks.
 
+## Github Access Token
+
+If you are a member of ApeWorX and would like to install private plugins, 
+[create a Github access token](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token).
+
+Once you have your token, export it to your terminal session:
+
+```bash
+export GITHUB_ACCESS_TOKEN=<your-token>
+```
+
 ## Pull Requests
 
 Pull requests are welcomed! Please adhere to the following:

src/ape/utils.py

@@ -212,10 +212,6 @@ def ape_org(self):
 
     @cached_property
     def available_plugins(self) -> Set[str]:
-        if not self.has_auth:
-            logger.warning(f"${self.TOKEN_KEY} not set, unable to list all plugins.")
-            return set()
-
         return {
             repo.name.replace("-", "_")
             for repo in self.ape_org.get_repos()

tests/integration/cli/test_misc.py

@@ -1,4 +1,7 @@
 import pytest
+from github import RateLimitExceededException
+
+from .utils import skip_projects_except
 
 
 # NOTE: test all the things without a direct test elsewhere
@@ -13,10 +16,31 @@
         ["networks"],
         ["networks", "list"],
         ["plugins"],
-        ["plugins", "list"],
-        ["plugins", "list", "--all"],
     ),
 )
 def test_invocation(ape_cli, runner, args):
     result = runner.invoke(ape_cli, args)
     assert result.exit_code == 0
+
+
+# Only run these tests once (limited to single arbitrary project).
+# This is to prevent Github from rate limiting us and causing test failures.
+@skip_projects_except(["empty-config"])
+@pytest.mark.parametrize(
+    "args",
+    (
+        ["plugins", "list"],
+        ["plugins", "list", "--all"],
+    ),
+)
+def test_invocation_run_once(ape_cli, runner, args):
+    result = runner.invoke(ape_cli, args)
+
+    if result.exit_code != 0:
+        # Check if failed because we were rate-limited.
+        # If that is the case, consider the test as passing.
+        err = result.exception
+        if not isinstance(err, RateLimitExceededException):
+            assert False, result.output
+
+    # Test Passed