Fix INSERT IGNORE enum test expectations

elianddb · elianddb · commit 17e8eb1276b9 · 2025-07-08T17:42:32.000Z
- Update test to expect ErrDataTruncatedForColumn instead of ErrConvertingToEnum
- Removes TODO comment as the fix is now implemented
- Matches MySQL behavior for enum validation error messages
diff --git a/enginetest/queries/insert_queries.go b/enginetest/queries/insert_queries.go
@@ -2836,7 +2836,7 @@ var InsertIgnoreScripts = []ScriptTest{
 		Assertions: []ScriptTestAssertion{
 			{
 				Query:       "insert into test_table values (1, 'invalid'), (2, 'comparative politics'), (3, null)",
-				ExpectedErr: types.ErrConvertingToEnum, // TODO: should be ErrDataTruncatedForColumn
+				ExpectedErr: types.ErrDataTruncatedForColumn,
 			},
 			{
 				Query:    "insert ignore into test_table values (1, 'invalid'), (2, 'bye'), (3, null)",

Original file line number	Diff line number	Diff line change
`@@ -2836,7 +2836,7 @@ var InsertIgnoreScripts = []ScriptTest{`
`2836`	`2836`	`Assertions: []ScriptTestAssertion{`
`2837`	`2837`	`{`
`2838`	`2838`	`Query: "insert into test_table values (1, 'invalid'), (2, 'comparative politics'), (3, null)",`
`2839`		`- ExpectedErr: types.ErrConvertingToEnum, // TODO: should be ErrDataTruncatedForColumn`
	`2839`	`+ ExpectedErr: types.ErrDataTruncatedForColumn,`
`2840`	`2840`	`},`
`2841`	`2841`	`{`
`2842`	`2842`	`Query: "insert ignore into test_table values (1, 'invalid'), (2, 'bye'), (3, null)",`