Repository: incubator-sentry
Updated Branches:
  refs/heads/master 439d5f29a -> a83c094f2


SENTRY-958: TestGrantPrivilege fails on JDK8 (Colm O hEigeartaigh via Lenni 
Kuff)

Change-Id: I73bec3c587bb8520d370e1519fb3cb6e5f9a6523


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/a83c094f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/a83c094f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/a83c094f

Branch: refs/heads/master
Commit: a83c094f2025928cd59cfc53a3b11a3123606154
Parents: 439d5f2
Author: Lenni Kuff <lsk...@cloudera.com>
Authored: Thu Nov 19 22:03:14 2015 -0800
Committer: Lenni Kuff <lsk...@cloudera.com>
Committed: Thu Nov 19 22:03:14 2015 -0800

----------------------------------------------------------------------
 .../org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/a83c094f/sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java
----------------------------------------------------------------------
diff --git 
a/sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java
 
b/sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java
index bc9dd13..8c7753e 100644
--- 
a/sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java
+++ 
b/sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/TestGrantPrivilege.java
@@ -194,7 +194,9 @@ public class TestGrantPrivilege extends 
AbstractSqoopSentryTestBase {
     for (MPrivilege privilege : client.getPrivilegesByPrincipal(role4Princ, 
allConnector)) {
       actions.add(privilege.getAction().toLowerCase());
     }
-    assertEquals(Lists.newArrayList(SqoopActionConstant.READ, 
SqoopActionConstant.WRITE), actions);
+    assertEquals(2, actions.size());
+    assertTrue(actions.contains(SqoopActionConstant.READ));
+    assertTrue(actions.contains(SqoopActionConstant.WRITE));
 
     /**
      * admin user grant all privilege on connector all to role role4

Reply via email to