mv to SQLViewSuite

Author: ulysses-you

sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala

@@ -738,6 +738,21 @@ abstract class SQLViewSuite extends QueryTest with SQLTestUtils {
   }
 
   test("SPARK-32356: forbid null type in create view") {
+    // test sql
+    val sql1 = "create view v as select null as c"
+    val sql2 = "alter view v as select null as c"
+    val sql3 = "create temporary view v as select null as c"
+    val sql4 = "create global temporary view v as select null as c"
+    Seq(sql1, sql2, sql3, sql4).foreach { input =>
+      val msg = intercept[AnalysisException] {
+        sql(input)
+      }.getMessage
+      assert(msg.contains(s"Cannot create tables/views with ${NullType.simpleString} type."))
+    }
+  }
+
+  test("SPARK-32356: forbid null type in create view") {
+    // test df.createTempView
     val msg = intercept[AnalysisException] {
       sql("select null as c").createTempView("null_type_view")
     }.getMessage

sql/core/src/test/scala/org/apache/spark/sql/execution/command/PlanResolutionSuite.scala

@@ -38,7 +38,7 @@ import org.apache.spark.sql.execution.datasources.CreateTable
 import org.apache.spark.sql.execution.datasources.v2.DataSourceV2Relation
 import org.apache.spark.sql.internal.SQLConf
 import org.apache.spark.sql.sources.SimpleScanSource
-import org.apache.spark.sql.types.{CharType, DoubleType, HIVE_TYPE_STRING, IntegerType, LongType, MetadataBuilder, NullType, StringType, StructField, StructType}
+import org.apache.spark.sql.types.{CharType, DoubleType, HIVE_TYPE_STRING, IntegerType, LongType, MetadataBuilder, StringType, StructField, StructType}
 
 class PlanResolutionSuite extends AnalysisTest {
   import CatalystSqlParser._
@@ -1557,19 +1557,6 @@ class PlanResolutionSuite extends AnalysisTest {
     checkFailure("testcat.tab", "foo")
   }
 
-  test("SPARK-32356: forbid null type in create view") {
-    val sql1 = "create view v as select null as c"
-    val sql2 = "alter view v as select null as c"
-    val sql3 = "create temporary view v as select null as c"
-    val sql4 = "create global temporary view v as select null as c"
-    Seq(sql1, sql2, sql3, sql4).foreach { sql =>
-      val msg = intercept[AnalysisException] {
-        parseAndResolve(sql)
-      }.getMessage
-      assert(msg.contains(s"Cannot create tables/views with ${NullType.simpleString} type."))
-    }
-  }
-
   // TODO: add tests for more commands.
 }