Fix tests

imback82 · imback82 · commit 0c83e37a9fc9 · 2021-07-03T09:13:49.000-07:00
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
@@ -2305,7 +2305,8 @@ abstract class DDLSuite extends QueryTest with SQLTestUtils {
       val e = intercept[AnalysisException] {
         sql("ALTER TABLE tmp_v ADD COLUMNS (c3 INT)")
       }
-      assert(e.message.contains("'tmp_v' is a view not a table"))
+      assert(e.message.contains(
+        "tmp_v is a temp view. 'ALTER TABLE ... ADD COLUMNS' expects a table."))
     }
   }
 
@@ -2315,7 +2316,8 @@ abstract class DDLSuite extends QueryTest with SQLTestUtils {
       val e = intercept[AnalysisException] {
         sql("ALTER TABLE v1 ADD COLUMNS (c3 INT)")
       }
-      assert(e.message.contains("ALTER ADD COLUMNS does not support views"))
+      assert(e.message.contains(
+        "default.v1 is a view. 'ALTER TABLE ... ADD COLUMNS' expects a table."))
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -2305,7 +2305,8 @@ abstract class DDLSuite extends QueryTest with SQLTestUtils {`
`2305`	`2305`	`val e = intercept[AnalysisException] {`
`2306`	`2306`	`sql("ALTER TABLE tmp_v ADD COLUMNS (c3 INT)")`
`2307`	`2307`	`}`
`2308`		`- assert(e.message.contains("'tmp_v' is a view not a table"))`
	`2308`	`+ assert(e.message.contains(`
	`2309`	`+ "tmp_v is a temp view. 'ALTER TABLE ... ADD COLUMNS' expects a table."))`
`2309`	`2310`	`}`
`2310`	`2311`	`}`
`2311`	`2312`
`@@ -2315,7 +2316,8 @@ abstract class DDLSuite extends QueryTest with SQLTestUtils {`
`2315`	`2316`	`val e = intercept[AnalysisException] {`
`2316`	`2317`	`sql("ALTER TABLE v1 ADD COLUMNS (c3 INT)")`
`2317`	`2318`	`}`
`2318`		`- assert(e.message.contains("ALTER ADD COLUMNS does not support views"))`
	`2319`	`+ assert(e.message.contains(`
	`2320`	`+ "default.v1 is a view. 'ALTER TABLE ... ADD COLUMNS' expects a table."))`
`2319`	`2321`	`}`
`2320`	`2322`	`}`
`2321`	`2323`