Skip to content

Conversation

@Ermabo
Copy link
Contributor

@Ermabo Ermabo commented Jan 13, 2026

Summary

Adds a short note explaining that the terms fake, stub, and mock are used inconsistently across testing literature and that this section follows a common .NET usage.


Internal previews

📄 File 🔗 Preview link
docs/core/testing/unit-testing-best-practices.md Unit testing best practices for .NET

Adds a short note explaining that the terms fake, stub, and mock are used inconsistently across testing literature and that this section follows a common .NET usage.
@Ermabo Ermabo requested a review from meaghanlewis as a code owner January 13, 2026 13:32
Copilot AI review requested due to automatic review settings January 13, 2026 13:32
@Ermabo Ermabo requested a review from Youssef1313 as a code owner January 13, 2026 13:32
@dotnetrepoman dotnetrepoman bot added this to the January 2026 milestone Jan 13, 2026
@dotnet-policy-service dotnet-policy-service bot added dotnet-fundamentals/svc community-contribution Indicates PR is created by someone from the .NET community. labels Jan 13, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a NOTE callout to clarify the inconsistent usage of testing terminology (fake, stub, and mock) across different testing literature and tools. The note explains that the document follows common .NET usage and contrasts it with classic xUnit/test-double terminology from Fowler and Meszaros.

Changes:

  • Added a NOTE block in the "Unit testing terminology" section to provide context about terminology inconsistencies and explain the definitions used in this document

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@Ermabo Ermabo closed this Jan 13, 2026
@Ermabo Ermabo reopened this Jan 13, 2026
@Youssef1313 Youssef1313 requested a review from nohwnd January 13, 2026 14:22
@meaghanlewis meaghanlewis enabled auto-merge (squash) January 16, 2026 00:06
@meaghanlewis meaghanlewis merged commit 3ef9e4f into dotnet:main Jan 17, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-fundamentals/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants