@nbakka/mcp-appium 2.0.83 → 2.0.85

package/lib/server.js

@@ -614,10 +614,10 @@ tool(
 
 tool(
   "fetch_testcases_from_tcms",
-  "Before generating ticket for a jira ticket, always fetch existing test cases from TCMS tool for a specific folder",
+  "Before calling these tool, folder name can be analysed by data fetched from jira ticket info. Before generating test cases for a jira ticket, always fetch existing test cases from TCMS tool for a specific folder",
   {
-    projectKey: zod_1.z.string().describe("The project key (e.g., SCRUM)"),
-    folderName: zod_1.z.string().describe("The folder name to filter test cases (e.g., PDP)")
+    projectKey: zod_1.z.string().describe("The project key Default: SCRUM"),
+    folderName: zod_1.z.string().describe("The folder name to filter test cases (e.g., PDP), folder name can to be fetched from jira ticket info")
   },
   async ({ projectKey, folderName }) => {
     try {
@@ -940,11 +940,11 @@ tool(
     "Check the approval status of test cases review session with 25 second wait",
     {
         sessionId: zod_1.z.string().describe("The session ID returned from review_testcases tool")
-
+    },
+    async ({ sessionId }) => {
         const session = approvalSessions.get(sessionId);
 
         if (!session) {
-    },
             return JSON.stringify({
                 status: "error",
                 message: "Session not found. Invalid session ID."
@@ -955,42 +955,47 @@ tool(
         const elapsedTime = Math.floor((currentTime - session.startTime) / 1000);
 
         if (session.status === 'approved') {
-
             const approvedTestCases = session.finalTestCases || session.testCases;
-            approvalSessions.delete(sessionId);
 
             // Clean up session and close server
             if (session.server) {
                 session.server.close();
             }
-            return `Test cases have been approved by the user!
+            approvalSessions.delete(sessionId);
+
+            return JSON.stringify({
+                status: "approved",
                 message: "Test cases have been approved by the user.",
                 approvedTestCases: approvedTestCases,
                 elapsedTime: elapsedTime
-                session.server.close();
-            }
-            approvalSessions.delete(sessionId);
-
             });
         } else if (session.status === 'cancelled') {
             // Clean up session and close server
             if (session.server) {
-                status: "cancelled",
-                message: "Review was cancelled by the user.",
-                elapsedTime: elapsedTime
                 session.server.close();
             }
             approvalSessions.delete(sessionId);
 
+            return JSON.stringify({
+                status: "cancelled",
+                message: "Review was cancelled by the user.",
+                elapsedTime: elapsedTime
             });
         } else if (session.status === 'timeout' || elapsedTime > 300) { // 5 minutes
             // Clean up session and close server
             if (session.server) {
-        } else {
-Status: timeout
-Elapsed time: ${elapsedTime} seconds`;
+                session.server.close();
+            }
+            approvalSessions.delete(sessionId);
 
+            return JSON.stringify({
                 status: "timeout",
+                message: `Review session timed out. Elapsed time: ${elapsedTime} seconds`,
+                elapsedTime: elapsedTime
+            });
+        } else {
+            return JSON.stringify({
+                status: "pending",
                 message: "Test cases are still pending approval. Continue polling.",
                 elapsedTime: elapsedTime,
                 remainingTime: Math.max(0, 300 - elapsedTime)

package/lib/testcases-generation-context.txt

@@ -7,11 +7,16 @@ final output should be test in following format
 "Verify pay on credit banner exits on PDP", "Existing"
 "Verify overview tab on PDP", "New"
 "Verify financial services are displayed under financial tab", "Remove", "SCRUM-TC-2"
-"Verify xyz is present in overview section ", "Verify xyz is moved from overview to property tour section ", "Modify", "SCRUM-TC-2"
+"Verify xyz is present in overview section (this is a before test case fetched from tcms)", "Verify xyz is moved from overview to property tour section (this is a after test case generated)", "Modify", "SCRUM-TC-2"
 Existing - already exists in TCMS and no changes needed
 New - doesn't exist in TCMS and needs to be created
 Remove - to remove test cases from TCMS because it is no longer valid
-Modify - existing TCMS test case needs to be modified because of changes in the feature, only test cases
+Modify - existing TCMS test case needs to be modified because of changes in the feature,
+should contains before test case from tcms and after testcase that is generated, only test cases
 existing in TCMS and if change is to be done to it should be marked as Modify
 "SCRUM-TC-2" - id of existing test case in TCMS, only for Remove and Modify test cases
-if any new test case is to be created then don't mention any id against it
+if any new test case is to be created then don't mention any id against it
+for review_testcases test cases tool data should be in this format, strictly follow this format espcially for Modify test cases
+{"testCases":[["Verify that brochure section is visible under overview tab","Verify brochure section is removed from property tour tab","Modify","SCRUM-TC-1"],
+["Verify similar properties section is visible under over tab in pdp section","Remove","SCRUM-TC-2"],
+["Verify Project Brochure section appears above the 'Explore on map' section in Overview tab","New"]]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbakka/mcp-appium",
-  "version": "2.0.83",
+  "version": "2.0.85",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"