X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-rest-connector%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fsal%2Frestconf%2Fimpl%2Ftest%2FRestGetOperationTest.java;h=536d140cf1b78e05b7a3e77d38e43a32d8a1f8f4;hb=refs%2Fchanges%2F27%2F8927%2F3;hp=f0a232fba670539c5a872e5894a89bd7c214c778;hpb=e64d758b45360b2ed8b7d8967fc1c7caaa7aa58f;p=controller.git diff --git a/opendaylight/md-sal/sal-rest-connector/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/RestGetOperationTest.java b/opendaylight/md-sal/sal-rest-connector/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/RestGetOperationTest.java index f0a232fba6..536d140cf1 100644 --- a/opendaylight/md-sal/sal-rest-connector/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/RestGetOperationTest.java +++ b/opendaylight/md-sal/sal-rest-connector/src/test/java/org/opendaylight/controller/sal/restconf/impl/test/RestGetOperationTest.java @@ -7,8 +7,9 @@ */ package org.opendaylight.controller.sal.restconf.impl.test; -import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; import static org.mockito.Matchers.any; @@ -16,6 +17,8 @@ import static org.mockito.Matchers.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; +import com.google.common.collect.Lists; +import com.google.common.collect.Maps; import java.io.FileNotFoundException; import java.io.InputStream; import java.io.UnsupportedEncodingException; @@ -29,14 +32,12 @@ import java.util.List; import java.util.Map; import java.util.regex.Matcher; import java.util.regex.Pattern; - import javax.ws.rs.core.Application; import javax.ws.rs.core.MediaType; import javax.ws.rs.core.MultivaluedHashMap; import javax.ws.rs.core.MultivaluedMap; import javax.ws.rs.core.Response; import javax.ws.rs.core.UriInfo; - import org.glassfish.jersey.server.ResourceConfig; import org.glassfish.jersey.test.JerseyTest; import org.junit.BeforeClass; @@ -68,16 +69,13 @@ import org.w3c.dom.Document; import org.w3c.dom.Element; import org.w3c.dom.NodeList; -import com.google.common.collect.Lists; -import com.google.common.collect.Maps; - public class RestGetOperationTest extends JerseyTest { static class NodeData { Object key; Object data; // List for a CompositeNode, value Object for a SimpleNode - NodeData( Object key, Object data ) { + NodeData(final Object key, final Object data) { this.key = key; this.data = data; } @@ -119,7 +117,7 @@ public class RestGetOperationTest extends JerseyTest { resourceConfig = resourceConfig.registerInstances(restconfImpl, StructuredDataToXmlProvider.INSTANCE, StructuredDataToJsonProvider.INSTANCE, XmlToCompositeNodeProvider.INSTANCE, JsonToCompositeNodeProvider.INSTANCE); - resourceConfig.registerClasses( RestconfDocumentedExceptionMapper.class ); + resourceConfig.registerClasses(RestconfDocumentedExceptionMapper.class); return resourceConfig; } @@ -174,12 +172,11 @@ public class RestGetOperationTest extends JerseyTest { /** * MountPoint test. URI represents mount point. * - * Slashes in URI behind mount point. lst1 element with key - * GigabitEthernet0%2F0%2F0%2F0 (GigabitEthernet0/0/0/0) is requested via - * GET HTTP operation. It is tested whether %2F character is replaced with - * simple / in InstanceIdentifier parameter in method - * {@link BrokerFacade#readConfigurationDataBehindMountPoint(MountInstance, InstanceIdentifier)} - * which is called in method {@link RestconfImpl#readConfigurationData} + * Slashes in URI behind mount point. lst1 element with key GigabitEthernet0%2F0%2F0%2F0 (GigabitEthernet0/0/0/0) is + * requested via GET HTTP operation. It is tested whether %2F character is replaced with simple / in + * InstanceIdentifier parameter in method + * {@link BrokerFacade#readConfigurationDataBehindMountPoint(MountInstance, InstanceIdentifier)} which is called in + * method {@link RestconfImpl#readConfigurationData} * * * @throws ParseException @@ -214,7 +211,7 @@ public class RestGetOperationTest extends JerseyTest { parameters.add(new InstanceIdentifier.NodeIdentifier(qNameCont)); parameters.add(new InstanceIdentifier.NodeIdentifierWithPredicates(qNameList, qNameKeyList, "GigabitEthernet0/0/0/0")); - return new InstanceIdentifier(parameters); + return InstanceIdentifier.create(parameters); } @Test @@ -356,7 +353,7 @@ public class RestGetOperationTest extends JerseyTest { } - private Matcher validateOperationsResponseJson(String searchIn, String rpcName, String moduleName) { + private Matcher validateOperationsResponseJson(final String searchIn, final String rpcName, final String moduleName) { StringBuilder regex = new StringBuilder(); regex.append("^"); @@ -390,7 +387,7 @@ public class RestGetOperationTest extends JerseyTest { } - private Matcher validateOperationsResponseXml(String searchIn, String rpcName, String namespace) { + private Matcher validateOperationsResponseXml(final String searchIn, final String rpcName, final String namespace) { StringBuilder regex = new StringBuilder(); regex.append("^"); @@ -494,7 +491,7 @@ public class RestGetOperationTest extends JerseyTest { } - private void validateModulesResponseXml(Response response) { + private void validateModulesResponseXml(final Response response) { assertEquals(200, response.getStatus()); String responseBody = response.readEntity(String.class); @@ -506,7 +503,7 @@ public class RestGetOperationTest extends JerseyTest { .find()); } - private void validateModulesResponseJson(Response response) { + private void validateModulesResponseJson(final Response response) { assertEquals(200, response.getStatus()); String responseBody = response.readEntity(String.class); @@ -518,7 +515,8 @@ public class RestGetOperationTest extends JerseyTest { .find()); } - private Matcher prepareJsonRegex(String module, String revision, String namespace, String searchIn) { + private Matcher prepareJsonRegex(final String module, final String revision, final String namespace, + final String searchIn) { StringBuilder regex = new StringBuilder(); regex.append("^"); @@ -544,7 +542,8 @@ public class RestGetOperationTest extends JerseyTest { } - private Matcher prepareXmlRegex(String module, String revision, String namespace, String searchIn) { + private Matcher prepareXmlRegex(final String module, final String revision, final String namespace, + final String searchIn) { StringBuilder regex = new StringBuilder(); regex.append("^"); @@ -572,13 +571,14 @@ public class RestGetOperationTest extends JerseyTest { return ptrn.matcher(searchIn); } - private void prepareMockForModulesTest(ControllerContext mockedControllerContext) throws FileNotFoundException { + private void prepareMockForModulesTest(final ControllerContext mockedControllerContext) + throws FileNotFoundException { SchemaContext schemaContext = TestUtils.loadSchemaContext("/modules"); mockedControllerContext.setGlobalSchema(schemaContext); // when(mockedControllerContext.getGlobalSchema()).thenReturn(schemaContext); } - private int get(String uri, String mediaType) { + private int get(final String uri, final String mediaType) { return target(uri).request(mediaType).get().getStatus(); } @@ -615,314 +615,325 @@ public class RestGetOperationTest extends JerseyTest { return null; } + /** + * If includeWhiteChars URI parameter is set to false then no white characters can be included in returned output + * + * @throws UnsupportedEncodingException + */ + @Test + public void getDataWithUriIncludeWhiteCharsParameterTest() throws UnsupportedEncodingException { + getDataWithUriIncludeWhiteCharsParameter("config"); + getDataWithUriIncludeWhiteCharsParameter("operational"); + } + + private void getDataWithUriIncludeWhiteCharsParameter(String target) throws UnsupportedEncodingException { + mockReadConfigurationDataMethod(); + String uri = "/" + target + "/ietf-interfaces:interfaces/interface/eth0"; + Response response = target(uri).queryParam("prettyPrint", "false").request("application/xml").get(); + String xmlData = response.readEntity(String.class); + + Pattern pattern = Pattern.compile(".*(>\\s+|\\s+<).*", Pattern.DOTALL); + Matcher matcher = pattern.matcher(xmlData); + // XML element can't surrounded with white character (e.g "> " or + // " <") + assertFalse(matcher.matches()); + + response = target(uri).queryParam("prettyPrint", "false").request("application/json").get(); + String jsonData = response.readEntity(String.class); + pattern = Pattern.compile(".*(\\}\\s+|\\s+\\{|\\]\\s+|\\s+\\[|\\s+:|:\\s+).*", Pattern.DOTALL); + matcher = pattern.matcher(jsonData); + // JSON element can't surrounded with white character (e.g "} ", " {", + // "] ", " [", " :" or ": ") + assertFalse(matcher.matches()); + } + @Test public void getDataWithUriDepthParameterTest() throws UnsupportedEncodingException { - ControllerContext.getInstance().setGlobalSchema( schemaContextModules ); - - CompositeNode depth1Cont = toCompositeNode( - toCompositeNodeData( toNestedQName( "depth1-cont" ), - toCompositeNodeData( toNestedQName( "depth2-cont1" ), - toCompositeNodeData( toNestedQName( "depth3-cont1" ), - toCompositeNodeData( toNestedQName( "depth4-cont1" ), - toSimpleNodeData( toNestedQName( "depth5-leaf1" ), "depth5-leaf1-value" ) - ), - toSimpleNodeData( toNestedQName( "depth4-leaf1" ), "depth4-leaf1-value" ) - ), - toSimpleNodeData( toNestedQName( "depth3-leaf1" ), "depth3-leaf1-value" ) - ), - toCompositeNodeData( toNestedQName( "depth2-cont2" ), - toCompositeNodeData( toNestedQName( "depth3-cont2" ), - toCompositeNodeData( toNestedQName( "depth4-cont2" ), - toSimpleNodeData( toNestedQName( "depth5-leaf2" ), "depth5-leaf2-value" ) - ), - toSimpleNodeData( toNestedQName( "depth4-leaf2" ), "depth4-leaf2-value" ) - ), - toSimpleNodeData( toNestedQName( "depth3-leaf2" ), "depth3-leaf2-value" ) - ), - toSimpleNodeData( toNestedQName( "depth2-leaf1" ), "depth2-leaf1-value" ) - ) ); - - when( brokerFacade.readConfigurationData( any( InstanceIdentifier.class ) ) ) - .thenReturn( depth1Cont ); + ControllerContext.getInstance().setGlobalSchema(schemaContextModules); + + CompositeNode depth1Cont = toCompositeNode(toCompositeNodeData( + toNestedQName("depth1-cont"), + toCompositeNodeData( + toNestedQName("depth2-cont1"), + toCompositeNodeData( + toNestedQName("depth3-cont1"), + toCompositeNodeData(toNestedQName("depth4-cont1"), + toSimpleNodeData(toNestedQName("depth5-leaf1"), "depth5-leaf1-value")), + toSimpleNodeData(toNestedQName("depth4-leaf1"), "depth4-leaf1-value")), + toSimpleNodeData(toNestedQName("depth3-leaf1"), "depth3-leaf1-value")), + toCompositeNodeData( + toNestedQName("depth2-cont2"), + toCompositeNodeData( + toNestedQName("depth3-cont2"), + toCompositeNodeData(toNestedQName("depth4-cont2"), + toSimpleNodeData(toNestedQName("depth5-leaf2"), "depth5-leaf2-value")), + toSimpleNodeData(toNestedQName("depth4-leaf2"), "depth4-leaf2-value")), + toSimpleNodeData(toNestedQName("depth3-leaf2"), "depth3-leaf2-value")), + toSimpleNodeData(toNestedQName("depth2-leaf1"), "depth2-leaf1-value"))); + + when(brokerFacade.readConfigurationData(any(InstanceIdentifier.class))).thenReturn(depth1Cont); // Test config with depth 1 - Response response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "1" ) - .request( "application/xml" ).get(); + Response response = target("/config/nested-module:depth1-cont").queryParam("depth", "1") + .request("application/xml").get(); - verifyXMLResponse( response, expectEmptyContainer( "depth1-cont" ) ); + verifyXMLResponse(response, expectEmptyContainer("depth1-cont")); // Test config with depth 2 - response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "2" ) - .request( "application/xml" ).get(); + response = target("/config/nested-module:depth1-cont").queryParam("depth", "2").request("application/xml") + .get(); -// String xml="depth2-leaf1-value"; -// Response mr=mock(Response.class); -// when(mr.getEntity()).thenReturn( new java.io.StringBufferInputStream(xml) ); + // String + // xml="depth2-leaf1-value"; + // Response mr=mock(Response.class); + // when(mr.getEntity()).thenReturn( new + // java.io.StringBufferInputStream(xml) ); - verifyXMLResponse( response, - expectContainer( "depth1-cont", - expectEmptyContainer( "depth2-cont1" ), - expectEmptyContainer( "depth2-cont2" ), - expectLeaf( "depth2-leaf1", "depth2-leaf1-value" ) - ) ); + verifyXMLResponse( + response, + expectContainer("depth1-cont", expectEmptyContainer("depth2-cont1"), + expectEmptyContainer("depth2-cont2"), expectLeaf("depth2-leaf1", "depth2-leaf1-value"))); // Test config with depth 3 - response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "3" ) - .request( "application/xml" ).get(); - - verifyXMLResponse( response, - expectContainer( "depth1-cont", - expectContainer( "depth2-cont1", - expectEmptyContainer( "depth3-cont1" ), - expectLeaf( "depth3-leaf1", "depth3-leaf1-value" ) - ), - expectContainer( "depth2-cont2", - expectEmptyContainer( "depth3-cont2" ), - expectLeaf( "depth3-leaf2", "depth3-leaf2-value" ) - ), - expectLeaf( "depth2-leaf1", "depth2-leaf1-value" ) - ) ); + response = target("/config/nested-module:depth1-cont").queryParam("depth", "3").request("application/xml") + .get(); + + verifyXMLResponse( + response, + expectContainer( + "depth1-cont", + expectContainer("depth2-cont1", expectEmptyContainer("depth3-cont1"), + expectLeaf("depth3-leaf1", "depth3-leaf1-value")), + expectContainer("depth2-cont2", expectEmptyContainer("depth3-cont2"), + expectLeaf("depth3-leaf2", "depth3-leaf2-value")), + expectLeaf("depth2-leaf1", "depth2-leaf1-value"))); // Test config with depth 4 - response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "4" ) - .request( "application/xml" ).get(); - - verifyXMLResponse( response, - expectContainer( "depth1-cont", - expectContainer( "depth2-cont1", - expectContainer( "depth3-cont1", - expectEmptyContainer( "depth4-cont1" ), - expectLeaf( "depth4-leaf1", "depth4-leaf1-value" ) - ), - expectLeaf( "depth3-leaf1", "depth3-leaf1-value" ) - ), - expectContainer( "depth2-cont2", - expectContainer( "depth3-cont2", - expectEmptyContainer( "depth4-cont2" ), - expectLeaf( "depth4-leaf2", "depth4-leaf2-value" ) - ), - expectLeaf( "depth3-leaf2", "depth3-leaf2-value" ) - ), - expectLeaf( "depth2-leaf1", "depth2-leaf1-value" ) - ) ); + response = target("/config/nested-module:depth1-cont").queryParam("depth", "4").request("application/xml") + .get(); + + verifyXMLResponse( + response, + expectContainer( + "depth1-cont", + expectContainer( + "depth2-cont1", + expectContainer("depth3-cont1", expectEmptyContainer("depth4-cont1"), + expectLeaf("depth4-leaf1", "depth4-leaf1-value")), + expectLeaf("depth3-leaf1", "depth3-leaf1-value")), + expectContainer( + "depth2-cont2", + expectContainer("depth3-cont2", expectEmptyContainer("depth4-cont2"), + expectLeaf("depth4-leaf2", "depth4-leaf2-value")), + expectLeaf("depth3-leaf2", "depth3-leaf2-value")), + expectLeaf("depth2-leaf1", "depth2-leaf1-value"))); // Test config with depth 5 - response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "5" ) - .request( "application/xml" ).get(); - - verifyXMLResponse( response, - expectContainer( "depth1-cont", - expectContainer( "depth2-cont1", - expectContainer( "depth3-cont1", - expectContainer( "depth4-cont1", - expectLeaf( "depth5-leaf1", "depth5-leaf1-value" ) - ), - expectLeaf( "depth4-leaf1", "depth4-leaf1-value" ) - ), - expectLeaf( "depth3-leaf1", "depth3-leaf1-value" ) - ), - expectContainer( "depth2-cont2", - expectContainer( "depth3-cont2", - expectContainer( "depth4-cont2", - expectLeaf( "depth5-leaf2", "depth5-leaf2-value" ) - ), - expectLeaf( "depth4-leaf2", "depth4-leaf2-value" ) - ), - expectLeaf( "depth3-leaf2", "depth3-leaf2-value" ) - ), - expectLeaf( "depth2-leaf1", "depth2-leaf1-value" ) - ) ); + response = target("/config/nested-module:depth1-cont").queryParam("depth", "5").request("application/xml") + .get(); + + verifyXMLResponse( + response, + expectContainer( + "depth1-cont", + expectContainer( + "depth2-cont1", + expectContainer( + "depth3-cont1", + expectContainer("depth4-cont1", + expectLeaf("depth5-leaf1", "depth5-leaf1-value")), + expectLeaf("depth4-leaf1", "depth4-leaf1-value")), + expectLeaf("depth3-leaf1", "depth3-leaf1-value")), + expectContainer( + "depth2-cont2", + expectContainer( + "depth3-cont2", + expectContainer("depth4-cont2", + expectLeaf("depth5-leaf2", "depth5-leaf2-value")), + expectLeaf("depth4-leaf2", "depth4-leaf2-value")), + expectLeaf("depth3-leaf2", "depth3-leaf2-value")), + expectLeaf("depth2-leaf1", "depth2-leaf1-value"))); // Test config with depth unbounded - response = target( "/config/nested-module:depth1-cont" ).queryParam( "depth", "unbounded" ) - .request( "application/xml" ).get(); - - verifyXMLResponse( response, - expectContainer( "depth1-cont", - expectContainer( "depth2-cont1", - expectContainer( "depth3-cont1", - expectContainer( "depth4-cont1", - expectLeaf( "depth5-leaf1", "depth5-leaf1-value" ) - ), - expectLeaf( "depth4-leaf1", "depth4-leaf1-value" ) - ), - expectLeaf( "depth3-leaf1", "depth3-leaf1-value" ) - ), - expectContainer( "depth2-cont2", - expectContainer( "depth3-cont2", - expectContainer( "depth4-cont2", - expectLeaf( "depth5-leaf2", "depth5-leaf2-value" ) - ), - expectLeaf( "depth4-leaf2", "depth4-leaf2-value" ) - ), - expectLeaf( "depth3-leaf2", "depth3-leaf2-value" ) - ), - expectLeaf( "depth2-leaf1", "depth2-leaf1-value" ) - ) ); + response = target("/config/nested-module:depth1-cont").queryParam("depth", "unbounded") + .request("application/xml").get(); + + verifyXMLResponse( + response, + expectContainer( + "depth1-cont", + expectContainer( + "depth2-cont1", + expectContainer( + "depth3-cont1", + expectContainer("depth4-cont1", + expectLeaf("depth5-leaf1", "depth5-leaf1-value")), + expectLeaf("depth4-leaf1", "depth4-leaf1-value")), + expectLeaf("depth3-leaf1", "depth3-leaf1-value")), + expectContainer( + "depth2-cont2", + expectContainer( + "depth3-cont2", + expectContainer("depth4-cont2", + expectLeaf("depth5-leaf2", "depth5-leaf2-value")), + expectLeaf("depth4-leaf2", "depth4-leaf2-value")), + expectLeaf("depth3-leaf2", "depth3-leaf2-value")), + expectLeaf("depth2-leaf1", "depth2-leaf1-value"))); // Test operational - CompositeNode depth2Cont1 = toCompositeNode( - toCompositeNodeData( toNestedQName( "depth2-cont1" ), - toCompositeNodeData( toNestedQName( "depth3-cont1" ), - toCompositeNodeData( toNestedQName( "depth4-cont1" ), - toSimpleNodeData( toNestedQName( "depth5-leaf1" ), "depth5-leaf1-value" ) - ), - toSimpleNodeData( toNestedQName( "depth4-leaf1" ), "depth4-leaf1-value" ) - ), - toSimpleNodeData( toNestedQName( "depth3-leaf1" ), "depth3-leaf1-value" ) - ) ); - - when( brokerFacade.readOperationalData( any( InstanceIdentifier.class ) ) ) - .thenReturn( depth2Cont1 ); - - response = target( "/operational/nested-module:depth1-cont/depth2-cont1" ) - .queryParam( "depth", "3" ).request( "application/xml" ).get(); - - verifyXMLResponse( response, - expectContainer( "depth2-cont1", - expectContainer( "depth3-cont1", - expectEmptyContainer( "depth4-cont1" ), - expectLeaf( "depth4-leaf1", "depth4-leaf1-value" ) - ), - expectLeaf( "depth3-leaf1", "depth3-leaf1-value" ) - ) ); + CompositeNode depth2Cont1 = toCompositeNode(toCompositeNodeData( + toNestedQName("depth2-cont1"), + toCompositeNodeData( + toNestedQName("depth3-cont1"), + toCompositeNodeData(toNestedQName("depth4-cont1"), + toSimpleNodeData(toNestedQName("depth5-leaf1"), "depth5-leaf1-value")), + toSimpleNodeData(toNestedQName("depth4-leaf1"), "depth4-leaf1-value")), + toSimpleNodeData(toNestedQName("depth3-leaf1"), "depth3-leaf1-value"))); + + when(brokerFacade.readOperationalData(any(InstanceIdentifier.class))).thenReturn(depth2Cont1); + + response = target("/operational/nested-module:depth1-cont/depth2-cont1").queryParam("depth", "3") + .request("application/xml").get(); + + verifyXMLResponse( + response, + expectContainer( + "depth2-cont1", + expectContainer("depth3-cont1", expectEmptyContainer("depth4-cont1"), + expectLeaf("depth4-leaf1", "depth4-leaf1-value")), + expectLeaf("depth3-leaf1", "depth3-leaf1-value"))); } @Test public void getDataWithInvalidDepthParameterTest() { - ControllerContext.getInstance().setGlobalSchema( schemaContextModules ); + ControllerContext.getInstance().setGlobalSchema(schemaContextModules); - final MultivaluedMap paramMap = new MultivaluedHashMap<>(); - paramMap.putSingle( "depth", "1o" ); - UriInfo mockInfo = mock( UriInfo.class ); - when( mockInfo.getQueryParameters( false ) ).thenAnswer( - new Answer>() { - @Override - public MultivaluedMap answer( InvocationOnMock invocation ) { - return paramMap; - } - } ); + final MultivaluedMap paramMap = new MultivaluedHashMap<>(); + paramMap.putSingle("depth", "1o"); + UriInfo mockInfo = mock(UriInfo.class); + when(mockInfo.getQueryParameters(false)).thenAnswer(new Answer>() { + @Override + public MultivaluedMap answer(final InvocationOnMock invocation) { + return paramMap; + } + }); - getDataWithInvalidDepthParameterTest( mockInfo ); + getDataWithInvalidDepthParameterTest(mockInfo); - paramMap.putSingle( "depth", "0" ); - getDataWithInvalidDepthParameterTest( mockInfo ); + paramMap.putSingle("depth", "0"); + getDataWithInvalidDepthParameterTest(mockInfo); - paramMap.putSingle( "depth", "-1" ); - getDataWithInvalidDepthParameterTest( mockInfo ); + paramMap.putSingle("depth", "-1"); + getDataWithInvalidDepthParameterTest(mockInfo); } - private void getDataWithInvalidDepthParameterTest( UriInfo uriInfo ) { + private void getDataWithInvalidDepthParameterTest(final UriInfo uriInfo) { try { - restconfImpl.readConfigurationData( "nested-module:depth1-cont", uriInfo ); - fail( "Expected RestconfDocumentedException" ); - } - catch( RestconfDocumentedException e ) { - assertTrue( "Unexpected error message: " + e.getErrors().get( 0 ).getErrorMessage(), - e.getErrors().get( 0 ).getErrorMessage().contains( "depth" ) ); + restconfImpl.readConfigurationData("nested-module:depth1-cont", uriInfo); + fail("Expected RestconfDocumentedException"); + } catch (RestconfDocumentedException e) { + assertTrue("Unexpected error message: " + e.getErrors().get(0).getErrorMessage(), e.getErrors().get(0) + .getErrorMessage().contains("depth")); } } - private void verifyXMLResponse( Response response, NodeData nodeData ) { + private void verifyXMLResponse(final Response response, final NodeData nodeData) { - Document doc = TestUtils.loadDocumentFrom( (InputStream) response.getEntity() ); - assertNotNull( "Could not parse XML document", doc ); + Document doc = TestUtils.loadDocumentFrom((InputStream) response.getEntity()); + assertNotNull("Could not parse XML document", doc); - //System.out.println(TestUtils.getDocumentInPrintableForm( doc )); + // System.out.println(TestUtils.getDocumentInPrintableForm( doc )); - verifyContainerElement( doc.getDocumentElement(), nodeData ); + verifyContainerElement(doc.getDocumentElement(), nodeData); } @SuppressWarnings("unchecked") - private void verifyContainerElement( Element element, NodeData nodeData ) { + private void verifyContainerElement(final Element element, final NodeData nodeData) { - assertEquals( "Element local name", nodeData.key, element.getNodeName() ); + assertEquals("Element local name", nodeData.key, element.getNodeName()); NodeList childNodes = element.getChildNodes(); - if( nodeData.data == null ) { // empty container - assertTrue( "Expected no child elements for \"" + element.getNodeName() + "\"", - childNodes.getLength() == 0 ); + if (nodeData.data == null) { // empty container + assertTrue("Expected no child elements for \"" + element.getNodeName() + "\"", childNodes.getLength() == 0); return; } - Map expChildMap = Maps.newHashMap(); - for( NodeData expChild: (List)nodeData.data ) { - expChildMap.put( expChild.key.toString(), expChild ); + Map expChildMap = Maps.newHashMap(); + for (NodeData expChild : (List) nodeData.data) { + expChildMap.put(expChild.key.toString(), expChild); } - for( int i = 0; i < childNodes.getLength(); i++ ) { - org.w3c.dom.Node actualChild = childNodes.item( i ); - if( !( actualChild instanceof Element ) ) { + for (int i = 0; i < childNodes.getLength(); i++) { + org.w3c.dom.Node actualChild = childNodes.item(i); + if (!(actualChild instanceof Element)) { continue; } - Element actualElement = (Element)actualChild; - NodeData expChild = expChildMap.remove( actualElement.getNodeName() ); - assertNotNull( "Unexpected child element for parent \"" + element.getNodeName() + - "\": " + actualElement.getNodeName(), expChild ); - - if( expChild.data == null || expChild.data instanceof List ) { - verifyContainerElement( actualElement, expChild ); - } - else { - assertEquals( "Text content for element: " + actualElement.getNodeName(), - expChild.data, actualElement.getTextContent() ); + Element actualElement = (Element) actualChild; + NodeData expChild = expChildMap.remove(actualElement.getNodeName()); + assertNotNull( + "Unexpected child element for parent \"" + element.getNodeName() + "\": " + + actualElement.getNodeName(), expChild); + + if (expChild.data == null || expChild.data instanceof List) { + verifyContainerElement(actualElement, expChild); + } else { + assertEquals("Text content for element: " + actualElement.getNodeName(), expChild.data, + actualElement.getTextContent()); } } - if( !expChildMap.isEmpty() ) { - fail( "Missing elements for parent \"" + element.getNodeName() + - "\": " + expChildMap.keySet() ); + if (!expChildMap.isEmpty()) { + fail("Missing elements for parent \"" + element.getNodeName() + "\": " + expChildMap.keySet()); } } - private NodeData expectContainer( String name, NodeData... childData ) { - return new NodeData( name, Lists.newArrayList( childData ) ); + private NodeData expectContainer(final String name, final NodeData... childData) { + return new NodeData(name, Lists.newArrayList(childData)); } - private NodeData expectEmptyContainer( String name ) { - return new NodeData( name, null ); + private NodeData expectEmptyContainer(final String name) { + return new NodeData(name, null); } - private NodeData expectLeaf( String name, Object value ) { - return new NodeData( name, value ); + private NodeData expectLeaf(final String name, final Object value) { + return new NodeData(name, value); } - private QName toNestedQName( String localName ) { - return QName.create( "urn:nested:module", "2014-06-3", localName ); + private QName toNestedQName(final String localName) { + return QName.create("urn:nested:module", "2014-06-3", localName); } @SuppressWarnings("unchecked") - private CompositeNode toCompositeNode( NodeData nodeData ) { + private CompositeNode toCompositeNode(final NodeData nodeData) { CompositeNodeBuilder builder = ImmutableCompositeNode.builder(); - builder.setQName( (QName) nodeData.key ); + builder.setQName((QName) nodeData.key); - for( NodeData child: (List)nodeData.data ) { - if( child.data instanceof List ) { - builder.add( toCompositeNode( child ) ); - } - else { - builder.addLeaf( (QName) child.key, child.data ); + for (NodeData child : (List) nodeData.data) { + if (child.data instanceof List) { + builder.add(toCompositeNode(child)); + } else { + builder.addLeaf((QName) child.key, child.data); } } return builder.toInstance(); } - private NodeData toCompositeNodeData( QName key, NodeData... childData ) { - return new NodeData( key, Lists.newArrayList( childData ) ); + private NodeData toCompositeNodeData(final QName key, final NodeData... childData) { + return new NodeData(key, Lists.newArrayList(childData)); } - private NodeData toSimpleNodeData( QName key, Object value ) { - return new NodeData( key, value ); + private NodeData toSimpleNodeData(final QName key, final Object value) { + return new NodeData(key, value); } }