Skip to content

test: DH-20954: Added a test-writing skill and updated mcp tool tests accordingly#296

Merged
bmingles merged 6 commits intomainfrom
DH-20954-10_mcp-test-skill
Feb 10, 2026
Merged

test: DH-20954: Added a test-writing skill and updated mcp tool tests accordingly#296
bmingles merged 6 commits intomainfrom
DH-20954-10_mcp-test-skill

Conversation

@bmingles
Copy link
Collaborator

@bmingles bmingles commented Feb 9, 2026

DH-20954:

  • I moved AGENTS writing test instructions into a dedicated test-writing skill
  • I then prompted the agent to use the test-writing skill to fix any mcp tool tests that were not in conformance with the skill

Testing

There shouldn't be any app logic that changed, and tests are refactors of pre-existing tests, so tests passing in CI should suffice

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

No docs changes detected for 99aeb73

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

End-to-end Test Summary

Tests 📝Passed ✅Failed ❌Skipped ⏭️Pending ⏳Other ❓Flaky 🍂Duration ⏱️
330000017:34:09
A ctrf plugin

Failed Test Summary

No failed tests ✨

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Unit Test Summary

Tests 📝Passed ✅Failed ❌Skipped ⏭️Pending ⏳Other ❓Flaky 🍂Duration ⏱️
6036030000000:00:00
A ctrf plugin

Failed Test Summary

No failed tests ✨

@bmingles bmingles requested a review from mofojed February 9, 2026 22:51
@bmingles bmingles merged commit 5f02d35 into main Feb 10, 2026
6 checks passed
@bmingles bmingles deleted the DH-20954-10_mcp-test-skill branch February 10, 2026 20:32
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