- New: mcp_server.py — FastMCP stdio server exposing 7 tools: gitea_create_issue, gitea_create_pr, gitea_close_issue, gitea_list_issues, gitea_view_issue, gitea_mark_issue, gitea_mirror_refs - New: auth.py — shared authentication and API helpers (get_credentials, get_auth_header, api_request, repo_api_url) - Refactored: create_pr.py, create_issue.py, manage_labels.py to use shared auth module (eliminates credential duplication) - New: tests/test_mcp_server.py — 17 tests for all MCP tools - Updated: tests/test_credentials.py — now tests auth.py directly - Updated: tests/test_create_issue.py — adapted for refactored imports - New: requirements.txt — frozen venv deps (mcp[cli], pytest) - Updated: README.md — MCP server as primary interface - Config: added gitea-tools to mcp_config.json Closes #1. Resolves #2, #5. Relates to #7.
This commit is contained in:
+33
-59
@@ -1,7 +1,7 @@
|
||||
"""Tests for create_issue.py.
|
||||
|
||||
Every test mocks `get_credentials` and `urllib.request.urlopen` so no real
|
||||
network calls or keychain access are performed.
|
||||
Every test mocks auth functions so no real network calls or keychain
|
||||
access are performed.
|
||||
|
||||
Note: create_issue.py may be inaccessible due to macOS sandbox restrictions.
|
||||
If so, these tests are automatically skipped.
|
||||
@@ -28,17 +28,6 @@ except (ImportError, PermissionError, OSError) as exc:
|
||||
FAKE_CREDS = ("testuser", "testpass")
|
||||
|
||||
|
||||
def _mock_urlopen(status=200, body=None):
|
||||
"""Return a mock context-manager for urllib.request.urlopen."""
|
||||
if body is None:
|
||||
body = {"number": 42, "html_url": "https://gitea.example.com/issues/42"}
|
||||
resp = MagicMock()
|
||||
resp.read.return_value = json.dumps(body).encode("utf-8")
|
||||
resp.__enter__ = lambda s: s
|
||||
resp.__exit__ = MagicMock(return_value=False)
|
||||
return resp
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Argument parsing
|
||||
# ---------------------------------------------------------------------------
|
||||
@@ -46,9 +35,9 @@ def _mock_urlopen(status=200, body=None):
|
||||
class TestArgParsing(unittest.TestCase):
|
||||
"""Ensure argparse accepts the expected flags and rejects bad input."""
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.api_request", return_value={"number": 1, "html_url": "http://x/1"})
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_minimal_args(self, _cred, _url):
|
||||
def test_minimal_args(self, _cred, _api):
|
||||
rc = create_issue.main(["--title", "Hello"])
|
||||
self.assertEqual(rc, 0)
|
||||
|
||||
@@ -57,9 +46,9 @@ class TestArgParsing(unittest.TestCase):
|
||||
create_issue.main(["--body", "no title given"])
|
||||
self.assertNotEqual(ctx.exception.code, 0)
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.api_request", return_value={"number": 1, "html_url": "http://x/1"})
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_remote_choices(self, _cred, _url):
|
||||
def test_remote_choices(self, _cred, _api):
|
||||
for remote in ("dadeschools", "prgs"):
|
||||
rc = create_issue.main(["--remote", remote, "--title", "X"])
|
||||
self.assertEqual(rc, 0, f"--remote {remote} should be accepted")
|
||||
@@ -76,21 +65,21 @@ class TestArgParsing(unittest.TestCase):
|
||||
class TestRemoteResolution(unittest.TestCase):
|
||||
"""Verify the correct host/org/repo are selected per remote."""
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.api_request", return_value={"number": 1, "html_url": "http://x/1"})
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_dadeschools_default(self, mock_cred, mock_url):
|
||||
def test_dadeschools_default(self, mock_cred, _api):
|
||||
create_issue.main(["--title", "T"])
|
||||
mock_cred.assert_called_with("gitea.dadeschools.net")
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.api_request", return_value={"number": 1, "html_url": "http://x/1"})
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_prgs_remote(self, mock_cred, mock_url):
|
||||
def test_prgs_remote(self, mock_cred, _api):
|
||||
create_issue.main(["--remote", "prgs", "--title", "T"])
|
||||
mock_cred.assert_called_with("gitea.prgs.cc")
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.api_request", return_value={"number": 1, "html_url": "http://x/1"})
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_host_override(self, mock_cred, mock_url):
|
||||
def test_host_override(self, mock_cred, _api):
|
||||
create_issue.main(["--host", "custom.example.com", "--title", "T"])
|
||||
mock_cred.assert_called_with("custom.example.com")
|
||||
|
||||
@@ -104,27 +93,19 @@ class TestAPIPayload(unittest.TestCase):
|
||||
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_payload_title_and_body(self, _cred):
|
||||
with patch("create_issue.urllib.request.Request") as MockReq:
|
||||
MockReq.return_value = MagicMock()
|
||||
with patch("create_issue.urllib.request.urlopen",
|
||||
return_value=_mock_urlopen()):
|
||||
create_issue.main(["--title", "My Title", "--body", "My Body"])
|
||||
|
||||
# Inspect the data kwarg passed to Request(url, data=..., ...)
|
||||
call_kwargs = MockReq.call_args
|
||||
data = json.loads(call_kwargs[1]["data"].decode("utf-8"))
|
||||
self.assertEqual(data["title"], "My Title")
|
||||
self.assertEqual(data["body"], "My Body")
|
||||
with patch("create_issue.api_request",
|
||||
return_value={"number": 1, "html_url": "http://x/1"}) as mock_api:
|
||||
create_issue.main(["--title", "My Title", "--body", "My Body"])
|
||||
payload = mock_api.call_args[0][3]
|
||||
self.assertEqual(payload["title"], "My Title")
|
||||
self.assertEqual(payload["body"], "My Body")
|
||||
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_url_construction(self, _cred):
|
||||
with patch("create_issue.urllib.request.Request") as MockReq:
|
||||
MockReq.return_value = MagicMock()
|
||||
with patch("create_issue.urllib.request.urlopen",
|
||||
return_value=_mock_urlopen()):
|
||||
create_issue.main(["--remote", "prgs", "--title", "X"])
|
||||
|
||||
url = MockReq.call_args[0][0]
|
||||
with patch("create_issue.api_request",
|
||||
return_value={"number": 1, "html_url": "http://x/1"}) as mock_api:
|
||||
create_issue.main(["--remote", "prgs", "--title", "X"])
|
||||
url = mock_api.call_args[0][1]
|
||||
self.assertEqual(
|
||||
url,
|
||||
"https://gitea.prgs.cc/api/v1/repos/Scaled-Tech-Consulting/Timesheet/issues",
|
||||
@@ -137,19 +118,16 @@ class TestAPIPayload(unittest.TestCase):
|
||||
@unittest.skipIf(_SKIP, _REASON)
|
||||
class TestBodyFile(unittest.TestCase):
|
||||
|
||||
@patch("create_issue.urllib.request.urlopen", return_value=_mock_urlopen())
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_body_from_file(self, _cred, _url):
|
||||
def test_body_from_file(self, _cred):
|
||||
with tempfile.NamedTemporaryFile(mode="w", suffix=".md", delete=False) as f:
|
||||
f.write("File body content")
|
||||
f.flush()
|
||||
with patch("create_issue.urllib.request.Request") as MockReq:
|
||||
MockReq.return_value = MagicMock()
|
||||
with patch("create_issue.urllib.request.urlopen",
|
||||
return_value=_mock_urlopen()):
|
||||
create_issue.main(["--title", "T", "--body-file", f.name])
|
||||
data = json.loads(MockReq.call_args[1]["data"].decode("utf-8"))
|
||||
self.assertEqual(data["body"], "File body content")
|
||||
with patch("create_issue.api_request",
|
||||
return_value={"number": 1, "html_url": "http://x/1"}) as mock_api:
|
||||
create_issue.main(["--title", "T", "--body-file", f.name])
|
||||
payload = mock_api.call_args[0][3]
|
||||
self.assertEqual(payload["body"], "File body content")
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
@@ -165,19 +143,15 @@ class TestAuthFailure(unittest.TestCase):
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# HTTP error handling
|
||||
# API error handling
|
||||
# ---------------------------------------------------------------------------
|
||||
@unittest.skipIf(_SKIP, _REASON)
|
||||
class TestHTTPError(unittest.TestCase):
|
||||
class TestAPIError(unittest.TestCase):
|
||||
|
||||
@patch("create_issue.get_credentials", return_value=FAKE_CREDS)
|
||||
def test_http_error_returns_1(self, _cred):
|
||||
import urllib.error
|
||||
err = urllib.error.HTTPError(
|
||||
url="https://example.com", code=422, msg="Unprocessable",
|
||||
hdrs=None, fp=io.BytesIO(b'{"message":"duplicate"}'),
|
||||
)
|
||||
with patch("create_issue.urllib.request.urlopen", side_effect=err):
|
||||
def test_api_error_returns_1(self, _cred):
|
||||
with patch("create_issue.api_request",
|
||||
side_effect=RuntimeError("HTTP 422: duplicate")):
|
||||
rc = create_issue.main(["--title", "Dup"])
|
||||
self.assertEqual(rc, 1)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user