pradeepagrawal8184 commented on code in PR #768:
URL: https://github.com/apache/ranger/pull/768#discussion_r2621794098
##########
hbase-agent/src/test/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessorTest.java:
##########
@@ -517,95 +522,100 @@ public void
test27_preGetOp_commandStringIncludesFamiliesRowsAndFilter() throws
@Test
public void test28_masterOperations_cover() throws Exception {
- RangerAuthorizationCoprocessor cp = spy(new
RangerAuthorizationCoprocessor());
- ObserverContext<MasterCoprocessorEnvironment> mctx =
mock(ObserverContext.class);
- TableDescriptor td = mock(TableDescriptor.class);
- RegionInfo regionInfo = mock(RegionInfo.class);
- TableName tname = TableName.valueOf("t");
- when(td.getTableName()).thenReturn(tname);
- when(regionInfo.getTable()).thenReturn(tname);
- doReturn(null).when(cp).requirePermission(any(), anyString(),
any(byte[].class), any());
- doReturn(null).when(cp).requirePermission(any(), anyString(), any(),
any(), any(), any());
- doReturn(null).when(cp).requireGlobalPermission(any(), anyString(),
anyString(), any());
- cp.preCreateTable(mctx, td, new RegionInfo[0]);
- cp.preDeleteTable(mctx, tname);
- cp.preModifyTable(mctx, tname, td);
- cp.preEnableTable(mctx, tname);
- cp.preDisableTable(mctx, tname);
- cp.preAbortProcedure(mctx, 1L);
- cp.postGetProcedures(mctx);
- cp.preMove(mctx, regionInfo, mock(ServerName.class),
mock(ServerName.class));
- cp.preAssign(mctx, regionInfo);
- cp.preUnassign(mctx, regionInfo, false);
- cp.preRegionOffline(mctx, regionInfo);
- cp.preBalance(mctx, mock(BalanceRequest.class));
- cp.preBalanceSwitch(mctx, true);
- cp.preShutdown(mctx);
- cp.preStopMaster(mctx);
- cp.preSnapshot(mctx, mock(SnapshotDescription.class), td);
- cp.preCloneSnapshot(mctx, mock(SnapshotDescription.class), td);
- cp.preRestoreSnapshot(mctx, mock(SnapshotDescription.class), td);
- cp.preDeleteSnapshot(mctx, mock(SnapshotDescription.class));
- cp.postGetTableDescriptors(mctx, new ArrayList<TableName>(), new
ArrayList<TableDescriptor>(), "rgx");
- cp.postGetTableNames(mctx, new ArrayList<TableDescriptor>(), "rgx");
- cp.preCreateNamespace(mctx, mock(NamespaceDescriptor.class));
- cp.preDeleteNamespace(mctx, "ns");
- cp.preModifyNamespace(mctx, mock(NamespaceDescriptor.class));
- cp.postListNamespaceDescriptors(mctx, new
ArrayList<NamespaceDescriptor>());
- cp.preSetUserQuota(mctx, "u", mock(GlobalQuotaSettings.class));
- cp.preSetUserQuota(mctx, "u", tname, mock(GlobalQuotaSettings.class));
- cp.preSetUserQuota(mctx, "u", "ns", mock(GlobalQuotaSettings.class));
- cp.preSetTableQuota(mctx, tname, mock(GlobalQuotaSettings.class));
- cp.preSetNamespaceQuota(mctx, "ns", mock(GlobalQuotaSettings.class));
- Assertions.assertTrue(true);
+ try {
Review Comment:
revert the changes as it was done due to version upgrade of junit and
maven-surefire-plugin
##########
hbase-agent/src/test/java/org/apache/ranger/services/hbase/client/TestHBaseResourceMgr.java:
##########
@@ -88,7 +88,7 @@ public void test02_getHBaseResource_tableFlow() throws
Exception {
HBaseClient client = Mockito.mock(HBaseClient.class);
Mockito.when(mgrMock.getHBaseConnection(Mockito.eq("svc"),
Mockito.eq("type"), Mockito.any()))
.thenReturn(client);
-
Mockito.when(client.getTableList(Mockito.eq("tab.*".replaceAll("\\*", ".\\*")),
+ Mockito.when(client.getTableList(Mockito.eq("tab.*"),
Review Comment:
revert the changes as it was done due to version upgrade of junit and
maven-surefire-plugin
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
