]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.3] time: removed from tests now obsolete assumption about Austral...
authorAndrew Gerrand <adg@golang.org>
Mon, 29 Sep 2014 03:46:12 +0000 (13:46 +1000)
committerBrad Fitzpatrick <bradfitz@golang.org>
Mon, 29 Sep 2014 03:46:12 +0000 (13:46 +1000)
commit370a9c9c0ec8abe2cdffcd7876086b3329f8b668
tree1beb4673ee2c01f625fd55806aa141178679b1e0
parentf3c81ed821268e2f2e2945b0816f495809bbdf21
[release-branch.go1.3] time: removed from tests now obsolete assumption about Australian tz abbreviations

««« CL 130920043 / 7dba9475ee72
time: removed from tests now obsolete assumption about Australian tz abbreviations

Australian timezones abbreviation for standard and daylight saving time were recently
changed from EST for both to AEST and AEDT in the icann tz database (see changelog
on www.iana.org/time-zones).

A test in the time package was written to check that the ParseInLocation function
understand that Feb EST and Aug EST are different time zones, even though they are
both called EST. This is no longer the case, and the Date function now returns
AEST or AEDT for australian tz on every Linux system with an up to date tz database
(and this makes the test fail).

Since I wasn't able to find another country that 1) uses daylight saving and 2) has
the same abbreviation for both on tzdata, I changed the test to make sure that
ParseInLocation does not get confused when it parses, in different locations, two
dates with the same abbreviation (this was suggested in the mailing list).

Fixes #8547.

LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/130920043

»»»

LGTM=dsymonds
R=rsc, dsymonds
CC=golang-codereviews
https://golang.org/cl/153720043
src/pkg/time/format_test.go