Skip to content

[ZEPHYR] TM4J-14847: Add a new tool test execution update test steps#386

Merged
BenzoEnzo merged 16 commits intomainfrom
TM4J-14847-test-execution-update-test-steps
Mar 30, 2026
Merged

[ZEPHYR] TM4J-14847: Add a new tool test execution update test steps#386
BenzoEnzo merged 16 commits intomainfrom
TM4J-14847-test-execution-update-test-steps

Conversation

@BenzoEnzo
Copy link
Copy Markdown
Contributor

Goal

This PR will provide functionality for updating a Test Execution Steps for user

Design

To be in line with things done for other Zephyr integrations, introduce PUT call type

Changeset

  • Add new update-test-steps.ts tool for updating Test steps for Test Execution tool
  • Modify client.ts to register new tool
  • Modify Zephyr documentation zephyr-integration.md to include new Test Execution Steps update capability
  • Add update-test-steps.test.ts to test new tool for updating Test Steps for Test Execution

Testing

  • Unit tests
  • Verify new tool is properly recognized and loaded by Copilot Agent, then tested it with different prompts (simple ones giving lot of autonomy to the agent, and more complex one expecting the agent to make extra calls to build proper request payload)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 20, 2026

🎯 Coverage Target Met!

📈 Coverage Metrics

Metric Coverage Target Status
Lines 93.5% 80%
Functions 86.0% 80%
Branches 86.8% 80%
Statements 93.5% 80%

📊 Test Statistics

  • Total Lines: 20,789 / 22,220
  • Total Functions: 320 / 372
  • Total Branches: 946 / 1090
  • Total Statements: 20789 / 22220

🔍 Files Needing Coverage

File Coverage
src/reflect/prompt/sap-test.ts 9.5%
src/qmetry/client/testsuite.ts 10.3%
src/common/prompts.ts 16.7%
src/qmetry/client/issues.ts 20.1%
src/swagger/client/api.ts 30.0%

📝 Report generated on Node.js v24.14.0 • View workflow • Coverage by Vitest + v8

Copy link
Copy Markdown
Contributor

@assisblf assisblf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice @BenzoEnzo! I added some comments

testExecutionIdOrKey: "SA-E5",
steps: [
{
statusName: "Pass",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
statusName: "Pass",

The prompt asks to just update the status of the second step, meaning that no changes need to be performed in the first step.

@BenzoEnzo BenzoEnzo merged commit 051da0e into main Mar 30, 2026
46 checks passed
@BenzoEnzo BenzoEnzo deleted the TM4J-14847-test-execution-update-test-steps branch March 30, 2026 14:08
@assisblf assisblf mentioned this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants