You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by ja...@apache.org on 2014/06/11 07:39:16 UTC

git commit: SENTRY-291: Remove duplicate testSameGrantTwice

Repository: incubator-sentry
Updated Branches:
  refs/heads/master e8eb01203 -> ef624165b


SENTRY-291: Remove duplicate testSameGrantTwice

(Sravya Tirukkovalur via Jarek Jarcec Cecho)


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

Branch: refs/heads/master
Commit: ef624165b1b38d81b65b1df25da42486d7e7242f
Parents: e8eb012
Author: Jarek Jarcec Cecho <ja...@apache.org>
Authored: Tue Jun 10 22:37:03 2014 -0700
Committer: Jarek Jarcec Cecho <ja...@apache.org>
Committed: Tue Jun 10 22:37:03 2014 -0700

----------------------------------------------------------------------
 .../service/thrift/TestSentryServiceIntegration.java | 15 ---------------
 1 file changed, 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/ef624165/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 3c500db..f0bf127 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,19 +238,4 @@ 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());
-  }
 }