Mas/jm 260/entire class futute mocking - #256
Open
MladenStankov wants to merge 3 commits into
Open
Conversation
- Add futureMixinDatabase field to MocksRepository - Add InterceptFuture method mirroring InterceptStatics - Modify GetMockMixin to check futureMixinDatabase for instance calls - Add futureMixinDatabase.Clear() in Reset() for test isolation - Create Mock.SetupFuture.cs with four overloads gated by #if !LITE_EDITION Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…lesale interception Corrects the misleading 'Constructors are not intercepted' remark on all four SetupFuture overloads. SetupFuture uses EnableInterception which applies wholesale interception including constructors, consistent with SetupStatic. - Update remarks on SetupFuture<T>() and SetupFuture<T>(Behavior) to accurately describe constructor interception behavior and guidance to use CallOriginal if constructor execution is required. - Add <remarks> to the Type overloads (SetupFuture(Type) and SetupFuture(Type, Behavior)) which previously had no remarks section at all.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Mock.SetupFutureoverloads so tests can configure future mocking for all instance members of a target type.SetupFutureXML documentation to clarify that constructor interception is included and to document the expected behavior guidance.Testing
Checklist