Add TODO comment for enum empty string test MySQL compatibility

elianddb · claude · elianddb · commit ead1e1543384 · 2025-07-10T23:17:56.000Z
Need to fix error type to match MySQL exactly - should return ErrInvalidColumnDefaultValue instead of ErrIncompatibleDefaultType. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/enginetest/queries/script_queries.go b/enginetest/queries/script_queries.go
@@ -8809,7 +8809,7 @@ where
 	},
 	{
 		// This is with STRICT_TRANS_TABLES or STRICT_ALL_TABLES in sql_mode
-		Skip:    true,
+		Skip:    true, // TODO: Fix error type to match MySQL exactly (should be ErrInvalidColumnDefaultValue)
 		Name:    "enums with empty string",
 		Dialect: "mysql",
 		SetUpScript: []string{

Original file line number	Diff line number	Diff line change
`@@ -8809,7 +8809,7 @@ where`
`8809`	`8809`	`},`
`8810`	`8810`	`{`
`8811`	`8811`	`// This is with STRICT_TRANS_TABLES or STRICT_ALL_TABLES in sql_mode`
`8812`		`- Skip: true,`
	`8812`	`+ Skip: true, // TODO: Fix error type to match MySQL exactly (should be ErrInvalidColumnDefaultValue)`
`8813`	`8813`	`Name: "enums with empty string",`
`8814`	`8814`	`Dialect: "mysql",`
`8815`	`8815`	`SetUpScript: []string{`