(kyuubi) branch master updated: [KYUUBI #5642][FOLLOWUP] Fix unit test

[chengpan]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 88d0d1bad [KYUUBI #5642][FOLLOWUP] Fix unit test
88d0d1bad is described below

commit 88d0d1bad35c075920f53ea05e6d8a3869f616a0
Author: Angerszhuuuu <angers....@gmail.com>
AuthorDate: Thu Nov 9 14:28:19 2023 +0800

    [KYUUBI #5642][FOLLOWUP] Fix unit test
    
    ### _Why are the changes needed?_
    To close 5642
     Fix unit test
    Spark 3.1
    ```
    - CreateDatabaseCommand/AlterDatabaseSetLocationCommand *** FAILED ***
      "Hive [2.3.7 does not support altering database location]" did not equal 
"Hive [metastore does not support altering database location.]" 
(RangerSparkExtensionSuite.scala:1194)
      Analysis:
      "Hive [2.3.7 does not support altering database location]" -> "Hive 
[metastore does not support altering database location.]"
    ```
    
    Spark 3.2 & Spark 3.3
    ```
    - CreateDatabaseCommand/AlterDatabaseSetLocationCommand *** FAILED ***
      "Hive [2.3.9 does not support altering database location]" did not equal 
"Hive [metastore does not support altering database location.]" 
(RangerSparkExtensionSuite.scala:1194)
      Analysis:
      "Hive [2.3.9 does not support altering database location]" -> "Hive 
[metastore does not support altering database location.]"
    ```
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [x] [Run 
test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests)
 locally before make a pull request
    
    ### _Was this patch authored or co-authored using generative AI tooling?_
    No
    
    Closes #5659 from AngersZhuuuu/KYUUBI-5642-FOLLOWUP.
    
    Closes #5642
    
    d1c1a081a [Angerszhuuuu] Update RangerSparkExtensionSuite.scala
    f70597b4e [Angerszhuuuu] [KYUUBI #5642][FOLLOWUP] Fix unit test
    
    Authored-by: Angerszhuuuu <angers....@gmail.com>
    Signed-off-by: Cheng Pan <cheng...@apache.org>
---
 .../kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala
 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala
index 98c513c13..e8e4b0220 100644
--- 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala
+++ 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala
@@ -1191,8 +1191,7 @@ class HiveCatalogRangerSparkExtensionSuite extends 
RangerSparkExtensionSuite {
                 s"[write] privilege on [[$path2, $path2/]]")
             val e = intercept[UndeclaredThrowableException](
               doAs(admin, sql(s"ALTER DATABASE $db1 SET LOCATION '$path2'")))
-            assert(e.getCause.getMessage ==
-              "Hive metastore does not support altering database location.")
+            assert(e.getCause.getMessage.contains("does not support altering 
database location"))
           }
         }
       }