From 7b7d8beca8298ee55f79596fe8fd694cde248eb4 Mon Sep 17 00:00:00 2001 From: Varun Gudisena Date: Tue, 3 Oct 2017 18:04:34 -0500 Subject: [PATCH] Add unit test cases Add unit test cases Issue-Id: DMAAP-151 Change-Id: I8196e6fc0609e1365e0ad2bace5fa6a37ebf915c Signed-off-by: Varun Gudisena --- .../att/nsa/dmaap/DMaaPWebExceptionMapperTest.java | 90 ++++++++++++++++++++++ .../nsa/dmaap/service/TopicRestServiceTest.java | 17 ++++ .../att/nsa/dmaap/util/DMaaPAuthFilterTest.java | 20 ++++- 3 files changed, 126 insertions(+), 1 deletion(-) diff --git a/src/test/java/com/att/nsa/dmaap/DMaaPWebExceptionMapperTest.java b/src/test/java/com/att/nsa/dmaap/DMaaPWebExceptionMapperTest.java index 8569044..22c31c9 100644 --- a/src/test/java/com/att/nsa/dmaap/DMaaPWebExceptionMapperTest.java +++ b/src/test/java/com/att/nsa/dmaap/DMaaPWebExceptionMapperTest.java @@ -22,6 +22,13 @@ package com.att.nsa.dmaap; import static org.junit.Assert.*; +import javax.ws.rs.BadRequestException; +import javax.ws.rs.InternalServerErrorException; +import javax.ws.rs.NotAllowedException; +import javax.ws.rs.NotAuthorizedException; +import javax.ws.rs.NotFoundException; +import javax.ws.rs.ServiceUnavailableException; + import org.junit.After; import org.junit.Before; import org.junit.Test; @@ -49,4 +56,87 @@ public class DMaaPWebExceptionMapperTest { } + @Test + public void testToResponseNotFoundException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new NotFoundException()); + } catch (NullPointerException e) { + assertTrue(true); + } + + } + + @Test + public void testToResponseInternalServerErrorException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new InternalServerErrorException()); + + } catch (NullPointerException e) { + assertTrue(true); + } + + } + + @Test + public void testToResponseNotAuthorizedException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new NotAuthorizedException("Error", "Error")); + + } catch (NullPointerException e) { + assertTrue(true); + } + + } + + @Test + public void testToResponseBadRequestException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new BadRequestException()); + + } catch (NullPointerException e) { + assertTrue(true); + } + + } + + @Test + public void testToResponseNotAllowedException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new NotAllowedException("Not Allowed")); + + } catch (NullPointerException e) { + assertTrue(true); + } + + } + + @Test + public void testToResponseServiceUnavailableException() { + + DMaaPWebExceptionMapper mapper = new DMaaPWebExceptionMapper(); + + try { + mapper.toResponse(new ServiceUnavailableException()); + + } catch (NullPointerException e) { + assertTrue(true); + } + + } + } \ No newline at end of file diff --git a/src/test/java/com/att/nsa/dmaap/service/TopicRestServiceTest.java b/src/test/java/com/att/nsa/dmaap/service/TopicRestServiceTest.java index 5c1ce97..abaac0f 100644 --- a/src/test/java/com/att/nsa/dmaap/service/TopicRestServiceTest.java +++ b/src/test/java/com/att/nsa/dmaap/service/TopicRestServiceTest.java @@ -390,6 +390,23 @@ public class TopicRestServiceTest { topicService.permitConsumerForTopic("enfTopicNamePlusExtra", "consumerID"); } + + @Test + public void testPermitConsumerForTopicWithException() throws DMaaPAccessDeniedException, CambriaApiException, IOException, + TopicExistsException, AccessDeniedException { + + Assert.assertNotNull(topicService); + + when(dmaapContext.getRequest()).thenReturn(httpServReq); + when(dmaaPAuthenticator.authenticate(dmaapContext)).thenReturn(nsaSimpleApiKey); + when(configReader.getfSecurityManager()).thenReturn(dmaaPAuthenticator); + when(dmaapContext.getConfigReader()).thenReturn(configReader); + + TopicBean topicBean = new TopicBean(); + topicBean.setTopicName("enfTopicNamePlusExtra"); + + topicService.permitConsumerForTopic("enfTopicNamePlusExtra", "consumerID"); + } @Test public void testDenyConsumerForTopic() throws DMaaPAccessDeniedException, CambriaApiException, IOException, diff --git a/src/test/java/com/att/nsa/dmaap/util/DMaaPAuthFilterTest.java b/src/test/java/com/att/nsa/dmaap/util/DMaaPAuthFilterTest.java index ba197b3..6212543 100644 --- a/src/test/java/com/att/nsa/dmaap/util/DMaaPAuthFilterTest.java +++ b/src/test/java/com/att/nsa/dmaap/util/DMaaPAuthFilterTest.java @@ -20,8 +20,9 @@ package com.att.nsa.dmaap.util; +import static org.junit.Assert.assertTrue; + import java.io.IOException; -import java.util.logging.Level; import javax.servlet.FilterChain; import javax.servlet.ServletException; @@ -43,6 +44,8 @@ import com.att.ajsc.beans.PropertiesMapBean; import com.att.nsa.cambria.beans.DMaaPContext; import com.att.nsa.cambria.exception.DMaaPResponseCode; +import com.att.cadi.Access.Level; + @RunWith(PowerMockRunner.class) @PrepareForTest({ PropertiesMapBean.class, DMaaPResponseCode.class }) public class DMaaPAuthFilterTest { @@ -78,6 +81,7 @@ public class DMaaPAuthFilterTest { PowerMockito.when(req.getHeader("Authorization")).thenReturn("Authorization"); // when(dmaapContext.getResponse()).thenReturn(res); filter.doFilter(req, res, chain); + assertTrue(true); } @@ -89,7 +93,21 @@ public class DMaaPAuthFilterTest { // when(dmaapContext.getResponse()).thenReturn(res); filter.doFilter(req, res, chain); + assertTrue(true); } + @Test + public void testLog() { + String s[] = { "test1", "test2" }; + Object[] o = s; + filter.log(Level.AUDIT, o); + filter.log(Level.DEBUG, o); + filter.log(Level.ERROR, o); + filter.log(Level.INFO, o); + filter.log(Level.INIT, o); + filter.log(Level.WARN, o); + assertTrue(true); + } + } \ No newline at end of file -- 2.16.6