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

Fix for issue #92: Can't setup protected methods with nullable parameters #200

Merged
merged 4 commits into from
Mar 24, 2017

Conversation

RobSiklos
Copy link
Contributor

No description provided.

@kzu
Copy link
Member

kzu commented Sep 15, 2015

Any chance you could add a new unit test that exercises this new behavior? It's looking great :)

@RobSiklos
Copy link
Contributor Author

Any chance you could add a new unit test that exercises this new behavior? It's looking great :)

@kzu Sure. Should I put it in ProtectedMockFixture or IssueReportsFixture ?

@kzu
Copy link
Member

kzu commented Feb 11, 2016

sorry @RobSiklos. ProtectedMockFixture should be fine.

thanks a lot!

@RobSiklos
Copy link
Contributor Author

@kzu Ok, unit test added - you should be able to merge now.

@kzu kzu merged commit a8ad776 into devlooped:master Mar 24, 2017
@devlooped devlooped locked and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants