Address PR review feedback from jycor

- Use SqlMode struct instead of manual string parsing in isStrictMode
- Leverage LoadSqlMode() and ModeEnabled() methods for proper SQL mode detection
- Unskip enum default validation tests that now work with our implementation
- Update expected error types to ErrInvalidColumnDefaultValue for enum defaults

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: elianddb

enginetest/queries/script_queries.go

@@ -8465,23 +8465,20 @@ where
 				ExpectedErr: sql.ErrIncompatibleDefaultType,
 			},
 			{
-				Skip:        true,
 				Query:       "create table bad (e enum('a') default 0);",
-				ExpectedErr: sql.ErrIncompatibleDefaultType,
+				ExpectedErr: sql.ErrInvalidColumnDefaultValue,
 			},
 			{
 				Query:       "create table bad (e enum('a') default '');",
 				ExpectedErr: sql.ErrIncompatibleDefaultType,
 			},
 			{
-				Skip:        true,
 				Query:       "create table bad (e enum('a') default '1');",
-				ExpectedErr: sql.ErrIncompatibleDefaultType,
+				ExpectedErr: sql.ErrInvalidColumnDefaultValue,
 			},
 			{
-				Skip:        true,
 				Query:       "create table bad (e enum('a') default 1);",
-				ExpectedErr: sql.ErrIncompatibleDefaultType,
+				ExpectedErr: sql.ErrInvalidColumnDefaultValue,
 			},
 
 			{

sql/types/enum.go

@@ -219,12 +219,8 @@ func (t EnumType) Convert(ctx context.Context, v interface{}) (interface{}, sql.
 // isStrictMode checks if STRICT_TRANS_TABLES or STRICT_ALL_TABLES is enabled
 func (t EnumType) isStrictMode(ctx context.Context) bool {
 	if sqlCtx, ok := ctx.(*sql.Context); ok {
-		sysVal, err := sqlCtx.GetSessionVariable(sqlCtx, "sql_mode")
-		if err == nil {
-			if sqlMode, ok := sysVal.(string); ok {
-				return strings.Contains(sqlMode, "STRICT_TRANS_TABLES") || strings.Contains(sqlMode, "STRICT_ALL_TABLES")
-			}
-		}
+		sqlMode := sql.LoadSqlMode(sqlCtx)
+		return sqlMode.ModeEnabled("STRICT_TRANS_TABLES") || sqlMode.ModeEnabled("STRICT_ALL_TABLES")
 	}
 	return false
 }