fix: Update state with earlier charge complete and arrival times

[InTheDaylight14]

Added absolute value around the seconds comparison to allow for a state change when the charge complete time and the arrival time move earlier than before.

closes #565

[InTheDaylight14]

I've confirmed this change works in home assistant. I'm not sure how to get the state to update in the test after the times move earlier.

[InTheDaylight14]

I've confirmed this change works in home assistant. I'm not sure how to get the state to update in the test after the times move earlier.

Such a small logic change I think we are good w/o specific test

[alandtse]

My preference is that we have a test to help with regressions. HA should have loads of examples of testing around mocking time. I think they use timemachine but there are alternatives.

[InTheDaylight14]

I'm able to mock the time okay. The entity state does not update after the mocked time changes (I tired to increase and decrease the time to make sure, increasing works under the original logic). Is there a way to force HA to update the sensors?

Not working test sequence:

• Check sensor state: passes
• Change Tesla API value to arrive 3 minutes earlier than before
• Set time 2 minutes in future
• Re-check state: state has not changed from the first check. Should be 1 minute before the first check.

[alandtse]

I haven't done HA testing for a while, but again, this is probably something done in core a lot.

[InTheDaylight14]

I've dumped way too much time into this, I can't solve it.

I think I've narrowed down to, the mock controller does not grab new data and update the HA states during the test. Its not an issue with the changing time, that would be easy.

[alandtse]

Well, if you can't solve it just merge up to dev and I'll get you in the next release.