-
Notifications
You must be signed in to change notification settings - Fork 232
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
LocalDateTimeRangeRandomizer Min-Max Exception due time of day #361
Comments
johkin
added a commit
to johkin/easy-random
that referenced
this issue
Oct 21, 2019
The randomizer could not generate datetime if the range span midnight, since the min/max-check failed for the LocalTimeRangeRandomizer. The new implementation breaks the current seed-based generation. j-easy#361
johkin
added a commit
to johkin/easy-random
that referenced
this issue
Oct 22, 2019
Thanks for reporting this issue! I believe it should be resolved with #373 . You can already give it a try with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I have this data generator:
The issue is related to LocalDateTimeRangeRandomizer, some time the range gets yesterday, so the time of the day could be greater than now. As show is this example:
That second comparator is whats is causing the exception. First LocalDateDate is actually before the second, but not the dateTimes.
It would be ok to remove the LocalTime check?
The text was updated successfully, but these errors were encountered: