Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for mocking protected generic methods #967

Conversation

JmlSaul
Copy link
Contributor

@JmlSaul JmlSaul commented Dec 20, 2019

No description provided.

Copy link
Contributor

@stakx stakx left a comment

Choose a reason for hiding this comment

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

This looks good in principle. Just a few smallish details, see review comments below.

Please also add an entry to CHANGELOG.md of the form:

 The format is loosely based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).

+## Unreleased
+
+#### Added
+
+ <brief description of your change, e.g.: "New `.Protected().Setup` and `Protected().Verify` method overloads to deal with generic methods"> (@JmlSaul, #967)
+
+
 ## 4.13.1 (2019-10-19)

 #### Fixed

src/Moq/Protected/IProtectedMock.cs Outdated Show resolved Hide resolved
src/Moq/Protected/IProtectedMock.cs Outdated Show resolved Hide resolved
src/Moq/Protected/ProtectedMock.cs Outdated Show resolved Hide resolved
@stakx stakx merged commit 0a25803 into devlooped:master Mar 3, 2020
@stakx
Copy link
Contributor

stakx commented Mar 3, 2020

Thanks for your contribution, @JmlSaul!

@stakx stakx added this to the 4.13.2 milestone Mar 3, 2020
@stakx stakx modified the milestones: 4.13.2, 4.14.0 Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants