-
Notifications
You must be signed in to change notification settings - Fork 183
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
Move most proxy generator tests to specs #1495
Move most proxy generator tests to specs #1495
Conversation
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.
Thanks @blairconrad! I made a cursory review and found just one issue.
using FluentAssertions; | ||
using Xbehave; | ||
using Xunit; | ||
|
||
public static class FakingDelegates | ||
{ | ||
public delegate void VoidDelegateWithOutAndRefValues(out string outString, ref int refInt); | ||
|
||
public delegate void NonVoidDelegateWithOutAndRefValues(ref string refString, out int outInt); |
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.
Shouldn't this delegate return something, given its name?
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.
YES!
74c34f3
to
973a541
Compare
Thanks, @thomaslevesque. One more issue than I found! Hey! I never noticed the force-pushed note before: |
I never noticed it either. I think it's new! |
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.
LGTM, thanks @blairconrad!
Thank you, @thomaslevesque. |
This change has been released in FakeItEasy 5.0.0. |
The tests were very low-level, and I've some upcoming PRs that will needlessly break them.