Skip to content

Commit 8bacb5e

Browse files
sameeragarwalhvanhovell
authored andcommitted
[SC-5462] Fix tests in AclExtensionsSuite/ReflectionBackedAclClientSuite
## What changes were proposed in this pull request? Fixes `AclExtensionsSuite` and `ReflectionBackedAclClientSuite` ## How was this patch tested? Unit Tests Author: Sameer Agarwal <[email protected]> Closes apache#154 from sameeragarwal/fix-tests.
1 parent 00294fc commit 8bacb5e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

sql/core/src/test/scala/com/databricks/sql/acl/ReflectionBackedAclClientSuite.scala

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ class ReflectionBackedAclClientSuite extends SparkFunSuite with SharedSQLContext
3939
sparkContextToken("usr1")
4040
driverToken(null)
4141
val input = Seq((tbl1, Select), (tbl1, Modify))
42-
assert(input === client.getValidPermissions(input))
42+
assert(input.toSet === client.getValidPermissions(input))
4343
assert(AclClientBackend.token === "usr1")
4444
assert(AclClientBackend.lastCommandArguments === Seq(
4545
("/CATALOG/`default`/DATABASE/`db1`/TABLE/`tbl_x`", "SELECT"),
@@ -134,15 +134,15 @@ class AclClientBackend {
134134

135135
def getValidPermissions(
136136
token: String,
137-
requests: Traversable[(String, String)]): Set[(String, String)] = {
137+
requests: Seq[(String, String)]): Set[(String, String)] = {
138138
AclClientBackend.token = token
139139
AclClientBackend.lastCommandArguments = requests.toSeq
140140
requests.toSet
141141
}
142142

143143
def getOwners(
144144
token: String,
145-
securables: Traversable[String]): Map[String, String] = {
145+
securables: Seq[String]): Map[String, String] = {
146146
AclClientBackend.token = token
147147
AclClientBackend.lastCommandArguments = securables.toSeq
148148
securables.map(_ -> "USER_1").toMap

0 commit comments

Comments
 (0)