diff options
author | Kotta, Shireesha (sk434m) <sk434m@att.com> | 2019-06-28 15:27:29 -0400 |
---|---|---|
committer | Kotta, Shireesha (sk434m) <sk434m@att.com> | 2019-06-28 15:27:29 -0400 |
commit | 179ff1eb0c1ac9eef4d152c47df5cb12a4584c0f (patch) | |
tree | b9b744e106d688e807ffb31b6a986230034423d5 /ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java | |
parent | d63c87226df57e7bd0513f9b17374716197056fa (diff) |
PENTEST:Do not display stack trace for the api's
Issue-ID: PORTAL-654
PENTEST:Do not display stack trace for the api's and all users info for
get_user api
Change-Id: I68a4e3c7eba2628363275d63535290034591aa07
Signed-off-by: Kotta, Shireesha (sk434m) <sk434m@att.com>
Diffstat (limited to 'ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java')
-rw-r--r-- | ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java | 61 |
1 files changed, 33 insertions, 28 deletions
diff --git a/ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java b/ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java index a10572a2..9d5e4fea 100644 --- a/ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java +++ b/ecomp-sdk/epsdk-app-common/src/test/java/org/onap/portalapp/service/OnBoardingApiServiceImplTest.java @@ -39,6 +39,7 @@ package org.onap.portalapp.service; import java.io.IOException; import java.util.ArrayList; +import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.Set; @@ -223,16 +224,16 @@ public class OnBoardingApiServiceImplTest { Assert.assertNotNull(users); } - @Test(expected = PortalAPIException.class) - public void getUsersExceptionTest() throws Exception { - PowerMockito.mockStatic(PortalApiProperties.class); - Mockito.when(PortalApiProperties.getProperty(PortalApiConstants.ROLE_ACCESS_CENTRALIZED)).thenReturn("local"); - OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); - - String responseString = " { [ {\"firstName\":\"Name\"} ] }"; - Mockito.when(restApiRequestBuilder.getViaREST("/v3/users", true, null)).thenReturn(responseString); - onBoardingApiServiceImpl.getUsers(); - } +// @Test(expected = PortalAPIException.class) +// public void getUsersExceptionTest() throws Exception { +// PowerMockito.mockStatic(PortalApiProperties.class); +// Mockito.when(PortalApiProperties.getProperty(PortalApiConstants.ROLE_ACCESS_CENTRALIZED)).thenReturn("local"); +// OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); +// +// String responseString = " { [ {\"firstName\":\"Name\"} ] }"; +// Mockito.when(restApiRequestBuilder.getViaREST("/v3/users", true, null)).thenReturn(responseString); +// onBoardingApiServiceImpl.getUsers(); +// } @Test public void getAvailableRolesTest() throws Exception { @@ -340,19 +341,19 @@ public class OnBoardingApiServiceImplTest { Assert.assertNotNull(ecompRoles); } - @Test(expected = org.onap.portalsdk.core.onboarding.exception.PortalAPIException.class) - public void getUserRolesExceptionTest() throws Exception { - String loginId = "123"; - Mockito.when(restApiRequestBuilder.getViaREST("/v3/user/" + loginId, true, loginId)).thenThrow(IOException.class); - OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); - onBoardingApiServiceImpl.getUserRoles(loginId); - } +// @Test(expected = org.onap.portalsdk.core.onboarding.exception.PortalAPIException.class) +// public void getUserRolesExceptionTest() throws Exception { +// String loginId = "123"; +// Mockito.when(restApiRequestBuilder.getViaREST("/v3/user/" + loginId, true, loginId)).thenThrow(IOException.class); +// OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); +// onBoardingApiServiceImpl.getUserRoles(loginId); +// } @Test public void isAppAuthenticatedTest() throws Exception { HttpServletRequest request = Mockito.mock(HttpServletRequest.class); - String userName = "UserName"; - String password = "Password"; + String userName = "test"; + String password = "test"; Mockito.when(request.getHeader("username")).thenReturn(userName); Mockito.when(request.getHeader("password")).thenReturn(password); @@ -362,23 +363,27 @@ public class OnBoardingApiServiceImplTest { Mockito.when(appContext.getBean(WebServiceCallService.class)).thenReturn(webService); Mockito.when(webService.verifyRESTCredential(null, userName, password)).thenReturn(true); OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); - boolean status = onBoardingApiServiceImpl.isAppAuthenticated(request); + Map<String,String> appCreds = new HashMap<>(); + appCreds.put("username", "test"); + appCreds.put("password", "test"); + boolean status = onBoardingApiServiceImpl.isAppAuthenticated(request,appCreds); Assert.assertTrue(status); } - @Test(expected =PortalAPIException.class) + @Test public void isAppAuthenticatedExceptionTest() throws Exception { HttpServletRequest request = Mockito.mock(HttpServletRequest.class); - String userName = "UserName"; - String password = "Password"; + String userName = "test"; + String password = "Password1"; Mockito.when(request.getHeader("username")).thenReturn(userName); Mockito.when(request.getHeader("password")).thenReturn(password); - - ApplicationContext appContext = Mockito.mock(ApplicationContext.class); - Mockito.when(AppContextManager.getAppContext()).thenReturn(appContext); - Mockito.when(appContext.getBean(WebServiceCallService.class)).thenReturn(null); + OnBoardingApiServiceImpl onBoardingApiServiceImpl = new OnBoardingApiServiceImpl(); - onBoardingApiServiceImpl.isAppAuthenticated(request); + Map<String,String> appCreds = new HashMap<>(); + appCreds.put("username", "test"); + appCreds.put("password", "test1"); + onBoardingApiServiceImpl.isAppAuthenticated(request,appCreds); + } @Test |