Allow for class mocks and interface mocks to mock System.Object methods

[kolomanschaft]

• int object.GetHashCode()
• bool object.Equals(object obj)
• string object.ToString()

This fixes #248.

[abatishchev]

Make it HashSet<T> so Contains would work faster?

[kzu]

👍 to this. Straightforward since Tuple already implements GetHashCode

[kzu]

Wouldn't mind at all if you rebased on top of master, grab EditorConfig VS extension from the VS gallery, and auto-format again to get rid of all the whitespace changes ;)

[kzu]

if the two minor fixes are implemented, I'd gladly merge this

[kolomanschaft]

Very nice! I'll do it on Monday when I'm back at work.

[kolomanschaft]

@kzu All done :-).

[kzu]

Thanks! 👍

[kzu]

Heya @kolomanschaft, would you mind looking at the side effect of this feature at #273?

It seems to me that these overrides should always behave as loose even on strict mocks, to avoid breaking existing behavior (i.e. adding strict mocks to a dictionary would now break :( ).

[kolomanschaft]

@kzu sounds reasonable. I'll have a look at it tomorrow.

[kzu]

Thanks a lot!

On Mon, Aug 8, 2016 at 8:52 AM Martin Hammerschmied <
notifications@github.com> wrote:

@kzu https://github.com/kzu sounds reasonable. I'll have a look at it
tomorrow.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#250 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKW67ZycHPkGvtNxkbuwTtEM4UNL1Mrks5qdxhfgaJpZM4Hb4hA
.

/kzu from mobile