@mmerterden/multi-agent-pipeline 11.2.0 → 11.3.1

package/pipeline/skills/figma-ios/figma-to-component/scripts/lib/test_figma_update_http.py

@@ -1,149 +0,0 @@
-"""HTTP path tests for figma-update.py.
-
-Mocks http.client.HTTPSConnection so we can exercise fetch_file_last_modified
-and load_figma_token end-to-end without touching the real Figma API.
-
-Run: python3 -m unittest test_figma_update_http.py
-"""
-
-import importlib.util
-import json
-import os
-import sys
-import unittest
-from pathlib import Path
-from unittest.mock import MagicMock, patch
-
-SCRIPTS_DIR = Path(__file__).parent.parent
-spec = importlib.util.spec_from_file_location(
-    "figma_update", SCRIPTS_DIR / "figma-update.py"
-)
-figma_update = importlib.util.module_from_spec(spec)
-sys.modules["figma_update"] = figma_update
-spec.loader.exec_module(figma_update)
-
-
-def _make_mock_response(status: int, body: str) -> MagicMock:
-    resp = MagicMock()
-    resp.status = status
-    resp.read.return_value = body.encode("utf-8")
-    return resp
-
-
-def _make_mock_conn(response) -> MagicMock:
-    conn = MagicMock()
-    conn.getresponse.return_value = response
-    return conn
-
-
-class FetchFileLastModifiedTests(unittest.TestCase):
-    def test_happy_path_returns_last_modified(self):
-        body = json.dumps({"lastModified": "2026-04-11T02:14:00Z", "name": "X"})
-        resp = _make_mock_response(200, body)
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn):
-            result = figma_update.fetch_file_last_modified("FILEKEY", "TOKEN")
-
-        self.assertEqual(result, "2026-04-11T02:14:00Z")
-
-    def test_uses_correct_url_and_headers(self):
-        body = json.dumps({"lastModified": "2026-04-11T02:14:00Z"})
-        resp = _make_mock_response(200, body)
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn) as mock_https:
-            figma_update.fetch_file_last_modified("ABC123", "TOKEN_XYZ")
-
-        # Correct host + timeout
-        mock_https.assert_called_once_with(figma_update.FIGMA_API_HOST, timeout=30)
-
-        # Correct URL path and auth header
-        conn.request.assert_called_once()
-        args, kwargs = conn.request.call_args
-        method, path = args[0], args[1]
-        headers = kwargs.get("headers", {})
-        self.assertEqual(method, "GET")
-        self.assertIn("/v1/files/ABC123", path)
-        self.assertIn("depth=1", path)
-        self.assertEqual(headers["X-Figma-Token"], "TOKEN_XYZ")
-
-    def test_non_200_raises_runtime_error(self):
-        resp = _make_mock_response(403, '{"err":"Forbidden"}')
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn):
-            with self.assertRaises(RuntimeError) as ctx:
-                figma_update.fetch_file_last_modified("FILEKEY", "BADTOKEN")
-
-        self.assertIn("403", str(ctx.exception))
-
-    def test_missing_last_modified_field_raises(self):
-        body = json.dumps({"name": "file with no lastModified"})
-        resp = _make_mock_response(200, body)
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn):
-            with self.assertRaises(RuntimeError) as ctx:
-                figma_update.fetch_file_last_modified("FILEKEY", "TOKEN")
-
-        self.assertIn("lastModified", str(ctx.exception))
-
-    def test_connection_closed_on_success(self):
-        body = json.dumps({"lastModified": "2026-04-11T02:14:00Z"})
-        resp = _make_mock_response(200, body)
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn):
-            figma_update.fetch_file_last_modified("FILEKEY", "TOKEN")
-
-        conn.close.assert_called_once()
-
-    def test_connection_closed_on_error(self):
-        resp = _make_mock_response(500, '{"err":"boom"}')
-        conn = _make_mock_conn(resp)
-
-        with patch.object(figma_update.http.client, "HTTPSConnection", return_value=conn):
-            with self.assertRaises(RuntimeError):
-                figma_update.fetch_file_last_modified("FILEKEY", "TOKEN")
-
-        conn.close.assert_called_once()
-
-
-class LoadFigmaTokenTests(unittest.TestCase):
-    def test_prefers_env_var(self):
-        with patch.dict(os.environ, {"FIGMA_ACCESS_TOKEN": "env-token"}, clear=False):
-            token = figma_update.load_figma_token()
-        self.assertEqual(token, "env-token")
-
-    def test_falls_back_to_keychain(self):
-        # Remove env var, mock subprocess.run
-        env = {k: v for k, v in os.environ.items() if k != "FIGMA_ACCESS_TOKEN"}
-        fake_result = MagicMock()
-        fake_result.returncode = 0
-        fake_result.stdout = "keychain-token\n"
-        with patch.dict(os.environ, env, clear=True):
-            with patch.object(figma_update.subprocess, "run", return_value=fake_result):
-                token = figma_update.load_figma_token()
-        self.assertEqual(token, "keychain-token")
-
-    def test_returns_none_when_nothing_available(self):
-        env = {k: v for k, v in os.environ.items() if k != "FIGMA_ACCESS_TOKEN"}
-        fake_result = MagicMock()
-        fake_result.returncode = 1
-        fake_result.stdout = ""
-        with patch.dict(os.environ, env, clear=True):
-            with patch.object(figma_update.subprocess, "run", return_value=fake_result):
-                token = figma_update.load_figma_token()
-        self.assertIsNone(token)
-
-    def test_keychain_exception_returns_none(self):
-        env = {k: v for k, v in os.environ.items() if k != "FIGMA_ACCESS_TOKEN"}
-        with patch.dict(os.environ, env, clear=True):
-            with patch.object(figma_update.subprocess, "run", side_effect=FileNotFoundError("no security")):
-                token = figma_update.load_figma_token()
-        self.assertIsNone(token)
-
-
-if __name__ == "__main__":
-    unittest.main(verbosity=2)

package/pipeline/skills/figma-ios/figma-to-component/scripts/lib/test_phase_clis.py

@@ -1,281 +0,0 @@
-"""Integration tests for phase{4,5,6,7}-finalize.py CLI wrappers.
-
-Invokes the scripts via subprocess against a tempdir registry + fake
-Swift source file. Asserts the written JSON end-state for each
-scenario including the full Phase 4 → 5 → 6 → 7 happy path, the
-byte-identical no-op, and the yellow cascade on content change.
-
-Run: python3 -m unittest test_phase_clis.py
-"""
-
-import json
-import subprocess
-import sys
-import tempfile
-import unittest
-from pathlib import Path
-
-SCRIPTS_DIR = Path(__file__).parent.parent
-PHASE4 = SCRIPTS_DIR / "phase4-finalize.py"
-PHASE5 = SCRIPTS_DIR / "phase5-finalize.py"
-PHASE6 = SCRIPTS_DIR / "phase6-finalize.py"
-PHASE7 = SCRIPTS_DIR / "phase7-finalize.py"
-
-
-def _run(script: Path, *args: str) -> subprocess.CompletedProcess:
-    return subprocess.run(
-        ["python3", str(script), *args],
-        capture_output=True,
-        text=True,
-        check=False,
-    )
-
-
-class Phase4CliTests(unittest.TestCase):
-    def setUp(self):
-        self.tmp = tempfile.TemporaryDirectory()
-        self.root = Path(self.tmp.name)
-        self.registry = self.root / "reg"
-        self.registry.mkdir()
-        self.swift = self.root / "Button.swift"
-        self.swift.write_text("struct Button {}\n")
-        self.node_id = "5:17752"
-        self.registry_file = self.registry / "5-17752.json"
-
-    def tearDown(self):
-        self.tmp.cleanup()
-
-    def test_first_run_creates_and_sets_version_1(self):
-        r = _run(
-            PHASE4,
-            "--node-id", self.node_id,
-            "--component-name", "Button",
-            "--source-file", str(self.swift),
-            "--registry-dir", str(self.registry),
-        )
-        self.assertEqual(r.returncode, 0, msg=r.stderr)
-        data = json.loads(self.registry_file.read_text())
-        impl = data["status"]["ios"]["implemented"]
-        self.assertTrue(impl["value"])
-        self.assertEqual(impl["version"], 1)
-        self.assertIsNotNone(impl["hash"])
-        self.assertIn("sha256:", impl["hash"])
-
-    def test_byte_identical_rerun_is_noop(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        first = json.loads(self.registry_file.read_text())["status"]["ios"]["implemented"]
-
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        second = json.loads(self.registry_file.read_text())["status"]["ios"]["implemented"]
-
-        self.assertEqual(second["version"], 1)
-        self.assertEqual(second["hash"], first["hash"])
-        self.assertEqual(second["at"], first["at"])
-
-    def test_content_change_bumps_version(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        self.swift.write_text("struct Button { let x: Int }\n")
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        data = json.loads(self.registry_file.read_text())
-        self.assertEqual(data["status"]["ios"]["implemented"]["version"], 2)
-
-    def test_missing_source_file_errors(self):
-        r = _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", "/nope/missing.swift", "--registry-dir", str(self.registry),
-        )
-        self.assertEqual(r.returncode, 1)
-        self.assertIn("--source-file", r.stderr)
-
-    def test_missing_registry_dir_errors(self):
-        r = _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", "/nope",
-        )
-        self.assertEqual(r.returncode, 1)
-
-    def test_android_platform(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-            "--platform", "android",
-        )
-        data = json.loads(self.registry_file.read_text())
-        self.assertTrue(data["status"]["android"]["implemented"]["value"])
-        self.assertFalse(data["status"]["ios"]["implemented"]["value"])
-
-
-class Phase5CliTests(unittest.TestCase):
-    def setUp(self):
-        self.tmp = tempfile.TemporaryDirectory()
-        self.root = Path(self.tmp.name)
-        self.registry = self.root / "reg"
-        self.registry.mkdir()
-        self.swift = self.root / "Button.swift"
-        self.swift.write_text("struct Button {}\n")
-        self.node_id = "5:17752"
-        self.registry_file = self.registry / "5-17752.json"
-
-    def tearDown(self):
-        self.tmp.cleanup()
-
-    def test_tested_recorded_against_current_impl_version(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        r = _run(PHASE5, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-        self.assertEqual(r.returncode, 0, msg=r.stderr)
-        data = json.loads(self.registry_file.read_text())
-        self.assertTrue(data["status"]["ios"]["tested"]["value"])
-        self.assertEqual(data["status"]["ios"]["tested"]["against"], 1)
-
-
-class Phase6CliTests(unittest.TestCase):
-    def setUp(self):
-        self.tmp = tempfile.TemporaryDirectory()
-        self.root = Path(self.tmp.name)
-        self.registry = self.root / "reg"
-        self.registry.mkdir()
-        self.node_id = "99:88"
-        self.registry_file = self.registry / "99-88.json"
-
-    def tearDown(self):
-        self.tmp.cleanup()
-
-    def test_phase6_creates_legacy_shell_for_fresh_node(self):
-        r = _run(
-            PHASE6,
-            "--node-id", self.node_id,
-            "--file-key", "ABC123",
-            "--component-name", "StandaloneIcon",
-            "--registry-dir", str(self.registry),
-        )
-        self.assertEqual(r.returncode, 0, msg=r.stderr)
-        data = json.loads(self.registry_file.read_text())
-        # Legacy shell fields
-        self.assertIn("fileKey", data)
-        self.assertEqual(data["fileKey"], "ABC123")
-        self.assertIn("componentName", data)
-        self.assertEqual(data["componentName"]["ios"], "StandaloneIcon")
-        # Status block created via set_flag
-        self.assertTrue(data["status"]["ios"]["codeConnect"]["value"])
-        # Legacy mirror also kept
-        self.assertTrue(data["codeConnect"]["ios"])
-
-    def test_phase6_action_created_vs_updated(self):
-        # First run → created
-        r1 = _run(
-            PHASE6, "--node-id", self.node_id, "--file-key", "ABC",
-            "--component-name", "X", "--registry-dir", str(self.registry),
-        )
-        out1 = json.loads(r1.stdout)
-        self.assertEqual(out1["action"], "created")
-
-        # Second run → updated
-        r2 = _run(
-            PHASE6, "--node-id", self.node_id, "--file-key", "ABC",
-            "--component-name", "X", "--registry-dir", str(self.registry),
-        )
-        out2 = json.loads(r2.stdout)
-        self.assertEqual(out2["action"], "updated")
-
-
-class FullPipelineChainTests(unittest.TestCase):
-    def setUp(self):
-        self.tmp = tempfile.TemporaryDirectory()
-        self.root = Path(self.tmp.name)
-        self.registry = self.root / "reg"
-        self.registry.mkdir()
-        self.swift = self.root / "Button.swift"
-        self.swift.write_text("struct Button {}\n")
-        self.node_id = "5:17752"
-        self.registry_file = self.registry / "5-17752.json"
-
-    def tearDown(self):
-        self.tmp.cleanup()
-
-    def _data(self):
-        return json.loads(self.registry_file.read_text())
-
-    def test_phase_4_5_6_7_chain_all_green(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        _run(PHASE5, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-        _run(
-            PHASE6, "--node-id", self.node_id, "--file-key", "K",
-            "--component-name", "Button", "--registry-dir", str(self.registry),
-        )
-        _run(PHASE7, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-
-        d = self._data()
-        s = d["status"]["ios"]
-        self.assertEqual(s["implemented"]["version"], 1)
-        self.assertEqual(s["tested"]["against"], 1)
-        self.assertEqual(s["codeConnect"]["against"], 1)
-        self.assertEqual(s["wiki"]["against"], 1)
-        self.assertTrue(d["codeConnect"]["ios"])  # legacy mirror
-
-    def test_yellow_cascade_after_phase4_rerun(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        _run(PHASE5, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-        _run(
-            PHASE6, "--node-id", self.node_id, "--file-key", "K",
-            "--component-name", "Button", "--registry-dir", str(self.registry),
-        )
-        _run(PHASE7, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-
-        # Content change → impl version bumps, downstream goes yellow
-        self.swift.write_text("struct Button { let x: Int }\n")
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-
-        d = self._data()
-        s = d["status"]["ios"]
-        self.assertEqual(s["implemented"]["version"], 2)
-        # tested/codeConnect/wiki still have against=1
-        self.assertEqual(s["tested"]["against"], 1)
-        self.assertEqual(s["codeConnect"]["against"], 1)
-        self.assertEqual(s["wiki"]["against"], 1)
-
-    def test_rerun_phase_clears_yellow(self):
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        _run(PHASE5, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-
-        # Bump impl
-        self.swift.write_text("struct Button { let x: Int }\n")
-        _run(
-            PHASE4, "--node-id", self.node_id, "--component-name", "Button",
-            "--source-file", str(self.swift), "--registry-dir", str(self.registry),
-        )
-        # impl=2, tested.against=1 → yellow
-
-        # Re-run phase 5
-        _run(PHASE5, "--node-id", self.node_id, "--registry-dir", str(self.registry))
-        self.assertEqual(self._data()["status"]["ios"]["tested"]["against"], 2)
-
-
-if __name__ == "__main__":
-    unittest.main(verbosity=2)