add TODO and mv test

Author: elianddb

enginetest/queries/script_queries.go

@@ -121,52 +121,6 @@ type ScriptTestAssertion struct {
 // Unlike other engine tests, ScriptTests must be self-contained. No other tables are created outside the definition of
 // the tests.
 var ScriptTests = []ScriptTest{
-	{
-		// https://github.com/dolthub/dolt/pull/9830
-		Name: "CREATE SCHEMA without database selection falls back to CREATE DATABASE",
-		SetUpScript: []string{
-			"CREATE DATABASE tmp",
-			"USE tmp",
-		},
-		Dialect: "mysql",
-		Assertions: []ScriptTestAssertion{
-			{
-				Query: "DROP DATABASE tmp",
-			},
-			{
-				Query:    "CREATE SCHEMA NewDatabase",
-				Expected: []sql.Row{{types.OkResult{RowsAffected: 1}}},
-			},
-			{
-				Query:    "SHOW DATABASES",
-				Expected: []sql.Row{{"NewDatabase"}, {"information_schema"}, {"mydb"}, {"mysql"}},
-			},
-			{
-				Query:    "USE NewDatabase",
-				Expected: []sql.Row{},
-			},
-			{
-				Query:    "SELECT DATABASE()",
-				Expected: []sql.Row{{"NewDatabase"}},
-			},
-			{
-				Query:    "CREATE TABLE test_table (id INT PRIMARY KEY)",
-				Expected: []sql.Row{{types.OkResult{RowsAffected: 0}}},
-			},
-			{
-				Query:    "SHOW TABLES",
-				Expected: []sql.Row{{"test_table"}},
-			},
-			{
-				Query:    "USE mydb",
-				Expected: []sql.Row{},
-			},
-			{
-				Query:    "DROP DATABASE NewDatabase",
-				Expected: []sql.Row{{types.OkResult{RowsAffected: 1}}},
-			},
-		},
-	},
 	{
 		// https://github.com/dolthub/go-mysql-server/issues/3216
 		Name: "UNION ALL with BLOB columns",
@@ -13565,6 +13519,52 @@ var BrokenScriptTests = []ScriptTest{
 }
 
 var CreateDatabaseScripts = []ScriptTest{
+	{
+		// https://github.com/dolthub/dolt/pull/9830
+		Name: "CREATE SCHEMA without database selection falls back to CREATE DATABASE",
+		SetUpScript: []string{
+			"CREATE DATABASE tmp",
+			"USE tmp",
+		},
+		Dialect: "mysql",
+		Assertions: []ScriptTestAssertion{
+			{
+				Query: "DROP DATABASE tmp",
+			},
+			{
+				Query:    "CREATE SCHEMA NewDatabase",
+				Expected: []sql.Row{{types.OkResult{RowsAffected: 1}}},
+			},
+			{
+				Query:    "SHOW DATABASES",
+				Expected: []sql.Row{{"NewDatabase"}, {"information_schema"}, {"mydb"}, {"mysql"}},
+			},
+			{
+				Query:    "USE NewDatabase",
+				Expected: []sql.Row{},
+			},
+			{
+				Query:    "SELECT DATABASE()",
+				Expected: []sql.Row{{"NewDatabase"}},
+			},
+			{
+				Query:    "CREATE TABLE test_table (id INT PRIMARY KEY)",
+				Expected: []sql.Row{{types.OkResult{RowsAffected: 0}}},
+			},
+			{
+				Query:    "SHOW TABLES",
+				Expected: []sql.Row{{"test_table"}},
+			},
+			{
+				Query:    "USE mydb",
+				Expected: []sql.Row{},
+			},
+			{
+				Query:    "DROP DATABASE NewDatabase",
+				Expected: []sql.Row{{types.OkResult{RowsAffected: 1}}},
+			},
+		},
+	},
 	{
 		Name: "CREATE DATABASE and create table",
 		Assertions: []ScriptTestAssertion{

sql/rowexec/ddl.go

@@ -409,6 +409,7 @@ func (b *BaseBuilder) buildCreateSchema(ctx *sql.Context, n *plan.CreateSchema,
 	// If no database is selected, first try to fall back to CREATE DATABASE
 	// since CREATE SCHEMA is a synonym for CREATE DATABASE in MySQL
 	// https://dev.mysql.com/doc/refman/8.4/en/create-database.html
+	// TODO: For PostgreSQL, return an error if no database is selected.
 	if database == "" {
 		return b.buildCreateDB(ctx, &plan.CreateDB{
 			Catalog:     n.Catalog,