Skip to content

Commit

Permalink
[no-release-notes] fix tests from insert id change (#8196)
Browse files Browse the repository at this point in the history
  • Loading branch information
jycor authored Aug 2, 2024
1 parent 2dbb49c commit c290ade
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion go/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ require (
github.com/cespare/xxhash/v2 v2.2.0
github.com/creasty/defaults v1.6.0
github.com/dolthub/flatbuffers/v23 v23.3.3-dh.2
github.com/dolthub/go-mysql-server v0.18.2-0.20240801201951-0a543f7cabef
github.com/dolthub/go-mysql-server v0.18.2-0.20240802195141-c31af05771a8
github.com/dolthub/gozstd v0.0.0-20240423170813-23a2903bca63
github.com/dolthub/swiss v0.1.0
github.com/goccy/go-json v0.10.2
Expand Down
4 changes: 2 additions & 2 deletions go/go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -185,8 +185,8 @@ github.com/dolthub/fslock v0.0.3 h1:iLMpUIvJKMKm92+N1fmHVdxJP5NdyDK5bK7z7Ba2s2U=
github.com/dolthub/fslock v0.0.3/go.mod h1:QWql+P17oAAMLnL4HGB5tiovtDuAjdDTPbuqx7bYfa0=
github.com/dolthub/go-icu-regex v0.0.0-20230524105445-af7e7991c97e h1:kPsT4a47cw1+y/N5SSCkma7FhAPw7KeGmD6c9PBZW9Y=
github.com/dolthub/go-icu-regex v0.0.0-20230524105445-af7e7991c97e/go.mod h1:KPUcpx070QOfJK1gNe0zx4pA5sicIK1GMikIGLKC168=
github.com/dolthub/go-mysql-server v0.18.2-0.20240801201951-0a543f7cabef h1:reJ/BdyoZdLtGxhnnizNJU/CXBPLlfVBs76ipsucT1I=
github.com/dolthub/go-mysql-server v0.18.2-0.20240801201951-0a543f7cabef/go.mod h1:IvXmiR+fvWQs27nNmp0XuTO7ze6j6ekrVkhWw+3qo2s=
github.com/dolthub/go-mysql-server v0.18.2-0.20240802195141-c31af05771a8 h1:Z/ZlQ/K+2Z4NWVu75YfpNxkCqL7+OvB+TKSVHHh719w=
github.com/dolthub/go-mysql-server v0.18.2-0.20240802195141-c31af05771a8/go.mod h1:IvXmiR+fvWQs27nNmp0XuTO7ze6j6ekrVkhWw+3qo2s=
github.com/dolthub/gozstd v0.0.0-20240423170813-23a2903bca63 h1:OAsXLAPL4du6tfbBgK0xXHZkOlos63RdKYS3Sgw/dfI=
github.com/dolthub/gozstd v0.0.0-20240423170813-23a2903bca63/go.mod h1:lV7lUeuDhH5thVGDCKXbatwKy2KW80L4rMT46n+Y2/Q=
github.com/dolthub/ishell v0.0.0-20240701202509-2b217167d718 h1:lT7hE5k+0nkBdj/1UOSFwjWpNxf+LCApbRHgnCA17XE=
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,7 @@ end
{
Query: "insert into t values (1, 1);",
Expected: []sql.Row{
{gmstypes.OkResult{RowsAffected: 1, InsertID: 1}},
{gmstypes.OkResult{RowsAffected: 1, InsertID: 4}},
},
},
{
Expand Down
4 changes: 2 additions & 2 deletions go/libraries/doltcore/sqle/enginetest/dolt_queries.go
Original file line number Diff line number Diff line change
Expand Up @@ -5272,7 +5272,7 @@ var DoltAutoIncrementTests = []queries.ScriptTest{
},
{
Query: "insert into t (a, b) values (100, 100)",
Expected: []sql.Row{{types.OkResult{RowsAffected: 1, InsertID: 100}}},
Expected: []sql.Row{{types.OkResult{RowsAffected: 1, InsertID: 5}}},
},
{
Query: "alter table t auto_increment = 50",
Expand Down Expand Up @@ -7227,7 +7227,7 @@ var DoltTempTableScripts = []queries.ScriptTest{
{
Query: "insert into t values (100), (1000)",
Expected: []sql.Row{
{types.OkResult{RowsAffected: 2, InsertID: 0x64}},
{types.OkResult{RowsAffected: 2, InsertID: 1}},
},
},
{
Expand Down
4 changes: 2 additions & 2 deletions go/libraries/doltcore/sqle/enginetest/dolt_queries_merge.go
Original file line number Diff line number Diff line change
Expand Up @@ -1684,7 +1684,7 @@ var MergeScripts = []queries.ScriptTest{
},
{
Query: "INSERT INTO t VALUES (3,3),(NULL,6);",
Expected: []sql.Row{{types.OkResult{RowsAffected: 2, InsertID: 3}}},
Expected: []sql.Row{{types.OkResult{RowsAffected: 2, InsertID: 6}}},
},
{
Query: "SELECT * FROM t ORDER BY pk;",
Expand Down Expand Up @@ -1720,7 +1720,7 @@ var MergeScripts = []queries.ScriptTest{
},
{
Query: "INSERT INTO t VALUES (3,3),(NULL,7);",
Expected: []sql.Row{{types.OkResult{RowsAffected: 2, InsertID: 3}}},
Expected: []sql.Row{{types.OkResult{RowsAffected: 2, InsertID: 7}}},
},
{
Query: "SELECT * FROM t ORDER BY pk;",
Expand Down

0 comments on commit c290ade

Please sign in to comment.