-
Notifications
You must be signed in to change notification settings - Fork 74
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
shim for DateTime.UtcNow not working #67
Comments
@MelbourneDeveloper yeah, probably going to get as much feedback as me :-) Anyway, I've worked around it the same way I do usually, introduce an additional DateTime input parameter into the relevant method, which works just as well. I just wanted to give this a try when I saw it... |
Would love to see this resolved. To me, using a library like Pose is the correct way to test for DateTime (though likely against the thoughts of OOP purists). Feedback / community seems like a bit of a ghost town though, wonder where I can pitch in if at all... There's also a general problem with async / await code that appears to be unresolved |
@Driedas @princefishthrower Please see my comment on #63 |
Release v2.0 to NuGet Version 2.0 supports the following targets: * .NET Standard 2.0 * .NET Core 2.0 * .NET Core 3.0 * .NET Framework 4.8 * .NET 7 * .NET 8 Version 2.0 fixes the following issues: * tonerdo/pose#17 * tonerdo/pose#37 * tonerdo/pose#38 * tonerdo/pose#41 * tonerdo/pose#47 * tonerdo/pose#49 * tonerdo/pose#60 * tonerdo/pose#67 * tonerdo/pose#68 * tonerdo/pose#70 * tonerdo/pose#71 * tonerdo/pose#72 * tonerdo/pose#75 * tonerdo/pose#79
@princefishthrower For support for async/await please see Miista/pose#12 |
Hi,
I've stumbled upon this project looking for exactly this feature for unit testing DateTime.UtcNow (without having to resort to 3rd party libraries like NodaTime etc), have cloned the repository and added the following unit test, based on the documentation
The result I get is a fail however, with the message
Am I doing something wrong here? The unit test suite that comes with the project is all green, however the above should work as well, according to my understanding...
The text was updated successfully, but these errors were encountered: