Skip to content

Commit 921c809

Browse files
author
James Cor
committed
fixing more tests
1 parent e91478a commit 921c809

File tree

1 file changed

+7
-13
lines changed

1 file changed

+7
-13
lines changed

sql/expression/function/str_to_date_test.go

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,25 +20,22 @@ func TestStrToDate(t *testing.T) {
2020
fmtStr string
2121
expected interface{}
2222
}{
23-
{"standard", "Dec 26, 2000 2:13:15", "%b %e, %Y %T", "2000-12-26 02:13:15"},
24-
{"ymd", "20240101", "%Y%m%d", "2024-01-01"},
25-
{"ymd", "2024121", "%Y%m%d", "2024-12-01"},
23+
{"standard", "Dec 26, 2000 2:13:15", "%b %e, %Y %T", time.Date(2000, time.December, 26, 2, 13, 15, 0, time.UTC)},
24+
{"ymd", "20240101", "%Y%m%d", time.Date(2024, time.January, 1, 0, 0, 0, 0, time.UTC)},
25+
{"ymd", "2024121", "%Y%m%d", time.Date(2024, time.December, 1, 0, 0, 0, 0, time.UTC)},
2626
{"ymd", "20241301", "%Y%m%d", nil},
2727
{"ymd", "20240001", "%Y%m%d", nil},
28-
{"ymd-with-time", "2024010203:04:05", "%Y%m%d%T", "2024-01-02 03:04:05"},
29-
{"ymd-with-time", "202408122:03:04", "%Y%m%d%T", "2024-08-12 02:03:04"},
28+
{"ymd-with-time", "2024010203:04:05", "%Y%m%d%T", time.Date(2024, time.January, 2, 3, 4, 5, 0, time.UTC)},
29+
{"ymd-with-time", "202408122:03:04", "%Y%m%d%T", time.Date(2024, time.August, 12, 2, 3, 4, 0, time.UTC)},
3030
// TODO: It shoud be nil, but returns "2024-02-31"
3131
// {"ymd", "20240231", "%Y%m%d", nil},
3232
}
3333

3434
for _, tt := range testCases {
35-
f, err := NewStrToDate(
35+
f := NewStrToDate(
3636
expression.NewGetField(0, types.Text, "", true),
3737
expression.NewGetField(1, types.Text, "", true),
3838
)
39-
if err != nil {
40-
t.Fatal(err)
41-
}
4239
t.Run(tt.name, func(t *testing.T) {
4340
dtime := eval(t, f, sql.NewRow(tt.dateStr, tt.fmtStr))
4441
require.Equal(t, tt.expected, dtime)
@@ -60,13 +57,10 @@ func TestStrToDateFailure(t *testing.T) {
6057
}
6158

6259
for _, tt := range testCases {
63-
f, err := NewStrToDate(
60+
f := NewStrToDate(
6461
expression.NewGetField(0, types.Text, "", true),
6562
expression.NewGetField(1, types.Text, "", true),
6663
)
67-
if err != nil {
68-
t.Fatal(err)
69-
}
7064
t.Run(tt.name, func(t *testing.T) {
7165
dtime := eval(t, f, sql.NewRow(tt.dateStr, tt.fmtStr))
7266
require.Equal(t, nil, dtime)

0 commit comments

Comments
 (0)