diff options
author | Michael O'Brien <frank.obrien@amdocs.com> | 2018-07-23 18:11:05 -0400 |
---|---|---|
committer | Michael O'Brien <frank.obrien@amdocs.com> | 2018-07-23 22:44:15 -0400 |
commit | 3bb57a533bb0b022940c495cf68142c48c58136b (patch) | |
tree | fad4c54d6635f512a7d02f94dd99dc1c96c552ed /reference/logging-demo/src/test/java/org | |
parent | 268e848b1494dc09d614d8479f70bf9aefa69166 (diff) |
inc log aop code coverage
Change-Id: I004fdc5ea71eef412fb096207523d41a669fdc61
Issue-ID: LOG-148
Signed-off-by: Michael O'Brien <frank.obrien@amdocs.com>
Diffstat (limited to 'reference/logging-demo/src/test/java/org')
-rw-r--r-- | reference/logging-demo/src/test/java/org/onap/logging/demo/ApplicationServiceTest.java | 69 |
1 files changed, 68 insertions, 1 deletions
diff --git a/reference/logging-demo/src/test/java/org/onap/logging/demo/ApplicationServiceTest.java b/reference/logging-demo/src/test/java/org/onap/logging/demo/ApplicationServiceTest.java index aab2ba7..a263a1f 100644 --- a/reference/logging-demo/src/test/java/org/onap/logging/demo/ApplicationServiceTest.java +++ b/reference/logging-demo/src/test/java/org/onap/logging/demo/ApplicationServiceTest.java @@ -22,10 +22,17 @@ package org.onap.logging.demo; import static org.junit.Assert.*; +import java.util.Set; + import javax.servlet.http.HttpServletRequest; import org.junit.Test; +import org.onap.demo.logging.test.JoinPointMock; import org.onap.demo.logging.ApplicationService; +import org.onap.demo.logging.LoggingAspect; +import org.onap.demo.logging.RestApplication; +import org.onap.demo.logging.RestHealthServiceImpl; +import org.onap.demo.logging.RestServiceImpl; import org.springframework.mock.web.MockHttpServletRequest; import org.springframework.util.Assert; @@ -39,7 +46,67 @@ public class ApplicationServiceTest { Assert.notNull(servletRequest); boolean health = service.health(servletRequest); Assert.isTrue(health); - System.out.println("health : " + health); } + @Test + public final void testRestEndpointCoverageForRestHealthServiceImpl() { + // primarily for code coverage + RestHealthServiceImpl service = new RestHealthServiceImpl(); + Assert.notNull(service); + ApplicationService appService = new ApplicationService(); + Assert.notNull(appService); + service.setApplicationService(appService); + String health = service.getHealth(); + Assert.notNull(health); + Assert.isTrue(health.equalsIgnoreCase("true")); + } + + @Test + public final void testRestEndpointCoverageForRestServiceImpl() { + // primarily for code coverage + RestServiceImpl service = new RestServiceImpl(); + Assert.notNull(service); + ApplicationService appService = new ApplicationService(); + Assert.notNull(appService); + service.setApplicationService(appService); + String health = service.getTest(); + Assert.notNull(health); + } + + @Test + public final void testJAXRSFramework() { + // primarily for code coverage + RestApplication app = new RestApplication(); + Assert.notNull(app); + Set<Class<?>> classes = app.getClasses(); + Assert.notNull(classes); + Assert.isTrue(classes.size() > 1); + } + + @Test + public final void testLoggingAspect() { + // primarily for code coverage + LoggingAspect aspect = new LoggingAspect(); + Assert.notNull(aspect); + JoinPointMock joinPoint = new JoinPointMock(); + ApplicationService appService = new ApplicationService(); + Assert.notNull(appService); + joinPoint.setTarget(appService.getClass()); + joinPoint.getTarget(); + HttpServletRequest servletRequest = new MockHttpServletRequest(); + Assert.notNull(servletRequest); + HttpServletRequest[] args = new HttpServletRequest[1]; + args[0] = servletRequest; + joinPoint.setArgs(args); + aspect.logAfter(joinPoint); + aspect.logBefore(joinPoint); + // cover mock joinpoint - expecting null as only target and args needs to be implemented + joinPoint.getKind(); + joinPoint.getSignature(); + joinPoint.getSourceLocation(); + joinPoint.getStaticPart(); + joinPoint.getThis(); + joinPoint.toLongString(); + joinPoint.toShortString(); + } } |