DYN-10274 Add skill - dynamo-unit-testing#16960
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10274
There was a problem hiding this comment.
Pull request overview
Adds a new repository “skill” for writing Dynamo NUnit tests and updates the agent guidance index/pointers so AI tools can discover and follow the new testing patterns.
Changes:
- Added a new
.agentsskill:dynamo-unit-testing(skill doc + testing patterns + quality checklist). - Introduced a new top-level
AGENTS.mdguidance entry point and redirectedCLAUDE.md. - Updated agent documentation to include the new skill and fixed Jira-template link locations.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| CLAUDE.md | Replaced detailed Claude guidance with a pointer to AGENTS.md. |
| AGENTS.md | New top-level agent guidance: project context, conventions, and links to .agents skills. |
| .agents/skills/dynamo-unit-testing/SKILL.md | New unit testing skill definition and workflow. |
| .agents/skills/dynamo-unit-testing/assets/test-patterns.md | Added test base-class guidance and example patterns/snippets. |
| .agents/skills/dynamo-unit-testing/references/quality-checklist.md | Added a checklist + anti-patterns reference for test quality. |
| .agents/skills/dynamo-pr-description/SKILL.md | Updated Jira template link to the new assets/template.md location. |
| .agents/skills/dynamo-jira-ticket/assets/template.md | Added a structured Jira ticket template (new location). |
| .agents/README.md | Registered the new skill and updated links/paths for rules/templates. |
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| @@ -0,0 +1,60 @@ | |||
| --- | |||
| name: dynamo-unit-testing | |||
| description: Write comprehensive NUnit tests for the Dynamo codebase following Dynamo testing patterns, conventions, and architectural constraints. Use this skill when writing test methods, test classes, setting up test infrastructure, mocking dependencies, testing with .dyn files, or reviewing test code in Dynamo. | |||
There was a problem hiding this comment.
This is the part I dont think nested structure works for Github Copilot agent customization, I had a PR earlier for designer skill went in this way and I had to flat it
There was a problem hiding this comment.
Ah I see, I will flatten it out.
# Conflicts: # .agents/README.md # CLAUDE.md
|
Purpose
Add skill - dynamo-unit-testing
Declarations
Check these if you believe they are true
Release Notes
N/A
Reviewers
(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)
(FILL ME IN, optional) Any additional notes to reviewers or testers.
FYIs
(FILL ME IN, Optional) Names of anyone else you wish to be notified of