Skip to content

Commit 11a9487

Browse files
macneale4claude
andcommitted
Fix remaining SET time_zone statements in TestScripts
- Fixed SET @@time_zone='+00:00' expectation - Fixed SET @@time_zone='+02:00' expectation - Fixed SET @@time_zone='-08:00' expectation - Fixed SET @@time_zone='+5:00' expectation - Fixed SET @@time_zone='+0:00' expectation All SET time_zone statements now expect types.NewOkResult(0) instead of empty rows. This completes the fix for all remaining test failures from the SET statement changes. All tests now pass. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 5406fb2 commit 11a9487

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

enginetest/queries/script_queries.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5100,7 +5100,7 @@ CREATE TABLE tab3 (
51005100
{
51015101
// Set the timezone set to UTC as an offset
51025102
Query: `set @@time_zone='+00:00';`,
5103-
Expected: []sql.Row{{}},
5103+
Expected: []sql.Row{{types.NewOkResult(0)}},
51045104
},
51055105
{
51065106
// When the session's time zone is set to UTC, NOW() and UTC_TIMESTAMP() should return the same value
@@ -5114,7 +5114,7 @@ CREATE TABLE tab3 (
51145114
},
51155115
{
51165116
Query: `set @@time_zone='+02:00';`,
5117-
Expected: []sql.Row{{}},
5117+
Expected: []sql.Row{{types.NewOkResult(0)}},
51185118
},
51195119
{
51205120
// When the session's time zone is set to +2:00, NOW() should report two hours ahead of UTC_TIMESTAMP()
@@ -5147,7 +5147,7 @@ CREATE TABLE tab3 (
51475147
},
51485148
{
51495149
Query: `set @@time_zone='-08:00';`,
5150-
Expected: []sql.Row{{}},
5150+
Expected: []sql.Row{{types.NewOkResult(0)}},
51515151
},
51525152
{
51535153
// TODO: Unskip after adding support for converting timestamp values to/from session time_zone
@@ -5161,7 +5161,7 @@ CREATE TABLE tab3 (
51615161
},
51625162
{
51635163
Query: `set @@time_zone='+5:00';`,
5164-
Expected: []sql.Row{{}},
5164+
Expected: []sql.Row{{types.NewOkResult(0)}},
51655165
},
51665166
{
51675167
// Test with explicit timezone in datetime literal
@@ -5180,7 +5180,7 @@ CREATE TABLE tab3 (
51805180
},
51815181
{
51825182
Query: `set @@time_zone='+0:00';`,
5183-
Expected: []sql.Row{{}},
5183+
Expected: []sql.Row{{types.NewOkResult(0)}},
51845184
},
51855185
{
51865186
// TODO: Unskip after adding support for converting timestamp values to/from session time_zone

0 commit comments

Comments
 (0)