Skip to content

feat: Add Test Double (Meszaros) semantic anchor#142

Merged
rdmueller merged 1 commit intoLLM-Coding:mainfrom
Nantero1:add-anchor-test-double
Mar 9, 2026
Merged

feat: Add Test Double (Meszaros) semantic anchor#142
rdmueller merged 1 commit intoLLM-Coding:mainfrom
Nantero1:add-anchor-test-double

Conversation

@Nantero1
Copy link

@Nantero1 Nantero1 commented Mar 7, 2026

Summary

  • Adds Test Double (Meszaros) as a new semantic anchor
  • Covers Gerard Meszaros' 5-type taxonomy from xUnit Test Patterns (2007)
  • Includes anchor file, all-anchors include, and AgentSkill catalog entry

Why this anchor?

The catalog has London School TDD ("mock-heavy") and Chicago School TDD ("minimal mocking"), but neither defines what kind of double you're using. Meszaros' taxonomy is the canonical classification:

  • Dummy — passed but never used
  • Stub — canned responses, no verification
  • Spy — records calls for later assertion
  • Mock — pre-programmed expectations, verifies interactions
  • Fake — working simplified implementation

This is orthogonal to both TDD schools: they define when/how much to use doubles, Meszaros defines what the double IS. London School TDD already mentions "test doubles (mocks, stubs)" in passing but never defines the taxonomy — this anchor fills that gap.

Files changed

  • docs/anchors/test-double-meszaros.adoc — new anchor file
  • docs/all-anchors.adoc — added include in Testing & Quality section
  • skill/semantic-anchor-translator/references/catalog.md — added catalog entry

Adds Meszaros' test double taxonomy (Dummy, Stub, Spy, Mock, Fake)
as a standalone anchor. Orthogonal to both TDD schools — they define
when/how much to use doubles, this defines what the double IS.
@coderabbitai
Copy link

coderabbitai bot commented Mar 7, 2026

Warning

Rate limit exceeded

@Nantero1 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 5 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 339924d1-05a2-4ac4-9cf6-8349bee772e5

📥 Commits

Reviewing files that changed from the base of the PR and between aefa0b4 and 77982ca.

📒 Files selected for processing (3)
  • docs/all-anchors.adoc
  • docs/anchors/test-double-meszaros.adoc
  • skill/semantic-anchor-translator/references/catalog.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@raifdmueller
Copy link
Contributor

Review: ✅ Excellent anchor, merge-ready.

Test Double (Meszaros) is a textbook semantic anchor. The Dummy/Stub/Spy/Mock/Fake taxonomy is exactly the kind of precise vocabulary that improves LLM communication — saying "use a Stub here, not a Mock" triggers fundamentally different behavior. The "vocabulary precision" concept is spot on.

Post-merge TODO:

  • Add German translation (test-double-meszaros.de.adoc)

Merging recommended, translation will follow in batch (see #149).

@rdmueller rdmueller merged commit 4555257 into LLM-Coding:main Mar 9, 2026
8 checks passed
raifdmueller added a commit to raifdmueller/Semantic-Anchors that referenced this pull request Mar 9, 2026
LLM-Coding#138, LLM-Coding#139)

Content from @Nantero1's original PRs LLM-Coding#138 and LLM-Coding#139, rebased on current main
after merging LLM-Coding#137, LLM-Coding#140, LLM-Coding#142, LLM-Coding#143.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants