You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by sr...@apache.org on 2014/06/11 07:19:23 UTC
git commit: SENTRY-181: Add a test case for duplicate privileges (
Jarek Jarcec Cecho via Sravya Tirukkovalur)
Repository: incubator-sentry
Updated Branches:
refs/heads/master af2368461 -> 439cd651a
SENTRY-181: Add a test case for duplicate privileges ( Jarek Jarcec Cecho via Sravya Tirukkovalur)
Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/439cd651
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/439cd651
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/439cd651
Branch: refs/heads/master
Commit: 439cd651ac83ae90f7ee0f45027254e9f90c18b2
Parents: af23684
Author: Sravya Tirukkovalur <sr...@clouera.com>
Authored: Tue Jun 10 22:18:44 2014 -0700
Committer: Sravya Tirukkovalur <sr...@clouera.com>
Committed: Tue Jun 10 22:18:44 2014 -0700
----------------------------------------------------------------------
.../service/thrift/TestSentryServiceIntegration.java | 15 +++++++++++++++
1 file changed, 15 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/439cd651/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
index f0bf127..3c500db 100644
--- a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
+++ b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
@@ -238,4 +238,19 @@ public class TestSentryServiceIntegration extends SentryServiceIntegrationBase {
client.grantTablePrivilege(requestorUserName, roleName, "server", "db1", "table1", "ALL");
assertEquals(1, client.listAllPrivilegesByRoleName(requestorUserName, roleName).size());
}
+
+ // See SENTRY-181
+ @Test
+ public void testSameGrantTwice() throws Exception {
+ String requestorUserName = ADMIN_USER;
+ Set<String> requestorUserGroupNames = Sets.newHashSet(ADMIN_GROUP);
+ setLocalGroupMapping(requestorUserName, requestorUserGroupNames);
+ writePolicyFile();
+ String roleName = "admin_r1";
+
+ client.createRole(requestorUserName, roleName);
+ client.grantTablePrivilege(requestorUserName, roleName, "server", "db1", "table1", "ALL");
+ client.grantTablePrivilege(requestorUserName, roleName, "server", "db1", "table1", "ALL");
+ assertEquals(1, client.listAllPrivilegesByRoleName(requestorUserName, roleName).size());
+ }
}