Merge pull request #76 from Open-Athena/codex/github-api-error-status

Include HTTP status in GitHub API errors

Author: ethanholz

src/gha_runner/gh.py

@@ -92,7 +92,8 @@ def _do_request(self, func, endpoint, **kwargs):
         resp: requests.Response = func(endpoint_url, headers=headers, **kwargs)
         if not resp.ok:
             raise RuntimeError(
-                f"Error in API call for {endpoint_url}: " f"{resp.content}"
+                f"Error in API call for {endpoint_url}: "
+                f"{resp.status_code} {resp.reason}: {resp.content}"
             )
         else:
             try:

tests/test_gh.py

@@ -129,9 +129,16 @@ def test_get_runners_error(github_instance):
     responses.add(
         responses.GET,
         "https://api.github.com/repos/test/test/actions/runners",
+        body="server unavailable",
         status=500,
     )
-    with pytest.raises(RunnerListError, match="Error getting runners: *"):
+    with pytest.raises(
+        RunnerListError,
+        match=(
+            "Error getting runners: Error in API call for .*: "
+            "500 Internal Server Error: b'server unavailable'"
+        ),
+    ):
         github_instance.get_runners()