Accepting expected result as a part of TestCase step instead of hardcoding (#381)

_Replace_ by description of the work done

## GitHub issue number
#106 

## **Associated Risks**

_Replace_ by possible risks this pull request can bring you might have
thought of

## ✅ **PR Checklist**

- [x] **I have read the [contribution
guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CONTRIBUTING.md)**
- [x] **I have read the [code of conduct
guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CODE_OF_CONDUCT.md)**
- [x] Title of the pull request is clear and informative.
- [x] 👌 Code hygiene
- [x] 🔭 Telemetry added, updated, or N/A
- [x] 📄 Documentation added, updated, or N/A
- [x] 🛡️ Automated tests added, or N/A

## 🧪 **How did you test it?**

<img width="629" height="447" alt="image"
src="https://github.com/user-attachments/assets/b01aa46a-ddf2-4c2c-9f11-23c54e62d42a"
/>

<img width="981" height="193" alt="image"
src="https://github.com/user-attachments/assets/7b8bc3cf-f8fa-4c7c-92df-ed1fe0216f05"
/>


_Replace_ with use cases tested and models used

---------

Co-authored-by: Dan Hellem <[email protected]>

Author: skmanoj

src/tools/testplans.ts

@@ -115,7 +115,7 @@ function configureTestPlanTools(server: McpServer, tokenProvider: () => Promise<
     {
       project: z.string().describe("The unique identifier (ID or name) of the Azure DevOps project."),
       title: z.string().describe("The title of the test case."),
-      steps: z.string().optional().describe("The steps to reproduce the test case. Make sure to format each step as '1. Step one\\n2. Step two' etc."),
+      steps: z.string().optional().describe("The steps to reproduce the test case. Make sure to format each step as '1. Step one|Expected result one\n2. Step two|Expected result two"),
       priority: z.number().optional().describe("The priority of the test case."),
       areaPath: z.string().optional().describe("The area path for the test case."),
       iterationPath: z.string().optional().describe("The iteration path for the test case."),
@@ -227,20 +227,24 @@ function configureTestPlanTools(server: McpServer, tokenProvider: () => Promise<
  * Helper function to convert steps text to XML format required
  */
 function convertStepsToXml(steps: string): string {
+  // Accepts steps in the format: '1. Step one|Expected result one\n2. Step two|Expected result two'
   const stepsLines = steps.split("\n").filter((line) => line.trim() !== "");
 
   let xmlSteps = `<steps id="0" last="${stepsLines.length}">`;
 
   for (let i = 0; i < stepsLines.length; i++) {
     const stepLine = stepsLines[i].trim();
     if (stepLine) {
-      const stepMatch = stepLine.match(/^(\d+)\.\s*(.+)$/);
-      const stepText = stepMatch ? stepMatch[2] : stepLine;
+      // Split step and expected result by '|', fallback to default if not provided
+      const [stepPart, expectedPart] = stepLine.split("|").map((s) => s.trim());
+      const stepMatch = stepPart.match(/^(\d+)\.\s*(.+)$/);
+      const stepText = stepMatch ? stepMatch[2] : stepPart;
+      const expectedText = expectedPart || "Verify step completes successfully";
 
       xmlSteps += `
                 <step id="${i + 1}" type="ActionStep">
                     <parameterizedString isformatted="true">${escapeXml(stepText)}</parameterizedString>
-                    <parameterizedString isformatted="true">Verify step completes successfully</parameterizedString>
+                    <parameterizedString isformatted="true">${escapeXml(expectedText)}</parameterizedString>
                 </step>`;
     }
   }

test/src/tools/testplan.test.ts

@@ -194,6 +194,43 @@ describe("configureTestPlanTools", () => {
       );
     });
 
+    it("should create test case & expected result with proper parameters", async () => {
+      configureTestPlanTools(server, tokenProvider, connectionProvider);
+      const call = (server.tool as jest.Mock).mock.calls.find(([toolName]) => toolName === "testplan_create_test_case");
+      if (!call) throw new Error("testplan_create_test_case tool not registered");
+      const [, , , handler] = call;
+
+      (mockWitApi.createWorkItem as jest.Mock).mockResolvedValue({
+        id: 1001,
+        fields: {
+          "System.Title": "New Test Case",
+          "System.WorkItemType": "Test Case",
+        },
+      });
+
+      const params = {
+        project: "proj1",
+        title: "New Test Case",
+        steps: "1. Test step 1 | Expected result 1\n2. Test step 2 | Expected result 2",
+      };
+      const result = await handler(params);
+
+      expect(mockWitApi.createWorkItem).toHaveBeenCalledWith({}, expect.any(Array), "proj1", "Test Case");
+      expect(result.content[0].text).toBe(
+        JSON.stringify(
+          {
+            id: 1001,
+            fields: {
+              "System.Title": "New Test Case",
+              "System.WorkItemType": "Test Case",
+            },
+          },
+          null,
+          2
+        )
+      );
+    });
+
     it("should handle multiple steps in test case", async () => {
       configureTestPlanTools(server, tokenProvider, connectionProvider);
       const call = (server.tool as jest.Mock).mock.calls.find(([toolName]) => toolName === "testplan_create_test_case");