From 058d3ee08427fd4c57c088b92124d245c1dd85d5 Mon Sep 17 00:00:00 2001 From: Angerszhuuuu Date: Tue, 31 Oct 2023 17:09:08 +0800 Subject: [PATCH] [KYUUBI #5576][Bug] Fix wrong code in test case of dir command ### _Why are the changes needed?_ To close #5575 Fix wrong code in test case of dir command ### _How was this patch tested?_ - [x] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [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 #5577 from AngersZhuuuu/KYUUBI-5576. Closes #5576 60e2cb817 [Angerszhuuuu] [KYUUBI #5576][Bug] Fix wrong code in test case of dir command Authored-by: Angerszhuuuu Signed-off-by: Cheng Pan --- .../kyuubi/plugin/spark/authz/PrivilegesBuilderSuite.scala | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilderSuite.scala b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilderSuite.scala index 54b91eb2837..6967774418b 100644 --- a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilderSuite.scala +++ b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilderSuite.scala @@ -1450,7 +1450,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite { val directory = File(tableDirectory).createDirectory() val plan = sql( s""" - |INSERT OVERWRITE DIRECTORY '$directory.path' + |INSERT OVERWRITE DIRECTORY '${directory.path}' |USING parquet |SELECT * FROM $reusedPartTable""".stripMargin) .queryExecution.analyzed @@ -1574,7 +1574,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite { val directory = File(tableDirectory).createDirectory() val plan = sql( s""" - |INSERT OVERWRITE DIRECTORY '$directory.path' + |INSERT OVERWRITE DIRECTORY '${directory.path}' |USING parquet |SELECT * FROM $reusedPartTable""".stripMargin) .queryExecution.analyzed @@ -1599,7 +1599,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite { val directory = File(tableDirectory).createDirectory() val plan = sql( s""" - |INSERT OVERWRITE DIRECTORY '$directory.path' + |INSERT OVERWRITE DIRECTORY '${directory.path}' |ROW FORMAT DELIMITED FIELDS TERMINATED BY ',' |SELECT * FROM $reusedPartTable""".stripMargin) .queryExecution.analyzed