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 <[email protected]> Authored: Tue Jun 10 22:18:44 2014 -0700 Committer: Sravya Tirukkovalur <[email protected]> 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()); + } }
