Clock feedback

[josecelano]

hi @da2ce7, this is my feedback for this.

It's not finished yet.

• Prefer From trait over Into.
• Rename clock
• Allow mocking the time in tests with a given time.

Why did I rename the Clock?

The Clock return an instance of itself, and it's the time. I see the Clock as the service and the Time as the value object. In this implementation, you do not use the service, only a value object that instantiates by itself.

With the new name, you see:

With the previous name, you read:

It's a little weird that the variable current_time is a UnixClock.

[josecelano]

I forgot to mention that I prefer System over Local to refer to the operating system time because it's the name used in the standard library.