search

tobyweston / temperature-machine

Data logger for multiple DS18B20 temperature sensors on one or more machines
Apache License 2.0
67 stars 22 forks source link

LogParserTest fails when using different timezones #55

Closed Quaxo76 closed 6 years ago

Quaxo76 commented 6 years ago

I live in Italy and have set my local timezone accordingly. I have to do that, because otherwise, the graphs will show the wrong time. But with the changed timezone, the LogParserTest always fails:

[info] LogParserTest
[info] 
[error] x Example inputs
[error]  LogMessage(2017-10-31T09:40:05.976Z,[main],INFO,Starting temperature-machine (server mode)...) != LogMessage(2017-10-31T10:40:05.976Z,[main],INFO,Starting temperature-machine (server mode)...) (LogParserTest.scala:97)
[error] Actual:   ...0-31T[09]:4...Z,[main],INFO...
[error] Expected: ...0-31T[10]:4...Z,[main],INFO...

I'm forced to revert the timezone to UTC to compile, then change it back (which is annoying) or to skip the tests altogether (which defeats the point of having those tests).

tobyweston commented 6 years ago

see #56 for wider concerns

tobyweston commented 6 years ago

This should be fixed now, ensure you delete any old logs else the UI will error displaying them.