X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=olm%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Ftransportpce%2Folm%2Fpower%2FPowerMgmtPowerMockTest.java;h=36c8d097f70926ab8918e954bf530854df5eee9f;hb=1d1ed9547322c81f92d173faacd9427ce6f57bf4;hp=109f336bc32b0119b149124535a4c1839d57ab63;hpb=c725528fb8a0b09ce24adac744c888a10cf90f72;p=transportpce.git diff --git a/olm/src/test/java/org/opendaylight/transportpce/olm/power/PowerMgmtPowerMockTest.java b/olm/src/test/java/org/opendaylight/transportpce/olm/power/PowerMgmtPowerMockTest.java index 109f336bc..36c8d097f 100644 --- a/olm/src/test/java/org/opendaylight/transportpce/olm/power/PowerMgmtPowerMockTest.java +++ b/olm/src/test/java/org/opendaylight/transportpce/olm/power/PowerMgmtPowerMockTest.java @@ -8,7 +8,7 @@ package org.opendaylight.transportpce.olm.power; -import java.math.BigDecimal; + import java.util.HashMap; import java.util.Map; import java.util.Optional; @@ -16,7 +16,6 @@ import org.junit.Assert; import org.junit.Before; import org.junit.Ignore; import org.junit.Test; -import org.junit.runner.RunWith; import org.mockito.ArgumentMatchers; import org.mockito.Mockito; import org.opendaylight.mdsal.binding.api.MountPoint; @@ -48,15 +47,13 @@ import org.opendaylight.transportpce.olm.util.OlmPowerServiceRpcImplUtil; import org.opendaylight.transportpce.olm.util.OlmUtils; import org.opendaylight.transportpce.test.AbstractTest; import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.olm.rev210618.ServicePowerSetupInput; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.Network; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.OpenroadmNodeVersion; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.mapping.Mapping; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.mapping.MappingBuilder; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.mapping.MappingKey; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.network.Nodes; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.network.NodesBuilder; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.network.NodesKey; -import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev210426.network.nodes.NodeInfoBuilder; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.OpenroadmNodeVersion; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.mapping.Mapping; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.mapping.MappingBuilder; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.mapping.MappingKey; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.network.Nodes; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.network.NodesBuilder; +import org.opendaylight.yang.gen.v1.http.org.opendaylight.transportpce.portmapping.rev220316.network.nodes.NodeInfoBuilder; import org.opendaylight.yang.gen.v1.http.org.openroadm.common.types.rev161014.OpticalControlMode; import org.opendaylight.yang.gen.v1.http.org.openroadm.common.types.rev161014.RatioDB; import org.opendaylight.yang.gen.v1.http.org.openroadm.device.rev170206.interfaces.grp.InterfaceBuilder; @@ -64,16 +61,9 @@ import org.opendaylight.yang.gen.v1.http.org.openroadm.device.types.rev191129.No import org.opendaylight.yang.gen.v1.http.org.openroadm.optical.transport.interfaces.rev161014.Interface1Builder; import org.opendaylight.yang.gen.v1.http.org.openroadm.optical.transport.interfaces.rev161014.ots.container.Ots; import org.opendaylight.yang.gen.v1.http.org.openroadm.optical.transport.interfaces.rev161014.ots.container.OtsBuilder; -import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; -import org.powermock.api.mockito.PowerMockito; -import org.powermock.core.classloader.annotations.PowerMockIgnore; -import org.powermock.core.classloader.annotations.PrepareForTest; -import org.powermock.modules.junit4.PowerMockRunner; +import org.opendaylight.yangtools.yang.common.Decimal64; @Ignore -@RunWith(PowerMockRunner.class) -@PrepareForTest({OlmUtils.class,PowerMgmtVersion121.class}) -@PowerMockIgnore("org.opendaylight.yang.gen.v1.http.org.openroadm.device.rev170206.org.openroadm.device.container.*") public class PowerMgmtPowerMockTest extends AbstractTest { private MountPoint mountPoint; @@ -96,7 +86,7 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Before public void setUp() { - this.mountPoint = new MountPointStub(this.getDataBroker()); + this.mountPoint = new MountPointStub(getDataBroker()); this.mountPointService = new MountPointServiceStub(mountPoint); this.mappingUtils = new MappingUtilsImpl(getDataBroker()); this.mappingUtils = Mockito.spy(new MappingUtilsImpl(getDataBroker())); @@ -107,31 +97,30 @@ public class PowerMgmtPowerMockTest extends AbstractTest { this.crossConnectImpl22 = new CrossConnectImpl221(deviceTransactionManager); this.crossConnect = new CrossConnectImpl(deviceTransactionManager, this.mappingUtils, this.crossConnectImpl121, this.crossConnectImpl22, this.crossConnectImpl710); + this.portMappingVersion22 = new PortMappingVersion221(getDataBroker(), deviceTransactionManager); + this.portMappingVersion121 = new PortMappingVersion121(getDataBroker(), deviceTransactionManager); + this.portMappingVersion710 = new PortMappingVersion710(getDataBroker(), deviceTransactionManager); + this.portMapping = new PortMappingImpl(getDataBroker(), this.portMappingVersion710, + this.portMappingVersion22, this.portMappingVersion121); this.openRoadmInterfacesImpl121 = new OpenRoadmInterfacesImpl121(deviceTransactionManager); - this.openRoadmInterfacesImpl22 = new OpenRoadmInterfacesImpl221(deviceTransactionManager); - this.openRoadmInterfacesImpl710 = new OpenRoadmInterfacesImpl710(deviceTransactionManager); + this.openRoadmInterfacesImpl22 = new OpenRoadmInterfacesImpl221(deviceTransactionManager, this.portMapping, + this.portMappingVersion22); + this.openRoadmInterfacesImpl710 = new OpenRoadmInterfacesImpl710(deviceTransactionManager, this.portMapping, + this.portMappingVersion710); this.openRoadmInterfaces = new OpenRoadmInterfacesImpl((this.deviceTransactionManager), this.mappingUtils,this.openRoadmInterfacesImpl121,this.openRoadmInterfacesImpl22, this.openRoadmInterfacesImpl710); this.openRoadmInterfaces = Mockito.spy(this.openRoadmInterfaces); - this.portMappingVersion22 = - new PortMappingVersion221(getDataBroker(), deviceTransactionManager, this.openRoadmInterfaces); - this.portMappingVersion121 = - new PortMappingVersion121(getDataBroker(), deviceTransactionManager, this.openRoadmInterfaces); - this.portMappingVersion710 = - new PortMappingVersion710(getDataBroker(), deviceTransactionManager, this.openRoadmInterfaces); - this.portMapping = new PortMappingImpl(getDataBroker(), this.portMappingVersion710, - this.portMappingVersion22, this.portMappingVersion121); this.portMapping = Mockito.spy(this.portMapping); - this.powerMgmt = new PowerMgmtImpl(this.getDataBroker(), this.openRoadmInterfaces, this.crossConnect, - this.deviceTransactionManager); + this.powerMgmt = new PowerMgmtImpl(getDataBroker(), this.openRoadmInterfaces, this.crossConnect, + this.deviceTransactionManager, this.portMapping); } @Test public void testSetPowerMockingUtil() { - PowerMockito.mockStatic(OlmUtils.class); - PowerMockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getXpdrNodesFromNodesBuilderDeg())); + Mockito.mockStatic(OlmUtils.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getXpdrNodesFromNodesBuilderDeg()); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput2(); boolean output = this.powerMgmt.setPower(input); Assert.assertEquals(false, output); @@ -140,17 +129,17 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerMockingUtilNetwokType() throws OpenRoadmInterfaceException { - PowerMockito.mockStatic(OlmUtils.class); - PowerMockito.mockStatic(PowerMgmtVersion121.class); - PowerMockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getXpdrNodesFromNodesBuilderNetwork())); + Mockito.mockStatic(OlmUtils.class); + Mockito.mockStatic(PowerMgmtVersion121.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getXpdrNodesFromNodesBuilderNetwork()); Map txPowerRangeMap = new HashMap<>(); - PowerMockito.when(PowerMgmtVersion121.getXponderPowerRange( + Mockito.when(PowerMgmtVersion121.getXponderPowerRange( Mockito.anyString(), Mockito.anyString(), Mockito.anyString(), ArgumentMatchers.eq(deviceTransactionManager))) .thenReturn(txPowerRangeMap); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput2(); - OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = PowerMockito.mock(OpenRoadmInterfacesImpl121.class); + OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = Mockito.mock(OpenRoadmInterfacesImpl121.class); Mockito.when(openRoadmInterfacesImpl121Spy.getInterface(Mockito.anyString(), Mockito.anyString())) .thenReturn(Optional.empty()); PowerMgmtImpl powerMgmtImpl = getNewPowerMgmt(openRoadmInterfacesImpl121Spy,this.crossConnect); @@ -161,17 +150,17 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerMockingUtilNetwokTypeMoreThanOneNode() throws OpenRoadmInterfaceException { - PowerMockito.mockStatic(OlmUtils.class); - PowerMockito.mockStatic(PowerMgmtVersion121.class); - PowerMockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getXpdrNodesFromNodesBuilderNetwork())); + Mockito.mockStatic(OlmUtils.class); + Mockito.mockStatic(PowerMgmtVersion121.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getXpdrNodesFromNodesBuilderNetwork()); Map txPowerRangeMap = new HashMap<>(); - PowerMockito.when(PowerMgmtVersion121 + Mockito.when(PowerMgmtVersion121 .getXponderPowerRange(Mockito.anyString(), Mockito.anyString(), Mockito.anyString(), ArgumentMatchers.eq(deviceTransactionManager))) .thenReturn(txPowerRangeMap); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput2(); - OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = PowerMockito.mock(OpenRoadmInterfacesImpl121.class); + OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = Mockito.mock(OpenRoadmInterfacesImpl121.class); Mockito.when(openRoadmInterfacesImpl121Spy.getInterface(Mockito.anyString(), Mockito.anyString())) .thenReturn(Optional.empty()); PowerMgmtImpl powerMgmtImpl = getNewPowerMgmt(openRoadmInterfacesImpl121Spy,this.crossConnect); @@ -182,9 +171,9 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerXpdrNodes() { - PowerMockito.mockStatic(OlmUtils.class); - PowerMockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getXpdrNodesFromNodesBuilderDeg())); + Mockito.mockStatic(OlmUtils.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getXpdrNodesFromNodesBuilderDeg()); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput3(); boolean output = this.powerMgmt.setPower(input); Assert.assertEquals(true, output); @@ -193,11 +182,11 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerRdmNodesReturnInterfaceEmpty() throws OpenRoadmInterfaceException { - PowerMockito.mockStatic(OlmUtils.class); - Mockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getRdmNodesFromNodesBuilder())); + Mockito.mockStatic(OlmUtils.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getRdmNodesFromNodesBuilder()); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput3(); - OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = PowerMockito.mock(OpenRoadmInterfacesImpl121.class); + OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = Mockito.mock(OpenRoadmInterfacesImpl121.class); Mockito.when(openRoadmInterfacesImpl121Spy.getInterface(Mockito.anyString(), Mockito.anyString())) .thenReturn(Optional.empty()); PowerMgmtImpl powerMgmtImpl = getNewPowerMgmt(openRoadmInterfacesImpl121Spy,this.crossConnect); @@ -207,11 +196,11 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerRdmNodesThrowsException() throws OpenRoadmInterfaceException { - PowerMockito.mockStatic(OlmUtils.class); - Mockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getRdmNodesFromNodesBuilder())); + Mockito.mockStatic(OlmUtils.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getRdmNodesFromNodesBuilder()); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput3(); - OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = PowerMockito.mock(OpenRoadmInterfacesImpl121.class); + OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = Mockito.mock(OpenRoadmInterfacesImpl121.class); Mockito.when(openRoadmInterfacesImpl121Spy.getInterface(Mockito.anyString(), Mockito.anyString())) .thenThrow(new OpenRoadmInterfaceException("error thrown by unit tests ")); PowerMgmtImpl powerMgmtImpl = getNewPowerMgmt(openRoadmInterfacesImpl121Spy,this.crossConnect); @@ -221,13 +210,13 @@ public class PowerMgmtPowerMockTest extends AbstractTest { @Test public void testSetPowerRdmNodesReturnInterface() throws OpenRoadmInterfaceException { - PowerMockito.mockStatic(OlmUtils.class); - Mockito.when(OlmUtils.getNode(Mockito.anyString(), ArgumentMatchers.eq(getDataBroker()))) - .thenReturn(Optional.of(getRdmNodesFromNodesBuilder())); + Mockito.mockStatic(OlmUtils.class); + Mockito.when(this.portMapping.getNode(Mockito.anyString())) + .thenReturn(getRdmNodesFromNodesBuilder()); ServicePowerSetupInput input = OlmPowerServiceRpcImplUtil.getServicePowerSetupInput3(); - OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = PowerMockito.mock(OpenRoadmInterfacesImpl121.class); + OpenRoadmInterfacesImpl121 openRoadmInterfacesImpl121Spy = Mockito.mock(OpenRoadmInterfacesImpl121.class); - Ots ots = new OtsBuilder().setSpanLossTransmit(new RatioDB(new BigDecimal(23))).build(); + Ots ots = new OtsBuilder().setSpanLossTransmit(new RatioDB(Decimal64.valueOf("23"))).build(); Interface1Builder intf1Builder = new Interface1Builder(); Mockito.when(openRoadmInterfacesImpl121Spy.getInterface(Mockito.anyString(), Mockito.anyString())).thenReturn( Optional.of(new InterfaceBuilder().addAugmentation(intf1Builder.setOts(ots).build()) @@ -246,9 +235,9 @@ public class PowerMgmtPowerMockTest extends AbstractTest { OpenRoadmInterfacesImpl openRoadmInterfacesSpy = new OpenRoadmInterfacesImpl((this.deviceTransactionManager), this.mappingUtils, openRoadmInterfacesImpl121Spy, this.openRoadmInterfacesImpl22, this.openRoadmInterfacesImpl710); - openRoadmInterfacesSpy = PowerMockito.spy(openRoadmInterfacesSpy); - return new PowerMgmtImpl(this.getDataBroker(), openRoadmInterfacesSpy, crossConnectMock, - this.deviceTransactionManager); + openRoadmInterfacesSpy = Mockito.spy(openRoadmInterfacesSpy); + return new PowerMgmtImpl(getDataBroker(), openRoadmInterfacesSpy, crossConnectMock, + this.deviceTransactionManager, this.portMapping); } private Nodes getXpdrNodesFromNodesBuilderDeg() { @@ -283,33 +272,24 @@ public class PowerMgmtPowerMockTest extends AbstractTest { } private MappingBuilder getMappingBuilderDeg() { - MappingBuilder mappingBuilder = new MappingBuilder(); - mappingBuilder.withKey(new MappingKey("deg")); - mappingBuilder.setLogicalConnectionPoint("logicalConnPoint"); - mappingBuilder.setSupportingOts("OTS"); - mappingBuilder.setSupportingCircuitPackName("2/0"); - mappingBuilder.setSupportingOms("OMS"); - mappingBuilder.setSupportingPort("8080"); - mappingBuilder.setSupportingCircuitPackName("circuit1"); - InstanceIdentifier portMappingIID = - InstanceIdentifier.builder(Network.class).child(Nodes.class, new NodesKey("node 1")) - .child(Mapping.class, new MappingKey("deg")).build(); - return mappingBuilder; + return new MappingBuilder() + .withKey(new MappingKey("deg")) + .setLogicalConnectionPoint("logicalConnPoint") + .setSupportingOts("OTS") + .setSupportingCircuitPackName("2/0") + .setSupportingOms("OMS") + .setSupportingPort("8080") + .setSupportingCircuitPackName("circuit1"); } private MappingBuilder getMappingBuilderNetWork() { - MappingBuilder mappingBuilder = new MappingBuilder(); - mappingBuilder.withKey(new MappingKey("network")); - mappingBuilder.setLogicalConnectionPoint("logicalConnPoint"); - mappingBuilder.setSupportingOts("OTS"); - mappingBuilder.setSupportingCircuitPackName("2/0"); - mappingBuilder.setSupportingOms("OMS"); - mappingBuilder.setSupportingPort("8080"); - mappingBuilder.setSupportingCircuitPackName("circuit1"); - InstanceIdentifier portMappingIID = - InstanceIdentifier.builder(Network.class).child(Nodes.class, new NodesKey("node 1")) - .child(Mapping.class, new MappingKey("network")).build(); - return mappingBuilder; + return new MappingBuilder() + .withKey(new MappingKey("network")) + .setLogicalConnectionPoint("logicalConnPoint") + .setSupportingOts("OTS") + .setSupportingCircuitPackName("2/0") + .setSupportingOms("OMS") + .setSupportingPort("8080") + .setSupportingCircuitPackName("circuit1"); } - }