Graphhopper: RealtimeIt#testExtraTrip fails for some local time zones

Created on 4 Nov 2017  路  8Comments  路  Source: graphhopper/graphhopper

I just ran the tests for the current master using the time one UTC+10:00 and the test RealtimeIt#testExtraTrip failed. If I change my time zone to Shanghai UTC+08:00 it works. If I change my time zone to Perth AWST (same time as UTC+08:00 right now), it fails.

Maybe @michaz you have an idea what might be happening here? In the test we define the time zone, but maybe the server itself runs on a different time zone?

bug public transport

Most helpful comment

That was a really helpful ticket, thanks for reminding me!

All 8 comments

Haha, nice. I probably left some debug or mock thing in there which uses the "current" time zone. Good that we are a global team. :)

(Will look at it today)

As we stumbled over this recently #1209 maybe we should use something like the policeman tools?

https://github.com/policeman-tools/forbidden-apis/wiki
http://blog.thetaphi.de/2012/07/default-locales-default-charsets-and.html

maybe we should use something like the policeman tools?

I second that! I think this is a good idea :). I created a PR #1213 testing the policeman tools.

Any updated on this @michaz? Could we ignore this test until the problem is solved?

Just a second..

That was a really helpful ticket, thanks for reminding me!

Thanks so much for fixing this @michaz!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

andreddosantos picture andreddosantos  路  8Comments

devemux86 picture devemux86  路  9Comments

mc51 picture mc51  路  5Comments

karussell picture karussell  路  9Comments

karussell picture karussell  路  9Comments