-
Notifications
You must be signed in to change notification settings - Fork 642
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 mutable test clock #2655
Comments
I can take that issue, too |
Great! |
its the java.time.Clock that we want to implement. |
@pientaa did you still intend to do this ? |
@sksamuel yes, I'm sorry, I was busy recently |
No problem, just wondering :) |
@sksamuel after looking at |
You can't set an arbitrary value on them after creation. Unless i missed it.
…On Thu, 13 Jan 2022, 12:44 Łukasz Pięta, ***@***.***> wrote:
@sksamuel <https://github.com/sksamuel> after looking at java.time.Clock
I noticed that there are already implemented such classes: OffsetClock
and TickClock. And they seem to be exactly what you asked for. Or maybe
there is something that I missed or misunderstood?
—
Reply to this email directly, view it on GitHub
<#2655 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFVSGQ7H4STNU6TSWUD76TUV4MRZANCNFSM5IJWGVCQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Would be nice if we had a fixed clock but that could be advanced manually.
The text was updated successfully, but these errors were encountered: