@skyramp/mcp 0.3.2-rc.pom-3 → 0.3.2

package/build/services/TestDiscoveryService.test.js

@@ -1,983 +0,0 @@
-// @ts-nocheck - Jest ESM type inference issues
-import { jest } from "@jest/globals";
-// Mock heavy dependencies
-jest.mock("simple-git", () => ({
-    simpleGit: jest.fn(() => ({
-        checkIsRepo: jest.fn().mockResolvedValue(false),
-    })),
-}));
-jest.unstable_mockModule("../utils/logger.js", () => ({
-    logger: {
-        info: jest.fn(),
-        debug: jest.fn(),
-        error: jest.fn(),
-        warning: jest.fn(),
-    },
-}));
-// fast-glob must return real file paths from the temp directory.
-// We use the real implementation by not mocking it.
-import * as fs from "fs";
-import * as path from "path";
-import * as os from "os";
-import { TestSource } from "../types/TestAnalysis.js";
-const { TestDiscoveryService } = await import("./TestDiscoveryService.js");
-describe("TestDiscoveryService", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "test-discovery-"));
-    });
-    afterEach(() => {
-        fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-    function writeFile(relPath, content) {
-        const fullPath = path.join(tmpDir, relPath);
-        fs.mkdirSync(path.dirname(fullPath), { recursive: true });
-        fs.writeFileSync(fullPath, content, "utf-8");
-        return fullPath;
-    }
-    // ── classifyTestFiles ──
-    describe("classifyTestFiles", () => {
-        it("classifies files with Skyramp marker as skyramp", async () => {
-            writeFile("tests/test_orders_smoke.py", '# Generated by Skyramp\nimport skyramp\nskyramp generate smoke rest');
-            const result = await service.discoverTests(tmpDir);
-            const skyrampTests = result.tests.filter((t) => t.source === TestSource.Skyramp);
-            expect(skyrampTests.length).toBe(1);
-            expect(skyrampTests[0].testFile).toContain("test_orders_smoke.py");
-        });
-        it("classifies test files without Skyramp marker as external", async () => {
-            writeFile("tests/test_orders.py", 'import pytest\nimport requests\nresponse = requests.get("/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            expect(externalTests.length).toBe(1);
-            expect(externalTests[0].testFile).toContain("test_orders.py");
-        });
-        it("skips non-test files that lack the Skyramp marker", async () => {
-            writeFile("src/utils.py", 'def helper():\n    return True');
-            const result = await service.discoverTests(tmpDir);
-            expect(result.tests.length).toBe(0);
-        });
-        it("handles both Skyramp and external tests in the same repo", async () => {
-            writeFile("tests/test_orders_smoke.py", '# Generated by Skyramp\nskyramp generate smoke rest');
-            writeFile("tests/test_api.py", 'import pytest\nrequests.get("/api/users")');
-            const result = await service.discoverTests(tmpDir);
-            expect(result.tests.length).toBe(2);
-            expect(result.tests.filter((t) => t.source === TestSource.Skyramp).length).toBe(1);
-            expect(result.tests.filter((t) => t.source === TestSource.External).length).toBe(1);
-        });
-    });
-    // ── isExternalTestFile (via classification) ──
-    describe("external test file detection", () => {
-        const patterns = [
-            ["test_orders.py", true],
-            ["orders_test.py", true],
-            ["orders.test.ts", true],
-            ["orders.spec.js", true],
-            ["OrdersTest.java", true],
-            ["orders_test.go", true],
-            ["orders_spec.rb", true],
-            ["OrdersTest.kt", true],
-            ["OrdersTests.cs", true],
-            ["orders_test.rs", true],
-            ["OrdersTest.php", true],
-            ["test_orders.rb", true],
-            ["utils.py", false],
-            ["main.ts", false],
-            ["main.go", false],
-        ];
-        it.each(patterns)("file %s → isExternal: %s", async (filename, expected) => {
-            writeFile(filename, 'import pytest\ndef test_something():\n    pass');
-            const result = await service.discoverTests(tmpDir);
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            if (expected) {
-                expect(externalTests.length).toBe(1);
-            }
-            else {
-                expect(externalTests.length).toBe(0);
-            }
-        });
-        it("detects files in test directories", async () => {
-            writeFile("tests/helpers.py", 'import requests\nrequests.get("/api/health")');
-            const result = await service.discoverTests(tmpDir);
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            expect(externalTests.length).toBe(1);
-        });
-        it("detects files in __tests__ directory", async () => {
-            writeFile("src/__tests__/api.ts", 'import axios from "axios";\naxios.get("/api/items");');
-            const result = await service.discoverTests(tmpDir);
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            expect(externalTests.length).toBe(1);
-        });
-    });
-    // ── extractExternalEndpoints ──
-    describe("extractExternalEndpoints", () => {
-        it("extracts endpoints from Python requests library", async () => {
-            writeFile("test_api.py", 'import requests\nrequests.get("/api/orders")\nrequests.post("/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-            expect(ext.apiEndpoint).toContain("POST /api/orders");
-        });
-        it("extracts endpoints from Python test client", async () => {
-            writeFile("test_api.py", 'def test_orders(client):\n    client.get("/api/orders")\n    client.post("/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-        });
-        it("extracts endpoints from axios", async () => {
-            writeFile("api.test.ts", 'import axios from "axios";\naxios.get("/api/items");\naxios.post("/api/items");');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/items");
-            expect(ext.apiEndpoint).toContain("POST /api/items");
-        });
-        it("extracts endpoints from supertest", async () => {
-            writeFile("api.test.ts", 'import supertest from "supertest";\nconst req = supertest(app);\nreq.get("/api/users");\nreq.post("/api/users");');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/users");
-            expect(ext.apiEndpoint).toContain("POST /api/users");
-        });
-        it("extracts endpoints from fetch with explicit method", async () => {
-            writeFile("api.test.ts", 'fetch("/api/orders", { method: "POST" });\nfetch("/api/orders");');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("POST /api/orders");
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-        });
-        it("strips base URL from endpoints", async () => {
-            writeFile("test_api.py", 'import requests\nrequests.get("http://localhost:8000/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-            expect(ext.apiEndpoint).not.toContain("localhost");
-        });
-        it("extracts endpoints from Java RestAssured", async () => {
-            writeFile("OrdersTest.java", 'import io.restassured.*;\ngiven().when().get("/api/orders").then();');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-        });
-        it("extracts endpoints from Go http.NewRequest", async () => {
-            writeFile("orders_test.go", 'package main\nimport "testing"\nimport "net/http"\nfunc TestOrders(t *testing.T) {\n  req, _ := http.NewRequest("GET", "/api/orders", nil)\n  http.NewRequest("POST", "/api/orders", body)\n}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-            expect(ext.apiEndpoint).toContain("POST /api/orders");
-        });
-        it("extracts endpoints from C# HttpClient", async () => {
-            writeFile("OrdersTests.cs", 'using Xunit;\npublic class OrdersTests {\n  [Fact]\n  public async Task GetOrders() {\n    await client.GetAsync("/api/orders");\n    await client.PostAsync("/api/orders", content);\n  }\n}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-            expect(ext.apiEndpoint).toContain("POST /api/orders");
-        });
-        it("extracts endpoints from Ruby Net::HTTP", async () => {
-            writeFile("orders_spec.rb", 'require "rspec"\nRSpec.describe "Orders" do\n  it "lists" do\n    Net::HTTP.get("/api/orders")\n    Net::HTTP.post("/api/orders", body)\n  end\nend');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext).toBeDefined();
-            expect(ext.apiEndpoint).toContain("GET /api/orders");
-        });
-    });
-    // ── detectExternalTestType ──
-    describe("detectExternalTestType", () => {
-        it("detects integration from filename", async () => {
-            writeFile("test_orders_integration.py", "import pytest\ndef test(): pass");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.testType).toBe("integration");
-        });
-        it("detects e2e from filename", async () => {
-            writeFile("test_e2e_flow.py", "import pytest\ndef test(): pass");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.testType).toBe("e2e");
-        });
-        it("detects contract from filename", async () => {
-            writeFile("test_contract.py", "import pytest\ndef test(): pass");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.testType).toBe("contract");
-        });
-        it("falls back to integration for files with HTTP calls", async () => {
-            writeFile("test_api.py", 'import requests\nrequests.get("/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.testType).toBe("integration");
-        });
-        it("returns unknown when no heuristic matches", async () => {
-            writeFile("test_utils.py", "def test_helper():\n    assert True");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.testType).toBe("unknown");
-        });
-    });
-    // ── detectExternalFramework ──
-    describe("detectExternalFramework", () => {
-        it("detects pytest", async () => {
-            writeFile("test_api.py", "import pytest\ndef test(): pass");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("pytest");
-        });
-        it("detects jest", async () => {
-            writeFile("api.test.ts", 'import { jest } from "jest";\ndescribe("api", () => {});');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("jest");
-        });
-        it("detects junit", async () => {
-            writeFile("OrdersTest.java", "import org.junit.Test;\n@Test\npublic void test() {}");
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("junit");
-        });
-        it("detects supertest", async () => {
-            writeFile("api.test.ts", 'import supertest from "supertest";\nconst req = supertest(app);');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("supertest");
-        });
-        it("detects go-testing", async () => {
-            writeFile("orders_test.go", 'package main\nimport "testing"\nfunc TestOrders(t *testing.T) {}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("go-testing");
-        });
-        it("detects rspec", async () => {
-            writeFile("orders_spec.rb", 'require "rspec"\nRSpec.describe "Orders" do\n  it "works" do; end\nend');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("rspec");
-        });
-        it("detects xunit", async () => {
-            writeFile("OrdersTests.cs", 'using Xunit;\npublic class OrdersTests {\n  [Fact]\n  public void Test() {}\n}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("xunit");
-        });
-        it("detects phpunit", async () => {
-            writeFile("OrdersTest.php", '<?php\nuse PHPUnit\\Framework\\TestCase;\nclass OrdersTest extends TestCase {\n  public function testList() {}\n}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("phpunit");
-        });
-        it("detects rust-test", async () => {
-            writeFile("orders_test.rs", '#[cfg(test)]\nmod tests {\n  #[test]\n  fn test_orders() {}\n}');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext.framework).toBe("rust-test");
-        });
-    });
-    // ── source field ──
-    describe("source field tagging", () => {
-        it("tags Skyramp tests with source: skyramp", async () => {
-            writeFile("test_smoke.py", '# Generated by Skyramp\nskyramp generate smoke rest');
-            const result = await service.discoverTests(tmpDir);
-            expect(result.tests[0].source).toBe(TestSource.Skyramp);
-        });
-        it("tags external tests with source: external", async () => {
-            writeFile("test_api.py", "import pytest\ndef test(): pass");
-            const result = await service.discoverTests(tmpDir);
-            expect(result.tests[0].source).toBe(TestSource.External);
-        });
-    });
-    // ── relevance-based partitioning ──
-    describe("relevance-based external test partitioning", () => {
-        it("returns all external test files without a hard cap in PR mode", async () => {
-            for (let i = 0; i < 60; i++) {
-                writeFile(`test_file_${i}.py`, `def test_${i}(): pass`);
-            }
-            const result = await service.discoverTests(tmpDir, { changedResources: ["file"] });
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            expect(externalTests.length).toBe(60);
-        });
-        it("extracts endpoints only for relevant files when changedResources provided", async () => {
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir, { changedResources: ["orders"] });
-            const externalTests = result.tests.filter((t) => t.source === TestSource.External);
-            // Only the relevant file (orders) is included — low-relevance files are excluded entirely.
-            expect(externalTests.length).toBe(1);
-            const ordersTest = externalTests.find(t => t.testFile.includes("orders"));
-            expect(ordersTest?.apiEndpoint).toContain("GET /api/orders");
-            // Low-relevance products test is not in result.tests at all.
-            expect(externalTests.find(t => t.testFile.includes("products"))).toBeUndefined();
-        });
-        it("returns relevantExternalTestPaths including files where extraction may have failed", async () => {
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_orders_helpers.py", 'def create_order(): pass'); // no extractable endpoints
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir, { changedResources: ["orders"] });
-            // Both orders-related files should be in relevantExternalTestPaths
-            expect(result.relevantExternalTestPaths.some(p => p.includes("test_orders_api"))).toBe(true);
-            expect(result.relevantExternalTestPaths.some(p => p.includes("test_orders_helpers"))).toBe(true);
-            // Products file should NOT be in relevantExternalTestPaths
-            expect(result.relevantExternalTestPaths.some(p => p.includes("test_products_api"))).toBe(false);
-        });
-        it("includes a file matched only by changedSymbols content grep (SKYR-3924 schema-only)", async () => {
-            // Path/URL do not token-match the resource; the file is relevant only because it
-            // references the changed symbol by name — the schema/DTO-only diff case.
-            writeFile("test_schema.py", "from app.schemas import DeploymentCreate\ndef test_ctor():\n    DeploymentCreate(name='x')");
-            writeFile("test_unrelated.py", "def test_noop(): pass");
-            const result = await service.discoverTests(tmpDir, {
-                changedResources: ["workpool"],
-                changedSymbols: ["DeploymentCreate"],
-            });
-            expect(result.relevantExternalTestPaths.some((p) => p.includes("test_schema"))).toBe(true);
-            expect(result.relevantExternalTestPaths.some((p) => p.includes("test_unrelated"))).toBe(false);
-        });
-        it("keeps direct endpoint-call matches uncapped when preciseResources is set", async () => {
-            // MAX_CONTENT_PROMOTED = 5; create 7 files that each POST the changed endpoint but whose
-            // names do not token-match the resource, so they route through endpoint-call matching.
-            for (let i = 0; i < 7; i++) {
-                writeFile(`test_case_${i}.py`, 'import requests\nrequests.post("/api/deployments/")');
-            }
-            const precise = await service.discoverTests(tmpDir, {
-                changedResources: ["deployments"],
-                preciseResources: true,
-            });
-            expect(precise.relevantExternalTestPaths.length).toBe(7);
-            // Without preciseResources the strong endpoint-call matches are capped together with
-            // loose mentions at MAX_CONTENT_PROMOTED.
-            const capped = await service.discoverTests(tmpDir, { changedResources: ["deployments"] });
-            expect(capped.relevantExternalTestPaths.length).toBe(5);
-        });
-        it("does full extraction for all files when no changedResources provided", async () => {
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            const result = await service.discoverTests(tmpDir);
-            const ext = result.tests.find((t) => t.source === TestSource.External);
-            expect(ext?.apiEndpoint).toContain("GET /api/orders");
-        });
-        it("caps full-repo mode at MAX_EXTERNAL_FULL_REPO for full extraction", async () => {
-            // Create 12 external test files to test the boundary (MAX_EXTERNAL_FULL_REPO=100 is too
-            // large to create in a unit test, so we verify the partitioning logic is wired correctly)
-            for (let i = 0; i < 12; i++) {
-                writeFile(`test_api_${i}.py`, `import requests\nrequests.get("/api/items/${i}")`);
-            }
-            const result = await service.discoverTests(tmpDir);
-            const externalTests = result.tests.filter(t => t.source === TestSource.External);
-            // All 12 files should be discovered
-            expect(externalTests.length).toBe(12);
-            // In full-repo mode (no changedResources) all files under cap go to relevantExternal
-            expect(result.relevantExternalTestPaths.length).toBe(12);
-            // All relevant files get full extraction (apiEndpoint populated)
-            const withEndpoints = externalTests.filter(t => t.apiEndpoint !== "");
-            expect(withEndpoints.length).toBe(12);
-        });
-        it("returns zero external tests when changedResources is empty array (PR mode, no endpoints)", async () => {
-            // Simulate PR mode where a parsed diff produced no detected endpoints:
-            // newEndpoints=[], modifiedEndpoints=[], and removedEndpoints=[] → changedResources = []
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir, { changedResources: [] });
-            const externalTests = result.tests.filter(t => t.source === TestSource.External);
-            // Empty changedResources = PR mode with no detected endpoints → zero external tests
-            expect(externalTests.length).toBe(0);
-            expect(result.relevantExternalTestPaths.length).toBe(0);
-        });
-        it("still returns external tests in full-repo mode (changedResources undefined)", async () => {
-            // Full-repo mode: changedResources not provided → should use capped full-repo behavior
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir); // no options → undefined
-            const externalTests = result.tests.filter(t => t.source === TestSource.External);
-            expect(externalTests.length).toBe(2);
-            expect(result.relevantExternalTestPaths.length).toBe(2);
-        });
-        it("Skyramp tests are unaffected by empty changedResources", async () => {
-            writeFile("tests/test_orders_smoke.py", '# Generated by Skyramp\nskyramp generate smoke rest');
-            writeFile("test_external.py", 'import pytest\ndef test(): pass');
-            const result = await service.discoverTests(tmpDir, { changedResources: [] });
-            const skyrampTests = result.tests.filter(t => t.source === TestSource.Skyramp);
-            const externalTests = result.tests.filter(t => t.source === TestSource.External);
-            // Skyramp tests always discovered regardless of changedResources
-            expect(skyrampTests.length).toBe(1);
-            // External tests suppressed in PR-mode-no-endpoints
-            expect(externalTests.length).toBe(0);
-        });
-        it("returns no external tests with ['unknown'] sentinel (unresolvable resources)", async () => {
-            // When diff endpoints exist but all paths resolve to "unknown" (e.g. decorator-relative
-            // paths like "/{order_id}"), changedResources = ["unknown"]. No external tests score > 0
-            // since "unknown" doesn't match any filename tokens — low-relevance files are excluded.
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir, { changedResources: ["unknown"] });
-            const externalTests = result.tests.filter(t => t.source === TestSource.External);
-            expect(externalTests.length).toBe(0);
-            expect(result.relevantExternalTestPaths.length).toBe(0);
-        });
-        it("low-relevance files are excluded from result.tests in PR mode", async () => {
-            writeFile("test_orders_api.py", 'import requests\nrequests.get("/api/orders")');
-            writeFile("test_products_api.py", 'import requests\nrequests.get("/api/products")');
-            const result = await service.discoverTests(tmpDir, { changedResources: ["orders"] });
-            // products scored 0 — excluded entirely, not even as name-only entry
-            const productsTest = result.tests.find(t => t.testFile.includes("test_products_api"));
-            expect(productsTest).toBeUndefined();
-        });
-    });
-});
-// ---------------------------------------------------------------------------
-// Framework detection ordering — private method behaviour tested via discoverTests
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — detectExternalFramework ordering", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-fw-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("pytest-playwright file → playwright-ui (not pytest)", async () => {
-        writeFile("tests/test_login.py", "import pytest\nfrom playwright.sync_api import Page\ndef test_login(page: Page):\n  page.goto('http://localhost')");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["login"] });
-        const t = result.tests.find(f => f.testFile.includes("test_login.py"));
-        expect(t?.framework).toBe("playwright-ui");
-    });
-    it("Playwright API-only test (request, no page) → playwright (not playwright-ui)", async () => {
-        writeFile("tests/test_api.py", "from playwright.sync_api import APIRequestContext\ndef test_orders(request: APIRequestContext):\n  request.get('/api/orders')");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["orders"] });
-        const t = result.tests.find(f => f.testFile.includes("test_api.py"));
-        expect(t?.framework).toBe("playwright");
-    });
-    it("pure pytest file → pytest", async () => {
-        writeFile("tests/test_calc.py", "import pytest\ndef test_add():\n  assert 1 + 1 == 2");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["calc"] });
-        const t = result.tests.find(f => f.testFile.includes("test_calc.py"));
-        expect(t?.framework).toBe("pytest");
-    });
-    it("RTL file with it()/expect() → rtl (not jest)", async () => {
-        writeFile("src/Cart.test.tsx", "import { render } from '@testing-library/react';\nit('renders', () => { expect(true).toBe(true); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["cart"], changedFrontendFiles: ["src/Cart.tsx"] });
-        const t = result.tests.find(f => f.testFile.includes("Cart.test.tsx"));
-        expect(t?.framework).toBe("rtl");
-    });
-    it("vue-test-utils file → vue-test-utils (not jest)", async () => {
-        writeFile("src/Counter.test.ts", "import { mount } from '@testing-library/vue';\nit('works', () => { expect(true).toBe(true); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["counter"], changedFrontendFiles: ["src/Counter.tsx"] });
-        const t = result.tests.find(f => f.testFile.includes("Counter.test.ts"));
-        expect(t?.framework).toBe("vue-test-utils");
-    });
-    it("JS Playwright file using page.goto() → playwright-ui", async () => {
-        writeFile("e2e/login.spec.ts", "import { test } from '@playwright/test';\ntest('login', async ({ page }) => { await page.goto('/'); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["login"] });
-        const t = result.tests.find(f => f.testFile.includes("login.spec.ts"));
-        expect(t?.framework).toBe("playwright-ui");
-    });
-    it("JS Playwright using only request fixture → playwright", async () => {
-        writeFile("e2e/api.spec.ts", "import { test } from '@playwright/test';\ntest('api', async ({ request }) => { await request.get('/api'); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["api"] });
-        const t = result.tests.find(f => f.testFile.includes("api.spec.ts"));
-        expect(t?.framework).toBe("playwright");
-    });
-});
-// ---------------------------------------------------------------------------
-// detectExternalTestType path rules
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — detectExternalTestType path rules", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-type-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("file in playwright/ dir → testType=e2e", async () => {
-        writeFile("apps/web/playwright/login.spec.ts", "import { test } from '@playwright/test';\ntest('login', async ({ page }) => { await page.goto('/'); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["login"] });
-        const t = result.tests.find(f => f.testFile.includes("login.spec.ts"));
-        expect(t?.testType).toBe("e2e");
-    });
-    it("file with e2e in filename inside e2e/ dir → testType=e2e", async () => {
-        // .e2e.ts in an e2e/ dir is recognized both by dir pattern and filename
-        writeFile("apps/web/e2e/auth.e2e.ts", "import { test } from '@playwright/test';\ntest('auth', async ({ page }) => { await page.goto('/'); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["auth"] });
-        const t = result.tests.find(f => f.testFile.includes("auth.e2e.ts"));
-        expect(t?.testType).toBe("e2e");
-    });
-    it("file in api dir (no playwright in path) → NOT e2e", async () => {
-        writeFile("apps/api/v2/tests/login.spec.ts", "import { test } from '@jest/globals';\ntest('login', () => { expect(true).toBe(true); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["login"] });
-        const t = result.tests.find(f => f.testFile.includes("login.spec.ts"));
-        expect(t?.testType).not.toBe("e2e");
-    });
-    it("generic spec file (no e2e/playwright signal) → NOT e2e", async () => {
-        writeFile("modules/bookings/login.spec.ts", "import { test } from '@jest/globals';\ntest('login', () => {});");
-        const result = await service.discoverTests(tmpDir, { changedResources: ["login"] });
-        const t = result.tests.find(f => f.testFile.includes("login.spec.ts"));
-        expect(t?.testType).not.toBe("e2e");
-    });
-});
-// ---------------------------------------------------------------------------
-// isFrontendFile edge cases
-// ---------------------------------------------------------------------------
-const { isFrontendFile } = await import("../prompts/test-recommendation/scopeAssessment.js");
-describe("isFrontendFile edge cases", () => {
-    it("pages/api/orders.tsx → false (Next.js API route, /api/ blocks all)", () => {
-        expect(isFrontendFile("pages/api/orders.tsx")).toBe(false);
-    });
-    it("routes/product-card.tsx → true (.tsx in routes/ is page component)", () => {
-        expect(isFrontendFile("routes/product-card.tsx")).toBe(true);
-    });
-    it("routes/orders.ts → false (.ts in routes/ is route handler)", () => {
-        expect(isFrontendFile("routes/orders.ts")).toBe(false);
-    });
-    it("src/components/Cart.tsx → true", () => {
-        expect(isFrontendFile("src/components/Cart.tsx")).toBe(true);
-    });
-    it("src/utils/helpers.ts → false (ambiguous extension, not in frontend dir)", () => {
-        expect(isFrontendFile("src/utils/helpers.ts")).toBe(false);
-    });
-    it("app/modules/booking/BookingList.tsx → true", () => {
-        expect(isFrontendFile("app/modules/booking/BookingList.tsx")).toBe(true);
-    });
-});
-// ---------------------------------------------------------------------------
-// Content-based relevance cap (MAX_CONTENT_PROMOTED = 5)
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — content-based relevance cap", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-cap-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("promotes at most MAX_CONTENT_PROMOTED=5 files via content match", async () => {
-        // 10 test files that all reference /orders in their content
-        for (let i = 0; i < 10; i++) {
-            writeFile(`tests/test_feature_${i}.py`, `import requests\nrequests.get("/api/checkout_${i}")`);
-            // content-match: all contain "/orders"
-            fs.writeFileSync(path.join(tmpDir, `tests/test_feature_${i}.py`), `import requests\n# tests ordering\nrequests.get("/api/orders/item_${i}")`);
-        }
-        // changedResources=["unknown"] so all score 0 by filename; content scan fires
-        const result = await service.discoverTests(tmpDir, { changedResources: ["orders"] });
-        // At most 5 content-matched files should be in relevantExternalTestPaths
-        expect(result.relevantExternalTestPaths.length).toBeLessThanOrEqual(5);
-    });
-});
-// ---------------------------------------------------------------------------
-// scoreRelevanceByContent regex safety (dot/plus escaped)
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — scoreRelevanceByContent regex safety", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-regex-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("resource with dot (order.items) does not match orderXitems via unescaped dot", async () => {
-        writeFile("tests/test_order.py", `import requests\nrequests.get("/api/orderXitems")`);
-        // "order.items" in changedResources — dot must be escaped
-        const result = await service.discoverTests(tmpDir, { changedResources: ["order.items"] });
-        // should NOT match (orderXitems ≠ order.items)
-        expect(result.relevantExternalTestPaths.some(p => p.includes("test_order.py"))).toBe(false);
-    });
-    it("resource with dot matches correctly when content has /order.items", async () => {
-        writeFile("tests/test_order_items.py", `import requests\nrequests.get("/api/order.items/list")`);
-        const result = await service.discoverTests(tmpDir, { changedResources: ["order.items"] });
-        expect(result.relevantExternalTestPaths.some(p => p.includes("test_order_items.py"))).toBe(true);
-    });
-    it("resource with plus does not throw SyntaxError", async () => {
-        writeFile("tests/test_v2.py", `import requests\nrequests.get("/api/v2/items")`);
-        // Should not throw
-        await expect(service.discoverTests(tmpDir, { changedResources: ["v2+api"] })).resolves.toBeDefined();
-    });
-});
-// ---------------------------------------------------------------------------
-// UI promotion gating: no changed frontend files ⇒ no .e2e.ts promotion
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — UI promotion gating (changedFrontendFiles)", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-nofe-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("does not promote .e2e.ts files when no changed frontend files are supplied", async () => {
-        writeFile("apps/web/playwright/onboarding.e2e.ts", "import { test } from '@playwright/test';\ntest('onboarding', async ({ page }) => { await page.goto('/onboarding'); });");
-        // changedResources=[] (no API endpoints) + no changedFrontendFiles (config-only PR) ⇒ Step 2 skipped
-        const result = await service.discoverTests(tmpDir, { changedResources: [] });
-        expect(result.relevantExternalTestPaths.length).toBe(0);
-        expect(result.tests.length).toBe(0);
-    });
-    it("promotes .e2e.ts files relevant to a changed frontend file", async () => {
-        writeFile("apps/web/playwright/onboarding.e2e.ts", "import { test } from '@playwright/test';\ntest('onboarding', async ({ page }) => { await page.goto('/onboarding'); });");
-        const result = await service.discoverTests(tmpDir, { changedResources: [], changedFrontendFiles: ["apps/web/onboarding.tsx"] });
-        // .e2e.ts should be promoted (UI_TEST_EXT includes .e2e.ts; "onboarding" token matches)
-        expect(result.tests.some(t => t.testFile.includes("onboarding.e2e.ts"))).toBe(true);
-    });
-    // ── Step 3: API test promotion ─────────────────────────────────────────────
-    // Uses a minimal mirror of the prefect-testb file structure to validate that
-    // the content-scored promotion finds test_deployments.py (which imports from
-    // the changed module) but not unrelated schedule/events test files.
-    describe("Step 3 API promotion (hasApiChanges)", () => {
-        // Realistic import lines taken from prefect-testb actual files.
-        // test_deployments.py imports from schemas.actions — matches the import-context
-        // pattern "schemas[./]actions" built when schemas/actions.py changes.
-        const TEST_DEPLOYMENTS_CONTENT = `\
-import pytest
-from httpx import AsyncClient
-from prefect.server.schemas.actions import DeploymentCreate, DeploymentUpdate
-from prefect.server import models, schemas
-
-class TestCreateDeployment:
-    async def test_create_deployment(self, client):
-        response = await client.post("/api/deployments/", json={"name": "test"})
-        assert response.status_code == 201
-
-class TestCreateFlowRunFromDeployment:
-    async def test_create_flow_run(self, client, deployment):
-        response = await client.post(f"/api/deployments/{deployment.id}/create_flow_run", json={})
-        assert response.status_code == 201
-`;
-        // test_deployment_schedules.py imports from schedules, NOT schemas.actions
-        const TEST_DEPLOYMENT_SCHEDULES_CONTENT = `\
-import pytest
-from prefect.server import models, schemas
-from prefect.server.schemas.core import DeploymentSchedule
-
-class TestReadDeploymentSchedules:
-    async def test_read_schedules(self, client, deployment):
-        response = await client.get(f"/api/deployments/{deployment.id}/schedules")
-        assert response.status_code == 200
-`;
-        // events/actions file has no deployment schema imports at all
-        const TEST_EVENTS_ACTIONS_CONTENT = `\
-import pytest
-from prefect.server.events import actions
-from prefect.server.events.clients import AssertingEventsClient
-
-class TestPausingWorkQueue:
-    async def test_pausing(self, session):
-        action = actions.PauseWorkQueue(work_queue_id="test")
-        await action.act(event=None)
-`;
-        // test_schedules.py — no deployment imports
-        const TEST_SCHEDULES_CONTENT = `\
-import pytest
-from prefect.server.schemas.schedules import CronSchedule, IntervalSchedule
-
-class TestCronSchedule:
-    def test_cron_parse(self):
-        s = CronSchedule(cron="0 * * * *")
-        assert s.cron == "0 * * * *"
-`;
-        it("content-matches test_deployments.py when schemas/actions.py changed", async () => {
-            // Mirror the Prefect directory structure
-            writeFile("tests/server/orchestration/api/test_deployments.py", TEST_DEPLOYMENTS_CONTENT);
-            writeFile("tests/server/orchestration/api/test_deployment_schedules.py", TEST_DEPLOYMENT_SCHEDULES_CONTENT);
-            writeFile("tests/events/server/actions/test_pausing_resuming_work_queue.py", TEST_EVENTS_ACTIONS_CONTENT);
-            writeFile("tests/test_schedules.py", TEST_SCHEDULES_CONTENT);
-            // changedResources = ["unknown"] because "actions" is filtered by GENERIC_SCHEMA_NAMES
-            // in analyzeChangesTool. Step 3 is skipped for ["unknown"] sentinel.
-            // So we use the import-context pattern directly via hasApiChanges + a non-unknown resource.
-            // Simulate: schemas/actions.py changed, but it produced a non-generic parent resource.
-            // For this test, use changedResources derived from the parent path "schemas":
-            // "schemas" is in GENERIC_SCHEMA_NAMES so would be filtered → ["unknown"].
-            // Step 3 should be skipped → test_deployments.py not promoted via Step 3.
-            const resultUnknown = await service.discoverTests(tmpDir, {
-                changedResources: ["unknown"],
-            });
-            // Step 3 skipped for ["unknown"] — no content-matched files
-            const promotedUnknown = resultUnknown.relevantExternalTestPaths;
-            expect(promotedUnknown.some(p => p.includes("test_deployments.py"))).toBe(false);
-        });
-        it("content-matches test_deployments.py when api/deployments.py changed (TM17 scenario)", async () => {
-            writeFile("tests/server/orchestration/api/test_deployments.py", TEST_DEPLOYMENTS_CONTENT);
-            writeFile("tests/server/orchestration/api/test_deployment_schedules.py", TEST_DEPLOYMENT_SCHEDULES_CONTENT);
-            writeFile("tests/events/server/actions/test_pausing_resuming_work_queue.py", TEST_EVENTS_ACTIONS_CONTENT);
-            writeFile("tests/test_schedules.py", TEST_SCHEDULES_CONTENT);
-            // changedResources = ["deployments"] (from api/deployments.py route file)
-            // Content pattern: /import\s+[^\n]*["']\/api\/deployments/i or /deployments/ prefix
-            // test_deployments.py calls POST /api/deployments/ → content-matched in Step 3
-            const result = await service.discoverTests(tmpDir, {
-                changedResources: ["deployments"],
-            });
-            const promoted = result.relevantExternalTestPaths;
-            // test_deployments.py: calls /api/deployments/ → content-matched → promoted unconditionally
-            expect(promoted.some(p => p.includes("test_deployments.py"))).toBe(true);
-            // test_deployment_schedules.py: contains /api/deployments/ in URL calls → also content-matched
-            // (the word "deployments" appears in its URL strings even though it tests schedules)
-            // This is expected — it IS in scope for a deployment endpoint change.
-            // test_schedules.py: no /deployments/ URL → scores 0 → fallback only (may or may not be in promoted)
-            // test_pausing_resuming: no /deployments/ URL → fallback only
-        });
-        it("API external tests no longer pre-loaded (agent greps instead)", async () => {
-            // External API tests are discovered by the agent via the grep instruction in the
-            // drift analysis prompt. discoverTests in PR mode (changedResources=[]) returns
-            // only Skyramp tests; external tests are excluded entirely.
-            writeFile("tests/server/orchestration/api/test_deployments.py", [
-                "from prefect.server import utils",
-                "",
-                "async def test_max_active_runs(client, deployment):",
-                "    count = utils.calculate_max_runs(deployment.id, db)",
-                "    assert count >= 0",
-            ].join("\n"));
-            const result = await service.discoverTests(tmpDir, {
-                changedResources: [], // PR mode, no pre-loading of external API tests
-            });
-            // No external tests pre-loaded — agent discovers via grep
-            expect(result.relevantExternalTestPaths.some(p => p.includes("test_deployments.py"))).toBe(false);
-        });
-        it("Step 3 does not run when changedResources is empty (no API changes)", async () => {
-            writeFile("tests/server/orchestration/api/test_deployments.py", TEST_DEPLOYMENTS_CONTENT);
-            const result = await service.discoverTests(tmpDir, {
-                changedResources: [],
-            });
-            // changedResources=[] means PR mode with no endpoints — external tests excluded entirely
-            expect(result.relevantExternalTestPaths).toHaveLength(0);
-        });
-        it("Step 1 path-token scoring finds test_work_queues.py for changedResources=[work_queues]", async () => {
-            // Pattern Q scenario: work_queues.py changed → changedResources=["work_queues"]
-            // test_work_queues.py scores >0 by name (has "work_queues" token)
-            writeFile("tests/server/orchestration/api/test_work_queues.py", 'import pytest\nfrom prefect.server.api import work_queues\n\nasync def test_read_work_queue_status(client, work_queue):\n    r = await client.get(f"/api/work_queues/{work_queue.id}/status")\n    assert r.status_code == 200\n');
-            writeFile("tests/server/orchestration/api/test_flows.py", "import pytest\nfrom prefect.server import schemas\n\nasync def test_create_flow(client): pass\n");
-            const result = await service.discoverTests(tmpDir, {
-                changedResources: ["work_queues"],
-            });
-            const promoted = result.relevantExternalTestPaths;
-            // test_work_queues.py → Step 1 name-scores it → in relevantExternal before Step 3
-            expect(promoted.some(p => p.includes("test_work_queues.py"))).toBe(true);
-            // test_flows.py → scores 0 → not promoted via Step 1
-            // (may appear in Step 3 fallback but is not guaranteed)
-        });
-    });
-});
-// ---------------------------------------------------------------------------
-// SKYR-3941: UI-test promotion (Step 2) is scoped by relevance to the changed
-// frontend files — a frontend PR must not pull in the whole monorepo's UI suite.
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — UI promotion scoping (SKYR-3941)", () => {
-    let service;
-    let tmpDir;
-    // Minimal RTL content so the file classifies as an external UI test.
-    const RTL = "import { render } from '@testing-library/react';\nit('renders', () => { expect(true).toBe(true); });";
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-fe-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    it("promotes UI tests relevant to the changed frontend files, not the whole suite", async () => {
-        writeFile("src/routes/orders/OrderSearch.test.tsx", RTL);
-        writeFile("packages/design-system/icons/__tests__/arrow-small-down.spec.tsx", RTL);
-        // No API resources (Step 1 promotes nothing) — promotion is purely Step 2.
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/routes/orders/OrderSearch.tsx"],
-        });
-        const promoted = result.relevantExternalTestPaths;
-        expect(promoted.some(p => p.includes("OrderSearch.test.tsx"))).toBe(true);
-        expect(promoted.some(p => p.includes("arrow-small-down.spec.tsx"))).toBe(false);
-    });
-    it("matches a camelCase changed component against a hyphenated test name", async () => {
-        writeFile("src/order-search.spec.tsx", RTL);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/OrderSearch.tsx"],
-        });
-        expect(result.relevantExternalTestPaths.some(p => p.includes("order-search.spec.tsx"))).toBe(true);
-    });
-    it("uses the parent directory as the signal when the changed file's basename is generic (index.tsx)", async () => {
-        writeFile("src/orders/OrdersTable.test.tsx", RTL);
-        writeFile("src/billing/BillingPanel.test.tsx", RTL);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/orders/index.tsx"], // stem "index" generic → parent "orders" is the token
-        });
-        const promoted = result.relevantExternalTestPaths;
-        expect(promoted.some(p => p.includes("OrdersTable.test.tsx"))).toBe(true);
-        expect(promoted.some(p => p.includes("BillingPanel.test.tsx"))).toBe(false);
-    });
-    it("caps scoped promotion at the backstop even when many tests match", async () => {
-        for (let i = 0; i < 55; i++)
-            writeFile(`src/widgets/widget-${i}.test.tsx`, RTL);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/widgets/widget.tsx"], // token "widget" matches all 55
-        });
-        const promotedWidgets = result.relevantExternalTestPaths.filter(p => p.includes("widget-"));
-        expect(promotedWidgets.length).toBe(50); // MAX_UI_PROMOTED backstop
-    });
-    it("promotes a UI test that imports a changed component even when its name shares no token (import-edge recall)", async () => {
-        // "Checkout" shares no token with "PriceLabel" — only the import edge connects them.
-        writeFile("src/features/Checkout.test.tsx", "import { render } from '@testing-library/react';\nimport { PriceLabel } from '../components/PriceLabel';\nit('shows price', () => { render(<PriceLabel />); });");
-        writeFile("src/features/Billing.test.tsx", RTL); // no relevant import, no token overlap
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/components/PriceLabel.tsx"],
-        });
-        const promoted = result.relevantExternalTestPaths;
-        expect(promoted.some(p => p.includes("Checkout.test.tsx"))).toBe(true); // via import edge
-        expect(promoted.some(p => p.includes("Billing.test.tsx"))).toBe(false);
-    });
-    it("does not let a generic-basename change (index.tsx) leak via import edge, but still matches on the parent dir", async () => {
-        writeFile("src/orders/OrdersList.test.tsx", RTL); // matches parent token "orders" (pass 1)
-        writeFile("src/widgets/Widget.test.tsx", "import { render } from '@testing-library/react';\nimport helpers from '../orders/index';\nit('x', () => {});"); // imports index — must NOT leak
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/orders/index.tsx"],
-        });
-        const promoted = result.relevantExternalTestPaths;
-        expect(promoted.some(p => p.includes("OrdersList.test.tsx"))).toBe(true); // parent-dir token
-        expect(promoted.some(p => p.includes("Widget.test.tsx"))).toBe(false); // "/index'" signature skipped
-    });
-    it("falls back to capped promote-all when only generic entrypoints changed — never degenerates to zero (SKYR-3941)", async () => {
-        // src/app/index.tsx: stem "index" and parent "app" are both generic → no tokens, and
-        // "index" is skipped for import signatures → no signal at all. A root/entry change is
-        // broadly impactful, so promotion must fall back rather than promote nothing.
-        writeFile("src/routes/orders/OrderList.test.tsx", RTL);
-        writeFile("src/products/ProductList.test.tsx", RTL);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [], changedFrontendFiles: ["src/app/index.tsx"],
-        });
-        const promoted = result.relevantExternalTestPaths;
-        // Neither test shares a token with "index"/"app" — without the fallback both would be
-        // dropped (score 0). The no-signal fallback promotes them instead.
-        expect(promoted.some(p => p.includes("OrderList.test.tsx"))).toBe(true);
-        expect(promoted.some(p => p.includes("ProductList.test.tsx"))).toBe(true);
-    });
-});
-// ---------------------------------------------------------------------------
-// Pass 3 — selector-edge promotion: page objects/specs couple to a changed
-// component by SELECTOR (data-testid / CSS class), not by filename or import.
-// ---------------------------------------------------------------------------
-describe("TestDiscoveryService — selector-edge promotion (changedSelectors)", () => {
-    let service;
-    let tmpDir;
-    beforeEach(() => {
-        service = new TestDiscoveryService();
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "tds-sel-"));
-    });
-    afterEach(() => { fs.rmSync(tmpDir, { recursive: true, force: true }); });
-    function writeFile(rel, content) {
-        const fp = path.join(tmpDir, rel);
-        fs.mkdirSync(path.dirname(fp), { recursive: true });
-        fs.writeFileSync(fp, content, "utf-8");
-        return fp;
-    }
-    // A Playwright page object that selects a component by CSS class. It is NOT a
-    // *.test/*.spec file (so passes 1-2 never consider it) and shares no name token
-    // with the changed component, nor does it import it — the exact POM shape that
-    // regressed after SKYR-3941. It still holds the OLD (renamed-away) selector.
-    const POM_CONTENT = `\
-import { Locator, Page } from '@playwright/test';
-
-export class TableComponent {
-  readonly columnFilter: Locator;
-  constructor(private page: Page) {
-    this.columnFilter = page.locator('.t--table-filter-columns-dropdown');
-  }
-  async openFilter() { await this.columnFilter.click(); }
-}
-`;
-    it("promotes a page object that references a changed selector (not name/import matched)", async () => {
-        writeFile("app/client/playwright/page-objects/components/table.component.ts", POM_CONTENT);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            // Changed component shares no token with "table.component" and the POM does not import it.
-            changedFrontendFiles: ["app/client/src/widgets/TableWidgetV2/component/header/actions/filter/CascadeFields.tsx"],
-            // Both the removed (old) and added (new) selector values, as the diff would carry them.
-            changedSelectors: ["t--table-filter-columns-dropdown", "t--tbl-filter-column-select"],
-        });
-        expect(result.tests.some(t => t.testFile.includes("table.component.ts"))).toBe(true);
-        expect(result.relevantExternalTestPaths.some(p => p.includes("table.component.ts"))).toBe(true);
-    });
-    it("does NOT promote the same page object when changedSelectors is absent (the regression baseline)", async () => {
-        writeFile("app/client/playwright/page-objects/components/table.component.ts", POM_CONTENT);
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            changedFrontendFiles: ["app/client/src/widgets/TableWidgetV2/component/header/actions/filter/CascadeFields.tsx"],
-            // no changedSelectors → passes 1-2 only → name/import miss → POM dropped
-        });
-        expect(result.tests.some(t => t.testFile.includes("table.component.ts"))).toBe(false);
-    });
-    it("does not promote a page object that references no changed selector", async () => {
-        writeFile("app/client/playwright/page-objects/components/unrelated.component.ts", "import { Page } from '@playwright/test';\nexport class Unrelated { constructor(p: Page) { p.locator('.t--some-other-widget-header'); } }");
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            changedFrontendFiles: ["app/client/src/widgets/TableWidgetV2/component/header/actions/filter/CascadeFields.tsx"],
-            changedSelectors: ["t--table-filter-columns-dropdown", "t--tbl-filter-column-select"],
-        });
-        expect(result.tests.some(t => t.testFile.includes("unrelated.component.ts"))).toBe(false);
-    });
-    it("promotes via a changed data-testid value (BEM/testid coupling)", async () => {
-        writeFile("e2e/pages/confirmation-modal.page.ts", "import { Page } from '@playwright/test';\nexport class ConfirmationModal { constructor(p: Page) { this.confirm = p.getByTestId('confirmation-modal-confirm-button'); } }");
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            changedFrontendFiles: ["src/modules/ui/layout/modal/components/ConfirmationModal.tsx"],
-            changedSelectors: ["confirmation-modal-confirm-button", "confirmation-modal-primary-button"],
-        });
-        expect(result.tests.some(t => t.testFile.includes("confirmation-modal.page.ts"))).toBe(true);
-    });
-    it("promotes a page object via a dynamic BEM base prefix (shared-component modifier)", async () => {
-        // The changed component composes its class dynamically (`ads-button--${kind}`), so only
-        // the base prefix `ads-button--` is extractable; the POM's concrete modifier classes
-        // (ads-button--primary/secondary) share that prefix and must still be promoted (SKYR-3943).
-        writeFile("app/client/playwright/page-objects/ads-button.page.ts", "import { Page } from '@playwright/test';\nexport class AdsButton { constructor(p: Page) { this.primary = p.locator('.ads-button--primary'); this.secondary = p.locator('.ads-button--secondary'); } }");
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            changedFrontendFiles: ["app/client/packages/design-system/ads/src/Button/Button.tsx"],
-            changedSelectors: ["ads-button--"],
-        });
-        expect(result.tests.some(t => t.testFile.includes("ads-button.page.ts"))).toBe(true);
-    });
-    it("caps pass-3 selector promotion at MAX_UI_PROMOTED (no re-flood)", async () => {
-        // 60 page objects all reference the same changed selector; pass 1/2 promote none of them
-        // (names/imports don't match), so pass 3 must cap total promotion at MAX_UI_PROMOTED (50).
-        for (let i = 0; i < 60; i++) {
-            writeFile(`app/client/playwright/page-objects/pom-${i}.page.ts`, `import { Page } from '@playwright/test';\nexport class P${i} { constructor(p: Page) { p.locator('.t--shared-widget-hook'); } }`);
-        }
-        const result = await service.discoverTests(tmpDir, {
-            changedResources: [],
-            changedFrontendFiles: ["app/client/src/widgets/SharedWidget.tsx"],
-            changedSelectors: ["t--shared-widget-hook"],
-        });
-        const promotedPoms = result.tests.filter(t => t.testFile.includes("/pom-")).length;
-        expect(promotedPoms).toBeGreaterThan(0);
-        expect(promotedPoms).toBeLessThanOrEqual(50);
-    });
-});