@skyramp/mcp 0.1.8 → 0.2.0-rc.2

package/build/services/TestHealthService.test.js

@@ -1,241 +0,0 @@
-import { TestHealthService } from "./TestHealthService.js";
-describe("TestHealthService", () => {
-    let service;
-    beforeEach(() => {
-        service = new TestHealthService();
-    });
-    describe("identifyIssues - endpoint rename detection", () => {
-        function identifyIssues(execution, drift) {
-            return service["identifyIssues"](execution, drift);
-        }
-        it("should create an endpoints_renamed issue when drift has endpoint_renamed changes", () => {
-            const drift = {
-                testFile: "products_smoke_test.py",
-                lastCommit: "abc123",
-                currentCommit: "def456",
-                driftScore: 30,
-                changes: [
-                    {
-                        type: "endpoint_renamed",
-                        file: "API Schema",
-                        description: "Endpoint renamed: get /api/v1/products -> /api/v1/items",
-                        severity: "high",
-                        details: "Path changed from /api/v1/products to /api/v1/items",
-                    },
-                ],
-                affectedFiles: { files: ["src/routers/product.py"] },
-                analysisTimestamp: new Date().toISOString(),
-            };
-            const issues = identifyIssues(undefined, drift);
-            const renameIssue = issues.find((i) => i.type === "endpoints_renamed");
-            expect(renameIssue).toBeDefined();
-            expect(renameIssue?.severity).toBe("high");
-            expect(renameIssue?.description).toContain("1 API endpoint(s) renamed");
-        });
-        it("should not create endpoints_renamed issue when no renames in drift", () => {
-            const drift = {
-                testFile: "products_smoke_test.py",
-                lastCommit: "abc123",
-                currentCommit: "def456",
-                driftScore: 15,
-                changes: [
-                    {
-                        type: "endpoint_removed",
-                        file: "API Schema",
-                        description: "1 endpoint(s) removed",
-                        severity: "high",
-                    },
-                ],
-                affectedFiles: { files: [] },
-                analysisTimestamp: new Date().toISOString(),
-            };
-            const issues = identifyIssues(undefined, drift);
-            const renameIssue = issues.find((i) => i.type === "endpoints_renamed");
-            expect(renameIssue).toBeUndefined();
-            const removeIssue = issues.find((i) => i.type === "endpoints_removed");
-            expect(removeIssue).toBeDefined();
-        });
-        it("should handle multiple rename changes", () => {
-            const drift = {
-                testFile: "products_smoke_test.py",
-                lastCommit: "abc123",
-                currentCommit: "def456",
-                driftScore: 40,
-                changes: [
-                    {
-                        type: "endpoint_renamed",
-                        file: "API Schema",
-                        description: "Endpoint renamed: get /api/v1/products -> /api/v1/items",
-                        severity: "high",
-                    },
-                    {
-                        type: "endpoint_renamed",
-                        file: "API Schema",
-                        description: "Endpoint renamed: post /api/v1/products -> /api/v1/items",
-                        severity: "high",
-                    },
-                ],
-                affectedFiles: { files: [] },
-                analysisTimestamp: new Date().toISOString(),
-            };
-            const issues = identifyIssues(undefined, drift);
-            const renameIssue = issues.find((i) => i.type === "endpoints_renamed");
-            expect(renameIssue).toBeDefined();
-            expect(renameIssue?.description).toContain("2 API endpoint(s) renamed");
-        });
-    });
-    describe("generateRecommendation - endpoint rename handling", () => {
-        function generateRecommendation(testFile, driftScore, execution, issues, apiEndpoint, apiSchemaChanges) {
-            const healthScore = service["calculateHealthScore"](execution
-                ? service["calculateExecutionScore"](execution).score
-                : undefined, driftScore);
-            return service["generateRecommendation"](testFile, healthScore, driftScore, execution, issues, apiEndpoint, apiSchemaChanges);
-        }
-        it("should return UPDATE action for endpoint renames regardless of drift score", () => {
-            const issues = [
-                {
-                    type: "endpoints_renamed",
-                    severity: "high",
-                    description: "1 API endpoint(s) renamed",
-                    details: "Endpoint renamed: get /api/v1/products -> /api/v1/items",
-                },
-            ];
-            const apiSchemaChanges = {
-                endpointsRemoved: [],
-                endpointsRenamed: [
-                    { oldPath: "/api/v1/products", newPath: "/api/v1/items", method: "get" },
-                ],
-                endpointsModified: [],
-                authenticationChanged: false,
-            };
-            // Even with low drift score, renames should trigger UPDATE
-            const rec = generateRecommendation("products_smoke_test.py", 12, // low drift
-            undefined, issues, undefined, apiSchemaChanges);
-            expect(rec.action).toBe("UPDATE");
-            expect(rec.priority).toBe("HIGH");
-            expect(rec.rationale).toContain("renamed");
-            expect(rec.estimatedWork).toBe("SMALL");
-        });
-        it("should include renamedEndpoints in recommendation details", () => {
-            const issues = [
-                {
-                    type: "endpoints_renamed",
-                    severity: "high",
-                    description: "1 API endpoint(s) renamed",
-                    details: "Endpoint renamed: get /api/v1/products -> /api/v1/items",
-                },
-            ];
-            const apiSchemaChanges = {
-                endpointsRemoved: [],
-                endpointsRenamed: [
-                    { oldPath: "/api/v1/products", newPath: "/api/v1/items", method: "get" },
-                ],
-                endpointsModified: [],
-                authenticationChanged: false,
-            };
-            const rec = generateRecommendation("products_smoke_test.py", 30, undefined, issues, undefined, apiSchemaChanges);
-            expect(rec.details?.endpointStatus).toBe("renamed");
-            expect(rec.details?.renamedEndpoints).toBeDefined();
-            expect(rec.details?.renamedEndpoints).toHaveLength(1);
-            expect(rec.details?.renamedEndpoints?.[0]).toEqual({
-                oldPath: "/api/v1/products",
-                newPath: "/api/v1/items",
-                method: "get",
-            });
-        });
-        it("should mention test failure in rationale when test is failing due to rename", () => {
-            const issues = [
-                {
-                    type: "endpoints_renamed",
-                    severity: "high",
-                    description: "1 API endpoint(s) renamed",
-                    details: "Endpoint renamed: get /api/v1/products -> /api/v1/items",
-                },
-            ];
-            const execution = {
-                testFile: "products_smoke_test.py",
-                executedAt: new Date().toISOString(),
-                passed: false,
-                duration: 10000,
-                errors: ["404 Not Found"],
-                warnings: [],
-                crashed: false,
-            };
-            const apiSchemaChanges = {
-                endpointsRemoved: [],
-                endpointsRenamed: [
-                    { oldPath: "/api/v1/products", newPath: "/api/v1/items", method: "get" },
-                ],
-                endpointsModified: [],
-                authenticationChanged: false,
-            };
-            const rec = generateRecommendation("products_smoke_test.py", 30, execution, issues, undefined, apiSchemaChanges);
-            expect(rec.action).toBe("UPDATE");
-            expect(rec.rationale).toContain("failing");
-        });
-        it("should escalate to UPDATE when drift is 10-30 and test is failing", () => {
-            const execution = {
-                testFile: "products_integration_test.py",
-                executedAt: new Date().toISOString(),
-                passed: false,
-                duration: 2770,
-                errors: ["POST returns 400: Input should be a valid integer"],
-                warnings: [],
-                crashed: false,
-            };
-            const rec = generateRecommendation("products_integration_test.py", 15, // low drift (10-30 range)
-            execution, []);
-            expect(rec.action).toBe("UPDATE");
-            expect(rec.priority).toBe("HIGH");
-            expect(rec.estimatedWork).toBe("MEDIUM");
-        });
-        it("should keep VERIFY when drift is 10-30 and test is passing", () => {
-            const execution = {
-                testFile: "products_fuzz_test.py",
-                executedAt: new Date().toISOString(),
-                passed: true,
-                duration: 75000,
-                errors: [],
-                warnings: [],
-                crashed: false,
-            };
-            const rec = generateRecommendation("products_fuzz_test.py", 15, execution, []);
-            expect(rec.action).toBe("VERIFY");
-            expect(rec.priority).toBe("LOW");
-        });
-        it("should not set renamedEndpoints when there are no renames", () => {
-            const rec = generateRecommendation("orders_smoke_test.py", 5, undefined, [], { exists: true }, undefined);
-            expect(rec.action).toBe("VERIFY");
-            expect(rec.details?.renamedEndpoints).toBeUndefined();
-            expect(rec.details?.endpointStatus).toBe("exists");
-        });
-        it("should prefer rename handling over high-drift REGENERATE", () => {
-            // If drift is > 70 but it's caused by a rename, we should UPDATE not REGENERATE
-            const issues = [
-                {
-                    type: "endpoints_renamed",
-                    severity: "high",
-                    description: "5 API endpoint(s) renamed",
-                    details: "Multiple renames",
-                },
-            ];
-            const apiSchemaChanges = {
-                endpointsRemoved: [],
-                endpointsRenamed: [
-                    { oldPath: "/api/v1/products", newPath: "/api/v1/items", method: "get" },
-                    { oldPath: "/api/v1/products", newPath: "/api/v1/items", method: "post" },
-                    { oldPath: "/api/v1/products/{id}", newPath: "/api/v1/items/{id}", method: "get" },
-                    { oldPath: "/api/v1/products/{id}", newPath: "/api/v1/items/{id}", method: "put" },
-                    { oldPath: "/api/v1/products/{id}", newPath: "/api/v1/items/{id}", method: "delete" },
-                ],
-                endpointsModified: [],
-                authenticationChanged: false,
-            };
-            const rec = generateRecommendation("products_smoke_test.py", 75, // would normally trigger REGENERATE
-            undefined, issues, undefined, apiSchemaChanges);
-            // Rename detection should take priority over drift threshold
-            expect(rec.action).toBe("UPDATE");
-            expect(rec.estimatedWork).toBe("SMALL"); // renames are simple substitutions
-        });
-    });
-});

package/build/types/TestDriftAnalysis.js

@@ -1 +0,0 @@
-export {};

package/build/types/TestHealth.js

@@ -1,4 +0,0 @@
-/**
- * Types for Test Health Analysis and Recommendations
- */
-export {};