Skip to content

tdd skill: point at the workspace Java TDD skill and the real test stack - #383

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/hopeful-pascal-9jlbqb
Oct 9, 2026
Merged

bernardladenthin merged 1 commit into
mainfrom
claude/hopeful-pascal-9jlbqb

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • .claude/skills/tdd/SKILL.md told the reader to follow .claude/skills/java-tdd-guide/SKILL.md, which in this repository has been only the pointer file since Share the java-tdd-guide pointer file across the four repositories (4-repo sync) #382; it now names the canonical skill in the sibling workspace repo (../workspace/.claude/skills/java-tdd-guide/SKILL.md) and the Java 21 supplements of the version chain.
  • The stack it described (JUnit 4, DataProviderRunner) is banned by the Maven Enforcer; the bullets now name JUnit Jupiter 6, Hamcrest 3, Mockito and @ParameterizedTest + @MethodSource.
  • ./mvnw replaced by mvn (there is no wrapper here, as CLAUDE.md says), with the hint to narrow -Dtest= while iterating.

Test plan

  • Documentation-only change (one skill file); no tests affected
  • CI is green on this branch
  • Docs / CHANGELOG updated where applicable (not a user-facing change, no CHANGELOG entry)

Related issues / PRs

Follow-up to #382 (the shared SKILL.pointer.md).

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes (if there are, I have notified the maintainer privately per SECURITY.md)

🤖 Generated with Claude Code

https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2


Generated by Claude Code

The skill told the reader to follow `.claude/skills/java-tdd-guide/SKILL.md`,
which in this repository is only the pointer file since #382, and described
the stack as JUnit 4 with DataProviderRunner, which the Maven Enforcer bans
(the suite runs on JUnit Jupiter 6, Hamcrest 3 and Mockito). It also showed
`./mvnw`, and there is no wrapper here (CLAUDE.md says so). Now it names the
workspace path, the Java 21 supplements of the version chain, the actual stack
and plain `mvn`.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2
@sonarqubecloud

sonarqubecloud Bot commented Oct 9, 2026

Copy link
Copy Markdown

@bernardladenthin
bernardladenthin merged commit dad1a2d into main Oct 9, 2026
27 of 30 checks passed
@bernardladenthin
bernardladenthin deleted the claude/hopeful-pascal-9jlbqb branch October 9, 2026 18:26

This branch had an error being deployed

1 failed and 1 active deployments
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