(spark) branch master updated: [SPARK-48358][SQL][TESTS][FOLLOWUP] Use `condition` instead of `errorClass` in `SqlScriptingInterpreterSuite`

[maxgekk]

This is an automated email from the ASF dual-hosted git repository.

maxgekk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 50955cb934ff [SPARK-48358][SQL][TESTS][FOLLOWUP] Use `condition` 
instead of `errorClass` in `SqlScriptingInterpreterSuite`
50955cb934ff is described below

commit 50955cb934ff3ac8d970f15b6e3e69f598235b08
Author: Max Gekk <max.g...@gmail.com>
AuthorDate: Wed Sep 11 11:27:01 2024 +0200

    [SPARK-48358][SQL][TESTS][FOLLOWUP] Use `condition` instead of `errorClass` 
in `SqlScriptingInterpreterSuite`
    
    ### What changes were proposed in this pull request?
    In the PR, I propose to replace `errorClass` by `condition` in 
`SqlScriptingInterpreterSuite`
    
    ### Why are the changes needed?
    The changes from the PR https://github.com/apache/spark/pull/47756 conflict 
to https://github.com/apache/spark/pull/48027
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    By running the modified test:
    ```
    $ build/sbt "test:testOnly *SqlScriptingInterpreterSuite"
    ```
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No.
    
    Closes #48072 from MaxGekk/fix-errorClass-REPEAT.
    
    Authored-by: Max Gekk <max.g...@gmail.com>
    Signed-off-by: Max Gekk <max.g...@gmail.com>
---
 .../apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
 
b/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
index b703e77d4d73..8d9cd1d8c780 100644
--- 
a/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
+++ 
b/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
@@ -670,7 +670,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with 
SharedSparkSession {
       exception = intercept[SqlScriptingException] (
         runSqlScript(commands)
       ),
-      errorClass = "INVALID_BOOLEAN_STATEMENT",
+      condition = "INVALID_BOOLEAN_STATEMENT",
       parameters = Map("invalidStatement" -> "1")
     )
   }
@@ -693,7 +693,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with 
SharedSparkSession {
         exception = intercept[SqlScriptingException] (
           runSqlScript(commands)
         ),
-        errorClass = "BOOLEAN_STATEMENT_WITH_EMPTY_ROW",
+        condition = "BOOLEAN_STATEMENT_WITH_EMPTY_ROW",
         parameters = Map("invalidStatement" -> "(SELECT * FROM T)")
       )
     }
@@ -719,7 +719,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with 
SharedSparkSession {
         exception = intercept[SparkException] (
           runSqlScript(commands)
         ),
-        errorClass = "SCALAR_SUBQUERY_TOO_MANY_ROWS",
+        condition = "SCALAR_SUBQUERY_TOO_MANY_ROWS",
         parameters = Map.empty,
         context = ExpectedContext(fragment = "(SELECT * FROM t)", start = 141, 
stop = 157)
       )


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org