-
Notifications
You must be signed in to change notification settings - Fork 179
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
Clarify use of WithArgumentsForConstructor expression #1855
Clarify use of WithArgumentsForConstructor expression #1855
Conversation
docs/creating-fakes.md
Outdated
// The constructor seen here is never actually invoked. It is an expression and it's purpose | ||
// is purely to communicate the constructor arguments which will be extracted from it | ||
// Since the constructor call seen here is an expression, it is not invoked at this point | ||
// in the code. (In fact no instance of `FooClass` will ever be constructed.) Instead, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact no instance of
FooClass
will ever be constructed
I think this could be confusing too. The fake will be an instance of the proxy class, but it's still "an instance of FooClass
", in the sense that fake is FooClass
is true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grumble grumble
working on it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
amended
6f12dde
to
4a6c8ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Thanks @blairconrad
Oh, thank you, @thomaslevesque. |
This change has been released as part of FakeItEasy 7.2.0. |
Inspired by #1854, where @Bludator had noted that the original documentation was confusing.