mockhttp
mockhttp copied to clipboard
Add improved VerifyNoOutstandingExpectation messages
Implementation for #78
I've added tests
@richardszalay would you mind taking a look at this?
Thanks for submitting this PR @hwoodiwiss, and apologies for taking so long to reply.
I'll need to publish this as a major/breaking change release due to the change in the public interfaces, but I think it's worth it.
No worries, and thanks for taking a look!
@richardszalay I've been thinking it over, it might be easiest for now to remove the CallerArgumentExpression stuff, and add that later as a separate feature, what do you think?
Closing this as I've had not had a response from the maintainer in a couple of years (not complaining, people have lives). If anyone else wants to look at implementing something similar in the future, feel free to use/adapt any of the code I've written here as needed.
Apologies for keeping you out of the loop, this functionality (though not this implementation) was included in 7.0.0.
Thanks for understanding re: availability, it's difficult to keep up with side projects when life gets in the way.
Rich
No problem, I'll have to have a play with 7.0.0!
Thank you for the library and the time you give to the community!
Thanks Hugo, and thank you again for the PR. I don't like not merging PRs, but from memory the reason I went for a different implementation is that I decided to also solve for some other usecases.