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
Add assertions for net6.0 DateOnly/TimeOnly #1754
Comments
@MullerWasHere can you confirm that you are working on this issue? |
@julealgon I was working on it previously, haven't progressed much and I'm lacking time. If you want you can open a PR for it. |
@jnyrup |
My initial thought was that adding .NET 6 could be done as part of adding these assertions, as there otherwise is no/little value in adding the extra target framework. |
Well, I tried to do so but it wasn't that straightforward. I think that these changes are not in scope for this specific feature implementation and should probably be performed by someone with better familiarity with the project. |
Oh, thought it would be more straight forward. |
@iliashkolyar Fluent Assertions now also target net6.0. |
Description
.NET 6 introduced two new interesting time structures,
DateOnly
andTimeOnly
, see this PR.Currently one can workaround the missing APIs in Fluent Assertions by converting
DateOnly
toDateTime
andTimeOnly
toTimeSpan
.It would be nice if we had native APIs for asserting on
DateOnly
andTimeOnly
.I believe most assertions can simply be copied from the
[Nullable]DateTimeAssertions
and[Nullable]SimpleTimeSpanAssertions
.The text was updated successfully, but these errors were encountered: