From 670feaeaa8695d139e326b635c1f6bd1aa5e0dcf Mon Sep 17 00:00:00 2001 From: Joss Armstrong Date: Wed, 9 Jan 2019 19:17:28 +0000 Subject: Test coverage for appc-provider-topology Increased line coverage from 0% to 86% Issue-ID: APPC-1317 Change-Id: I2297634d50a389c8c250981656e00a9bb2741ea3 Signed-off-by: Joss Armstrong --- .../appc/provider/topology/TopologyService.java | 7 +++--- .../provider/topology/TopologyServiceTest.java | 27 +++++++++------------- 2 files changed, 14 insertions(+), 20 deletions(-) diff --git a/appc-provider/appc-provider-bundle/src/main/java/org/onap/appc/provider/topology/TopologyService.java b/appc-provider/appc-provider-bundle/src/main/java/org/onap/appc/provider/topology/TopologyService.java index 74f3296d7..441aab624 100644 --- a/appc-provider/appc-provider-bundle/src/main/java/org/onap/appc/provider/topology/TopologyService.java +++ b/appc-provider/appc-provider-bundle/src/main/java/org/onap/appc/provider/topology/TopologyService.java @@ -5,6 +5,8 @@ * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Copyright (C) 2017 Amdocs + * ================================================================================ + * Modifications (C) 2019 Ericsson * ============================================================================= * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,13 +38,10 @@ import com.att.eelf.configuration.EELFLogger; import com.att.eelf.configuration.EELFManager; import com.att.eelf.i18n.EELFResourceManager; import java.net.InetAddress; -import java.text.DateFormat; -import java.text.SimpleDateFormat; import java.time.Instant; import java.time.temporal.ChronoUnit; import java.util.Date; import java.util.Properties; -import java.util.TimeZone; import org.onap.appc.Constants; import org.onap.appc.configuration.Configuration; import org.onap.appc.configuration.ConfigurationFactory; @@ -601,7 +600,7 @@ public class TopologyService { /*************************************************/ - private boolean callGraph(Properties props) { + protected boolean callGraph(Properties props) { String moduleName = configuration.getProperty(Constants.PROPERTY_MODULE_NAME); String methodName = configuration.getProperty(Constants.PROPERTY_TOPOLOGY_METHOD); String version = configuration.getProperty(Constants.PROPERTY_TOPOLOGY_VERSION); diff --git a/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/topology/TopologyServiceTest.java b/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/topology/TopologyServiceTest.java index f3014e264..a4aed5048 100644 --- a/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/topology/TopologyServiceTest.java +++ b/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/topology/TopologyServiceTest.java @@ -5,6 +5,8 @@ * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Copyright (C) 2017 Amdocs + * ================================================================================ + * Modifications (C) 2019 Ericsson * ============================================================================= * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -27,6 +29,7 @@ import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.mockito.Mock; +import org.mockito.Mockito; import org.opendaylight.yang.gen.v1.org.onap.appc.provider.rev160104.UUID; import org.opendaylight.yang.gen.v1.org.onap.appc.provider.rev160104.common.request.header.CommonRequestHeader; import org.opendaylight.yang.gen.v1.org.onap.appc.provider.rev160104.config.payload.ConfigPayload; @@ -34,21 +37,19 @@ import org.opendaylight.yang.gen.v1.org.onap.appc.provider.rev160104.vnf.resourc import org.onap.appc.configuration.Configuration; import org.onap.appc.configuration.ConfigurationFactory; import org.onap.appc.provider.AppcProvider; -import org.powermock.api.mockito.PowerMockito; import org.powermock.core.classloader.annotations.PrepareForTest; import org.powermock.modules.junit4.PowerMockRunner; -import static org.mockito.Matchers.any; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.internal.verification.VerificationModeFactory.times; import static org.powermock.api.mockito.PowerMockito.mockStatic; import static org.powermock.api.mockito.PowerMockito.spy; -import static org.powermock.api.mockito.PowerMockito.verifyPrivate; import static org.powermock.api.mockito.PowerMockito.when; +import java.util.Properties; @RunWith(PowerMockRunner.class) -@PrepareForTest({TopologyService.class, ConfigurationFactory.class}) +@PrepareForTest(ConfigurationFactory.class) public class TopologyServiceTest { @Mock private AppcProvider appcProvider; @@ -71,11 +72,10 @@ public class TopologyServiceTest { ConfigPayload configPayload = mock(ConfigPayload.class); doReturn("url").when(configPayload).getConfigUrl(); doReturn("configJson").when(configPayload).getConfigJson(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.modifyConfig(commonRequestHeader, configPayload); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } @Test @@ -86,11 +86,10 @@ public class TopologyServiceTest { UUID uuid = mock(UUID.class); doReturn("uuid-value").when(uuid).getValue(); doReturn(uuid).when(vnfResource).getVmId(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.migrate(commonRequestHeader, vnfResource); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } @Test @@ -101,11 +100,10 @@ public class TopologyServiceTest { UUID uuid = mock(UUID.class); doReturn("uuid-value").when(uuid).getValue(); doReturn(uuid).when(vnfResource).getVmId(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.restart(commonRequestHeader, vnfResource); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } @Test @@ -116,11 +114,10 @@ public class TopologyServiceTest { UUID uuid = mock(UUID.class); doReturn("uuid-value").when(uuid).getValue(); doReturn(uuid).when(vnfResource).getVmId(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.rebuild(commonRequestHeader, vnfResource); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } @Test @@ -131,11 +128,10 @@ public class TopologyServiceTest { UUID uuid = mock(UUID.class); doReturn("uuid-value").when(uuid).getValue(); doReturn(uuid).when(vnfResource).getVmId(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.snapshot(commonRequestHeader, vnfResource); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } @Test @@ -146,11 +142,10 @@ public class TopologyServiceTest { UUID uuid = mock(UUID.class); doReturn("uuid-value").when(uuid).getValue(); doReturn(uuid).when(vnfResource).getVmId(); - PowerMockito.doReturn(true).when(topologyService, "callGraph", any()); topologyService.vmstatuscheck(commonRequestHeader, vnfResource); - verifyPrivate(topologyService, times(1)).invoke("callGraph", any()); + Mockito.verify(topologyService, times(1)).callGraph(Mockito.any(Properties.class)); } } \ No newline at end of file -- cgit 1.2.3-korg