Skip to content

docs(test): add Given/When/AssertThat testing skill - #3448

Open
jjamroga wants to merge 2 commits into
kagent-dev:mainfrom
jjamroga:docs/given-when-assert-that-skill
Open

jjamroga wants to merge 2 commits into
kagent-dev:mainfrom
jjamroga:docs/given-when-assert-that-skill

Conversation

@jjamroga

@jjamroga jjamroga commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add a Go test-writing skill that starts from observable behavior and meaningful boundary cases, then names scenarios with Given/When/AssertThat.
  • Keep tests as ordinary go test cases, with optional fixtures rather than a BDD framework; add the standard per-skill Claude discovery symlink.
  • Make using the skill a repository-wide expectation before writing or changing Go behavior tests, including feature and bug-fix tests, in AGENTS.md.

Validation

  • git diff upstream-kagent/main...HEAD --check — passed.
  • test -f .claude/skills/given-when-assert-that/SKILL.md — passed (symlink resolves).
  • Go tests not run: this PR changes only agent guidance, with no code or test changes.

Adapted from solo-io/enterprise-kagent#695.

Signed-off-by: Jonathan Jamroga <jjamroga@gmail.com>
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 30, 2026
Signed-off-by: Jonathan Jamroga <jjamroga@gmail.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant