npm - @groupby/ai-dev - Versions diffs - 0.5.16 → 0.5.17 - Mend

@groupby/ai-dev 0.5.16 → 0.5.17

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (27) hide show

package/teams/firstspirit-caas/mcp/test_bamboo_artifacts.py ADDED Viewed

@@ -0,0 +1,315 @@
+"""Standalone tests for the Bamboo artifact tools.
+Run: python3 plugins/planetexpress/mcp/test_bamboo_artifacts.py
+"""
+import importlib.util
+import json
+import os
+import unittest
+from unittest.mock import MagicMock, patch
+os.environ.setdefault("JIRA_BASE_URL", "https://jira.example.com")
+os.environ.setdefault("JIRA_TOKEN", "test-token")
+os.environ.setdefault("BITBUCKET_BASE_URL", "https://bb.example.com")
+os.environ.setdefault("BITBUCKET_TOKEN", "test-token")
+os.environ.setdefault("BAMBOO_BASE_URL", "https://ci.example.com")
+os.environ.setdefault("BAMBOO_TOKEN", "test-token")
+_here = os.path.dirname(os.path.abspath(__file__))
+_spec = importlib.util.spec_from_file_location(
+    "jira_tools", os.path.join(_here, "jira-tools.py")
+)
+_mod = importlib.util.module_from_spec(_spec)
+_spec.loader.exec_module(_mod)
+def _mock_resp(body: bytes = b"", content_type: str = "application/json"):
+    resp = MagicMock()
+    resp.read.return_value = body
+    resp.headers = {"Content-Type": content_type}
+    resp.__enter__ = lambda s: s
+    resp.__exit__ = MagicMock(return_value=False)
+    return resp
+class TestBambooRequestHeaders(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_return_headers_gives_bytes_and_content_type(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"<html>x</html>", "text/html;charset=utf-8")
+        raw, ctype = _mod.bamboo_request("/artifact/x", accept_json=False, return_headers=True)
+        self.assertEqual(raw, b"<html>x</html>")
+        self.assertEqual(ctype, "text/html;charset=utf-8")
+    @patch("urllib.request.urlopen")
+    def test_default_contract_unchanged(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(json.dumps({"a": 1}).encode())
+        self.assertEqual(_mod.bamboo_request("/rest/x"), {"a": 1})
+# Mirrors real Bamboo output: the fetched URL is in /browse/ space, the parent
+# link is UNQUOTED with "Parent Directory" text, and child links are quoted
+# absolute /artifact/{plan}/build-N/ paths in a DIFFERENT prefix space.
+INDEX_HTML = """<HTML><HEAD><TITLE>build</TITLE></HEAD><BODY>
+<H1>build</H1><TABLE BORDER=0><TR><TD><A HREF=../label>Parent Directory</A></TD></TR>
+<TR><TD><img src="/images/icons/icon_folder.gif" alt="(dir)">&nbsp;<A HREF="/artifact/PLAN/BUILD/build-21/label/build/reports">reports</a>&nbsp;</TD></TR>
+<TR><TD><A HREF="/artifact/PLAN/BUILD/build-21/label/build/index.html">index.html</a></TD></TR>
+</TABLE></BODY></HTML>"""
+class TestParseArtifactIndex(unittest.TestCase):
+    def test_parses_children_across_url_space(self):
+        base = "https://ci.example.com/browse/PLAN-21/artifact/BUILD/label/build"
+        entries = _mod._parse_artifact_index(INDEX_HTML, base)
+        by_name = {e["name"]: e for e in entries}
+        self.assertEqual(set(by_name), {"reports", "index.html"})
+        self.assertTrue(by_name["reports"]["isDir"])
+        self.assertFalse(by_name["index.html"]["isDir"])
+        self.assertEqual(
+            by_name["reports"]["href"],
+            "https://ci.example.com/artifact/PLAN/BUILD/build-21/label/build/reports")
+    def test_skips_parent_link(self):
+        base = "https://ci.example.com/browse/PLAN-21/artifact/BUILD/label/build"
+        entries = _mod._parse_artifact_index(INDEX_HTML, base)
+        for e in entries:
+            self.assertNotIn("parent", e["name"].lower())
+            self.assertNotIn("label", e["name"].lower())
+def _result_with_artifacts():
+    return {
+        "buildResultKey": "PLAN-21",
+        "artifacts": {"size": 0, "artifact": []},
+        "stages": {"stage": [{"results": {"result": [{
+            "buildResultKey": "PLAN-BUILD-21",
+            "artifacts": {"size": 2, "artifact": [
+                {"name": "Build log", "shared": False,
+                 "link": {"href": "https://ci.example.com/download/PLAN-BUILD/build_logs/PLAN-BUILD-21.log"}},
+                {"name": "owasp dependency check report", "shared": False,
+                 "size": 2252875, "prettySizeDescription": "2 MB",
+                 "producerJobKey": "PLAN-BUILD-21",
+                 "link": {"href": "https://ci.example.com/browse/PLAN-21/artifact/BUILD/owasp-dependency-check-report/build"}},
+            ]},
+        }]}}]},
+    }
+# The dep-check artifact href (browse space) lists a child in /artifact/ space,
+# which in turn lists the report file — the real two-level, two-space tree.
+DIR_BUILD = """<HTML><BODY><TABLE><TR><TD><A HREF=../owasp-dependency-check-report>Parent Directory</A></TD></TR>
+<TR><TD><A HREF="/artifact/PLAN/BUILD/build-21/owasp-dependency-check-report/build/reports">reports</a></TD></TR></TABLE></BODY></HTML>"""
+DIR_REPORTS = """<HTML><BODY><TABLE><TR><TD><A HREF=..>Parent Directory</A></TD></TR>
+<TR><TD><A HREF="/artifact/PLAN/BUILD/build-21/owasp-dependency-check-report/build/reports/dependency-check-report.html">dependency-check-report.html</a></TD></TR></TABLE></BODY></HTML>"""
+class TestListBambooArtifacts(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_flattens_jobs_and_recurses_dir(self, mock_urlopen):
+        mock_urlopen.side_effect = [
+            _mock_resp(json.dumps(_result_with_artifacts()).encode()),
+            _mock_resp(DIR_BUILD.encode(), "text/html"),
+            _mock_resp(DIR_REPORTS.encode(), "text/html"),
+        ]
+        result = _mod.list_bamboo_artifacts("PLAN-21")
+        names = {a["name"]: a for a in result["artifacts"]}
+        self.assertIn("owasp dependency check report", names)
+        self.assertIn("Build log", names)
+        self.assertFalse(names["Build log"]["isDirectory"])
+        dep = names["owasp dependency check report"]
+        self.assertTrue(dep["isDirectory"])
+        leaves = [f["name"] for f in dep["files"]]
+        self.assertIn("reports/dependency-check-report.html", leaves)
+    def test_registered_in_tools(self):
+        self.assertIn("list_bamboo_artifacts", _mod.TOOLS)
+class TestGetBambooArtifact(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_fetch_by_href_inlines_small_text(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"hello report", "text/plain")
+        r = _mod.get_bamboo_artifact(
+            "PLAN-21",
+            href="https://ci.example.com/artifact/PLAN/BUILD/build-21/label/reports/x.txt",
+        )
+        self.assertEqual(r["content"], "hello report")
+        self.assertEqual(r["contentType"], "text/plain")
+        self.assertTrue(os.path.exists(r["localPath"]))
+        self.assertEqual(r["file"], "x.txt")
+    @patch("urllib.request.urlopen")
+    def test_oversized_text_is_path_only(self, mock_urlopen):
+        big = b"A" * 5000
+        mock_urlopen.return_value = _mock_resp(big, "text/html")
+        r = _mod.get_bamboo_artifact(
+            "PLAN-21",
+            href="https://ci.example.com/artifact/PLAN/BUILD/build-21/label/big.html",
+            max_inline_bytes="1000",
+        )
+        self.assertIsNone(r["content"])
+        self.assertIn("too large", r["note"])
+        self.assertTrue(os.path.exists(r["localPath"]))
+    @patch("urllib.request.urlopen")
+    def test_binary_is_path_only(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"\x89PNG\x00\x01", "image/png")
+        r = _mod.get_bamboo_artifact(
+            "PLAN-21",
+            href="https://ci.example.com/artifact/PLAN/BUILD/build-21/label/img.png",
+        )
+        self.assertIsNone(r["content"])
+        self.assertIn("binary", r["note"])
+    @patch("urllib.request.urlopen")
+    def test_fetch_by_name_and_file_from_directory(self, mock_urlopen):
+        mock_urlopen.side_effect = [
+            _mock_resp(json.dumps(_result_with_artifacts()).encode()),
+            _mock_resp(DIR_BUILD.encode(), "text/html"),
+            _mock_resp(DIR_REPORTS.encode(), "text/html"),
+            _mock_resp(b"<html>dep-check</html>", "text/html"),
+        ]
+        r = _mod.get_bamboo_artifact(
+            "PLAN-21",
+            name="owasp dependency check report",
+            file="dependency-check-report.html",
+        )
+        self.assertEqual(r["content"], "<html>dep-check</html>")
+        self.assertEqual(r["file"], "reports/dependency-check-report.html")
+    @patch("urllib.request.urlopen")
+    def test_unknown_name_lists_available(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(json.dumps(_result_with_artifacts()).encode())
+        with self.assertRaises(RuntimeError) as ctx:
+            _mod.get_bamboo_artifact("PLAN-21", name="nope")
+        self.assertIn("owasp dependency check report", str(ctx.exception))
+    def test_registered_in_tools(self):
+        self.assertIn("get_bamboo_artifact", _mod.TOOLS)
+class TestKeyFromRef(unittest.TestCase):
+    def test_bare_key_unchanged(self):
+        self.assertEqual(_mod._bamboo_key_from_ref("CAASCAASCONNECT-BUILD"), "CAASCAASCONNECT-BUILD")
+    def test_browse_url_stripped(self):
+        self.assertEqual(
+            _mod._bamboo_key_from_ref("https://ci.example.com/browse/CAASCAASCONNECT-BUILD"),
+            "CAASCAASCONNECT-BUILD")
+    def test_query_and_trailing_slash_stripped(self):
+        self.assertEqual(
+            _mod._bamboo_key_from_ref("https://ci.example.com/browse/CAAS-BUILD/?x=1#frag"),
+            "CAAS-BUILD")
+class TestListBambooBranchPlans(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_parses_branches_and_accepts_url(self, mock_urlopen):
+        body = {
+            "key": "PLAN-BUILD",
+            "branches": {"size": 2, "branch": [
+                {"shortName": "feature/x", "key": "PLAN-BUILD2", "enabled": True},
+                {"shortName": "feature/y", "key": "PLAN-BUILD3", "enabled": False},
+            ]},
+        }
+        mock_urlopen.return_value = _mock_resp(json.dumps(body).encode())
+        r = _mod.list_bamboo_branch_plans("https://ci.example.com/browse/PLAN-BUILD")
+        self.assertEqual(r["planKey"], "PLAN-BUILD")
+        self.assertEqual(r["branchCount"], 2)
+        self.assertEqual(r["branchPlans"][0],
+            {"shortName": "feature/x", "key": "PLAN-BUILD2", "enabled": True,
+             "url": "https://ci.example.com/browse/PLAN-BUILD2"})
+        called = mock_urlopen.call_args[0][0].full_url
+        self.assertIn("/rest/api/latest/plan/PLAN-BUILD/branch.json", called)
+    @patch("urllib.request.urlopen")
+    def test_master_only_plan_empty(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(json.dumps({"key": "PLAN-REL", "branches": {"size": 0}}).encode())
+        r = _mod.list_bamboo_branch_plans("PLAN-REL")
+        self.assertEqual(r["branchCount"], 0)
+        self.assertEqual(r["branchPlans"], [])
+    def test_registered_in_tools(self):
+        self.assertIn("list_bamboo_branch_plans", _mod.TOOLS)
+class TestListBambooPlans(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_parses_plans_and_accepts_url(self, mock_urlopen):
+        body = {
+            "key": "CAASCAASCONNECT",
+            "name": "caas-connect",
+            "plans": {"size": 1, "plan": [
+                {"shortName": "Build", "key": "CAASCAASCONNECT-BUILD", "name": "caas Build", "enabled": True},
+            ]},
+        }
+        mock_urlopen.return_value = _mock_resp(json.dumps(body).encode())
+        r = _mod.list_bamboo_plans("https://ci.example.com/browse/CAASCAASCONNECT")
+        self.assertEqual(r["projectKey"], "CAASCAASCONNECT")
+        self.assertEqual(r["planCount"], 1)
+        self.assertEqual(r["plans"][0]["key"], "CAASCAASCONNECT-BUILD")
+        self.assertEqual(r["plans"][0]["url"], "https://ci.example.com/browse/CAASCAASCONNECT-BUILD")
+        called = mock_urlopen.call_args[0][0].full_url
+        self.assertIn("/rest/api/latest/project/CAASCAASCONNECT.json", called)
+    def test_registered_in_tools(self):
+        self.assertIn("list_bamboo_plans", _mod.TOOLS)
+class TestListBambooPlanRuns(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_newest_and_oldest_when_all_retained_fetched(self, mock_urlopen):
+        body = {"results": {"size": 2, "result": [
+            {"buildNumber": 12, "key": "PLAN-BUILD-12", "state": "Failed",
+             "buildCompletedTime": "2024-01-02", "buildRelativeTime": "1 day ago"},
+            {"buildNumber": 11, "key": "PLAN-BUILD-11", "state": "Successful",
+             "buildCompletedTime": "2024-01-01", "buildRelativeTime": "2 days ago"},
+        ]}}
+        mock_urlopen.return_value = _mock_resp(json.dumps(body).encode())
+        r = _mod.list_bamboo_plan_runs("https://ci.example.com/browse/PLAN-BUILD")
+        self.assertEqual(r["retainedCount"], 2)
+        self.assertEqual(r["latestBuildNumber"], 12)
+        self.assertEqual(r["newest"]["buildNumber"], 12)
+        self.assertEqual(r["oldestRetained"]["buildNumber"], 11)
+        self.assertEqual(r["newest"]["url"], "https://ci.example.com/browse/PLAN-BUILD-12")
+        self.assertEqual(mock_urlopen.call_count, 1)
+    @patch("urllib.request.urlopen")
+    def test_oldest_fetched_via_tail_when_truncated(self, mock_urlopen):
+        page1 = {"results": {"size": 30, "result": [
+            {"buildNumber": 30, "key": "PLAN-BUILD-30", "state": "Successful"},
+        ]}}
+        tail = {"results": {"size": 30, "result": [
+            {"buildNumber": 1, "key": "PLAN-BUILD-1", "state": "Failed"},
+        ]}}
+        mock_urlopen.side_effect = [
+            _mock_resp(json.dumps(page1).encode()),
+            _mock_resp(json.dumps(tail).encode()),
+        ]
+        r = _mod.list_bamboo_plan_runs("PLAN-BUILD", max_results="1")
+        self.assertEqual(r["newest"]["buildNumber"], 30)
+        self.assertEqual(r["oldestRetained"]["buildNumber"], 1)
+        self.assertIn("start-index=29", mock_urlopen.call_args_list[1][0][0].full_url)
+    @patch("urllib.request.urlopen")
+    def test_empty_plan(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(json.dumps({"results": {"size": 0}}).encode())
+        r = _mod.list_bamboo_plan_runs("PLAN-BUILD508")
+        self.assertEqual(r["retainedCount"], 0)
+        self.assertIsNone(r["newest"])
+        self.assertIsNone(r["oldestRetained"])
+        self.assertIsNone(r["latestBuildNumber"])
+    def test_registered_in_tools(self):
+        self.assertIn("list_bamboo_plan_runs", _mod.TOOLS)
+if __name__ == "__main__":
+    unittest.main()

package/teams/firstspirit-caas/mcp/test_jira_links.py ADDED Viewed

@@ -0,0 +1,192 @@
+"""Standalone tests for the Jira issue-link tools.
+Run: python3 plugins/planetexpress/mcp/test_jira_links.py
+"""
+import importlib.util
+import json
+import os
+import unittest
+from unittest.mock import MagicMock, patch
+os.environ.setdefault("JIRA_BASE_URL", "https://jira.example.com")
+os.environ.setdefault("JIRA_TOKEN", "test-token")
+os.environ.setdefault("BITBUCKET_BASE_URL", "https://bb.example.com")
+os.environ.setdefault("BITBUCKET_TOKEN", "test-token")
+os.environ.setdefault("BAMBOO_BASE_URL", "https://bamboo.example.com")
+os.environ.setdefault("BAMBOO_TOKEN", "test-token")
+_here = os.path.dirname(os.path.abspath(__file__))
+_spec = importlib.util.spec_from_file_location(
+    "jira_tools", os.path.join(_here, "jira-tools.py")
+)
+_mod = importlib.util.module_from_spec(_spec)
+_spec.loader.exec_module(_mod)
+def _mock_resp(body: bytes = b""):
+    resp = MagicMock()
+    resp.read.return_value = body
+    resp.__enter__ = lambda s: s
+    resp.__exit__ = MagicMock(return_value=False)
+    return resp
+# Standard link-type catalogue used across tests.
+LINK_TYPES = {
+    "issueLinkTypes": [
+        {"id": "10000", "name": "Blocks", "inward": "is blocked by", "outward": "blocks"},
+        {"id": "10001", "name": "Relates", "inward": "relates to", "outward": "relates to"},
+        {"id": "10002", "name": "Sequence", "inward": "follows on from", "outward": "precedes"},
+    ]
+}
+def _link_types_resp():
+    return _mock_resp(json.dumps(LINK_TYPES).encode())
+class TestGetJiraLinkTypes(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_returns_parsed_types(self, mock_urlopen):
+        mock_urlopen.return_value = _link_types_resp()
+        result = _mod.get_jira_link_types()
+        names = [t["name"] for t in result["link_types"]]
+        self.assertEqual(len(result["link_types"]), 3)
+        self.assertIn("Blocks", names)
+        self.assertEqual(result["link_types"][0]["outward"], "blocks")
+        self.assertEqual(result["link_types"][0]["inward"], "is blocked by")
+        req = mock_urlopen.call_args[0][0]
+        self.assertTrue(req.full_url.endswith("/rest/api/2/issueLinkType"))
+    def test_registered_in_tools(self):
+        self.assertIn("get_jira_link_types", _mod.TOOLS)
+        self.assertEqual(_mod.TOOLS["get_jira_link_types"]["parameters"], {})
+class TestLinkJiraIssues(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_outward_phrase(self, mock_urlopen):
+        mock_urlopen.side_effect = [_link_types_resp(), _mock_resp(b"")]
+        result = _mod.link_jira_issues("HP-1", "HP-2", "blocks")
+        self.assertEqual(result["type"], "Blocks")
+        self.assertEqual(result["summary"], "HP-1 blocks HP-2")
+        post_req = mock_urlopen.call_args_list[1][0][0]
+        self.assertEqual(post_req.method, "POST")
+        self.assertTrue(post_req.full_url.endswith("/rest/api/2/issueLink"))
+        sent = json.loads(post_req.data.decode())
+        self.assertEqual(sent["type"]["name"], "Blocks")
+        # Jira renders inwardIssue <outward> outwardIssue, so "HP-1 blocks HP-2"
+        # requires HP-1 in the inward slot.
+        self.assertEqual(sent["inwardIssue"]["key"], "HP-1")
+        self.assertEqual(sent["outwardIssue"]["key"], "HP-2")
+    @patch("urllib.request.urlopen")
+    def test_inward_phrase_swaps_direction(self, mock_urlopen):
+        mock_urlopen.side_effect = [_link_types_resp(), _mock_resp(b"")]
+        result = _mod.link_jira_issues("HP-1", "HP-2", "is blocked by")
+        sent = json.loads(mock_urlopen.call_args_list[1][0][0].data.decode())
+        self.assertEqual(sent["type"]["name"], "Blocks")
+        # "HP-1 is blocked by HP-2" == outwardIssue <inward> inwardIssue, so HP-1
+        # goes in the outward slot.
+        self.assertEqual(sent["outwardIssue"]["key"], "HP-1")
+        self.assertEqual(sent["inwardIssue"]["key"], "HP-2")
+        self.assertEqual(result["summary"], "HP-1 is blocked by HP-2")
+    @patch("urllib.request.urlopen")
+    def test_bare_type_name_defaults_outward(self, mock_urlopen):
+        mock_urlopen.side_effect = [_link_types_resp(), _mock_resp(b"")]
+        result = _mod.link_jira_issues("HP-1", "HP-2", "Relates")
+        sent = json.loads(mock_urlopen.call_args_list[1][0][0].data.decode())
+        self.assertEqual(sent["type"]["name"], "Relates")
+        self.assertEqual(sent["inwardIssue"]["key"], "HP-1")
+        self.assertEqual(result["relationship"], "relates to")
+    @patch("urllib.request.urlopen")
+    def test_symmetric_phrase_not_ambiguous(self, mock_urlopen):
+        mock_urlopen.side_effect = [_link_types_resp(), _mock_resp(b"")]
+        result = _mod.link_jira_issues("HP-1", "HP-2", "relates to")
+        self.assertEqual(result["type"], "Relates")
+        sent = json.loads(mock_urlopen.call_args_list[1][0][0].data.decode())
+        self.assertEqual(sent["type"]["name"], "Relates")
+    @patch("urllib.request.urlopen")
+    def test_no_match_raises_listing_phrases(self, mock_urlopen):
+        mock_urlopen.side_effect = [_link_types_resp()]
+        with self.assertRaises(ValueError) as ctx:
+            _mod.link_jira_issues("HP-1", "HP-2", "supersedes")
+        msg = str(ctx.exception)
+        self.assertIn("blocks", msg)
+        self.assertIn("relates to", msg)
+    @patch("urllib.request.urlopen")
+    def test_ambiguous_phrase_raises(self, mock_urlopen):
+        ambiguous = {"issueLinkTypes": [
+            {"id": "1", "name": "Cloners", "inward": "is cloned by", "outward": "clones"},
+            {"id": "2", "name": "Duplicate", "inward": "is duplicated by", "outward": "clones"},
+        ]}
+        mock_urlopen.side_effect = [_mock_resp(json.dumps(ambiguous).encode())]
+        with self.assertRaises(ValueError) as ctx:
+            _mod.link_jira_issues("HP-1", "HP-2", "clones")
+        self.assertIn("ambiguous", str(ctx.exception).lower())
+    def test_empty_args_raise(self):
+        with self.assertRaises(ValueError):
+            _mod.link_jira_issues("", "HP-2", "blocks")
+        with self.assertRaises(ValueError):
+            _mod.link_jira_issues("HP-1", "HP-2", "  ")
+    def test_registered_in_tools(self):
+        self.assertIn("link_jira_issues", _mod.TOOLS)
+        params = _mod.TOOLS["link_jira_issues"]["parameters"]
+        self.assertEqual(set(params), {"from_issue", "to_issue", "relationship"})
+class TestGetTicketLinks(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_extracts_outward_and_inward_links(self, mock_urlopen):
+        issue = {
+            "key": "HP-1",
+            "fields": {
+                "summary": "Parent",
+                "issuelinks": [
+                    {
+                        "id": "1",
+                        "type": {"name": "Blocks", "inward": "is blocked by", "outward": "blocks"},
+                        "outwardIssue": {
+                            "key": "HP-2",
+                            "fields": {"summary": "Downstream", "status": {"name": "Open"}},
+                        },
+                    },
+                    {
+                        "id": "2",
+                        "type": {"name": "Blocks", "inward": "is blocked by", "outward": "blocks"},
+                        "inwardIssue": {
+                            "key": "HP-3",
+                            "fields": {"summary": "Upstream", "status": {"name": "Done"}},
+                        },
+                    },
+                ],
+            },
+        }
+        mock_urlopen.return_value = _mock_resp(json.dumps(issue).encode())
+        result = _mod.get_ticket("HP-1")
+        links = result["links"]
+        self.assertEqual(len(links), 2)
+        self.assertEqual(links[0]["relationship"], "blocks")
+        self.assertEqual(links[0]["key"], "HP-2")
+        self.assertEqual(links[0]["status"], "Open")
+        self.assertEqual(links[1]["relationship"], "is blocked by")
+        self.assertEqual(links[1]["key"], "HP-3")
+    @patch("urllib.request.urlopen")
+    def test_no_links_yields_empty_list(self, mock_urlopen):
+        issue = {"key": "HP-9", "fields": {"summary": "Lonely"}}
+        mock_urlopen.return_value = _mock_resp(json.dumps(issue).encode())
+        result = _mod.get_ticket("HP-9")
+        self.assertEqual(result["links"], [])
+if __name__ == "__main__":
+    unittest.main()

package/teams/firstspirit-caas/mcp/test_update_jira_ticket.py ADDED Viewed

@@ -0,0 +1,161 @@
+"""Standalone tests for update_jira_ticket.
+Run: python3 plugins/planetexpress/mcp/test_update_jira_ticket.py
+"""
+import importlib.util
+import json
+import os
+import unittest
+from unittest.mock import MagicMock, patch
+os.environ.setdefault("JIRA_BASE_URL", "https://jira.example.com")
+os.environ.setdefault("JIRA_TOKEN", "test-token")
+os.environ.setdefault("BITBUCKET_BASE_URL", "https://bb.example.com")
+os.environ.setdefault("BITBUCKET_TOKEN", "test-token")
+os.environ.setdefault("BAMBOO_BASE_URL", "https://bamboo.example.com")
+os.environ.setdefault("BAMBOO_TOKEN", "test-token")
+_here = os.path.dirname(os.path.abspath(__file__))
+_spec = importlib.util.spec_from_file_location(
+    "jira_tools", os.path.join(_here, "jira-tools.py")
+)
+_mod = importlib.util.module_from_spec(_spec)
+_spec.loader.exec_module(_mod)
+def _mock_resp(body: bytes = b""):
+    resp = MagicMock()
+    resp.read.return_value = body
+    resp.__enter__ = lambda s: s
+    resp.__exit__ = MagicMock(return_value=False)
+    return resp
+class TestUpdateJiraTicket(unittest.TestCase):
+    @patch("urllib.request.urlopen")
+    def test_update_description_only(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"")
+        result = _mod.update_jira_ticket("HP-1234", description="Why and what.")
+        self.assertEqual(result["key"], "HP-1234")
+        self.assertTrue(result["updated"])
+        self.assertIn("HP-1234", result["link"])
+        req = mock_urlopen.call_args[0][0]
+        self.assertIn("HP-1234", req.full_url)
+        self.assertEqual(req.method, "PUT")
+        sent = json.loads(req.data.decode())
+        self.assertEqual(sent["fields"]["description"], "Why and what.")
+        self.assertNotIn("summary", sent["fields"])
+    @patch("urllib.request.urlopen")
+    def test_update_summary_only(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"")
+        _mod.update_jira_ticket("HP-1234", summary="New title")
+        sent = json.loads(mock_urlopen.call_args[0][0].data.decode())
+        self.assertEqual(sent["fields"]["summary"], "New title")
+        self.assertNotIn("description", sent["fields"])
+    @patch("urllib.request.urlopen")
+    def test_update_both_fields(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"")
+        _mod.update_jira_ticket("HP-1234", description="Why.", summary="New title")
+        sent = json.loads(mock_urlopen.call_args[0][0].data.decode())
+        self.assertEqual(sent["fields"]["description"], "Why.")
+        self.assertEqual(sent["fields"]["summary"], "New title")
+    def test_neither_field_raises(self):
+        with self.assertRaises(ValueError):
+            _mod.update_jira_ticket("HP-1234")
+    def test_empty_issue_key_raises(self):
+        with self.assertRaises(ValueError):
+            _mod.update_jira_ticket("", description="Why.")
+    @patch("urllib.request.urlopen")
+    def test_add_labels_uses_update_verb(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"")
+        result = _mod.update_jira_ticket("HP-1234", add_labels="cve, security")
+        sent = json.loads(mock_urlopen.call_args[0][0].data.decode())
+        self.assertNotIn("fields", sent)
+        self.assertEqual(
+            sent["update"]["labels"], [{"add": "cve"}, {"add": "security"}]
+        )
+        self.assertEqual(result["added_labels"], ["cve", "security"])
+    @patch("urllib.request.urlopen")
+    def test_add_and_remove_labels_with_description(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(b"")
+        _mod.update_jira_ticket(
+            "HP-1234", description="Why.", add_labels="a", remove_labels="b"
+        )
+        sent = json.loads(mock_urlopen.call_args[0][0].data.decode())
+        self.assertEqual(sent["fields"]["description"], "Why.")
+        self.assertEqual(
+            sent["update"]["labels"], [{"add": "a"}, {"remove": "b"}]
+        )
+    def test_labels_only_no_field_does_not_raise(self):
+        with patch("urllib.request.urlopen", return_value=_mock_resp(b"")):
+            result = _mod.update_jira_ticket("HP-1234", add_labels="x")
+        self.assertTrue(result["updated"])
+    def test_tool_registered_in_tools_dict(self):
+        self.assertIn("update_jira_ticket", _mod.TOOLS)
+        params = _mod.TOOLS["update_jira_ticket"]["parameters"]
+        self.assertIn("issue_key", params)
+        self.assertIn("description", params)
+        self.assertIn("summary", params)
+        self.assertIn("add_labels", params)
+        self.assertIn("remove_labels", params)
+class TestTransitionJiraTicket(unittest.TestCase):
+    _TRANSITIONS = json.dumps({
+        "transitions": [
+            {"id": "11", "name": "Backlog", "to": {"name": "Backlog"}},
+            {"id": "21", "name": "Commit", "to": {"name": "Committed"}},
+        ]
+    }).encode()
+    @patch("urllib.request.urlopen")
+    def test_get_transitions(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(self._TRANSITIONS)
+        result = _mod.get_jira_transitions("PX-386")
+        self.assertEqual(result["key"], "PX-386")
+        self.assertEqual(
+            result["transitions"],
+            [
+                {"id": "11", "name": "Backlog", "to_status": "Backlog"},
+                {"id": "21", "name": "Commit", "to_status": "Committed"},
+            ],
+        )
+    @patch("urllib.request.urlopen")
+    def test_transition_matches_target_status(self, mock_urlopen):
+        mock_urlopen.side_effect = [
+            _mock_resp(self._TRANSITIONS),
+            _mock_resp(b""),
+        ]
+        result = _mod.transition_jira_ticket("PX-386", "committed")
+        self.assertEqual(result["status"], "Committed")
+        self.assertEqual(result["transition"], "Commit")
+        post_req = mock_urlopen.call_args_list[1][0][0]
+        self.assertEqual(post_req.method, "POST")
+        self.assertIn("PX-386/transitions", post_req.full_url)
+        sent = json.loads(post_req.data.decode())
+        self.assertEqual(sent["transition"]["id"], "21")
+    @patch("urllib.request.urlopen")
+    def test_transition_no_match_raises_with_options(self, mock_urlopen):
+        mock_urlopen.return_value = _mock_resp(self._TRANSITIONS)
+        with self.assertRaises(ValueError) as ctx:
+            _mod.transition_jira_ticket("PX-386", "Done")
+        self.assertIn("Committed", str(ctx.exception))
+    def test_transition_tools_registered(self):
+        self.assertIn("get_jira_transitions", _mod.TOOLS)
+        self.assertIn("transition_jira_ticket", _mod.TOOLS)
+if __name__ == "__main__":
+    unittest.main()