*/
package org.opendaylight.netconf.sal.connect.netconf.sal;
-import static org.mockito.Matchers.any;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertThrows;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
-import com.google.common.util.concurrent.CheckedFuture;
import com.google.common.util.concurrent.Futures;
import java.util.Collection;
-import java.util.Collections;
-import java.util.Set;
-import org.junit.Assert;
+import java.util.concurrent.ExecutionException;
+import org.junit.AfterClass;
import org.junit.Before;
+import org.junit.BeforeClass;
import org.junit.Test;
import org.mockito.ArgumentCaptor;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
-import org.opendaylight.controller.config.util.xml.XmlUtil;
-import org.opendaylight.controller.md.sal.dom.api.DOMRpcAvailabilityListener;
-import org.opendaylight.controller.md.sal.dom.api.DOMRpcException;
-import org.opendaylight.controller.md.sal.dom.api.DOMRpcIdentifier;
-import org.opendaylight.controller.md.sal.dom.api.DOMRpcResult;
-import org.opendaylight.mdsal.binding.generator.impl.ModuleInfoBackedContext;
+import org.opendaylight.mdsal.binding.runtime.spi.BindingRuntimeHelpers;
+import org.opendaylight.mdsal.dom.api.DOMRpcAvailabilityListener;
+import org.opendaylight.mdsal.dom.api.DOMRpcIdentifier;
+import org.opendaylight.mdsal.dom.api.DOMRpcResult;
import org.opendaylight.netconf.api.NetconfMessage;
+import org.opendaylight.netconf.api.xml.XmlUtil;
+import org.opendaylight.netconf.sal.connect.api.MessageTransformer;
import org.opendaylight.netconf.sal.connect.api.RemoteDeviceCommunicator;
+import org.opendaylight.netconf.sal.connect.netconf.AbstractBaseSchemasTest;
import org.opendaylight.netconf.sal.connect.netconf.schema.mapping.NetconfMessageTransformer;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.netconf.base._1._0.rev110601.$YangModuleInfoImpl;
+import org.opendaylight.netconf.sal.connect.netconf.util.NetconfMessageTransformUtil;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.netconf.base._1._0.rev110601.IetfNetconfService;
+import org.opendaylight.yangtools.rcf8528.data.util.EmptyMountPointContext;
import org.opendaylight.yangtools.yang.common.QName;
import org.opendaylight.yangtools.yang.common.RpcResult;
import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
+import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier;
import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode;
-import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
+import org.opendaylight.yangtools.yang.data.api.schema.DOMSourceAnyxmlNode;
+import org.opendaylight.yangtools.yang.data.api.schema.DataContainerChild;
import org.opendaylight.yangtools.yang.data.impl.schema.Builders;
+import org.opendaylight.yangtools.yang.model.api.EffectiveModelContext;
import org.opendaylight.yangtools.yang.model.api.RpcDefinition;
-import org.opendaylight.yangtools.yang.model.api.SchemaContext;
-import org.opendaylight.yangtools.yang.model.api.SchemaPath;
+import org.w3c.dom.Node;
-public class NetconfDeviceRpcTest {
+public class NetconfDeviceRpcTest extends AbstractBaseSchemasTest {
+ private static EffectiveModelContext SCHEMA_CONTEXT;
@Mock
private DOMRpcAvailabilityListener listener;
private RemoteDeviceCommunicator<NetconfMessage> communicator;
private NetconfDeviceRpc rpc;
- private SchemaPath path;
+ private QName type;
private DOMRpcResult expectedReply;
- private SchemaContext schema;
+
+ @BeforeClass
+ public static void beforeClass() {
+ SCHEMA_CONTEXT = BindingRuntimeHelpers.createEffectiveModel(IetfNetconfService.class);
+ }
+
+ @AfterClass
+ public static void afterClass() {
+ SCHEMA_CONTEXT = null;
+ }
@Before
public void setUp() throws Exception {
MockitoAnnotations.initMocks(this);
- schema = getSchema();
- NetconfMessageTransformer transformer = new NetconfMessageTransformer(schema, true);
+ NetconfMessageTransformer transformer = new NetconfMessageTransformer(
+ new EmptyMountPointContext(SCHEMA_CONTEXT), true, BASE_SCHEMAS.getBaseSchema());
final NetconfMessage reply = new NetconfMessage(XmlUtil.readXmlToDocument(
"<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"no\"?>\n"
+ "<rpc-reply xmlns=\"urn:ietf:params:xml:ns:netconf:base:1.0\" message-id=\"101\">\n"
RpcResult<NetconfMessage> result = RpcResultBuilder.success(reply).build();
doReturn(Futures.immediateFuture(result))
.when(communicator).sendRequest(any(NetconfMessage.class), any(QName.class));
- rpc = new NetconfDeviceRpc(schema, communicator, transformer);
+ rpc = new NetconfDeviceRpc(SCHEMA_CONTEXT, communicator, transformer);
- path = SchemaPath
- .create(true, QName.create("urn:ietf:params:xml:ns:netconf:base:1.0", "2011-06-01", "get-config"));
- expectedReply = transformer.toRpcResult(reply, path);
+ type = QName.create("urn:ietf:params:xml:ns:netconf:base:1.0", "2011-06-01", "get-config");
+ expectedReply = transformer.toRpcResult(reply, type);
+ }
+
+ @Test
+ public void testDeadlock() throws Exception {
+ // when rpc is successful, but transformer fails for some reason
+ final MessageTransformer<NetconfMessage> failingTransformer = mock(MessageTransformer.class);
+ final RemoteDeviceCommunicator<NetconfMessage> communicatorMock = mock(RemoteDeviceCommunicator.class);
+ final NetconfMessage msg = null;
+ final RpcResult<NetconfMessage> result = RpcResultBuilder.success(msg).build();
+ when(communicatorMock.sendRequest(any(), any())).thenReturn(Futures.immediateFuture(result));
+ when(failingTransformer.toRpcResult(any(), any())).thenThrow(new RuntimeException("FAIL"));
+ final NetconfDeviceRpc failingRpc = new NetconfDeviceRpc(SCHEMA_CONTEXT, communicatorMock, failingTransformer);
+ assertThrows(ExecutionException.class, () -> failingRpc.invokeRpc(type, mock(ContainerNode.class)).get());
+ assertThrows(ExecutionException.class, () -> failingRpc.invokeRpc(type, null).get());
}
@Test
public void testInvokeRpc() throws Exception {
- NormalizedNode<?, ?> input = createNode("urn:ietf:params:xml:ns:netconf:base:1.0", "2011-06-01", "filter");
- final CheckedFuture<DOMRpcResult, DOMRpcException> future = rpc.invokeRpc(path, input);
- final DOMRpcResult result = future.checkedGet();
- Assert.assertEquals(expectedReply, result);
+ ContainerNode input = createNode("urn:ietf:params:xml:ns:netconf:base:1.0", "2011-06-01", "filter");
+ final DOMRpcResult result = rpc.invokeRpc(type, input).get();
+ assertEquals(expectedReply.getResult().getIdentifier(), result.getResult().getIdentifier());
+ assertEquals(resolveNode(expectedReply), resolveNode(result));
+ }
+
+ private static Node resolveNode(final DOMRpcResult result) {
+ DataContainerChild<?, ?> value = ((ContainerNode) result.getResult())
+ .getChild(NetconfMessageTransformUtil.NETCONF_DATA_NODEID).get();
+ Node node = ((DOMSourceAnyxmlNode)value).getValue().getNode();
+ assertNotNull(node);
+ return node;
}
@Test
verify(listener).onRpcAvailable(argument.capture());
final Collection<DOMRpcIdentifier> argValue = argument.getValue();
- final Set<RpcDefinition> operations = schema.getOperations();
- Assert.assertEquals(argValue.size(), operations.size());
+ final Collection<? extends RpcDefinition> operations = SCHEMA_CONTEXT.getOperations();
+ assertEquals(argValue.size(), operations.size());
for (RpcDefinition operation : operations) {
- final DOMRpcIdentifier domRpcIdentifier = DOMRpcIdentifier.create(operation.getPath());
- Assert.assertTrue(argValue.contains(domRpcIdentifier));
+ final DOMRpcIdentifier domRpcIdentifier = DOMRpcIdentifier.create(operation.getQName());
+ assertTrue(argValue.contains(domRpcIdentifier));
}
}
- private static ContainerNode createNode(String namespace, String date, String localName) {
+ private static ContainerNode createNode(final String namespace, final String date, final String localName) {
return Builders.containerBuilder().withNodeIdentifier(
- new YangInstanceIdentifier.NodeIdentifier(QName.create(namespace, date, localName))).build();
- }
-
- private static SchemaContext getSchema() {
- final ModuleInfoBackedContext moduleInfoBackedContext = ModuleInfoBackedContext.create();
- moduleInfoBackedContext.addModuleInfos(Collections.singleton($YangModuleInfoImpl.getInstance()));
- return moduleInfoBackedContext.tryToCreateSchemaContext().get();
+ new NodeIdentifier(QName.create(namespace, date, localName))).build();
}
}
Code Review / netconf.git / blobdiff