Skip to content

Commit 340ed0c

Browse files
committed
RANGER-4168: fixed NPE in recently added unit tests - #2
1 parent ce3c3b8 commit 340ed0c

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

security-admin/src/test/java/org/apache/ranger/biz/TestTagDBStore.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import org.apache.ranger.db.XXServiceResourceDao;
3232
import org.apache.ranger.db.XXServiceVersionInfoDao;
3333
import org.apache.ranger.db.XXTagDao;
34+
import org.apache.ranger.db.XXTagDefDao;
3435
import org.apache.ranger.db.XXTagResourceMapDao;
3536
import org.apache.ranger.entity.XXService;
3637
import org.apache.ranger.entity.XXServiceResource;
@@ -852,14 +853,18 @@ public void testGetTagResourceMaps() throws Exception {
852853
@Test
853854
public void testGetServiceTagsIfUpdated() throws Exception {
854855
XXServiceVersionInfo serviceVersionInfoDbObj = createXXServiceVersionInfo();
856+
XXTagDefDao xxTagDefDao = Mockito.mock(XXTagDefDao.class);
855857
XXServiceVersionInfoDao xxServiceVersionInfoDao = Mockito.mock(XXServiceVersionInfoDao.class);
856858
XXServiceDao xxServiceDao = Mockito.mock(XXServiceDao.class);
859+
XXServiceResourceDao xxServiceResourceDao = Mockito.mock(XXServiceResourceDao.class);
857860
XXService xxService = createXXService();
858861
RangerServiceDef rangerServiceDef = createRangerServiceDef();
859862

860863
Mockito.when(daoManager.getXXServiceVersionInfo()).thenReturn(xxServiceVersionInfoDao);
861864
Mockito.when(xxServiceVersionInfoDao.findByServiceName(serviceName)).thenReturn(serviceVersionInfoDbObj);
865+
Mockito.when(daoManager.getXXTagDef()).thenReturn(xxTagDefDao);
862866
Mockito.when(daoManager.getXXService()).thenReturn(xxServiceDao);
867+
Mockito.when(daoManager.getXXServiceResource()).thenReturn(xxServiceResourceDao);
863868
Mockito.when(xxServiceDao.findIdByName(serviceName)).thenReturn(id);
864869
Mockito.when(xxServiceDao.findByName(serviceName)).thenReturn(xxService);
865870
Mockito.when(svcStore.getServiceDef(Mockito.any())).thenReturn(rangerServiceDef);
@@ -874,14 +879,18 @@ public void testGetServiceTagsIfUpdated() throws Exception {
874879
@Test
875880
public void testGetServiceTags() throws Exception {
876881
XXServiceVersionInfo serviceVersionInfoDbObj = createXXServiceVersionInfo();
882+
XXTagDefDao xxTagDefDao = Mockito.mock(XXTagDefDao.class);
877883
XXServiceVersionInfoDao xxServiceVersionInfoDao = Mockito.mock(XXServiceVersionInfoDao.class);
878884
XXServiceDao xxServiceDao = Mockito.mock(XXServiceDao.class);
885+
XXServiceResourceDao xxServiceResourceDao = Mockito.mock(XXServiceResourceDao.class);
879886
XXService xxService = createXXService();
880887
RangerServiceDef rangerServiceDef = createRangerServiceDef();
881888

882889
Mockito.when(daoManager.getXXServiceVersionInfo()).thenReturn(xxServiceVersionInfoDao);
883890
Mockito.when(xxServiceVersionInfoDao.findByServiceName(serviceName)).thenReturn(serviceVersionInfoDbObj);
891+
Mockito.when(daoManager.getXXTagDef()).thenReturn(xxTagDefDao);
884892
Mockito.when(daoManager.getXXService()).thenReturn(xxServiceDao);
893+
Mockito.when(daoManager.getXXServiceResource()).thenReturn(xxServiceResourceDao);
885894
Mockito.when(xxServiceDao.findByName(serviceName)).thenReturn(xxService);
886895
Mockito.when(svcStore.getServiceDef(Mockito.any())).thenReturn(rangerServiceDef);
887896

0 commit comments

Comments
 (0)