Added Support for Async nullables (but failing test)

UnoSD / Moq.Dapper

Moq extensions for Dapper methods.

GNU General Public License v2.0

173 stars 78 forks source link

Added Support for Async nullables (but failing test) #11

Closed ross2411 closed 6 years ago

ross2411 commented 6 years ago

I've copied the code for the nullables support into the async extensions. This appears to work (ie my original issue is fixed) but I can't get the unit test to pass. Let me know if you think this is any good and if so what's wrong with my unit test Thanks for the speedy turn around!

UnoSD commented 6 years ago

Hi @ross2411, the test was failing because it was mocking and testing a method taking a Type[] as parameter and that value was passed as null, I changed it to use the QueryAsync<T>(string) with all its default parameters and now the test passes.

UnoSD commented 6 years ago

Thank you for the contribution it's merged.