diff options
author | fraenkel <michael.fraenkel@gmail.com> | 2018-04-11 21:18:58 -0400 |
---|---|---|
committer | Brad Fitzpatrick <bradfitz@golang.org> | 2018-04-12 04:56:57 +0000 |
commit | 94ef1dafbd9b490bdf7fc40b33df8399eea03b8e (patch) | |
tree | 9532a9b5b10dc26e76ccc02f2b7213f23dbee72e /src/time/time_test.go | |
parent | 2dfb423e6e84b64acb0dc741cba5b1fa5fe03c64 (diff) | |
download | go-94ef1dafbd9b490bdf7fc40b33df8399eea03b8e.tar.gz go-94ef1dafbd9b490bdf7fc40b33df8399eea03b8e.zip |
time: don't panic when stringifying Weekday
Fixes #24692
Change-Id: I14058cd3968d08fbcfc275f1b13b6dba9e3c5068
Reviewed-on: https://go-review.googlesource.com/106535
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/time/time_test.go')
-rw-r--r-- | src/time/time_test.go | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/src/time/time_test.go b/src/time/time_test.go index 867e61ea33..dee144fe13 100644 --- a/src/time/time_test.go +++ b/src/time/time_test.go @@ -673,7 +673,7 @@ var gobTests = []Time{ Date(0, 1, 2, 3, 4, 5, 6, UTC), Date(7, 8, 9, 10, 11, 12, 13, FixedZone("", 0)), Unix(81985467080890095, 0x76543210), // Time.sec: 0x0123456789ABCDEF - {}, // nil location + {}, // nil location Date(1, 2, 3, 4, 5, 6, 7, FixedZone("", 32767*60)), Date(1, 2, 3, 4, 5, 6, 7, FixedZone("", -32768*60)), } @@ -1319,6 +1319,16 @@ func TestZeroMonthString(t *testing.T) { } } +// Issue 24692: Out of range weekday panics +func TestWeekdayString(t *testing.T) { + if got, want := Weekday(Tuesday).String(), "Tuesday"; got != want { + t.Errorf("Tuesday weekday = %q; want %q", got, want) + } + if got, want := Weekday(14).String(), "%!Weekday(14)"; got != want { + t.Errorf("14th weekday = %q; want %q", got, want) + } +} + func TestReadFileLimit(t *testing.T) { const zero = "/dev/zero" if _, err := os.Stat(zero); err != nil { |